Patchwork [Natty,1/1] pata_marvell: Add support for 88SE91A0, 88SE91A4

login
register
mail settings
Submitter Leann Ogasawara
Date Aug. 1, 2011, 3:44 p.m.
Message ID <1312213477.20595.13.camel@adamo>
Download mbox | patch
Permalink /patch/107775/
State New
Headers show

Comments

Leann Ogasawara - Aug. 1, 2011, 3:44 p.m.
http://bugs.launchpad.net/bugs/777325

SRU Justification:

Impact: Enable support for Marvell IDE PATA controllers found on Asus
P8P67LE motherboard.

Fix: Upstream commit f920fe1cb74191a780d88937f36994231a8faba1

Patch was cleanly cherry-picked and has been successfully tested by the
bug reporter.  I'm submitting this for Natty SRU since there are no
further 2.6.38.y stable updates coming form upstream.

Thanks,
Leann

From 3f41ebebd86e05528b1e56ddd93eee220eba5218 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Pawe=C5=82=20Drewniak?= <czajernia@gmail.com>
Date: Fri, 24 Jun 2011 02:07:35 -0400
Subject: [PATCH] pata_marvell: Add support for 88SE91A0, 88SE91A4
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

BugLink: http://bugs.launchpad.net/bugs/777325

This patch enables support for Marvell IDE PATA controllers found on
Asus P8P67LE motherboard.

The formatting has been corrected and I also received a report from two
users of this motherboard that the patch works.

Signed-off-by: Paweł Drewniak <czajernia@gmail.com>
Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
(cherry picked from commit f920fe1cb74191a780d88937f36994231a8faba1)

Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
---
 drivers/ata/pata_marvell.c |    3 +++
 1 files changed, 3 insertions(+), 0 deletions(-)
Seth Forshee - Aug. 1, 2011, 4:14 p.m.
On Mon, Aug 01, 2011 at 08:44:36AM -0700, Leann Ogasawara wrote:
> http://bugs.launchpad.net/bugs/777325
> 
> SRU Justification:
> 
> Impact: Enable support for Marvell IDE PATA controllers found on Asus
> P8P67LE motherboard.
> 
> Fix: Upstream commit f920fe1cb74191a780d88937f36994231a8faba1
> 
> Patch was cleanly cherry-picked and has been successfully tested by the
> bug reporter.  I'm submitting this for Natty SRU since there are no
> further 2.6.38.y stable updates coming form upstream.
> 
> Thanks,
> Leann
> 
> From 3f41ebebd86e05528b1e56ddd93eee220eba5218 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Pawe=C5=82=20Drewniak?= <czajernia@gmail.com>
> Date: Fri, 24 Jun 2011 02:07:35 -0400
> Subject: [PATCH] pata_marvell: Add support for 88SE91A0, 88SE91A4
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> 
> BugLink: http://bugs.launchpad.net/bugs/777325
> 
> This patch enables support for Marvell IDE PATA controllers found on
> Asus P8P67LE motherboard.
> 
> The formatting has been corrected and I also received a report from two
> users of this motherboard that the patch works.
> 
> Signed-off-by: Paweł Drewniak <czajernia@gmail.com>
> Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
> (cherry picked from commit f920fe1cb74191a780d88937f36994231a8faba1)
> 
> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>

Low risk of regressions and positive test results. Looks good.

Acked-by: Seth Forshee <seth.forshee@canonical.com>
Andy Whitcroft - Aug. 1, 2011, 5:57 p.m.
On Mon, Aug 01, 2011 at 08:44:36AM -0700, Leann Ogasawara wrote:
> http://bugs.launchpad.net/bugs/777325
> 
> SRU Justification:
> 
> Impact: Enable support for Marvell IDE PATA controllers found on Asus
> P8P67LE motherboard.
> 
> Fix: Upstream commit f920fe1cb74191a780d88937f36994231a8faba1
> 
> Patch was cleanly cherry-picked and has been successfully tested by the
> bug reporter.  I'm submitting this for Natty SRU since there are no
> further 2.6.38.y stable updates coming form upstream.
> 
> Thanks,
> Leann
> 
> From 3f41ebebd86e05528b1e56ddd93eee220eba5218 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Pawe=C5=82=20Drewniak?= <czajernia@gmail.com>
> Date: Fri, 24 Jun 2011 02:07:35 -0400
> Subject: [PATCH] pata_marvell: Add support for 88SE91A0, 88SE91A4
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> 
> BugLink: http://bugs.launchpad.net/bugs/777325
> 
> This patch enables support for Marvell IDE PATA controllers found on
> Asus P8P67LE motherboard.
> 
> The formatting has been corrected and I also received a report from two
> users of this motherboard that the patch works.
> 
> Signed-off-by: Paweł Drewniak <czajernia@gmail.com>
> Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
> (cherry picked from commit f920fe1cb74191a780d88937f36994231a8faba1)
> 
> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
> ---
>  drivers/ata/pata_marvell.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ata/pata_marvell.c b/drivers/ata/pata_marvell.c
> index dd38083..9f4648f 100644
> --- a/drivers/ata/pata_marvell.c
> +++ b/drivers/ata/pata_marvell.c
> @@ -161,6 +161,9 @@ static const struct pci_device_id marvell_pci_tbl[] = {
>  	{ PCI_DEVICE(0x11AB, 0x6121), },
>  	{ PCI_DEVICE(0x11AB, 0x6123), },
>  	{ PCI_DEVICE(0x11AB, 0x6145), },
> +	{ PCI_DEVICE(0x1B4B, 0x91A0), },
> +	{ PCI_DEVICE(0x1B4B, 0x91A4), },
> +
>  	{ }	/* terminate list */
>  };

