Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-23 Thread David Miller
From: Arnd Bergmann Date: Thu, 22 Jan 2015 11:18:50 +0100 > I see. In this case, I think merging your new driver is not a good idea: +1 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-22 Thread Arnd Bergmann
On Thursday 22 January 2015 00:20:59 wstep...@codeaurora.org wrote: > > Right. For review purposes, I think it would be helpful to split this > > huge patch into several steps then: > > > > - add a base driver > > - add the overlay interface > > - add the nss driver > > > > Ideally more of them. >

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-21 Thread wstephen
> Right. For review purposes, I think it would be helpful to split this > huge patch into several steps then: > > - add a base driver > - add the overlay interface > - add the nss driver > > Ideally more of them. The nss-drv driver is open sourced but we are currently not planning to upstream to l

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-20 Thread Arnd Bergmann
On Monday 19 January 2015 21:58:09 wstep...@codeaurora.org wrote: > > On Thursday 15 January 2015 08:12:51 wstep...@codeaurora.org wrote: > > > >> The nss-gmac driver is designed to work standalone or with the nss-drv > >> driver so the switchable data plane overlay was implemented. When it > >> wo

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-19 Thread wstephen
> On Thursday 15 January 2015 08:12:51 wstep...@codeaurora.org wrote: >> >> The nss-gmac driver is for the internal GMAC IP in the Qualcomm IPQ806x >> SoC. There are 2 ARM cores and 2 NSS cores inside the IPQ806x SoC. The >> main purpose of these NSS cores is to offload the networking stack from >>

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-15 Thread Stephen Hemminger
On Thu, 8 Jan 2015 14:03:46 -0800 Stephen Wang wrote: > +static int32_t nss_gmac_setup_tx_desc_queue(struct nss_gmac_dev *gmacdev, > + struct device *dev, > + uint32_t no_of_desc, > +

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-15 Thread Stephen Hemminger
On Thu, 8 Jan 2015 14:03:46 -0800 Stephen Wang wrote: > + struct nss_gmac_data_plane_ops *data_plane_ops; > + /* ops to send messages to nss-drv*/ This should be a const pointer. -- To unsubscribe from this list: send the line "unsubscribe linux-kerne

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-15 Thread Stephen Hemminger
On Thu, 8 Jan 2015 14:03:46 -0800 Stephen Wang wrote: > + uint32_t rx_bytes; /**< Number of RX bytes */ > + uint32_t rx_packets;/**< Number of RX packets */ 32 bit packet stats are a problem with lots of traffic. -- To unsubscribe from this list: send the line "

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-15 Thread Arnd Bergmann
On Thursday 15 January 2015 08:12:51 wstep...@codeaurora.org wrote: > > The nss-gmac driver is for the internal GMAC IP in the Qualcomm IPQ806x > SoC. There are 2 ARM cores and 2 NSS cores inside the IPQ806x SoC. The > main purpose of these NSS cores is to offload the networking stack from > the A

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-15 Thread wstephen
Hi Arnd, Francois The nss-gmac driver is for the internal GMAC IP in the Qualcomm IPQ806x SoC. There are 2 ARM cores and 2 NSS cores inside the IPQ806x SoC. The main purpose of these NSS cores is to offload the networking stack from the ARM cores to achieve high performance at routing/ipsec..etc w

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-09 Thread Mark Rutland
On Thu, Jan 08, 2015 at 10:03:46PM +, Stephen Wang wrote: > This driver adds support for the internal GMACs on IPQ806x SoCs. > It supports the device-tree and will register up to 4 ethernet > interfaces. > > Signed-off-by: Stephen Wang > --- > drivers/net/ethernet/atheros/Kconfig

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-08 Thread Francois Romieu
Stephen Wang : > This driver adds support for the internal GMACs on IPQ806x SoCs. > It supports the device-tree and will register up to 4 ethernet > interfaces. > > Signed-off-by: Stephen Wang > --- [...] > +/* > + * NSS GMAC data plane ops, default would be slowpath and can be override by > + *

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-08 Thread Arnd Bergmann
On Thursday 08 January 2015 14:03:46 Stephen Wang wrote: > This driver adds support for the internal GMACs on IPQ806x SoCs. > It supports the device-tree and will register up to 4 ethernet > interfaces. > > Signed-off-by: Stephen Wang Just a very brief review here. The driver is far too long to

Re: [PATCH] ethernet: atheros: Add nss-gmac driver

2015-01-08 Thread Joe Perches
On Thu, 2015-01-08 at 14:03 -0800, Stephen Wang wrote: > This driver adds support for the internal GMACs on IPQ806x SoCs. > It supports the device-tree and will register up to 4 ethernet > interfaces. Trivial: You might try running the code through checkpatch. -- To unsubscribe from this list: