[PATCH] ata: libahci: do not complain in case of deferred probe

[PATCH] ata: libahci: do not complain in case of deferred probe

[Miquel Raynal]

Retrieving PHYs can deffer the probe, do not spawn an error when
-EPROBE_DEFER is returned, it is normal behavior.

Fixes: b1a9edbda040 ("ata: libahci: allow to use multiple PHYs")
Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>
---
 drivers/ata/libahci_platform.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c
index 72312ad2e142..c25cdbf817f1 100644
--- a/drivers/ata/libahci_platform.c
+++ b/drivers/ata/libahci_platform.c
@@ -338,6 +338,9 @@ static int ahci_platform_get_phy(struct ahci_host_priv *hpriv, u32 port,
 		hpriv->phys[port] = NULL;
 		rc = 0;
 		break;
+	case -EPROBE_DEFER:
+		/* Do not complain yet */
+		break;
 
 	default:
 		dev_err(dev,
-- 
2.20.1

Re: [PATCH] ata: libahci: do not complain in case of deferred probe

[Hans de Goede]

Hi,

On 31-07-19 14:26, Miquel Raynal wrote:
> Retrieving PHYs can deffer the probe, do not spawn an error when
> -EPROBE_DEFER is returned, it is normal behavior.
> 
> Fixes: b1a9edbda040 ("ata: libahci: allow to use multiple PHYs")
> Signed-off-by: Miquel Raynal <miquel.raynal@bootlin.com>

Patch looks good to me:

Reviewed-by: Hans de Goede <hdegoede@redhat.com>

Regards,

Hans



> ---
>   drivers/ata/libahci_platform.c | 3 +++
>   1 file changed, 3 insertions(+)
> 
> diff --git a/drivers/ata/libahci_platform.c b/drivers/ata/libahci_platform.c
> index 72312ad2e142..c25cdbf817f1 100644
> --- a/drivers/ata/libahci_platform.c
> +++ b/drivers/ata/libahci_platform.c
> @@ -338,6 +338,9 @@ static int ahci_platform_get_phy(struct ahci_host_priv *hpriv, u32 port,
>   		hpriv->phys[port] = NULL;
>   		rc = 0;
>   		break;
> +	case -EPROBE_DEFER:
> +		/* Do not complain yet */
> +		break;
>   
>   	default:
>   		dev_err(dev,
> 

Re: [PATCH] ata: libahci: do not complain in case of deferred probe

[Jens Axboe]

On 7/31/19 6:26 AM, Miquel Raynal wrote:
> Retrieving PHYs can deffer the probe, do not spawn an error when
> -EPROBE_DEFER is returned, it is normal behavior.

Applied, thanks.

-- 
Jens Axboe