Premi, Sanjeev wrote: >> -----Original Message----- >> From: u-boot-boun...@lists.denx.de >> [mailto:u-boot-boun...@lists.denx.de] On Behalf Of Premi, Sanjeev >> Sent: Monday, April 27, 2009 8:49 PM >> To: Dirk Behme >> Cc: u-boot@lists.denx.de >> Subject: Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates >> >> >>> -----Original Message----- >>> From: Dirk Behme [mailto:dirk.be...@googlemail.com] >>> Sent: Monday, April 27, 2009 8:47 PM >>> To: Premi, Sanjeev >>> Cc: u-boot@lists.denx.de >>> Subject: Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates >>> >>> Hi Premi, >>> >>> Sanjeev Premi wrote: >>>> This series contains 3 specific updates: >>>> - Use common API to print cpu and board >>>> related information. >>>> - Remove unused board type definitions. >>>> - Print correct silicon revision in the >>>> board information >>>> >>>> These updates have been tested on OMAP3EVM >>>> with ES 3.0 and ES3.1 silicon versions. >>>> >>>> Thanks to Dirk Behme [dirk.be...@googlemail.com] >>>> for converting a quick hack into complete >>>> solution. >>>> >>>> Sanjeev Premi (3): >>>> OMAP3: Use functions print_cpuinfo() and checkboard() >>>> OMAP3: Remove unused board-types >>>> OMAP3: Print correct silicon revision >>> Applying all 3 patches I get >>> >>> sys_info.c: In function 'print_cpuinfo': >>> sys_info.c:297: warning: no return statement in function returning >>> non-void >>> board.c: In function 'checkboard': >>> board.c:349: warning: implicit declaration of function 'is_mem_sdr' >>> board.c:357: warning: no return statement in function >>> returning non-void >>> >> I usually do a distclean followed by make; but let me do this again. > > Yeah. found it... a miss in interactive commit 3/3. > > Should I re-submit the whole series? > OR > Is it okay to re-send just the last one.
I'm not a custodian, but last one marked with 'v4' should be sufficient from my point of view. Best regards Dirk > ~sanjeev > >> ~sanjeev >> >>> Not sure which patch it is, though. >>> >>> Best regards >>> >>> Dirk >>> >>> Btw.: You can easily check this by doing a >>> >>> ./MAKEALL ARM_CORTEX_A8 >>> >>> >>> >> _______________________________________________ >> U-Boot mailing list >> U-Boot@lists.denx.de >> http://lists.denx.de/mailman/listinfo/u-boot >> >> _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot