[SRU B/C] sky2: Increase D3 delay again

Stefan Bader stefan.bader at canonical.com
Mon Apr 8 11:05:51 UTC 2019


On 08.04.19 12:50, Stefan Bader wrote:
> From: Kai-Heng Feng <kai.heng.feng at canonical.com>
> 
> BugLink: https://bugs.launchpad.net/bugs/1798921

For applying to the kernels ^this line can be dropped. I failed to notice that
it is already present from the upstream commit (below).

-Stefan

> 
> Another platform requires even longer delay to make the device work
> correctly after S3.
> 
> So increase the delay to 300ms.
> 
> BugLink: https://bugs.launchpad.net/bugs/1798921
> 
> Signed-off-by: Kai-Heng Feng <kai.heng.feng at canonical.com>
> Signed-off-by: David S. Miller <davem at davemloft.net>
> 
> (cherry picked from commit 1765f5dcd00963e33f1b8a4e0f34061fbc0e2f7f)
> Signed-off-by: Stefan Bader <stefan.bader at canonical.com>
> ---
> 
> Note: Was picked up by Xenial via upstream stable, proposing to get
> into Bionic and Cosmic, too.
> 
> -Stefan
> 
>  drivers/net/ethernet/marvell/sky2.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/net/ethernet/marvell/sky2.c b/drivers/net/ethernet/marvell/sky2.c
> index f3a5fa84860f..57727fe1501e 100644
> --- a/drivers/net/ethernet/marvell/sky2.c
> +++ b/drivers/net/ethernet/marvell/sky2.c
> @@ -5073,7 +5073,7 @@ static int sky2_probe(struct pci_dev *pdev, const struct pci_device_id *ent)
>  	INIT_WORK(&hw->restart_work, sky2_restart);
>  
>  	pci_set_drvdata(pdev, hw);
> -	pdev->d3_delay = 200;
> +	pdev->d3_delay = 300;
>  
>  	return 0;
>  
> 


-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: OpenPGP digital signature
URL: <https://lists.ubuntu.com/archives/kernel-team/attachments/20190408/66fd5719/attachment.sig>


More information about the kernel-team mailing list