On Sat, Apr 13, 2024 at 10:08:40AM +0200, Krzysztof Kozlowski wrote:
> On 09/04/2024 11:46, Diogo Ivo wrote:
> > Further streamline this function by moving the delay post-processing
> > to the callers, leaving it only with the task of returing the measured
> > delay values.
> >
> > Signed-off-by: Diogo Ivo <diogo.ivo@xxxxxxxxxxxxxxxxxx>
> > ---
> > drivers/memory/tegra/tegra210-emc-cc-r21021.c | 120 +++++++-----------
> > 1 file changed, 46 insertions(+), 74 deletions(-)
> >
> > diff --git a/drivers/memory/tegra/tegra210-emc-cc-r21021.c b/drivers/memory/tegra/tegra210-emc-cc-r21021.c
> > index ec2f84758d55..5e2c84fc835c 100644
> > --- a/drivers/memory/tegra/tegra210-emc-cc-r21021.c
> > +++ b/drivers/memory/tegra/tegra210-emc-cc-r21021.c
> > @@ -105,7 +105,7 @@ enum {
> > next->ptfv_list[w])) / \
> > (next->ptfv_list[w] + 1); \
> > \
> > - emc_dbg(emc, EMA_UPDATES, "%s: (s=%lu) EMA: %u\n", \
> > + emc_dbg(emc, EMA_UPDATES, "%s: (s=%u) EMA: %u\n", \
>
> Does not look related.
This was necessary to avoid compiler warnings as before when we were calling
this macro from update_clock_tree_delay() the value we were passing in was
declared as an unsigned long and now it is declared as u32, which it still
big enough for the values we are dealing with here.
Best regards,
Diogo
