On Thu, Nov 15, 2018 at 8:28 PM Alexander Graf <ag...@suse.de> wrote:
> On 15.11.18 18:58, Andy Shevchenko wrote:
> > Cache the value of the reg-io-width property for the future use.

> >  struct ns16550_platdata {
> >       unsigned long base;
> > +     int reg_width;
> >       int reg_shift;
> > -     int clock;
> >       int reg_offset;
> > +     int clock;
>
> Why move clock?

To group reg_* parameters together as it might make sense in the
future to have some other struct which would be used inside
serial_device_info and here.
If it's a big deal, don't pay attention to it, I would not touch it by request.

P.S. Why to overquoute?

-- 
With Best Regards,
Andy Shevchenko
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot

Reply via email to