linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] ata_piix: update map 10b for ich8m
@ 2007-08-06 17:43 Tejun Heo
  2007-08-06 17:51 ` Kristen Carlson Accardi
                   ` (2 more replies)
  0 siblings, 3 replies; 6+ messages in thread
From: Tejun Heo @ 2007-08-06 17:43 UTC (permalink / raw)
  To: Jeff Garzik, linux-ide, kristen.c.accardi, stable

Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.

Signed-off-by: Tejun Heo <htejun@gmail.com>
Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com
---
Many users can be hit by this.  Please consider for -stable.  Thanks.

 drivers/ata/ata_piix.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
index a78832e..c5b4509 100644
--- a/drivers/ata/ata_piix.c
+++ b/drivers/ata/ata_piix.c
@@ -436,7 +436,7 @@ static const struct piix_map_db ich8_map_db = {
 		/* PM   PS   SM   SS       MAP */
 		{  P0,  P2,  P1,  P3 }, /* 00b (hardwired when in AHCI) */
 		{  RV,  RV,  RV,  RV },
-		{  IDE,  IDE,  NA,  NA }, /* 10b (IDE mode) */
+		{  P0,  P2, IDE, IDE }, /* 10b (IDE mode) */
 		{  RV,  RV,  RV,  RV },
 	},
 };

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH] ata_piix: update map 10b for ich8m
  2007-08-06 17:43 [PATCH] ata_piix: update map 10b for ich8m Tejun Heo
@ 2007-08-06 17:51 ` Kristen Carlson Accardi
  2007-08-07  7:53 ` [stable] " Greg KH
  2007-08-08  0:57 ` Jeff Garzik
  2 siblings, 0 replies; 6+ messages in thread
From: Kristen Carlson Accardi @ 2007-08-06 17:51 UTC (permalink / raw)
  To: Tejun Heo; +Cc: Jeff Garzik, linux-ide, stable

On Tue, 7 Aug 2007 02:43:27 +0900
Tejun Heo <htejun@gmail.com> wrote:

> Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.
> 
> Signed-off-by: Tejun Heo <htejun@gmail.com>
> Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com

Acked-by: Kristen Carlson Accardi <kristen.c.accardi@intel.com>

> ---
> Many users can be hit by this.  Please consider for -stable.  Thanks.
> 
>  drivers/ata/ata_piix.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index a78832e..c5b4509 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -436,7 +436,7 @@ static const struct piix_map_db ich8_map_db = {
>  		/* PM   PS   SM   SS       MAP */
>  		{  P0,  P2,  P1,  P3 }, /* 00b (hardwired when in AHCI) */
>  		{  RV,  RV,  RV,  RV },
> -		{  IDE,  IDE,  NA,  NA }, /* 10b (IDE mode) */
> +		{  P0,  P2, IDE, IDE }, /* 10b (IDE mode) */
>  		{  RV,  RV,  RV,  RV },
>  	},
>  };
> 

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [stable] [PATCH] ata_piix: update map 10b for ich8m
  2007-08-06 17:43 [PATCH] ata_piix: update map 10b for ich8m Tejun Heo
  2007-08-06 17:51 ` Kristen Carlson Accardi
@ 2007-08-07  7:53 ` Greg KH
  2007-08-07  8:09   ` Tejun Heo
  2007-08-08  0:57 ` Jeff Garzik
  2 siblings, 1 reply; 6+ messages in thread
From: Greg KH @ 2007-08-07  7:53 UTC (permalink / raw)
  To: Tejun Heo; +Cc: Jeff Garzik, linux-ide, kristen.c.accardi, stable

On Tue, Aug 07, 2007 at 02:43:27AM +0900, Tejun Heo wrote:
> Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.
> 
> Signed-off-by: Tejun Heo <htejun@gmail.com>
> Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com
> ---
> Many users can be hit by this.  Please consider for -stable.  Thanks.
> 
>  drivers/ata/ata_piix.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index a78832e..c5b4509 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -436,7 +436,7 @@ static const struct piix_map_db ich8_map_db = {
>  		/* PM   PS   SM   SS       MAP */
>  		{  P0,  P2,  P1,  P3 }, /* 00b (hardwired when in AHCI) */
>  		{  RV,  RV,  RV,  RV },
> -		{  IDE,  IDE,  NA,  NA }, /* 10b (IDE mode) */
> +		{  P0,  P2, IDE, IDE }, /* 10b (IDE mode) */
>  		{  RV,  RV,  RV,  RV },
>  	},
>  };

Did this ever make it into Linus's tree?

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [stable] [PATCH] ata_piix: update map 10b for ich8m
  2007-08-07  8:09   ` Tejun Heo
