[PATCH][v2] driver/FSL SATA:Fix wrong Device Error Register usage

David Laight David.Laight at ACULAB.COM
Tue Mar 8 22:39:02 EST 2011


 
>  Case when ffs return will never arise.This scenario is 
> already been discussed
>  on linuxppc-dev at lists.ozlabs.org. Please see below explanation:
>  sata_fsl_error_intr() is called during device error.The 
> mentioned scenario
>  will never comes. It can be observed via code:-
>         if (cereg) {   --> cereg is set on command error. 
> Means there is at least 1 device present.

This all relies on the hardware working properly, and
nothing else accidentally clearing the device 'error'
bit mask.

Being slightly defensive when the out of range value
causes an array subscript error seems sensible.

It isn't as though the code size or execution time
matters here.

	David




More information about the Linuxppc-dev mailing list