On 2/22/22 18:48, Sergey Shtylyov wrote:
> On 2/22/22 3:37 AM, Damien Le Moal wrote:
>
>>> HPT371 chips physically have only one channel, the secondary one, however
>>> the primary channel registers do exist! Thus we have to manually disable
>>> the non-existing channel (if the BIOS hasn't done this already). Alan Cox
>>> has only added such code to the 'pata_hpt3x2n' driver, forgetting about
>>> this one... :-/
>>
>> No need to assign blame by name :) Something more neutral like:
>>
>> "Similarly to the pata_hpt3x2n driver, always disable the primary channel."
>
> Should I now repost?
>
>> will do. Also, do you have a fixes tag for this one ? Is it again:
>>
>> Fixes: 669a5db411d8 ("[libata] Add a bunch of PATA drivers.")> ?
>
> Yes.
Applied to for-5.17-fixes with the commit message fixes. Thanks !
--
Damien Le Moal
Western Digital Research