@ 2007-08-07  8:07     ` Greg KH
  0 siblings, 0 replies; 6+ messages in thread
From: Greg KH @ 2007-08-07  8:07 UTC (permalink / raw)
  To: Tejun Heo; +Cc: Jeff Garzik, linux-ide, kristen.c.accardi, stable

On Tue, Aug 07, 2007 at 05:09:08PM +0900, Tejun Heo wrote:
> Greg KH wrote:
> > On Tue, Aug 07, 2007 at 02:43:27AM +0900, Tejun Heo wrote:
> >> Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.
> >>
> >> Signed-off-by: Tejun Heo <htejun@gmail.com>
> >> Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com
> >> ---
> >> Many users can be hit by this.  Please consider for -stable.  Thanks.
> >>
> >>  drivers/ata/ata_piix.c |    2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> >> index a78832e..c5b4509 100644
> >> --- a/drivers/ata/ata_piix.c
> >> +++ b/drivers/ata/ata_piix.c
> >> @@ -436,7 +436,7 @@ static const struct piix_map_db ich8_map_db = {
> >>  		/* PM   PS   SM   SS       MAP */
> >>  		{  P0,  P2,  P1,  P3 }, /* 00b (hardwired when in AHCI) */
> >>  		{  RV,  RV,  RV,  RV },
> >> -		{  IDE,  IDE,  NA,  NA }, /* 10b (IDE mode) */
> >> +		{  P0,  P2, IDE, IDE }, /* 10b (IDE mode) */
> >>  		{  RV,  RV,  RV,  RV },
> >>  	},
> >>  };
> > 
> > Did this ever make it into Linus's tree?
> 
> Not yet.  Hmmm... Should I have mailed -stable separately after Jeff's ACK?

Yeah, that would be best, as we need to wait until the patches go into
Linus's tree before adding them to ours to make sure we don't get out of
sync.

thanks,

greg k-h

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [stable] [PATCH] ata_piix: update map 10b for ich8m
  2007-08-07  7:53 ` [stable] " Greg KH
@ 2007-08-07  8:09   ` Tejun Heo
  2007-08-07  8:07     ` Greg KH
  0 siblings, 1 reply; 6+ messages in thread
From: Tejun Heo @ 2007-08-07  8:09 UTC (permalink / raw)
  To: Greg KH; +Cc: Jeff Garzik, linux-ide, kristen.c.accardi, stable

Greg KH wrote:
> On Tue, Aug 07, 2007 at 02:43:27AM +0900, Tejun Heo wrote:
>> Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.
>>
>> Signed-off-by: Tejun Heo <htejun@gmail.com>
>> Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com
>> ---
>> Many users can be hit by this.  Please consider for -stable.  Thanks.
>>
>>  drivers/ata/ata_piix.c |    2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
>> index a78832e..c5b4509 100644
>> --- a/drivers/ata/ata_piix.c
>> +++ b/drivers/ata/ata_piix.c
>> @@ -436,7 +436,7 @@ static const struct piix_map_db ich8_map_db = {
>>  		/* PM   PS   SM   SS       MAP */
>>  		{  P0,  P2,  P1,  P3 }, /* 00b (hardwired when in AHCI) */
>>  		{  RV,  RV,  RV,  RV },
>> -		{  IDE,  IDE,  NA,  NA }, /* 10b (IDE mode) */
>> +		{  P0,  P2, IDE, IDE }, /* 10b (IDE mode) */
>>  		{  RV,  RV,  RV,  RV },
>>  	},
>>  };
> 
> Did this ever make it into Linus's tree?

Not yet.  Hmmm... Should I have mailed -stable separately after Jeff's ACK?

Thanks.

-- 
tejun

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH] ata_piix: update map 10b for ich8m
  2007-08-06 17:43 [PATCH] ata_piix: update map 10b for ich8m Tejun Heo
  2007-08-06 17:51 ` Kristen Carlson Accardi
  2007-08-07  7:53 ` [stable] " Greg KH
@ 2007-08-08  0:57 ` Jeff Garzik
  2 siblings, 0 replies; 6+ messages in thread
From: Jeff Garzik @ 2007-08-08  0:57 UTC (permalink / raw)
  To: Tejun Heo; +Cc: linux-ide, kristen.c.accardi, stable

Tejun Heo wrote:
> Fix map entry 10b for ich8.  It's [P0 P2 IDE IDE] like ich6 / ich6m.
> 
> Signed-off-by: Tejun Heo <htejun@gmail.com>
> Cc: <Kristen Carlson Accardi> kristen.c.accardi@intel.com
> ---
> Many users can be hit by this.  Please consider for -stable.  Thanks.
> 
>  drivers/ata/ata_piix.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

applied



^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2007-08-08  0:57 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-08-06 17:43 [PATCH] ata_piix: update map 10b for ich8m Tejun Heo
2007-08-06 17:51 ` Kristen Carlson Accardi
2007-08-07  7:53 ` [stable] " Greg KH
2007-08-07  8:09   ` Tejun Heo
2007-08-07  8:07     ` Greg KH
2007-08-08  0:57 ` Jeff Garzik

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).