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

[Tom Rini]

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...@gmail.com/
> 
> Resend the right patch.
> 
> [...]

Applied to u-boot/master, thanks!

-- 
Tom

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

[Stefan Roese]

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...@gmail.com/

Resend the right patch.

Signed-off-by: Tony Dinh 


Reviewed-by: Stefan Roese 

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

--
DENX Software Engineering GmbH,  Managing Director: Erika Unter
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-51 Fax: (+49)-8142-66989-80 Email: s...@denx.de

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

[Simon Glass]

On Fri, 6 Oct 2023 at 21: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...@gmail.com/
>
> Resend the right patch.
>
> Signed-off-by: Tony Dinh 
> ---
>
>  drivers/ata/sata.c | 6 ++
>  1 file changed, 6 insertions(+)

Reviewed-by: Simon Glass 

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

[Tony Dinh]

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...@gmail.com/

Resend the right patch.

Signed-off-by: Tony Dinh 
---

 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;
 }
 
-- 
2.39.2