On May 21, 2010, at 10:57 AM, Timur Tabi wrote:

> On Fri, May 21, 2010 at 4:18 AM, Kumar Gala <ga...@kernel.crashing.org> wrote:
> 
>> +#ifdef CONFIG_FSL_NGPIXIS
>> +#include "ngpixis.h"
>> +#define PIXIS_VSYSCLK0         offsetof(struct ngpixis, sclk[0])
>> +#define PIXIS_VSYSCLK1         offsetof(struct ngpixis, sclk[1])
>> +#define PIXIS_VSYSCLK2         offsetof(struct ngpixis, sclk[2])
>> +#define PIXIS_VDDRCLK0         offsetof(struct ngpixis, dclk[0])
>> +#define PIXIS_VDDRCLK1         offsetof(struct ngpixis, dclk[1])
>> +#define PIXIS_VDDRCLK2         offsetof(struct ngpixis, dclk[2])
>> +#endif
> 
> Yuck.  I specifically created the ngpixis_t structure because Wolfgang
> objected to the use of macros for offsets.
> 
> But I understand why you did it.  It's the easiest way to maintain
> compatibility between pixis and ngpixis.

Wolfgang, is this an issue?

The other option is have 3 different structs:

On p2020: #define PIXIS_VSYSCLK0         offsetof(struct ngpixis, sclk[0])
on  8572: #define PIXIS_VSYSCLK0        0x1C
on  8536: #define PIXIS_VSYSCLK0        0x1A

- k
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Reply via email to