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

["Ilpo Järvinen" <ilpo.jarvinen@linux.intel.com>]

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@ideasonboard.com
> Reviewed-by: Hans de Goede johannes.goede@oss.qualcomm.com

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@linux.intel.com> 
> Sent: 01 April 2026 06:11 PM
> To: T, Arun <arun.t@intel.com>
> Cc: johannes.goede@oss.qualcomm.com; Kao, Arec <arec.kao@intel.com>; ilpo.jarvinen@linux.intel.com; dan.scally@ideasonboard.com; platform-driver-x86@vger.kernel.org; linux-media@vger.kernel.org; linux-kernel@vger.kernel.org; Djait, Mehdi <mehdi.djait@intel.com>
> 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@intel.com>
> > Reviewed-by: Daniel Scally <dan.scally@ideasonboard.com>
> > Reviewed-by: Sakari Ailus <sakari.ailus@linux.intel.com>
> 
> 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@oss.qualcomm.com
> 
> --
> Regards,
> 
> Sakari Ailus
>