On 04/20/2015 02:35 PM, [email protected] wrote: > Swap the whole 32 bits we read from the hardware instead of swapping > just the 16bits we care about in place later. > > Signed-off-by: Christoph Hellwig <[email protected]> > --- > drivers/scsi/megaraid/megaraid_sas_base.c | 5 +++-- > 1 files changed, 3 insertions(+), 2 deletions(-) > > diff --git a/drivers/scsi/megaraid/megaraid_sas_base.c > b/drivers/scsi/megaraid/megaraid_sas_base.c > index e863701..fa6c3c9 100644 > --- a/drivers/scsi/megaraid/megaraid_sas_base.c > +++ b/drivers/scsi/megaraid/megaraid_sas_base.c > @@ -4923,8 +4923,9 @@ megasas_register_aen(struct megasas_instance *instance, > u32 seq_num, > > if (instance->aen_cmd) { > > - prev_aen.word = instance->aen_cmd->frame->dcmd.mbox.w[1]; > - prev_aen.members.locale = le16_to_cpu(prev_aen.members.locale); > + prev_aen.word = > + le32_to_cpu(instance->aen_cmd->frame->dcmd.mbox.w[1]); > + prev_aen.members.locale = prev_aen.members.locale;
This^ looks odd, do we need the last assignment? Tomas > > /* > * A class whose enum value is smaller is inclusive of all -- To unsubscribe from this list: send the line "unsubscribe linux-scsi" in the body of a message to [email protected] More majordomo info at http://vger.kernel.org/majordomo-info.html
