[U-Boot] [PATCH v2 1/2] ppc4xx: Add 440EPx SPI driver
Ben Warren
biggerbadderben at gmail.com
Tue Dec 9 07:38:08 CET 2008
Wolfgang Denk wrote:
> Dear Stefan Roese,
>
> In message <200812090638.36452.sr at denx.de> you wrote:
>
>>> Regarding Ben's comments:
>>>
>>> First, thanks for reviewing. You gave me two different approaches to
>>> consider, 1) using in/out functions, and 2) using a structure. So, for
>>> this version, I've followed the style of the example you gave, using the
>>> structure approach.
>>>
>> No, you _need_ to use the in_/out_() accessor functions. This is unrelated to
>> using plain addresses and offsets vs. a structure. I really welcome the
>> change to the structure in this version, so please just add the in_()/out_()
>> functions upon accessing the SoC registers.
>>
>
> Hm... Seems this is a reply to an off-list message?
>
> Steven, please make sure to keep the ML on Cc: !
>
> Best regards,
>
> Wolfgang Denk
>
>
Looks on-list to me:
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/51194
regards,
Ben
More information about the U-Boot
mailing list