svn commit: r228785 - in head/sys/dev/ath/ath_hal: ar5210 ar5211

[John Baldwin]

On Wednesday, December 21, 2011 4:52:04 pm Adrian Chadd wrote:
> Erm, why did you do this without first getting clearance from someone
> who has the hardware to test it?
> 
> Just because it looks obviously wrong to you, doesn't at all mean that
> it's "wrong". It's quite possible that the driver _requires_ those
> bits to be written to the hardware as 0.
> 
> 
> I'd appreciate it if would please revert this and other ath/hal
> changes until I've had time to research them and test them out.

I agree it should be reviewed, but if you are seriously depending on
the fact that the shifted values are beyond the edge of the word boundary
and so the result "wraps" to zero, then I'd question the sanity of your code.

-- 
John Baldwin