Re: [PATCH 3/7] OPP: Rework _set_required_devs() to manage a single device per call

From: Viresh Kumar
Date: Thu Jul 11 2024 - 09:08:51 EST


On 11-07-24, 12:19, Ulf Hansson wrote:
> Unless I am mistaken, I don't think that is a scenario we should care
> about here.
>
> _opp_set_required_dev() is being called for a device that is about to
> be attached to its corresponding genpd.
>
> Yes, in some cases, we attach a genpd provider's device to its
> genpd-parent, but that is not to control the required-opps.

I understand and I am okay with what you are suggesting, we can fix it later on
if required anyway.

But just to give my reasoning behind that is that we want to avoid a very
specific case here and allow everything else. The special case being genpd
propagation, so I would normally not do a blanket ban but just that case.

But as I said, its okay :)

--
viresh