<div dir="auto"><div>Hi,<div dir="auto"><br></div><div dir="auto"><br></div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Jul 12, 2022, 11:41 Chen-Yu Tsai <<a href="mailto:wenst@chromium.org">wenst@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Tue, Jul 12, 2022 at 4:08 PM Robert Foss <<a href="mailto:robert.foss@linaro.org" target="_blank" rel="noreferrer">robert.foss@linaro.org</a>> wrote:<br>
><br>
> Hi Allen,<br>
><br>
> On Thu, 7 Jul 2022 at 10:06, allen <<a href="mailto:allen.chen@ite.com.tw" target="_blank" rel="noreferrer">allen.chen@ite.com.tw</a>> wrote:<br>
> ><br>
> > From: allen chen <<a href="mailto:allen.chen@ite.com.tw" target="_blank" rel="noreferrer">allen.chen@ite.com.tw</a>><br>
> ><br>
> > Change power sequence to meet it6505 data sheet requirement when boot on.<br>
> ><br>
> > Signed-off-by: Pin-Yen Lin <<a href="mailto:treapking@chromium.org" target="_blank" rel="noreferrer">treapking@chromium.org</a>><br>
> > Signed-off-by: Allen Chen <<a href="mailto:allen.chen@ite.com.tw" target="_blank" rel="noreferrer">allen.chen@ite.com.tw</a>><br>
> ><br>
> > ---<br>
> >  drivers/gpu/drm/bridge/ite-it6505.c | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/drivers/gpu/drm/bridge/ite-it6505.c b/drivers/gpu/drm/bridge/ite-it6505.c<br>
> > index 2d119e3016b3..aa5e0aa1af85 100644<br>
> > --- a/drivers/gpu/drm/bridge/ite-it6505.c<br>
> > +++ b/drivers/gpu/drm/bridge/ite-it6505.c<br>
> > @@ -3029,7 +3029,7 @@ static int it6505_init_pdata(struct it6505 *it6505)<br>
> >                 return PTR_ERR(pdata->ovdd);<br>
> >         }<br>
> ><br>
> > -       pdata->gpiod_reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_HIGH);<br>
> > +       pdata->gpiod_reset = devm_gpiod_get(dev, "reset", GPIOD_OUT_LOW);<br>
><br>
> Making this change is problematic since it requires a corresponding<br>
> change in all of the device trees that use this device. It's against<br>
> policy to change this interface after it has been introduced.<br>
><br>
> Unless anyone thinks otherwise, I would like to see this patch dropped.<br>
<br>
I think you're confusing GPIO_ACTIVE_{LOW,HIGH} flags in the device tree<br>
vs GPIOD_OUT_{LOW,HIGH}, which just sets the "default" state the GPIO<br>
should be in when it is requested.<br>
<br>
This change doesn't impact the device tree.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Thanks ChenYu, you are 100% right.</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">Reviewed-by: Robert Foss <<a href="mailto:robert.foss@linaro.org">robert.foss@linaro.org</a>></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
Regards<br>
ChenYu<br>
<br>
> >         if (IS_ERR(pdata->gpiod_reset)) {<br>
> >                 dev_err(dev, "gpiod_reset gpio not found");<br>
> >                 return PTR_ERR(pdata->gpiod_reset);<br>
> > --<br>
> > 2.25.1<br>
> ><br>
</blockquote></div></div></div>