RE: [PATCH v7 2/2] media: ov13b10: Support multiple regulators

[Ilpo Järvinen]

On Thu, 2 Apr 2026, T, Arun wrote:

> Hi Sakarai, 
> 
> Thank you for reviewing my patches and for your comments. Based on that, I added you as a reviewer.
> 
> The other reviewers explicitly asked me by email to add their Reviewed-by tags, so I included:
> Reviewed-by: Daniel Scally dan.scally@xxxxxxxxxxxxxxxx
> Reviewed-by: Hans de Goede johannes.goede@xxxxxxxxxxxxxxxx

There are both misformatted.

-- 
 i.

> Sorry for the mistake. If you do not want your name included as a 
> reviewer, I will remove it and send a new v8 patch series. 
> -
> Regards,
> Arun T
> 
> -----Original Message-----
> From: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx> 
> Sent: 01 April 2026 06:11 PM
> To: T, Arun <arun.t@xxxxxxxxx>
> Cc: johannes.goede@xxxxxxxxxxxxxxxx; Kao, Arec <arec.kao@xxxxxxxxx>; ilpo.jarvinen@xxxxxxxxxxxxxxx; dan.scally@xxxxxxxxxxxxxxxx; platform-driver-x86@xxxxxxxxxxxxxxx; linux-media@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; Djait, Mehdi <mehdi.djait@xxxxxxxxx>
> Subject: Re: [PATCH v7 2/2] media: ov13b10: Support multiple regulators
> 
> Hi Arun,
> 
> On Wed, Apr 01, 2026 at 05:50:30PM +0530, Arun T wrote:
> > The OV13B10 sensor driver currently handles a single regulator called 
> > avdd, however the sensor can be supplied by up to three regulators.
> > Update the driver to handle all of them together using the regulator 
> > bulk API.
> > 
> > Signed-off-by: Arun T <arun.t@xxxxxxxxx>
> > Reviewed-by: Daniel Scally <dan.scally@xxxxxxxxxxxxxxxx>
> > Reviewed-by: Sakari Ailus <sakari.ailus@xxxxxxxxxxxxxxx>
> 
> You can't add Reviewed-by: tags that haven't been explicitly (or otherwise) given. I haven't given mine on either of the two patches, for instance.
> 
> What about the other two?
> 
> > Reviewed-by: Hans de Goede <johannes.goede@xxxxxxxxxxxxxxxx
> 
> --
> Regards,
> 
> Sakari Ailus
>