[U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Sanjeev Premi
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

 board/omap3/beagle/beagle.h|2 -
 board/omap3/evm/evm.h  |2 -
 board/omap3/overo/overo.h  |2 -
 board/omap3/pandora/pandora.h  |2 -
 board/omap3/zoom1/zoom1.h  |2 -
 cpu/arm_cortexa8/cpu.c |4 +-
 cpu/arm_cortexa8/omap3/board.c |   24 -
 cpu/arm_cortexa8/omap3/clock.c |5 +-
 cpu/arm_cortexa8/omap3/sys_info.c  |  145 +++-
 include/asm-arm/arch-omap3/omap3.h |   22 ++---
 include/asm-arm/arch-omap3/sys_proto.h |4 -
 include/configs/omap3_beagle.h |6 ++
 include/configs/omap3_evm.h|7 ++-
 include/configs/omap3_overo.h  |6 ++
 include/configs/omap3_pandora.h|6 ++
 include/configs/omap3_zoom1.h  |6 ++
 16 files changed, 131 insertions(+), 114 deletions(-)

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot


Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Dirk Behme
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

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


Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Premi, Sanjeev

 -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.

~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


Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Premi, Sanjeev
 -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.

~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


Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Dirk Behme
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


Re: [U-Boot] [PATCHv3 0/3] OMAP3: Board specific updates

2009-04-27 Thread Wolfgang Denk
Dear Dirk Behme,

In message 49f5d1b5.8070...@googlemail.com you wrote:

  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.

ACK, this is sufficient. Anything else would just be a waste of
bandwidth and storage space.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH, MD: Wolfgang Denk  Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: w...@denx.de
I think there's a world market for about five computers.
 -- attr. Thomas J. Watson (Chairman of the Board, IBM), 1943
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot