[PATCH 04/13] soc: samsung: Add Exynos USI driver

[Sam Protsenko]

On Wed, Dec 27, 2023 at 11:49 AM Simon Glass <sjg at chromium.org> wrote:
>
> Hi Sam,
>

[snip]

>
> Just a few nits here
>
> Reviewed-by: Simon Glass <sjg at chromium.org>
>

[snip]

> > +
> > +struct exynos_usi {
> > +       struct udevice *dev;
>
> Can we drop this? It doesn't seem very useful and we try to avoid
> having bidirectional pointers. since it is possible to get the 'priv'
> pointer from the device.
>

Sure. I tried to keep the driver as close as possible to Linux
kernel's version, where I borrowed it from. But if it's the current
preference in U-Boot, I'll fix this in v2.

[snip]

> > +static int exynos_usi_parse_dt(struct exynos_usi *usi)
>
> Use of_to_plat() method?
>

Will do in v2. Thanks for the review!

[snip]