[PATCH v2] soc: mediatek: Introduce mediatek-regulator-coupler driver

Dmitry Osipenko dmitry.osipenko at collabora.com
Tue Jun 28 04:49:06 PDT 2022


Hi Angelo,

On 6/28/22 14:01, AngeloGioacchino Del Regno wrote:
> +		mrc->vsram_rdev = rdev;
> +	} else if (!strstr(rdev_name, "vgpu") && !strstr(rdev_name, "Vgpu"))
> +		return 1;

Nit: checkpatch --strict should warn about the missing {} here. In
general braces are always needed for each condition if they were needed
once in the if-else.

Reviewed-by: Dmitry Osipenko <dmitry.osipenko at collabora.com>

-- 
Best regards,
Dmitry



More information about the Linux-mediatek mailing list