[U-Boot] [PATCH 2/5] board_r: run scsi init() on ARM too

Marek Vasut marex at denx.de
Fri Jul 4 17:44:21 CEST 2014


On Friday, July 04, 2014 at 05:25:10 PM, Ian Campbell wrote:
> On Fri, 2014-07-04 at 16:59 +0200, Marek Vasut wrote:
> > On Saturday, May 31, 2014 at 06:36:12 PM, Ian Campbell wrote:
> > > This has been disabled for ARM in initr_scsi since that function was
> > > introduced. However it works fine for me on Cubieboard and Cubietruck
> > > (with the upcoming AHCI glue patch).
> > 
> > Does this break bisectability ?
> 
> Not if this patch goes in first.
> 
> > > I also tested on two random ARM platforms which seem to define
> > > CONFIG_CMD_SCSI: - highbank worked fine (on midway hardware)
> > > 
> > >  - omap5_uevm built OK and I confirmed using objdump that things were
> > >  as
> > >  
> > >    expected (i.e. the default weak scsi_init nop was used).
> > > 
> > > While there remove the mismatched comment from the #endif (omitting the
> > > comment seems to be the prevailing style in this file) and add a
> > > missing return to initr_doc which I happened to spot while editing.
> > > 
> > > Signed-off-by: Ian Campbell <ijc at hellion.org.uk>
> > > Cc: Simon Glass <sjg at chromium.org>
> > 
> > Make sure to check no ARM boards call scsi_init() from their board files.
> 
> I grepped around and couldn't find any.

OK, thanks!

Best regards,
Marek Vasut


More information about the U-Boot mailing list