| Message ID | 20091125170436.5446.83994.sendpatchset@localhost |
|---|---|
| State | Not Applicable |
| Delegated to: | David Miller |
| Headers | show |
On Wed, 25 Nov 2009 18:04:36 +0100 Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote: > From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > Subject: [PATCH] pata_cmd640: document known issues > > Document known issues with the driver to aid distribution makers, > users and developers in making informed decisions instead of wasting > their time needlessly. Umm. its a) under CONFIG_PCI, and b) says CMD640 PCI IDE... -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 25 November 2009 06:26:33 pm Alan Cox wrote: > On Wed, 25 Nov 2009 18:04:36 +0100 > Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote: > > > From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > > Subject: [PATCH] pata_cmd640: document known issues > > > > Document known issues with the driver to aid distribution makers, > > users and developers in making informed decisions instead of wasting > > their time needlessly. > > Umm. its a) under CONFIG_PCI, and b) says CMD640 PCI IDE... c) * This drives only the PCI version of the controller. If you have a * VLB one then we have enough docs to support it but you can write * your own code. Then what has happened to CMD640 VLB IDE support in libata? -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 11/25/2009 12:26 PM, Alan Cox wrote: > On Wed, 25 Nov 2009 18:04:36 +0100 > Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: > >> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> >> Subject: [PATCH] pata_cmd640: document known issues >> >> Document known issues with the driver to aid distribution makers, >> users and developers in making informed decisions instead of wasting >> their time needlessly. > > Umm. its a) under CONFIG_PCI, and b) says CMD640 PCI IDE... Agreed, that's a bit redundant. Also, as I noted in another email, known issues should be listed in the driver source code at the top of the file, not in Kconfig. That way the list of known issues actually follows the implementation, as the source code is copied around various kernel trees. Jeff -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wednesday 25 November 2009 06:37:19 pm Jeff Garzik wrote: > On 11/25/2009 12:26 PM, Alan Cox wrote: > > On Wed, 25 Nov 2009 18:04:36 +0100 > > Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> wrote: > > > >> From: Bartlomiej Zolnierkiewicz<bzolnier@gmail.com> > >> Subject: [PATCH] pata_cmd640: document known issues > >> > >> Document known issues with the driver to aid distribution makers, > >> users and developers in making informed decisions instead of wasting > >> their time needlessly. > > > > Umm. its a) under CONFIG_PCI, and b) says CMD640 PCI IDE... > > Agreed, that's a bit redundant. > > Also, as I noted in another email, known issues should be listed in the > driver source code at the top of the file, not in Kconfig. Agreed, since virtually everybody should be looking into source code of each single driver before compiling kernel and/or deciding to include it into distribution kernel. -- Bartlomiej Zolnierkiewicz -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, 25 Nov 2009 18:34:04 +0100 Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote: > On Wednesday 25 November 2009 06:26:33 pm Alan Cox wrote: > > On Wed, 25 Nov 2009 18:04:36 +0100 > > Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> wrote: > > > > > From: Bartlomiej Zolnierkiewicz <bzolnier@gmail.com> > > > Subject: [PATCH] pata_cmd640: document known issues > > > > > > Document known issues with the driver to aid distribution makers, > > > users and developers in making informed decisions instead of wasting > > > their time needlessly. > > > > Umm. its a) under CONFIG_PCI, and b) says CMD640 PCI IDE... > > c) > > * This drives only the PCI version of the controller. If you have a > * VLB one then we have enough docs to support it but you can write > * your own code. > > Then what has happened to CMD640 VLB IDE support in libata? There is none. Nobody has complained about that either, and in fact I've not even been able to find a board on ebay to test either driver with VLB ! -- To unsubscribe from this list: send the line "unsubscribe linux-ide" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Index: b/drivers/ata/Kconfig =================================================================== --- a/drivers/ata/Kconfig +++ b/drivers/ata/Kconfig @@ -287,8 +287,11 @@ config PATA_CMD640_PCI depends on PCI && EXPERIMENTAL help This option enables support for the CMD640 PCI IDE - interface chip. Only the primary channel is currently - supported. + interface chip. + + Known issues: + - only the primary channel is currently supported + - only the PCI chip interface is currently supported If unsure, say N.