<div dir="ltr">Hi Sam,<div>Thank you again for your support. I forgot that point, which is really important to know.</div><div><br></div><div>    Molly</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Sam Ravnborg <<a href="mailto:sam@ravnborg.org">sam@ravnborg.org</a>> 于2022年7月10日周日 18:13写道：<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Molly,<br>
<br>
On Sun, Jul 10, 2022 at 02:19:41PM +0800, Molly Sophia wrote:<br>
> Hi Sam,<br>
> <br>
> Thanks for your suggestions.<br>
> <br>
> Sam Ravnborg <<a href="mailto:sam@ravnborg.org" target="_blank">sam@ravnborg.org</a>> 于 2022年7月10日周日 上午4:47写道：<br>
> <br>
> > Hi Molly,<br>
> ><br>
> > thanks for the quick response to the review comments.<br>
> ><br>
> > On Sat, Jul 09, 2022 at 10:11:35PM +0800, MollySophia wrote:<br>
> > > Add documentation for "novatek,nt35596s" panel.<br>
> > ><br>
> > > Signed-off-by: MollySophia <<a href="mailto:mollysophia379@gmail.com" target="_blank">mollysophia379@gmail.com</a>><br>
> > The s-o-b needs your real name - guess the above is a concatenation of<br>
> > first name and surname.<br>
> ><br>
> > The binding included in this patch fails the check:<br>
> > $ make DT_CHECKER_FLAGS=-m dt_binding_check<br>
> ><br>
> > You may need to run:<br>
> > $ pip3 install dtschema --upgrade<br>
> ><br>
> > Or you may have to install some dependencies first.<br>
> > The problem is that the patch is missing a "reset-gpios: true"<br>
> ><br>
> > On top of this I looked at the binding - and the description<br>
> > this is copied from is almost identical.<br>
> > So another approach would be to extend the existing binding like<br>
> > in the following.<br>
> ><br>
> > And this also gives a good hint that maybe this can be embedded in<br>
> > the existing driver - and there is no need for a new driver.<br>
> > Could you try to give this a spin and get back on this.<br>
> ><br>
> <br>
> That's reasonable. Actually, this driver was modified from<br>
> novatek,nt35596s, with different panel initialization commands, and it<br>
> seems easy to be embedded in<br>
> the existing driver. However, I wonder what the driver file name would<br>
> be...? "panel-novatek-nt35596s-nt36672a.c" or something else?<br>
<br>
Just keep the current driver name - we cannot embed all the supported HW<br>
in one driver name anyway. And then you do not break currents users in<br>
case they have hardwired the current driver name.<br>
<br>
        Sam<br>
</blockquote></div>