Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Simon Glass
Hi, On Sun, Oct 9, 2011 at 6:03 PM, Mike Frysinger wrote: > On Sunday 09 October 2011 15:28:16 Wolfgang Denk wrote: >> Simon Glass wrote: >> > 0 errors, 44 warnings for >> > 0001-sandbox-Add-architecture-header-files.patch: warning: >> > arch/sandbox/include/asm/bitops.h,30: Use of volatile is us

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Mike Frysinger
On Sunday 09 October 2011 15:28:16 Wolfgang Denk wrote: > Simon Glass wrote: > > 0 errors, 44 warnings for > > 0001-sandbox-Add-architecture-header-files.patch: warning: > > arch/sandbox/include/asm/bitops.h,30: Use of volatile is usually wrong: > > see Documentation/volatile-considered-harmful.txt

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Simon Glass
Hi Wolfgang, On Sun, Oct 9, 2011 at 12:28 PM, Wolfgang Denk wrote: > Dear Simon Glass, > > In message <1318031631-13643-2-git-send-email-...@chromium.org> you wrote: >> This adds required header files for the sandbox architecture, and a basic >> description of what sandbox is (README.sandbox). >>

Re: [U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-09 Thread Wolfgang Denk
Dear Simon Glass, In message <1318031631-13643-2-git-send-email-...@chromium.org> you wrote: > This adds required header files for the sandbox architecture, and a basic > description of what sandbox is (README.sandbox). > > This commit generates a list of 44 checkpatch warnings: This should go t

[U-Boot] [PATCH v5 01/20] sandbox: Add architecture header files

2011-10-07 Thread Simon Glass
This adds required header files for the sandbox architecture, and a basic description of what sandbox is (README.sandbox). This commit generates a list of 44 checkpatch warnings: 0 errors, 44 warnings for 0001-sandbox-Add-architecture-header-files.patch: warning: arch/sandbox/include/asm/bitops.h