On Fri, Sep 26, 2025 at 12:03:11PM +0200, Boris Brezillon wrote: >On Fri, 26 Sep 2025 17:07:22 +0800 >Rain Yang <[email protected]> wrote: > >> From: Rain Yang <[email protected]> >> >> The regulator is optional, skip the setup instead of returning an >> error if it is not present > >AFAICT, it's not flagged optional in the DT bindings yet, so I'd prefer >to have this change and the DT bindings update in the same patch series >(both will go through the drm-misc tree anway). >
thanks, Boris. I will create a patch in dt-binding to make mali-supply required only applied to to rk3588 >> >> Signed-off-by: Rain Yang <[email protected]> >> --- >> drivers/gpu/drm/panthor/panthor_devfreq.c | 3 +-- >> 1 file changed, 1 insertion(+), 2 deletions(-) >> >> diff --git a/drivers/gpu/drm/panthor/panthor_devfreq.c >> b/drivers/gpu/drm/panthor/panthor_devfreq.c >> index 3686515d368d..2df1d76d84a0 100644 >> --- a/drivers/gpu/drm/panthor/panthor_devfreq.c >> +++ b/drivers/gpu/drm/panthor/panthor_devfreq.c >> @@ -146,10 +146,9 @@ int panthor_devfreq_init(struct panthor_device *ptdev) >> ptdev->devfreq = pdevfreq; >> >> ret = devm_pm_opp_set_regulators(dev, reg_names); >> - if (ret) { >> + if (ret && ret != -ENODEV) { >> if (ret != -EPROBE_DEFER) >> DRM_DEV_ERROR(dev, "Couldn't set OPP regulators\n"); >> - >> return ret; >> } >> >
