[PATCH] ixp4xx: Support the all multicast flag on the NPE devices.
Mikael Pettersson
mikpe at it.uu.se
Mon May 31 12:09:33 EDT 2010
Richard Cochran writes:
> On Mon, May 31, 2010 at 04:09:13PM +0200, Krzysztof Halasa wrote:
> > Looks good, though I'd prefer a bit of "const" and "static" in the
> > allmulti[] declaration. Would you please repost? TIA.
>
> Okay, here it is.
>
> Thanks,
> Richard
>
> This patch adds support for the IFF_ALLMULTI flag. Previously only the
> IFF_PROMISC flag was supported.
>
> Signed-off-by: Richard Cochran <richard.cochran at omicron.at>
> ---
> drivers/net/arm/ixp4xx_eth.c | 11 +++++++++++
> 1 files changed, 11 insertions(+), 0 deletions(-)
>
> diff --git a/drivers/net/arm/ixp4xx_eth.c b/drivers/net/arm/ixp4xx_eth.c
> index 6be8b09..42b4361 100644
> --- a/drivers/net/arm/ixp4xx_eth.c
> +++ b/drivers/net/arm/ixp4xx_eth.c
> @@ -739,6 +739,17 @@ static void eth_set_mcast_list(struct net_device *dev)
> struct dev_mc_list *mclist;
> u8 diffs[ETH_ALEN], *addr;
> int i;
> + static const u8 allmulti[] = { 0x01, 0x00, 0x00, 0x00, 0x00, 0x00 };
> +
> + if (dev->flags & IFF_ALLMULTI) {
> + for (i = 0; i < ETH_ALEN; i++) {
> + __raw_writel(allmulti[i], &port->regs->mcast_addr[i]);
> + __raw_writel(allmulti[i], &port->regs->mcast_mask[i]);
Seems a bit excessive to define a lookup table for a computation
that amounts to nothing more than "i ? 0 : 1".
Something like the following would IMO be cleaner:
if (...) {
for (...) {
u8 multi = i ? 0x00 : 0x01;
__raw_writel(multi, ...);
...
}
}
More information about the linux-arm-kernel
mailing list