Re: [XEN PATCH v2] misra: add more entires to exclude-list.json

[Julien Grall <julien@xxxxxxx>]

Hi,

On 17/08/2023 11:13, Federico Serafini wrote:
> On 17/08/23 12:02, Federico Serafini wrote:
> > On 17/08/23 11:57, Julien Grall wrote:
> > > Hi,
> > >
> > > On 17/08/2023 10:55, Federico Serafini wrote:
> > > > Add entries to the exclude-list.json for those files that need to be
> > > > excluded from the analysis scan.
> > > >
> > > > Signed-off-by: Federico Serafini <federico.serafini@xxxxxxxxxxx>
> > > > ---
> > > > Changes in v2:
> > > > - fixed indentation.
> > > What's the difference with the other v2 you posted ~10 min ago?
> > >
> > > Is this the patch we need to review? If so, can you review on the 
> > > previous one to explicitely say it should be ignored.
> > > Cheers,
> > There is no difference,
> > it seems that two mails have been sent,
> > sorry for that.
> The patches to be reviewed are:
> [XEN PATCH v2] misra: add more entires to exclude-list.json
> [XEN PATCH 1/2] automation/eclair: update configuration
>
> When a v2 of a patch within a series is needed,
> should I send a v2 of the whole series?
It depends on the situation. In your case here, I would have commented 
on the patch that the indentation is wrong and wait for more input on 
the rest of the series.
If this was the only change, then most likely it could have been dealt 
on commit. Otherwise you could have sent a new revision for the full series.
Cheers,

--
Julien Grall