[PATCH] drm/bridge: thc63lvd1024: Fix regulator_get_optional() misuse
Laurent Pinchart
laurent.pinchart at ideasonboard.com
Tue Dec 15 20:55:45 UTC 2020
Hi Mark,
Thank you for the patch.
On Fri, Nov 08, 2019 at 05:32:08PM +0000, Mark Brown wrote:
> The thc63lvd1024 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 handling at all 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.
>
> Signed-off-by: Mark Brown <broonie at kernel.org>
Reviewed-by: Laurent Pinchart <laurent.pinchart at ideasonboard.com>
and queued in my tree for v5.12.
> ---
> drivers/gpu/drm/bridge/thc63lvd1024.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/gpu/drm/bridge/thc63lvd1024.c b/drivers/gpu/drm/bridge/thc63lvd1024.c
> index 3d74129b2995..ffca28ccc2c4 100644
> --- a/drivers/gpu/drm/bridge/thc63lvd1024.c
> +++ b/drivers/gpu/drm/bridge/thc63lvd1024.c
> @@ -200,7 +200,7 @@ static int thc63_probe(struct platform_device *pdev)
> thc63->dev = &pdev->dev;
> platform_set_drvdata(pdev, thc63);
>
> - thc63->vcc = devm_regulator_get_optional(thc63->dev, "vcc");
> + thc63->vcc = devm_regulator_get(thc63->dev, "vcc");
> if (IS_ERR(thc63->vcc)) {
> if (PTR_ERR(thc63->vcc) == -EPROBE_DEFER)
> return -EPROBE_DEFER;
--
Regards,
Laurent Pinchart
More information about the dri-devel
mailing list