On Wed, Oct 18, 2023 at 8:28 AM Qi Feng <qi.f...@arm.com> wrote: > From: Wei Chen <wei.c...@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.c...@arm.com> > Signed-off-by: Qi Feng <qi.f...@arm.com>
Looks like a reasonable solution to me! Reviewed-by: Linus Walleij <linus.wall...@linaro.org> Yours, Linus Walleij