[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Date: Thu, 3 Feb 2022 08:49:42 +0900
From: Damien Le Moal <damien.lemoal@...nsource.wdc.com>
To: Paul Menzel <pmenzel@...gen.mpg.de>
Cc: linux-ide@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3] ata: ahci: Skip 200 ms debounce delay for Marvell
88SE9235
On 2/3/22 08:05, Damien Le Moal wrote:
>>> ata: ahci: Add support for Marvell 88SE9235 adapter
>>>
>>> Since it is exactly what this patch is doing by adding a PCI ID.
>>
>> Thank you for applying the patch. I saw the summary/title change also
>> with the other patch. I am sorry, but I totally disagree. Reading that
>> summary/title in `git log --oneline`, it’s not clear at all, what the
>> patch does, and the full description or diff has to be read. “Add
>> support” for me means, that it was unsupported before, which is not true
>> at all as the defaults were used.
>
> Right... I did not consider the final entry in the ahci_pci_tbl table
> which matches against the device against the PCI AHCI storage class.
> So right you are. I will restore your original commit title/message.
>
> Thanks for checking.
Fixed now !
--
Damien Le Moal
Western Digital Research
Powered by blists - more mailing lists