[U-Boot] [PATCH] SATA: do not auto-initialize during boot

[Kumar Gala]

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