On Dec 11, 2008, at 2:03 PM, Mike Frysinger wrote:
On Thursday 11 December 2008 11:17:56 Kumar Gala wrote:
On Dec 11, 2008, at 4:32 AM, Mike Frysinger wrote:
Rather than have the board code initialize SATA automatically during boot, make the user manually run "sata init". This brings the SATA subsystem in line with common U-Boot policy.
Along these lines, the "is_sata_supported()" hook is no longer needed, so scrub it from the tree.
Signed-off-by: Mike Frysinger vapier@gentoo.org
board/freescale/mpc8536ds/mpc8536ds.c | 11 ----------- common/cmd_sata.c | 7 +++++-- include/sata.h | 2 -- lib_ppc/board.c | 20 -------------------- 4 files changed, 5 insertions(+), 35 deletions(-)
Why should we make the user have to do 'sata init'? That seems annoying and easy to forget.
it's already standard behavior with most mass storage things (like mmc and sf and ...), so it isnt a big deal imo. i asked if Wolfgang wanted it automated and he preferred making the user do it themselves.
This seems backwards to me..
'sata init' isn't safe. It seems like you should only be able to call it once. However I think we can keep issuing it and cause bad things to happen.
Also, in the code you removed we do a runtime check on 8536 to see if SATA is even available. That check is still valid.
- k