Em 21-10-2010 19:06, Jarod Wilson escreveu:
> On Oct 21, 2010, at 4:38 PM, Mauro Carvalho Chehab wrote:
>
>> Em 21-10-2010 18:06, Jarod Wilson escreveu:
>>>> @@ -265,6 +265,7 @@ struct mceusb_dev {
>>>> u32 connected:1;
>>>> u32 tx_mask_inverted:1;
>>>> u32 microsoft_gen1:1;
>>>> + u32 is_polaris:1;
>>>> u32 reserved:29;
>>>
>>> reserved should be decremented by 1 here if adding another flag.
>>
>> Ok. By curiosity, why are you reserving space on a bit array like that?
>
> Legacy, mostly, I guess. Its been that way going back ages. I suppose we could just convert them all to bools and/or leave them as is and drop the reserved part entirely...
Agreed. If this is not part of any kabi (and it isn't, as far as I see), we can just
drop the reserved, as gcc will pad it accordingly.
> Yeah, this sorta grew from merging the old lirc_mceusb and lirc_mceusb2 drivers into one, and believe it or not, its actually cleaner than it used to be... :)
>
> You've got a test patch in your inbox that attempts to merge the logic in this patch into the existing loop, and I'll see if I can beat on it with all three generations of stand-alone mceusb devices tonight...
>
I'll do some tests and send you a feedback.
Cheers,
Mauro
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
