[U-Boot] [PATCH 18/29] dm: ahci: Unwind the confusing init code
Bin Meng
bmeng.cn at gmail.com
Tue Jun 13 03:14:04 UTC 2017
On Tue, Jun 6, 2017 at 3:15 AM, Simon Glass <sjg at chromium.org> wrote:
> Two AHCI drivers use SCSI with CONFIG_DM_SCSI. The SCSI uclass calls
> scsi_low_level_init() which is implemented by ahci.c. If
> CONFIG_SCSI_AHCI_PLAT is defined it does one thing and if it is not
> it does something else.
>
> We don't need to call through scsi_low_level_init() to get the init
> completed. Instead, adjust the two drivers to call into AHCI directly.
> Drop the post-probe init in the SCSI uclass. This means that driver model
> doesn't need to use scsi_low_level_init(). It is a legacy function and
> driver model should use a driver's probe() method instead.
>
> While we are hear, add a comment to the top of the file explaining what
typo: we are 'here'
> ahci.c does.
>
> Signed-off-by: Simon Glass <sjg at chromium.org>
> ---
>
> drivers/ata/ahci.c | 26 ++++++++++++++++++++------
> drivers/ata/dwc_ahci.c | 6 +++++-
> drivers/ata/sata_ceva.c | 3 ++-
> drivers/scsi/scsi-uclass.c | 8 --------
> include/ahci.h | 16 ++++++++++++++++
> include/scsi.h | 4 +---
> 6 files changed, 44 insertions(+), 19 deletions(-)
>
Reviewed-by: Bin Meng <bmeng.cn at gmail.com>
More information about the U-Boot
mailing list