[U-Boot] Checkpatch warnings for "volatile"

[Prabhakar Lad]

Hi Wolfgang

On 10/15/11, Wolfgang Denk <wd at denx.de> wrote:
> Dear Prabhakar Lad,
>
> In message
> <CA+V-a8sYRZJDZojEpQ55ZGRZ6--Niq0ThKVV8e_RtQrRuShE8A at mail.gmail.com> you
> wrote:
>>
>> > I've explained this a number of times recently - there are actually
>> > very, very few occasions where "volatile" actually makes sense.
>> >
>> >      Agreed, but I see a piece of code where virtual address are
>> > compared.
>>      For example in arch/arm/cpu/arm926ejs/davinci/cpu.c
>>      In this function static inline unsigned pll_prediv(unsigned pllbase)
>> and
>>      also in this static inline unsigned pll_postdiv(unsigned pllbase)
>>
>>      Any suggestion on this on how to tackle or let it remain stagnant?
>
> I cannot see a justification for any of the ""volatile" in this file.
>
> Of course, all these ugly REG() calls should be converted to proper
> use of I/O accessors.
>
  Thanks for the reply.

Regards
--Prabhakar Lad

> Best regards,
>
> Wolfgang Denk
>
> --
> DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
> HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
> Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
> Hindsight is an exact science.
>