[PATCH 1/2] misc: vexpress_config: Use member .priv_auto to set the private data

Linus Walleij linus.walleij at linaro.org
Sat Oct 21 20:49:35 CEST 2023


On Wed, Oct 18, 2023 at 8:28 AM Qi Feng <qi.feng at arm.com> wrote:

> From: Wei Chen <wei.chen at arm.com>
>
> In current vexpress_config_probe code, it sets the uclass private data
> directly. This will cause one compilation error:
> drivers/misc/vexpress_config.c:114:27: error: lvalue required as left operand of assignment
>   114 |  dev_get_uclass_priv(dev) = priv;
>       |                           ^
>
> In this patch we set the uclass private data through struct member
> .priv_auto, and this compilation error disappears.
>
> Signed-off-by: Wei Chen <wei.chen at arm.com>
> Signed-off-by: Qi Feng <qi.feng at arm.com>

Looks like a reasonable solution to me!
Reviewed-by: Linus Walleij <linus.walleij at linaro.org>

Yours,
Linus Walleij


More information about the U-Boot mailing list