<div dir="auto">Hi Sam,</div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, 8 Nov 2020 at 9:47 pm, Sam Ravnborg <<a href="mailto:sam@ravnborg.org">sam@ravnborg.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Russell,<br>
<br>
On Sun, Nov 08, 2020 at 09:57:25AM +0000, Russell King - ARM Linux admin wrote:<br>
> On Sun, Nov 08, 2020 at 10:53:22AM +0100, Sam Ravnborg wrote:<br>
> > Russell,<br>
> > <br>
> > On Sat, Oct 31, 2020 at 07:17:47PM +1100, Jonathan Liu wrote:<br>
> > > It has been observed that resetting force in the detect function can<br>
> > > result in the PHY being powered down in response to hot-plug detect<br>
> > > being asserted, even when the HDMI connector is forced on.<br>
> > > <br>
> > > Enabling debug messages and adding a call to dump_stack() in<br>
> > > dw_hdmi_phy_power_off() shows the following in dmesg:<br>
> > > [  160.637413] dwhdmi-rockchip ff940000.hdmi: EVENT=plugin<br>
> > > [  160.637433] dwhdmi-rockchip ff940000.hdmi: PHY powered down in 0 iterations<br>
> > > <br>
> > > Call trace:<br>
> > > dw_hdmi_phy_power_off<br>
> > > dw_hdmi_phy_disable<br>
> > > dw_hdmi_update_power<br>
> > > dw_hdmi_detect<br>
> > > dw_hdmi_connector_detect<br>
> > > drm_helper_probe_detect_ctx<br>
> > > drm_helper_hpd_irq_event<br>
> > > dw_hdmi_irq<br>
> > > irq_thread_fn<br>
> > > irq_thread<br>
> > > kthread<br>
> > > ret_from_fork<br>
> > > <br>
> > > Fixes: 381f05a7a842 ("drm: bridge/dw_hdmi: add connector mode forcing")<br>
> > > Signed-off-by: Jonathan Liu <<a href="mailto:net147@gmail.com" target="_blank">net147@gmail.com</a>><br>
> > <br>
> > you are the original author of this code - any comments on this patch?<br>
> <br>
> No further comments beyond what has already been discussed, and the<br>
> long and short of it is it's been so long that I don't remember why<br>
> that code was there. Given that, I'm not even in a position to ack<br>
> the change. Sorry.<br>
Thanks for the quick reply.<br>
<br>
Given that this fixes a problem for Jonathan I will apply this to -fixes<br>
if there is no other feedback the next couple of days.<br>
If it introduces regression we can take it from there.<br>
<br>
Jonathan - please ping me if I forget.<br>
<br>
        Sam</blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex" dir="auto"><br>
</blockquote></div></div><div dir="auto">Ping.</div><div dir="auto"><br></div><div dir="auto">Regards,</div><div dir="auto">Jonathan</div>