Julia Lawall <julia.law...@lip6.fr> wrote:

>The function rtl2830_init contains the code:
>
>         buf[0] = tmp << 6;
>         buf[0] = (if_ctl >> 16) & 0x3f;
>         buf[1] = (if_ctl >>  8) & 0xff;
>         buf[2] = (if_ctl >>  0) & 0xff;
>
>Is there any purpose to initializing buf[0] twice?
>
>julia
>--
>To unsubscribe from this list: send the line "unsubscribe linux-media"
>in
>the body of a message to majord...@vger.kernel.org
>More majordomo info at  http://vger.kernel.org/majordomo-info.html

Hmm.  Since 0x3f is the lowest 6 bits, it looks like the second line should use 
|= instead of = .   I don't know anything about the rt2830 though.

-Andy
--
To unsubscribe from this list: send the line "unsubscribe linux-media" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to