Hi, On Tue, Jun 9, 2020 at 5:05 AM Harigovindan P <harigovi@xxxxxxxxxxxxxx> wrote: > > ti-sn65dsi86 bridge is enumerated as a runtime device. When > suspend is triggered, PM core adds a refcount on all the > devices and calls device suspend, since usage count is > already incremented, runtime suspend will not be called > and it kept the bridge regulators and gpios ON which resulted > in platform not entering into XO shutdown. > > Add changes to force suspend on the runtime device during pm sleep. > > Signed-off-by: Harigovindan P <harigovi@xxxxxxxxxxxxxx> > --- > > Changes in v2: > - Include bridge name in the commit message and > remove dependent patchwork link from the commit > text as bridge is independent of OEM(Stephen Boyd) > > Changes in v3: > - Updating changelog to explain the need for patch > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 2 ++ > 1 file changed, 2 insertions(+) I think this patch is good to go now (has both Stephen's and my reviews). I noticed that Neil landed my other patches to this driver recently (thanks!) and wondered why he didn't land this one. Then, I realized that you didn't send it to him or the other bridge maintainer. :( Have you tried running get_maintainer? $ ./scripts/get_maintainer.pl -f drivers/gpu/drm/bridge/ti-sn65dsi86.c Andrzej Hajda <a.hajda@xxxxxxxxxxx> (maintainer:DRM DRIVERS FOR BRIDGE CHIPS) Neil Armstrong <narmstrong@xxxxxxxxxxxx> (maintainer:DRM DRIVERS FOR BRIDGE CHIPS) Laurent Pinchart <Laurent.pinchart@xxxxxxxxxxxxxxxx> (reviewer:DRM DRIVERS FOR BRIDGE CHIPS) Jonas Karlman <jonas@xxxxxxxxx> (reviewer:DRM DRIVERS FOR BRIDGE CHIPS) Jernej Skrabec <jernej.skrabec@xxxxxxxx> (reviewer:DRM DRIVERS FOR BRIDGE CHIPS) David Airlie <airlied@xxxxxxxx> (maintainer:DRM DRIVERS) Daniel Vetter <daniel@xxxxxxxx> (maintainer:DRM DRIVERS) dri-devel@xxxxxxxxxxxxxxxxxxxxx (open list:DRM DRIVERS) linux-kernel@xxxxxxxxxxxxxxx (open list) In any case, unless someone has extra feedback on this patch I think it's ready to land. Neil: If you're willing to land this patch too, can you let Harigovindan know if it needs to be re-sent with you in the "To:" list or if you can find it on the dri-devel list? Thanks! -Doug
