* pata_ali patch not merged?
@ 2007-06-25 23:13 Chuck Ebbert
2007-07-03 7:47 ` Tejun Heo
2007-07-03 14:07 ` Jeff Garzik
0 siblings, 2 replies; 4+ messages in thread
From: Chuck Ebbert @ 2007-06-25 23:13 UTC (permalink / raw)
To: IDE/ATA development list; +Cc: Alan
This patch was found to fix some of the problems with the
pata_ali driver. Is it going to be merged in 2.6.22?
[https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=156482]
--- linux-2.6.22-rc4/drivers/ata/pata_ali.c 2007-06-04 23:05:17.000000000 -0700
+++ devel/drivers/ata/pata_ali.c 2007-06-06 20:49:03.000000000 -0700
@@ -544,34 +628,34 @@ static int ali_init_one(struct pci_dev *
/* Revision 0xC2 adds UDMA66 */
static const struct ata_port_info info_c2 = {
.sht = &ali_sht,
.flags = ATA_FLAG_SLAVE_POSS | ATA_FLAG_SRST | ATA_FLAG_PIO_LBA48,
.pio_mask = 0x1f,
.mwdma_mask = 0x07,
.udma_mask = 0x1f,
.port_ops = &ali_c2_port_ops
};
- /* Revision 0xC3 is UDMA100 */
+ /* Revision 0xC3 is UDMA66 for now */
static const struct ata_port_info info_c3 = {
.sht = &ali_sht,
.flags = ATA_FLAG_SLAVE_POSS | ATA_FLAG_SRST | ATA_FLAG_PIO_LBA48,
.pio_mask = 0x1f,
.mwdma_mask = 0x07,
- .udma_mask = 0x3f,
+ .udma_mask = 0x1f,
.port_ops = &ali_c2_port_ops
};
- /* Revision 0xC4 is UDMA133 */
+ /* Revision 0xC4 is UDMA100 */
static const struct ata_port_info info_c4 = {
.sht = &ali_sht,
.flags = ATA_FLAG_SLAVE_POSS | ATA_FLAG_SRST | ATA_FLAG_PIO_LBA48,
.pio_mask = 0x1f,
.mwdma_mask = 0x07,
- .udma_mask = 0x7f,
+ .udma_mask = 0x3f,
.port_ops = &ali_c2_port_ops
};
/* Revision 0xC5 is UDMA133 with LBA48 DMA */
static const struct ata_port_info info_c5 = {
.sht = &ali_sht,
.flags = ATA_FLAG_SLAVE_POSS | ATA_FLAG_SRST,
.pio_mask = 0x1f,
.mwdma_mask = 0x07,
.udma_mask = 0x7f,
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: pata_ali patch not merged?
2007-06-25 23:13 pata_ali patch not merged? Chuck Ebbert
@ 2007-07-03 7:47 ` Tejun Heo
2007-07-03 12:13 ` Alan Cox
2007-07-03 14:07 ` Jeff Garzik
1 sibling, 1 reply; 4+ messages in thread
From: Tejun Heo @ 2007-07-03 7:47 UTC (permalink / raw)
To: Chuck Ebbert, Jeff Garzik; +Cc: IDE/ATA development list, Alan
Chuck Ebbert wrote:
> This patch was found to fix some of the problems with the
> pata_ali driver. Is it going to be merged in 2.6.22?
>
> [https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=156482]
Jeff, this looks like a worthy fix for 2.6.22.
--
tejun
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: pata_ali patch not merged?
2007-07-03 7:47 ` Tejun Heo
@ 2007-07-03 12:13 ` Alan Cox
0 siblings, 0 replies; 4+ messages in thread
From: Alan Cox @ 2007-07-03 12:13 UTC (permalink / raw)
To: Tejun Heo; +Cc: Chuck Ebbert, Jeff Garzik, IDE/ATA development list
On Tue, 03 Jul 2007 16:47:33 +0900
Tejun Heo <htejun@gmail.com> wrote:
> Chuck Ebbert wrote:
> > This patch was found to fix some of the problems with the
> > pata_ali driver. Is it going to be merged in 2.6.22?
> >
> > [https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=156482]
>
> Jeff, this looks like a worthy fix for 2.6.22.
It's currently in -mm and seems to have sorted all those chipsets out so
its a good candidate to push on further.
Alan
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: pata_ali patch not merged?
2007-06-25 23:13 pata_ali patch not merged? Chuck Ebbert
2007-07-03 7:47 ` Tejun Heo
@ 2007-07-03 14:07 ` Jeff Garzik
1 sibling, 0 replies; 4+ messages in thread
From: Jeff Garzik @ 2007-07-03 14:07 UTC (permalink / raw)
To: Chuck Ebbert; +Cc: IDE/ATA development list, Alan
Chuck Ebbert wrote:
> This patch was found to fix some of the problems with the
> pata_ali driver. Is it going to be merged in 2.6.22?
>
> [https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=156482]
applied
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2007-07-03 14:07 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2007-06-25 23:13 pata_ali patch not merged? Chuck Ebbert
2007-07-03 7:47 ` Tejun Heo
2007-07-03 12:13 ` Alan Cox
2007-07-03 14:07 ` 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).