Re: [PATCH] drm/panfrost: Fix regulator_get_optional() misuse

From: Rob Herring
Date: Thu Sep 05 2019 - 04:21:39 EST


On Wed, Sep 4, 2019 at 1:30 PM Mark Brown <broonie@xxxxxxxxxx> wrote:
> The panfrost driver requests a supply using regulator_get_optional()
> but both the name of the supply and the usage pattern suggest that it is
> being used for the main power for the device and is not at all optional
> for the device for function, there is no meaningful handling for absent
> supplies. Such regulators should use the vanilla regulator_get()
> interface, it will ensure that even if a supply is not described in the
> system integration one will be provided in software.

I guess commits e21dd290881b ("drm/panfrost: Enable devfreq to work
without regulator") and c90f30812a79 ("drm/panfrost: Add missing check
for pfdev->regulator")
in -next should be reverted or partially reverted?