From mboxrd@z Thu Jan 1 00:00:00 1970 From: Tom Date: Tue, 29 Sep 2009 07:24:26 -0500 Subject: [U-Boot] [PATCH] TI: DaVinci DM355: Fix Compilation warning In-Reply-To: <1254187262-7269-1-git-send-email-s-paulraj@ti.com> References: <1254187262-7269-1-git-send-email-s-paulraj@ti.com> Message-ID: <4AC1FC7A.1000700@windriver.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de s-paulraj at ti.com wrote: > From: Sandeep Paulraj > > This patch fixes a compilation warning while building DM355 > > Signed-off-by: Sandeep Paulraj > --- > board/davinci/dm355evm/dm355evm.c | 8 ++++---- > 1 files changed, 4 insertions(+), 4 deletions(-) > > diff --git a/board/davinci/dm355evm/dm355evm.c b/board/davinci/dm355evm/dm355evm.c > index 0a44748..5dd3b0d 100644 > --- a/board/davinci/dm355evm/dm355evm.c > +++ b/board/davinci/dm355evm/dm355evm.c > @@ -96,11 +96,11 @@ static void nand_dm355evm_select_chip(struct mtd_info *mtd, int chip) > u32 rbase = (u32) this->IO_ADDR_R; > > if (chip == 1) { > - __set_bit(14, &wbase); > - __set_bit(14, &rbase); > + __set_bit(14, (void *)&wbase); > + __set_bit(14, (void *)&rbase); > } else { > - __clear_bit(14, &wbase); > - __clear_bit(14, &rbase); > + __clear_bit(14, (void *)&wbase); > + __clear_bit(14, (void *)&rbase); > } > this->IO_ADDR_W = (void *)wbase; > this->IO_ADDR_R = (void *)rbase; It looks like __clear_bit is #defined to generic_clear_bit which has this declaration. generic_clear_bit(int nr, volatile unsigned long addr) So maybe changing u32 to volatile unsigned long will work? Also it looks like dm365evm has the same logic for chip select. At the least, it should also be fixed. Since they look like copies, it would be better if there was a common function. Tom