Message ID | 20130529012231.GB2874@mtj.dyndns.org |
---|---|
State | Not Applicable |
Delegated to: | David Miller |
Headers | show |
> Applied to libata/for-3.10-fixes. In the future, please base patches > on top of git://git.kernel.org/pub/scm/linux/kernel/git/tj/libata.git > for-XXX branches. As you saw, it didn't initially occur to me to prepare a patch at all; it seemed such a trivial copy/paste/one-character-edit exercise. After diagnosis, the most difficult part was writing a meaningful commit message. Which I presumed would be easier for you; a lot of the difficulty for me was trying to figure out one line that would fit the ongoing narrative that is the kernel changelog. I was actually surprised when you asked for a full patch, so I just sent you a copy of my private commit, with an improved commit message and S-o-b. (Should I have added "Cc: stable" to it?) But since I now know you prefer even trivial changes like this in formal patch form, I'll try to do it right in future. -- 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
Hello, On Wed, May 29, 2013 at 08:36:27AM -0400, George Spelvin wrote: > As you saw, it didn't initially occur to me to prepare a patch at all; > it seemed such a trivial copy/paste/one-character-edit exercise. > After diagnosis, the most difficult part was writing a meaningful > commit message. > > Which I presumed would be easier for you; a lot of the difficulty for me > was trying to figure out one line that would fit the ongoing narrative > that is the kernel changelog. > > I was actually surprised when you asked for a full patch, so I just sent > you a copy of my private commit, with an improved commit message and S-o-b. > (Should I have added "Cc: stable" to it?) > > But since I now know you prefer even trivial changes like this in formal > patch form, I'll try to do it right in future. It's partly for convenience on my part but more to attribute changes correctly. This is something you did so if at all possible I'd like you to take the credit and (however minute it may be) the accompanying responsibility. It doesn't have to be perfect and I'm happy to edit the commit messages / patches as necessary and adding cc to stable. Well, at least unless you're gonna be submitting large volume of patches. Thanks.
diff --git a/drivers/ata/ahci.c b/drivers/ata/ahci.c index 2180876..2b50dfd 100644 --- a/drivers/ata/ahci.c +++ b/drivers/ata/ahci.c @@ -423,6 +423,8 @@ static const struct pci_device_id ahci_pci_tbl[] = { .driver_data = board_ahci_yes_fbs }, /* 88se9125 */ { PCI_DEVICE(PCI_VENDOR_ID_MARVELL_EXT, 0x917a), .driver_data = board_ahci_yes_fbs }, /* 88se9172 */ + { PCI_DEVICE(PCI_VENDOR_ID_MARVELL_EXT, 0x9172), + .driver_data = board_ahci_yes_fbs }, /* 88se9172 */ { PCI_DEVICE(PCI_VENDOR_ID_MARVELL_EXT, 0x9192), .driver_data = board_ahci_yes_fbs }, /* 88se9172 on some Gigabyte */ { PCI_DEVICE(PCI_VENDOR_ID_MARVELL_EXT, 0x91a3),