[RESEND PATCH] bootstd: sata: bootdev scanning for ahci sata with no drive attached

It's normal to have no SATA drive attached to the controller, so return a successful status when there is no block device found after probing.
Note: this patch depends on the previous patch https://patchwork.ozlabs.org/project/uboot/patch/20230917230649.30357-1-mibo...
Resend the right patch.
Signed-off-by: Tony Dinh mibodhi@gmail.com ---
drivers/ata/sata.c | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/drivers/ata/sata.c b/drivers/ata/sata.c index f126b84e05..dcb5fcf476 100644 --- a/drivers/ata/sata.c +++ b/drivers/ata/sata.c @@ -79,6 +79,12 @@ int sata_rescan(bool verbose)
ret = uclass_probe_all(UCLASS_AHCI);
+ if (ret == -ENODEV) { + if (verbose) + printf("No SATA block device found\n"); + return 0; + } + return ret; }

On Fri, 6 Oct 2023 at 21:34, Tony Dinh mibodhi@gmail.com wrote:
It's normal to have no SATA drive attached to the controller, so return a successful status when there is no block device found after probing.
Note: this patch depends on the previous patch https://patchwork.ozlabs.org/project/uboot/patch/20230917230649.30357-1-mibo...
Resend the right patch.
Signed-off-by: Tony Dinh mibodhi@gmail.com
drivers/ata/sata.c | 6 ++++++ 1 file changed, 6 insertions(+)
Reviewed-by: Simon Glass sjg@chromium.org

On 10/7/23 05:34, Tony Dinh wrote:
It's normal to have no SATA drive attached to the controller, so return a successful status when there is no block device found after probing.
Note: this patch depends on the previous patch https://patchwork.ozlabs.org/project/uboot/patch/20230917230649.30357-1-mibo...
Resend the right patch.
Signed-off-by: Tony Dinh mibodhi@gmail.com
Reviewed-by: Stefan Roese sr@denx.de
Thanks, Stefan
drivers/ata/sata.c | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/drivers/ata/sata.c b/drivers/ata/sata.c index f126b84e05..dcb5fcf476 100644 --- a/drivers/ata/sata.c +++ b/drivers/ata/sata.c @@ -79,6 +79,12 @@ int sata_rescan(bool verbose)
ret = uclass_probe_all(UCLASS_AHCI);
- if (ret == -ENODEV) {
if (verbose)
printf("No SATA block device found\n");
return 0;
- }
- return ret; }
Viele Grüße, Stefan Roese

On Fri, 06 Oct 2023 20:34:28 -0700, Tony Dinh wrote:
It's normal to have no SATA drive attached to the controller, so return a successful status when there is no block device found after probing.
Note: this patch depends on the previous patch https://patchwork.ozlabs.org/project/uboot/patch/20230917230649.30357-1-mibo...
Resend the right patch.
[...]
Applied to u-boot/master, thanks!
participants (4)
-
Simon Glass
-
Stefan Roese
-
Tom Rini
-
Tony Dinh