Re: [PATCH] i2c: designware: Add base addr info

From: Andy Shevchenko
Date: Sat Mar 27 2021 - 15:04:44 EST


On Sat, Mar 27, 2021 at 8:18 PM Daniel Gomez <daniel@xxxxxxxx> wrote:
> On Fri, 26 Mar 2021 at 13:28, Andy Shevchenko
> <andriy.shevchenko@xxxxxxxxxxxxxxx> wrote:
> > On Fri, Mar 26, 2021 at 11:35:08AM +0100, Daniel Gomez wrote:

...

> > It's a good question. On one hand we know that resource_size_t is a simple
> > redefinition of phys_addr_t, but it might be changed in the future. OTOH,
> > struct resource has types of resource_size_t. In any case it's a type that is
> > platform dependent (like long, size_t). Hence, the special specifier is needed.
>
> This 'issue' occurs in other subsystems like iio but I can see the
> patches are quite old in comparison with the i2c-tegra one.
> Also, the same happens when they print the variable (wrong specifier).

Now since you have better understanding you may clean those up!

--
With Best Regards,
Andy Shevchenko