Messages in this thread | | | From | Kalle Valo <> | Subject | Re: [PATCH 2/2] adm8211: fix the possible pci cache line sizes inside switch-case | Date | Wed, 06 May 2015 07:59:04 +0300 |
| |
Okash Khawaja <okash.khawaja@gmail.com> writes:
> The PCI cache line size value was being compared against decimal values prefixed with 0x. > > Fixed the literals to use the correct hex values. > > > Signed-off-by: Okash Khawaja <okash.khawaja@gmail.com>
[...]
> @@ -1101,10 +1101,10 @@ static void adm8211_hw_init(struct ieee80211_hw *dev) > case 0x8: > reg |= (0x1 << 14); > break; > - case 0x16: > + case 0x10: > reg |= (0x2 << 14); > break; > - case 0x32: > + case 0x20: > reg |= (0x3 << 14); > break; > default:
Did you test this? How certain can we be that this doesn't break anything?
-- Kalle Valo
| |