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
Attachment:
signature.asc
Description: PGP signature