[PATCH] drm/tidss: Power up attached PM domains on probe

Maxime Ripard mripard at kernel.org
Thu Oct 12 12:40:56 UTC 2023


Hi,

On Thu, Oct 12, 2023 at 05:10:06PM +0530, Devarsh Thakkar wrote:
> On 09/10/23 13:20, Devarsh Thakkar wrote:
> > Some SoC's such as AM62P have dedicated power domains
> > for OLDI which need to be powered on separetely along
> > with display controller.
> > 
> > So during driver probe, power up all attached PM domains
> > enumerated in devicetree node for DSS.
> > 
> > This also prepares base to add display support for AM62P.
> > 
> 
> NAK, for this patch, as discussed with team there are already plans
> to have separate OLDI bridge driver which should eventually handle
> the additional power domains.

Just for the record in case your current plan doesn't work out and we
need to revisit this patch: my point was that it's something that
deviates by a margin from what drivers are usually expected to do, so we
should document why that deviation is there.

The patch itself looks reasonable to me otherwise.

Maxime
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 228 bytes
Desc: not available
URL: <https://lists.freedesktop.org/archives/dri-devel/attachments/20231012/fb73e6a2/attachment-0001.sig>


More information about the dri-devel mailing list