Looks simple enough to me, low chance of regression.

Acked-by: Andy Whitcroft <apw@canonical.com>

-apw
Leann Ogasawara - Aug. 1, 2011, 6:17 p.m.
Applied to Natty master-next.

Thanks,
Leann

On Mon, 2011-08-01 at 08:44 -0700, Leann Ogasawara wrote:
> http://bugs.launchpad.net/bugs/777325
> 
> SRU Justification:
> 
> Impact: Enable support for Marvell IDE PATA controllers found on Asus
> P8P67LE motherboard.
> 
> Fix: Upstream commit f920fe1cb74191a780d88937f36994231a8faba1
> 
> Patch was cleanly cherry-picked and has been successfully tested by the
> bug reporter.  I'm submitting this for Natty SRU since there are no
> further 2.6.38.y stable updates coming form upstream.
> 
> Thanks,
> Leann
> 
> From 3f41ebebd86e05528b1e56ddd93eee220eba5218 Mon Sep 17 00:00:00 2001
> From: =?UTF-8?q?Pawe=C5=82=20Drewniak?= <czajernia@gmail.com>
> Date: Fri, 24 Jun 2011 02:07:35 -0400
> Subject: [PATCH] pata_marvell: Add support for 88SE91A0, 88SE91A4
> MIME-Version: 1.0
> Content-Type: text/plain; charset=UTF-8
> Content-Transfer-Encoding: 8bit
> 
> BugLink: http://bugs.launchpad.net/bugs/777325
> 
> This patch enables support for Marvell IDE PATA controllers found on
> Asus P8P67LE motherboard.
> 
> The formatting has been corrected and I also received a report from two
> users of this motherboard that the patch works.
> 
> Signed-off-by: Paweł Drewniak <czajernia@gmail.com>
> Signed-off-by: Jeff Garzik <jgarzik@redhat.com>
> (cherry picked from commit f920fe1cb74191a780d88937f36994231a8faba1)
> 
> Signed-off-by: Leann Ogasawara <leann.ogasawara@canonical.com>
> ---
>  drivers/ata/pata_marvell.c |    3 +++
>  1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/drivers/ata/pata_marvell.c b/drivers/ata/pata_marvell.c
> index dd38083..9f4648f 100644
> --- a/drivers/ata/pata_marvell.c
> +++ b/drivers/ata/pata_marvell.c
> @@ -161,6 +161,9 @@ static const struct pci_device_id marvell_pci_tbl[] = {
>  	{ PCI_DEVICE(0x11AB, 0x6121), },
>  	{ PCI_DEVICE(0x11AB, 0x6123), },
>  	{ PCI_DEVICE(0x11AB, 0x6145), },
> +	{ PCI_DEVICE(0x1B4B, 0x91A0), },
> +	{ PCI_DEVICE(0x1B4B, 0x91A4), },
> +
>  	{ }	/* terminate list */
>  };
>  
> -- 
> 1.7.4.1
> 
> 
> 
>

Patch

diff --git a/drivers/ata/pata_marvell.c b/drivers/ata/pata_marvell.c
index dd38083..9f4648f 100644
--- a/drivers/ata/pata_marvell.c
+++ b/drivers/ata/pata_marvell.c
@@ -161,6 +161,9 @@  static const struct pci_device_id marvell_pci_tbl[] = {
 	{ PCI_DEVICE(0x11AB, 0x6121), },
 	{ PCI_DEVICE(0x11AB, 0x6123), },
 	{ PCI_DEVICE(0x11AB, 0x6145), },
+	{ PCI_DEVICE(0x1B4B, 0x91A0), },
+	{ PCI_DEVICE(0x1B4B, 0x91A4), },
+
 	{ }	/* terminate list */
 };