Re: [PATCH 2/2] staging: keucr: use kernel byteorder functions

From: Jack Stone
Date: Thu Mar 03 2011 - 02:59:05 EST


> @@ -455,10 +457,10 @@ int MS_LibProcessBootBlock(struct us_data *us, WORD PhyBlock, BYTE *PageData)
> }
>
> idi = &((MemStickBootBlockCIS_IDI *)(PageBuffer + (EntryOffset % MS_BYTES_PER_PAGE)))->idi.idi;
> - if (LittleEndianWORD(idi->wIDIgeneralConfiguration) != MS_IDI_GENERAL_CONF)
> + if (le16_to_cpu(idi->wIDIgeneralConfiguration) != MS_IDI_GENERAL_CONF)
> goto exit;
>
> - us->MS_Lib.BytesPerSector = LittleEndianWORD(idi->wIDIbytesPerSector);
> + us->MS_Lib.BytesPerSector = be16_to_cpu(idi->wIDIbytesPerSector);

Shouldn't this be le16_to_cpu?

> if (us->MS_Lib.BytesPerSector != MS_BYTES_PER_PAGE)
> goto exit;
> }

Thanks,

Jack
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/