Re: [U-Boot] [PATCH v2 1/1] TI: Davinci: NAND Driver Cleanup

2010-03-17 Thread Chemparathy, Cyril
Hi Nick, > I didn't check, but I would assume checkpatch would complain about the > spaces that have crept in here? Interestingly checkpatch complains only if it finds 8 or more spaces at the beginning of the line (below). For some reason, vim cindent inserted 7 spaces in there, and that kept

Re: [U-Boot] [PATCH v2 1/1] TI: Davinci: NAND Driver Cleanup

2010-03-17 Thread Nick Thompson
On 16/03/10 18:51, Cyril Chemparathy wrote: > #define davinci_emif_regs \ > - ((struct davinci_emif_regs *)DAVINCI_ASYNC_EMIF_CNTRL_BASE) > + ((struct davinci_emif_regs *)DAVINCI_ASYNC_EMIF_CNTRL_BASE) > + I didn't check, but I would assume checkpatch would complain about the spaces tha

[U-Boot] [PATCH v2 1/1] TI: Davinci: NAND Driver Cleanup

2010-03-16 Thread Cyril Chemparathy
Modified to use IO accessor routines consistently. Eliminated volatile usage to keep checkpatch.pl happy. Signed-off-by: Cyril Chemparathy --- 1. Added fixes for DA830 EVM and Davinci Schmoogie boards 2. Reverted to include davinci_emif_regs macro definition board/davinci/da830evm/da830evm.c