RE: [PATCH v2 1/2] dt-bindings: backlight: Add MPS MP3309C

From: Flavio Suligoi
Date: Mon Sep 18 2023 - 08:32:10 EST


HI Rob,

...

> Subject: Re: [PATCH v2 1/2] dt-bindings: backlight: Add MPS MP3309C
>

...

> > +
> > + mps,overvoltage-protection-microvolt:
> > + description: Overvoltage protection (13.5V, 24V or 35.5V). If missing, the
> > + hardware default of 35.5V is used.
>
> default: 35500000
>
> instead of prose saying the same thing.
>
> With that,
>
> Reviewed-by: Rob Herring <robh@xxxxxxxxxx>

Ok, right, I'll fix it in my next version.

I prefer to wait a few days before sending the next version,
to wait if there are other comments on the device driver file as well.

Just a question about the procedure: in my next version, I have to add the sentence:

"Reviewed-by: Rob Herring <robh@xxxxxxxxxx>"

in my commit or it will be added again by you?

Thanks for your help,
Flavio