On Mon, Mar 16, 2009 at 3:58 AM, Hans Verkuil <hverk...@xs4all.nl> wrote:
> Looks great!
>
> I have one small final note (doesn't prevent this from going in):
>
> In au8522_decoder.c you can remove the 'normal_i2c[]' array and
> the I2C_CLIENT_INSMOD macro. It's only relevant for kernels < 2.6.22.
>
> Note that you should replace the normal_i2c array with a comment telling the
> reader which i2c address is used by this device. Or, alternatively, put that
> info in au8522.h if it isn't there already (didn't check this).

No problem.  I had suspected that might need to be removed.  I will
add a patch to a series I have queuing up of stuff unrelated to the
HVR-950q, and will submit a PULL request Tuesday night.

Thanks for all your help,

Devin

-- 
Devin J. Heitmueller
http://www.devinheitmueller.com
AIM: devinheitmueller
--
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