Stefan Richter wrote:
> James Bottomley wrote:
>>> This is all spurious bracket addition; it doesn't really have any place
>>> in a code fix.
[...]
> Before:
>
> if (upsg->sg[i].count > (dev->adapter_info.options & AAC_OPT_NEW_COMM)
> ? (dev->scsi_host_ptr->max_sectors << 9) : 65536) {
Doesn't make sense. dev->adapter_info.options is a __le32 bitfield.
> After:
>
> if (upsg->sg[i].count > (
> (dev->adapter_info.options & AAC_OPT_NEW_COMM)
> ? (dev->scsi_host_ptr->max_sectors << 9) : 65536
> )) {
Makes sense. Don't know if it's correct.
--
Stefan Richter
-=====-==--= =-== --=--
http://arcgraph.de/sr/
--
To unsubscribe from this list: send the line "unsubscribe linux-scsi" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
