Re: [U-Boot] [PATCH 1/1] dm: scsi: Scan the actual number of ports

2019-08-26 Thread Tom Rini
On Tue, Aug 20, 2019 at 04:47:42PM +, Park, Aiden wrote:

> The scsi_scan_dev() is looping over the number of uc_plat->max_id.
> The number of actual ports a AHCI controller has can be greater than
> max_id. Update uc_plat->max_id to make SCSI scan all detected ports.
> 
> Signed-off-by: Aiden Park 
> Reviewed-by: Bin Meng 

Applied to u-boot/master, thanks!

-- 
Tom


signature.asc
Description: PGP signature
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


Re: [U-Boot] [PATCH 1/1] dm: scsi: Scan the actual number of ports

2019-08-21 Thread Bin Meng
On Wed, Aug 21, 2019 at 12:47 AM Park, Aiden  wrote:
>
> The scsi_scan_dev() is looping over the number of uc_plat->max_id.
> The number of actual ports a AHCI controller has can be greater than
> max_id. Update uc_plat->max_id to make SCSI scan all detected ports.
>
> Signed-off-by: Aiden Park 
> ---
>  drivers/ata/ahci.c | 8 
>  1 file changed, 8 insertions(+)
>

Reviewed-by: Bin Meng 
___
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot


[U-Boot] [PATCH 1/1] dm: scsi: Scan the actual number of ports

2019-08-20 Thread Park, Aiden
The scsi_scan_dev() is looping over the number of uc_plat->max_id.
The number of actual ports a AHCI controller has can be greater than
max_id. Update uc_plat->max_id to make SCSI scan all detected ports.

Signed-off-by: Aiden Park 
---
 drivers/ata/ahci.c | 8 
 1 file changed, 8 insertions(+)

diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c
index e3135bb75f..9a08575053 100644
--- a/drivers/ata/ahci.c
+++ b/drivers/ata/ahci.c
@@ -1167,6 +1167,14 @@ int ahci_probe_scsi(struct udevice *ahci_dev, ulong base)
if (ret)
return ret;
 
+   /*
+* scsi_scan_dev() scans devices up-to the number of max_id.
+* Update max_id if the number of detected ports exceeds max_id.
+* This allows SCSI to scan all detected ports.
+*/
+   uc_plat->max_id = max_t(unsigned long, uc_priv->n_ports,
+   uc_plat->max_id);
+
return 0;
 }
 
-- 
2.20.1

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