Re: [PATCH] ide: hpt366: fix incorrect mask when checking at cmd_high_time

From: Sergei Shtylyov
Date: Tue Jul 12 2016 - 07:09:59 EST


On 7/12/2016 2:04 PM, Sergei Shtylyov wrote:

From: Colin Ian King <colin.king@xxxxxxxxxxxxx>

According to the HPT366 data sheet, PCI config space dword 0x40-0x43
bits 11:8 specify the primary drive cmd_high_time, however,
currently just 3 bits of the 4 are being used because the mask
is 0x07 and not 0x0f. Fix the mask, allowing for the 40MHz clock
to be detected.

Also add in missing space between switch and parenthesis to clean
up a checkpatch warning.

Signed-off-by: Colin Ian King <colin.king@xxxxxxxxxxxxx>

Acked-by: Sergei Shtylyov <sergei.shtylyov@xxxxxxxxxxxxxxxxxx>

BTW, drivers/ata/pata_hpt366.c has the same bug...

MBR, Sergei