Re: [PATCH v1] drm/panthor: skip regulator setup if no such prop

[Rain Yang]

On Fri, Sep 26, 2025 at 12:03:11PM +0200, Boris Brezillon wrote:
>On Fri, 26 Sep 2025 17:07:22 +0800
>Rain Yang <jiyu.yang@xxxxxxxxxxx> wrote:
>
>> From: Rain Yang <jiyu.yang@xxxxxxx>
>> 
>> 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 <jiyu.yang@xxxxxxx>
>> ---
>>  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;
>>  	}
>>  
>