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

[T, Arun]

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

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