Re: [PATCH] net: via-rhine: Drop revision property, use quirks instead

From: David Miller
Date: Fri May 02 2014 - 15:56:41 EST


From: Alexey Charkov <alchark@xxxxxxxxx>
Date: Wed, 30 Apr 2014 22:21:09 +0400

> This adds two new flags to quirks and thus removes the need to carry
> revision in rhine_private. As a result, the init logic is simplified
> a bit.
>
> This also fixes a compiler warning in OF code on 64bit due to pointer
> casting:
>
> drivers/net/ethernet/via/via-rhine.c: In function ʽrhine_init_one_platformʼ:
> drivers/net/ethernet/via/via-rhine.c:1132:13: warning: cast from pointer to integer of different size [-Wpointer-to-int-cast]
> revision = (u32)match->data;
> ^
>
> That code was added in commit 2d283862dc62daead9db0dc89cd0d0351e91f765
> ("net: via-rhine: add OF bus binding").
>
> Tested in platform configuration on a VIA WM8950 APC Rock board.
>
> Reported-by: Jan Moskyto Matejka <mq@xxxxxxx>
> Signed-off-by: Alexey Charkov <alchark@xxxxxxxxx>

This looks a lot better, applied, thanks.