Liu Dave wrote: >> +#if defined(CONFIG_MPC834X) || \ >> + defined(CONFIG_MPC8313) || \ >> + defined(CONFIG_MPC8315) || \ >> + defined(CONFIG_MPC837X) >> + >> +typedef struct spi8xxx >> +{ >> + u8 res0[0x20]; /* 0x0-0x01f reserved */ >> + u32 mode; /* mode register */ >> + u32 event; /* event register */ >> + u32 mask; /* mask register */ >> + u32 com; /* command register */ >> + u32 tx; /* transmit register */ >> + u32 rx; /* receive register */ >> + u8 res1[0xC8]; /* fill up to 0x100 */ >> +} spi8xxx_t; >> > > Ben, the sizeof(struct spi8xxx_t) should be 0x1000, > So, This will break 83xx immap, the DMA and PCI will be failed. > I will send one patch to fix it. > > Oh jeez. Sorry. I guess it should be res1[0xfc8]. Thanks for taking care of this Dave.
regards, Ben ------------------------------------------------------------------------- This SF.net email is sponsored by: Microsoft Defy all challenges. Microsoft(R) Visual Studio 2008. http://clk.atdmt.com/MRT/go/vse0120000070mrt/direct/01/ _______________________________________________ U-Boot-Users mailing list U-Boot-Users@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/u-boot-users