On 30 September 2015 at 14:46, Inki Dae <inki.dae@xxxxxxxxxxx> wrote: > Hi Emil, > > On 2015년 09월 30일 21:19, Emil Velikov wrote: >> Hi Inki, >> >> On 30 September 2015 at 12:21, Inki Dae <inki.dae@xxxxxxxxxxx> wrote: >>> This patch removes unnecessary pm suspend/resume functions. >>> >>> All kms sub drivers will be controlled by top of Exynos drm driver >>> and connector dpms so these sub drivers shouldn't have their own >>> pm interfaces. >>> >> Not sure if you've noticed but this patch seems to do the opposite of >> what Gustavo was aiming with an earlier series [1]. > > I removed just the interfaces related to sleep pm not runtime pm. From > long ago, Linux DRM drivers - especially ARM DRM drivers - have been > stepping forward to a integrated DRM driver so it'd be reasonable for > sleep pm operations of all KMS drivers are controlled by top of DRM > driver. It means that Exynos drm driver has already sleep pm interfaces. > > However, I think a power domain of each kms device couldn't be > controlled by the top in case of ARM SoC because kms devices can use a > different power domain each other according to Vendor SoC so runtime pm > should be controlled by each kms driver, which will be triggered by DRM top. > > And Gustavo's patch set you mentioned - now being reviewed - will add > only runtime pm interfaces to each kms driver. > Ack. Seems like I misread your patch. Thank you for the comprehensive answer and apologies for the noise. Thanks Emil _______________________________________________ dri-devel mailing list dri-devel@xxxxxxxxxxxxxxxxxxxxx http://lists.freedesktop.org/mailman/listinfo/dri-devel
