[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-Id: <20110921.153146.2033377674238625359.davem@davemloft.net>
Date: Wed, 21 Sep 2011 15:31:46 -0400 (EDT)
From: David Miller <davem@...emloft.net>
To: fthain@...egraphics.com.au
Cc: linux-m68k@...r.kernel.org, netdev@...r.kernel.org
Subject: Re: [PATCH] macmace, macsonic: cleanup
From: Finn Thain <fthain@...egraphics.com.au>
Date: Wed, 14 Sep 2011 03:30:25 +1000 (EST)
>
> We check ether_type before registering the platform device in
> arch/m68k/mac/config.c. Doing the same test again in the driver is
> redundant so remove it.
>
> Multiple probes should not happen since the conversion to platform devices,
> so lose that test too.
>
> Then macmace.c need not include macintosh.h, so remove that and irq.h and
> include linux/interrupt.h explicitly.
>
> Tested on PowerBook 520, Quadra 660av, LC 630.
>
> Signed-off-by: Finn Thain <fthain@...egraphics.com.au>
Applied to net-next, thanks.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Powered by blists - more mailing lists