On 8 May 2007, at 02:19, Krzysztof Halasa wrote:
> Adds a driver for built-in IXP4xx Ethernet MAC and HSS ports

quoted text
>

> Signed-off-by: Krzysztof Halasa

>

> diff --git a/arch/arm/mach-ixp4xx/ixdp425-setup.c b/arch/arm/mach-

> ixp4xx/ixdp425-setup.c

> index ec4f079..f20d39d 100644

> --- a/arch/arm/mach-ixp4xx/ixdp425-setup.c

> +++ b/arch/arm/mach-ixp4xx/ixdp425-setup.c

> @@ -101,10 +101,35 @@ static struct platform_device ixdp425_uart = {

>  	.resource		= ixdp425_uart_resources

>  };

>

> +/* Built-in 10/100 Ethernet MAC interfaces */

> +static struct mac_plat_info ixdp425_plat_mac[] = {

> +	{

> +		.phy		= 0,

> +		.rxq		= 3,

> +	}, {

> +		.phy		= 1,

> +		.rxq		= 4,

> +	}

> +};

> +

> +static struct platform_device ixdp425_mac[] = {

> +	{

> +		.name			= "ixp4xx_eth",

> +		.id			= IXP4XX_ETH_NPEB,

> +		.dev.platform_data	= ixdp425_plat_mac,

> +	}, {

> +		.name			= "ixp4xx_eth",

> +		.id			= IXP4XX_ETH_NPEC,

> +		.dev.platform_data	= ixdp425_plat_mac + 1,

> +	}

> +};

> +

>  static struct platform_device *ixdp425_devices[] __initdata = {

>  	&ixdp425_i2c_controller,

>  	&ixdp425_flash,

> -	&ixdp425_uart

> +	&ixdp425_uart,

> +	&ixdp425_mac[0],

> +	&ixdp425_mac[1],

>  };

>

>  static void __init ixdp425_init(void)

A final submission should probably have this platform data separated

from the net driver and sent upstream via Russell's patch tracking

system rather than netdev.
> diff --git a/drivers/net/arm/Kconfig b/drivers/net/arm/Kconfig

quoted text
> index 678e4f4..5e2acb6 100644

> --- a/drivers/net/arm/Kconfig

> +++ b/drivers/net/arm/Kconfig

> @@ -46,3 +46,13 @@ config EP93XX_ETH

>  	help

>  	  This is a driver for the ethernet hardware included in EP93xx

> CPUs.

>  	  Say Y if you are building a kernel for EP93xx based devices.

> +

> +config IXP4XX_ETH

> +	tristate "IXP4xx Ethernet support"

> +	depends on NET_ETHERNET && ARM && ARCH_IXP4XX

> +	select IXP4XX_NPE

> +	select IXP4XX_QMGR

> +	select MII

> +	help

> +	  Say Y here if you want to use built-in Ethernet ports

> +	  on IXP4xx processor.

> diff --git a/drivers/net/arm/Makefile b/drivers/net/arm/Makefile

> index a4c8682..7c812ac 100644

> --- a/drivers/net/arm/Makefile

> +++ b/drivers/net/arm/Makefile

> @@ -9,3 +9,4 @@ obj-$(CONFIG_ARM_ETHER3)	+= ether3.o

>  obj-$(CONFIG_ARM_ETHER1)	+= ether1.o

>  obj-$(CONFIG_ARM_AT91_ETHER)	+= at91_ether.o

>  obj-$(CONFIG_EP93XX_ETH)	+= ep93xx_eth.o

> +obj-$(CONFIG_IXP4XX_ETH)	+= ixp4xx_eth.o

> diff --git a/drivers/net/arm/ixp4xx_eth.c b/drivers/net/arm/

> ixp4xx_eth.c

> new file mode 100644

> index 0000000..dcea6e5

> --- /dev/null

> +++ b/drivers/net/arm/ixp4xx_eth.c

> @@ -0,0 +1,1002 @@

> +/*

> + * Intel IXP4xx Ethernet driver for Linux

> + *

> + * Copyright (C) 2007 Krzysztof Halasa

> + *

> + * This program is free software; you can redistribute it and/or

> modify it

> + * under the terms of version 2 of the GNU General Public License

> + * as published by the Free Software Foundation.

> + *

> + * Ethernet port config (0x00 is not present on IXP42X):

> + *

> + * logical port		0x00		0x10		0x20

> + * NPE			0 (NPE-A)	1 (NPE-B)	2 (NPE-C)

> + * physical PortId	2		0		1

> + * TX queue		23		24		25

> + * RX-free queue	26		27		28

> + * TX-done queue is always 31, RX queue is configurable

> + */

> +

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +

> +#ifndef __ARMEB__

> +#warning Little endian mode not supported

> +#endif

This has gone from error to warning - fair play but if are planning

to put this upstream this cycle (anything's possible :) ) you'll want

to declare this driver broken on ARMEB in Kconfig please.
Personally I'd like LE ethernet tested and working before we push.
> +

quoted text
> +#define DEBUG_QUEUES	0

> +#define DEBUG_RX	0

> +#define DEBUG_TX	0

> +#define DEBUG_PKT_BYTES	0

> +#define DEBUG_MDIO	0

> +

> +#define DRV_NAME	"ixp4xx_eth"

> +#define DRV_VERSION	"0.04"

> +

> +#define TX_QUEUE_LEN	16 /* dwords */

> +#define PKT_DESCS	64 /* also length of queues: TX-done, RX-ready,

> RX */

> +

> +#define POOL_ALLOC_SIZE		(sizeof(struct desc) * (PKT_DESCS))

> +#define REGS_SIZE		0x1000

> +#define MAX_MRU			1536

> +

> +#define MDIO_INTERVAL		(3 * HZ)

> +#define MAX_MDIO_RETRIES	100 /* microseconds, typically 30 cycles */

> +

> +#define NPE_ID(port)		((port)->id >> 4)

> +#define PHYSICAL_ID(port)	((NPE_ID(port) + 2) % 3)

> +#define TX_QUEUE(plat)		(NPE_ID(port) + 23)

> +#define RXFREE_QUEUE(plat)	(NPE_ID(port) + 26)

> +#define TXDONE_QUEUE		31

> +

> +/* TX Control Registers */

> +#define TX_CNTRL0_TX_EN		BIT(0)

> +#define TX_CNTRL0_HALFDUPLEX	BIT(1)

> +#define TX_CNTRL0_RETRY		BIT(2)

> +#define TX_CNTRL0_PAD_EN	BIT(3)

> +#define TX_CNTRL0_APPEND_FCS	BIT(4)

> +#define TX_CNTRL0_2DEFER	BIT(5)

> +#define TX_CNTRL0_RMII		BIT(6)	/* reduced MII */

> +#define TX_CNTRL1_RETRIES	0x0F /* 4 bits */

> +

> +/* RX Control Registers */

> +#define RX_CNTRL0_RX_EN		BIT(0)

> +#define RX_CNTRL0_PADSTRIP_EN	BIT(1)

> +#define RX_CNTRL0_SEND_FCS	BIT(2)

> +#define RX_CNTRL0_PAUSE_EN	BIT(3)

> +#define RX_CNTRL0_LOOP_EN	BIT(4)

> +#define RX_CNTRL0_ADDR_FLTR_EN	BIT(5)

> +#define RX_CNTRL0_RX_RUNT_EN	BIT(6)

> +#define RX_CNTRL0_BCAST_DIS	BIT(7)

> +#define RX_CNTRL1_DEFER_EN	BIT(0)

> +

> +/* Core Control Register */

> +#define CORE_RESET		BIT(0)

> +#define CORE_RX_FIFO_FLUSH	BIT(1)

> +#define CORE_TX_FIFO_FLUSH	BIT(2)

> +#define CORE_SEND_JAM		BIT(3)

> +#define CORE_MDC_EN		BIT(4) /* NPE-B ETH-0 only */

> +

> +/* Definitions for MII access routines */

> +#define MII_CMD_GO		BIT(31)

> +#define MII_CMD_WRITE		BIT(26)

> +#define MII_STAT_READ_FAILED	BIT(31)

> +

> +/* NPE message codes */

> +#define NPE_GETSTATUS			0x00

> +#define NPE_EDB_SETPORTADDRESS		0x01

> +#define NPE_EDB_GETMACADDRESSDATABASE	0x02

> +#define NPE_EDB_SETMACADDRESSSDATABASE	0x03

> +#define NPE_GETSTATS			0x04

> +#define NPE_RESETSTATS			0x05

> +#define NPE_SETMAXFRAMELENGTHS		0x06

> +#define NPE_VLAN_SETRXTAGMODE		0x07

> +#define NPE_VLAN_SETDEFAULTRXVID	0x08

> +#define NPE_VLAN_SETPORTVLANTABLEENTRY	0x09

> +#define NPE_VLAN_SETPORTVLANTABLERANGE	0x0A

> +#define NPE_VLAN_SETRXQOSENTRY		0x0B

> +#define NPE_VLAN_SETPORTIDEXTRACTIONMODE 0x0C

> +#define NPE_STP_SETBLOCKINGSTATE	0x0D

> +#define NPE_FW_SETFIREWALLMODE		0x0E

> +#define NPE_PC_SETFRAMECONTROLDURATIONID 0x0F

> +#define NPE_PC_SETAPMACTABLE		0x11

> +#define NPE_SETLOOPBACK_MODE		0x12

> +#define NPE_PC_SETBSSIDTABLE		0x13

> +#define NPE_ADDRESS_FILTER_CONFIG	0x14

> +#define NPE_APPENDFCSCONFIG		0x15

> +#define NPE_NOTIFY_MAC_RECOVERY_DONE	0x16

> +#define NPE_MAC_RECOVERY_START		0x17

> +

> +

Two returns? Defines make sense in-file here :)
> +struct eth_regs {

quoted text
> +	u32 tx_control[2], __res1[2];		/* 000 */

> +	u32 rx_control[2], __res2[2];		/* 010 */

> +	u32 random_seed, __res3[3];		/* 020 */

> +	u32 partial_empty_threshold, __res4;	/* 030 */

> +	u32 partial_full_threshold, __res5;	/* 038 */

> +	u32 tx_start_bytes, __res6[3];		/* 040 */

> +	u32 tx_deferral, rx_deferral,__res7[2];	/* 050 */

> +	u32 tx_2part_deferral[2], __res8[2];	/* 060 */

> +	u32 slot_time, __res9[3];		/* 070 */

> +	u32 mdio_command[4];			/* 080 */

> +	u32 mdio_status[4];			/* 090 */

> +	u32 mcast_mask[6], __res10[2];		/* 0A0 */

> +	u32 mcast_addr[6], __res11[2];		/* 0C0 */

> +	u32 int_clock_threshold, __res12[3];	/* 0E0 */

> +	u32 hw_addr[6], __res13[61];		/* 0F0 */

> +	u32 core_control;			/* 1FC */

> +};

> +

> +struct port {

> +	struct resource *mem_res;

> +	struct eth_regs __iomem *regs;

> +	struct npe *npe;

> +	struct net_device *netdev;

> +	struct net_device_stats stat;

> +	struct mii_if_info mii;

> +	struct delayed_work mdio_thread;

> +	struct mac_plat_info *plat;

> +	struct sk_buff *rx_skb_tab[PKT_DESCS];

> +	struct desc *rx_desc_tab; /* coherent */

> +	int id;			/* logical port ID */

> +	u32 rx_desc_tab_phys;

> +	u32 msg_enable;

> +};

> +

> +/* NPE message structure */

> +struct msg {

> +	union {

> +		struct {

> +			u8 cmd, eth_id, mac[ETH_ALEN];

> +		};

> +		struct {

> +			u8 cmd, eth_id, __byte2, byte3;

> +			u8 __byte4, byte5, __byte6, byte7;

> +		};

> +		struct {

> +			u8 cmd, eth_id, __b2, byte3;

> +			u32 data32;

> +		};

> +	};

> +};

> +

> +/* Ethernet packet descriptor */

> +struct desc {

> +	u32 next;		/* pointer to next buffer, unused */

> +	u16 buf_len;		/* buffer length */

> +	u16 pkt_len;		/* packet length */

> +	u32 data;		/* pointer to data buffer in RAM */

> +	u8 dest_id;

> +	u8 src_id;

> +	u16 flags;

> +	u8 qos;

> +	u8 padlen;

> +	u16 vlan_tci;

> +	u8 dest_mac[ETH_ALEN];

> +	u8 src_mac[ETH_ALEN];

> +};

> +

> +

> +#define rx_desc_phys(port, n)	((port)->rx_desc_tab_phys +		\

> +				 (n) * sizeof(struct desc))

> +#define tx_desc_phys(n)		(tx_desc_tab_phys + (n) * sizeof(struct

> desc))

> +

> +static spinlock_t mdio_lock;

> +static struct eth_regs __iomem *mdio_regs; /* mdio command and

> status only */

> +static struct npe *mdio_npe;

> +static int ports_open;

> +static struct dma_pool *dma_pool;

> +static struct sk_buff *tx_skb_tab[PKT_DESCS];

> +static struct desc *tx_desc_tab; /* coherent */

> +static u32 tx_desc_tab_phys;

> +

> +

> +static inline void set_regbits(u32 bits, u32 __iomem *reg)

> +{

> +	__raw_writel(__raw_readl(reg) | bits, reg);

> +}

> +static inline void clr_regbits(u32 bits, u32 __iomem *reg)

> +{

> +	__raw_writel(__raw_readl(reg) & ~bits, reg);

> +}

> +

> +

> +static u16 mdio_cmd(struct net_device *dev, int phy_id, int location,

> +		    int write, u16 cmd)

> +{

> +	int cycles = 0;

> +

> +	if (__raw_readl(&mdio_regs->mdio_command[3]) & 0x80) {

> +		printk("%s: MII not ready to transmit\n", dev->name);

> +		return 0;	/* not ready to transmit */

> +	}

> +

> +	if (write) {

> +		__raw_writel(cmd & 0xFF, &mdio_regs->mdio_command[0]);

> +		__raw_writel(cmd >> 8, &mdio_regs->mdio_command[1]);

> +	}

> +	__raw_writel(((phy_id << 5) | location) & 0xFF,

> +		     &mdio_regs->mdio_command[2]);

> +	__raw_writel((phy_id >> 3) | (write << 2) | 0x80 /* GO */,

> +		     &mdio_regs->mdio_command[3]);

> +

> +	while ((cycles < MAX_MDIO_RETRIES) &&

> +	       (__raw_readl(&mdio_regs->mdio_command[3]) & 0x80)) {

> +		udelay(1);

> +		cycles++;

> +	}

> +

> +	if (cycles == MAX_MDIO_RETRIES) {

> +		printk("%s: MII write failed\n", dev->name);

> +		return 0;

> +	}

> +

> +#if DEBUG_MDIO

> +	printk(KERN_DEBUG "mdio_cmd() took %i cycles\n", cycles);

> +#endif

> +

> +	if (write)

> +		return 0;

> +

> +	if (__raw_readl(&mdio_regs->mdio_status[3]) & 0x80) {

> +		printk("%s: MII read failed\n", dev->name);

> +		return 0;

> +	}

> +

> +	return (__raw_readl(&mdio_regs->mdio_status[0]) & 0xFF) |

> +		(__raw_readl(&mdio_regs->mdio_status[1]) << 8);

> +}

> +

> +static int mdio_read(struct net_device *dev, int phy_id, int

> location)

> +{

> +	unsigned long flags;

> +	u16 val;

> +

> +	spin_lock_irqsave(&mdio_lock, flags);

> +	val = mdio_cmd(dev, phy_id, location, 0, 0);

> +	spin_unlock_irqrestore(&mdio_lock, flags);

> +	return val;

> +}

> +

> +static void mdio_write(struct net_device *dev, int phy_id, int

> location,

> +		       int val)

> +{

> +	unsigned long flags;

> +

> +	spin_lock_irqsave(&mdio_lock, flags);

> +	mdio_cmd(dev, phy_id, location, 1, val);

> +	spin_unlock_irqrestore(&mdio_lock, flags);

> +}

> +

> +static void eth_set_duplex(struct port *port)

> +{

> +	if (port->mii.full_duplex)

> +		clr_regbits(TX_CNTRL0_HALFDUPLEX, &port->regs->tx_control[0]);

> +	else

> +		set_regbits(TX_CNTRL0_HALFDUPLEX, &port->regs->tx_control[0]);

> +}

> +

> +

> +static void mdio_thread(struct work_struct *work)

> +{

> +	struct port *port = container_of(work, struct port,

> mdio_thread.work);

> +

> +	if (mii_check_media(&port->mii, 1, 0))

> +		eth_set_duplex(port);

> +	schedule_delayed_work(&port->mdio_thread, MDIO_INTERVAL);

> +}

> +

> +

> +static inline void debug_skb(const char *func, struct sk_buff *skb)

> +{

> +#if DEBUG_PKT_BYTES

> +	int i;

> +

> +	printk(KERN_DEBUG "%s(%i): ", func, skb->len);

> +	for (i = 0; i < skb->len; i++) {

> +		if (i >= DEBUG_PKT_BYTES)

> +			break;

> +		printk("%s%02X",

> +		       ((i == 6) || (i == 12) || (i >= 14)) ? " " : "",

> +		       skb->data[i]);

> +	}

> +	printk("\n");

> +#endif

> +}

> +

> +

> +static inline void debug_desc(unsigned int queue, u32 desc_phys,

> +			      struct desc *desc, int is_get)

> +{

> +#if DEBUG_QUEUES

> +	const char *op = is_get ? "->" : "<-";

> +

> +	if (!desc_phys) {

> +		printk(KERN_DEBUG "queue %2i %s NULL\n", queue, op);

> +		return;

> +	}

> +	printk(KERN_DEBUG "queue %2i %s %X: %X %3X %3X %08X %2X < %2X %4X

> %X"

> +	       " %X %X %02X%02X%02X%02X%02X%02X < %02X%02X%02X%02X%02X%02X

> \n",

> +	       queue, op, desc_phys, desc->next, desc->buf_len, desc-

> >pkt_len,

> +	       desc->data, desc->dest_id, desc->src_id,

> +	       desc->flags, desc->qos,

> +	       desc->padlen, desc->vlan_tci,

> +	       desc->dest_mac[0], desc->dest_mac[1],

> +	       desc->dest_mac[2], desc->dest_mac[3],

> +	       desc->dest_mac[4], desc->dest_mac[5],

> +	       desc->src_mac[0], desc->src_mac[1],

> +	       desc->src_mac[2], desc->src_mac[3],

> +	       desc->src_mac[4], desc->src_mac[5]);

> +#endif

> +}

> +

> +static inline int queue_get_desc(unsigned int queue, struct port

> *port,

> +				 int is_tx)

> +{

> +	u32 phys, tab_phys, n_desc;

> +	struct desc *tab;

> +

> +	if (!(phys = qmgr_get_entry(queue))) {

> +		debug_desc(queue, phys, NULL, 1);

> +		return -1;

> +	}

> +

> +	phys &= ~0x1F; /* mask out non-address bits */

> +	tab_phys = is_tx ? tx_desc_phys(0) : rx_desc_phys(port, 0);

> +	tab = is_tx ? tx_desc_tab : port->rx_desc_tab;

> +	n_desc = (phys - tab_phys) / sizeof(struct desc);

> +	BUG_ON(n_desc >= PKT_DESCS);

> +

> +	debug_desc(queue, phys, &tab[n_desc], 1);

> +	BUG_ON(tab[n_desc].next);

> +	return n_desc;

> +}

> +

> +static inline void queue_put_desc(unsigned int queue, u32 desc_phys,

> +				  struct desc *desc)

> +{

> +	debug_desc(queue, desc_phys, desc, 0);

> +	BUG_ON(desc_phys & 0x1F);

> +	qmgr_put_entry(queue, desc_phys);

> +}

> +

> +

> +static void eth_rx_irq(void *pdev)

> +{

> +	struct net_device *dev = pdev;

> +	struct port *port = netdev_priv(dev);

> +

> +#if DEBUG_RX

> +	printk(KERN_DEBUG "eth_rx_irq() start\n");

> +#endif

> +	qmgr_disable_irq(port->plat->rxq);

> +	netif_rx_schedule(dev);

> +}

> +

> +static int eth_poll(struct net_device *dev, int *budget)

> +{

> +	struct port *port = netdev_priv(dev);

> +	unsigned int queue = port->plat->rxq;

> +	int quota = dev->quota, received = 0;

> +

> +#if DEBUG_RX

> +	printk(KERN_DEBUG "eth_poll() start\n");

> +#endif

> +	while (quota) {

> +		struct sk_buff *old_skb, *new_skb;

> +		struct desc *desc;

> +		u32 data;

> +		int n = queue_get_desc(queue, port, 0);

> +		if (n < 0) {	/* No packets received */

> +			dev->quota -= received;

> +			*budget -= received;

> +			received = 0;

> +			netif_rx_complete(dev);

> +			qmgr_enable_irq(queue);

> +			if (!qmgr_stat_empty(queue) &&

> +			    netif_rx_reschedule(dev, 0)) {

> +				qmgr_disable_irq(queue);

> +				continue;

> +			}

> +			return 0; /* all work done */

> +		}

> +

> +		desc = &port->rx_desc_tab[n];

> +

> +		if ((new_skb = netdev_alloc_skb(dev, MAX_MRU)) != NULL) {

> +#if 0

> +			skb_reserve(new_skb, 2);  /* FIXME */

> +#endif

> +			data = dma_map_single(&dev->dev, new_skb->data,

> +					      MAX_MRU, DMA_FROM_DEVICE);

> +		}

> +

> +		if (!new_skb || dma_mapping_error(data)) {

> +			if (new_skb)

> +				dev_kfree_skb(new_skb);

> +			port->stat.rx_dropped++;

> +			/* put the desc back on RX-ready queue */

> +			desc->buf_len = MAX_MRU;

> +			desc->pkt_len = 0;

> +			queue_put_desc(RXFREE_QUEUE(port->plat),

> +				       rx_desc_phys(port, n), desc);

> +			BUG_ON(qmgr_stat_overflow(RXFREE_QUEUE(port->plat)));

> +			continue;

> +		}

> +

> +		/* process received skb */

> +		old_skb = port->rx_skb_tab[n];

> +		dma_unmap_single(&dev->dev, desc->data,

> +				 MAX_MRU, DMA_FROM_DEVICE);

> +		skb_put(old_skb, desc->pkt_len);

> +

> +		debug_skb("eth_poll", old_skb);

> +

> +		old_skb->protocol = eth_type_trans(old_skb, dev);

> +		dev->last_rx = jiffies;

> +		port->stat.rx_packets++;

> +		port->stat.rx_bytes += old_skb->len;

> +		netif_receive_skb(old_skb);

> +

> +		/* put the new skb on RX-free queue */

> +		port->rx_skb_tab[n] = new_skb;

> +		desc->buf_len = MAX_MRU;

> +		desc->pkt_len = 0;

> +		desc->data = data;

> +		queue_put_desc(RXFREE_QUEUE(port->plat),

> +			       rx_desc_phys(port, n), desc);

> +		BUG_ON(qmgr_stat_overflow(RXFREE_QUEUE(port->plat)));

> +		quota--;

> +		received++;

> +	}

> +	dev->quota -= received;

> +	*budget -= received;

> +	return 1;		/* not all work done */

> +}

> +

> +static void eth_xmit_ready_irq(void *pdev)

> +{

> +	struct net_device *dev = pdev;

> +

> +#if DEBUG_TX

> +	printk(KERN_DEBUG "eth_xmit_empty() start\n");

> +#endif

> +	netif_start_queue(dev);

> +}

> +

> +static int eth_xmit(struct sk_buff *skb, struct net_device *dev)

> +{

> +	struct port *port = netdev_priv(dev);

> +	struct desc *desc;

> +	u32 phys;

> +	struct sk_buff *old_skb;

> +	int n;

> +

> +#if DEBUG_TX

> +	printk(KERN_DEBUG "eth_xmit() start\n");

> +#endif

> +	if (unlikely(skb->len > MAX_MRU)) {

> +		dev_kfree_skb(skb);

> +		port->stat.tx_errors++;

> +		return NETDEV_TX_OK;

> +	}

> +

> +	n = queue_get_desc(TXDONE_QUEUE, port, 1);

> +	BUG_ON(n < 0);

> +	desc = &tx_desc_tab[n];

> +	phys = tx_desc_phys(n);

> +

> +	if ((old_skb = tx_skb_tab[n]) != NULL) {

> +		dma_unmap_single(&dev->dev, desc->data,

> +				 desc->buf_len, DMA_TO_DEVICE);

> +		port->stat.tx_packets++;

> +		port->stat.tx_bytes += old_skb->len;

> +		dev_kfree_skb(old_skb);

> +	}

> +

> +	/* disable VLAN functions in NPE image for now */

> +	memset(desc, 0, sizeof(*desc));

> +	desc->buf_len = desc->pkt_len = skb->len;

> +	desc->data = dma_map_single(&dev->dev, skb->data,

> +				    skb->len, DMA_TO_DEVICE);

> +	if (dma_mapping_error(desc->data)) {

> +		desc->data = 0;

> +		dev_kfree_skb(skb);

> +		tx_skb_tab[n] = NULL;

> +		port->stat.tx_dropped++;

> +		/* put the desc back on TX-done queue */

> +		queue_put_desc(TXDONE_QUEUE, phys, desc);

> +		return 0;

> +	}

> +

> +	tx_skb_tab[n] = skb;

> +	debug_skb("eth_xmit", skb);

> +

> +	/* NPE firmware pads short frames with zeros internally */

> +	wmb();

> +	queue_put_desc(TX_QUEUE(port->plat), phys, desc);

> +	BUG_ON(qmgr_stat_overflow(TX_QUEUE(port->plat)));

> +	dev->trans_start = jiffies;

> +

> +	if (qmgr_stat_full(TX_QUEUE(port->plat))) {

> +		netif_stop_queue(dev);

> +		/* we could miss TX ready interrupt */

> +		if (!qmgr_stat_full(TX_QUEUE(port->plat))) {

> +			netif_start_queue(dev);

> +		}

> +	}

> +

> +#if DEBUG_TX

> +	printk(KERN_DEBUG "eth_xmit() end\n");

> +#endif

> +	return NETDEV_TX_OK;

> +}

> +

> +

> +static struct net_device_stats *eth_stats(struct net_device *dev)

> +{

> +	struct port *port = netdev_priv(dev);

> +	return &port->stat;

> +}

> +

> +static void eth_set_mcast_list(struct net_device *dev)

> +{

> +	struct port *port = netdev_priv(dev);

> +	struct dev_mc_list *mclist = dev->mc_list;

> +	u8 diffs[ETH_ALEN], *addr;

> +	int cnt = dev->mc_count, i;

> +

> +	if ((dev->flags & IFF_PROMISC) || !mclist || !cnt) {

> +		clr_regbits(RX_CNTRL0_ADDR_FLTR_EN,

> +			    &port->regs->rx_control[0]);

> +		return;

> +	}

> +

> +	memset(diffs, 0, ETH_ALEN);

> +	addr = mclist->dmi_addr; /* first MAC address */

> +

> +	while (--cnt && (mclist = mclist->next))

> +		for (i = 0; i < ETH_ALEN; i++)

> +			diffs[i] |= addr[i] ^ mclist->dmi_addr[i];

> +

> +	for (i = 0; i < ETH_ALEN; i++) {

> +		__raw_writel(addr[i], &port->regs->mcast_addr[i]);

> +		__raw_writel(~diffs[i], &port->regs->mcast_mask[i]);

> +	}

> +

> +	set_regbits(RX_CNTRL0_ADDR_FLTR_EN, &port->regs->rx_control[0]);

> +}

> +

> +

> +static int eth_ioctl(struct net_device *dev, struct ifreq *req,

> int cmd)

> +{

> +	struct port *port = netdev_priv(dev);

> +	unsigned int duplex_chg;

> +	int err;

> +

> +	if (!netif_running(dev))

> +		return -EINVAL;

> +	err = generic_mii_ioctl(&port->mii, if_mii(req), cmd, &duplex_chg);

> +	if (duplex_chg)

> +		eth_set_duplex(port);

> +	return err;

> +}

> +

> +

> +static int request_queues(struct port *port)

> +{

> +	int err;

> +

> +	err = qmgr_request_queue(RXFREE_QUEUE(port->plat), PKT_DESCS, 0, 0);

> +	if (err)

> +		return err;

> +

> +	err = qmgr_request_queue(port->plat->rxq, PKT_DESCS, 0, 0);

> +	if (err)

> +		goto rel_rxfree;

> +

> +	err = qmgr_request_queue(TX_QUEUE(port->plat), TX_QUEUE_LEN, 0, 0);

> +	if (err)

> +		goto rel_rx;

> +

> +	/* TX-done queue handles skbs sent out by the NPEs */

> +	if (!ports_open) {

> +		err = qmgr_request_queue(TXDONE_QUEUE, PKT_DESCS, 0, 0);

> +		if (err)

> +			goto rel_tx;

> +	}

> +	return 0;

> +

> +rel_tx:

> +	qmgr_release_queue(TX_QUEUE(port->plat));

> +rel_rx:

> +	qmgr_release_queue(port->plat->rxq);

> +rel_rxfree:

> +	qmgr_release_queue(RXFREE_QUEUE(port->plat));

> +	return err;

> +}

> +

> +static void release_queues(struct port *port)

> +{

> +	qmgr_release_queue(RXFREE_QUEUE(port->plat));

> +	qmgr_release_queue(port->plat->rxq);

> +	qmgr_release_queue(TX_QUEUE(port->plat));

> +

> +	if (!ports_open)

> +		qmgr_release_queue(TXDONE_QUEUE);

> +}

> +

> +static int init_queues(struct port *port)

> +{

> +	int i;

> +

> +	if (!dma_pool) {

> +		/* Setup TX descriptors - common to all ports */

> +		dma_pool = dma_pool_create(DRV_NAME, NULL, POOL_ALLOC_SIZE,

> +					   32, 0);

> +		if (!dma_pool)

> +			return -ENOMEM;

> +

> +		if (!(tx_desc_tab = dma_pool_alloc(dma_pool, GFP_KERNEL,

> +						   &tx_desc_tab_phys)))

> +			return -ENOMEM;

> +		memset(tx_desc_tab, 0, POOL_ALLOC_SIZE);

> +		memset(tx_skb_tab, 0, sizeof(tx_skb_tab)); /* static table */

> +

> +		for (i = 0; i < PKT_DESCS; i++) {

> +			queue_put_desc(TXDONE_QUEUE, tx_desc_phys(i),

> +				       &tx_desc_tab[i]);

> +			BUG_ON(qmgr_stat_overflow(TXDONE_QUEUE));

> +		}

> +	}

> +

> +	/* Setup RX buffers */

> +	if (!(port->rx_desc_tab = dma_pool_alloc(dma_pool, GFP_KERNEL,

> +						 &port->rx_desc_tab_phys)))

> +		return -ENOMEM;

> +	memset(port->rx_desc_tab, 0, POOL_ALLOC_SIZE);

> +	memset(port->rx_skb_tab, 0, sizeof(port->rx_skb_tab)); /* table */

> +

> +	for (i = 0; i < PKT_DESCS; i++) {

> +		struct desc *desc = &port->rx_desc_tab[i];

> +		struct sk_buff *skb;

> +

> +		if (!(skb = netdev_alloc_skb(port->netdev, MAX_MRU)))

> +			return -ENOMEM;

> +		port->rx_skb_tab[i] = skb;

> +		desc->buf_len = MAX_MRU;

> +#if 0

> +		skb_reserve(skb, 2);  /* FIXME */

> +#endif

Hallo :o)
> +		desc->data = dma_map_single(&port->netdev->dev, skb->data,

quoted text
> +					    MAX_MRU, DMA_FROM_DEVICE);

> +		if (dma_mapping_error(desc->data)) {

> +			desc->data = 0;

> +			return -EIO;

> +		}

> +		queue_put_desc(RXFREE_QUEUE(port->plat),

> +			       rx_desc_phys(port, i), desc);

> +		BUG_ON(qmgr_stat_overflow(RXFREE_QUEUE(port->plat)));

> +	}

> +	return 0;

> +}

> +

> +static void destroy_queues(struct port *port)

> +{

> +	int i;

> +

> +	while (queue_get_desc(RXFREE_QUEUE(port->plat), port, 0) >= 0)

> +		/* nothing to do here */;

> +	while (queue_get_desc(port->plat->rxq, port, 0) >= 0)

> +		/* nothing to do here */;

> +	while (queue_get_desc(TX_QUEUE(port->plat), port, 1) >= 0) {

> +		/* nothing to do here */;

> +	}

> +	if (!ports_open)

> +		while (queue_get_desc(TXDONE_QUEUE, port, 1) >= 0)

> +			/* nothing to do here */;

> +

> +	if (port->rx_desc_tab) {

> +		for (i = 0; i < PKT_DESCS; i++) {

> +			struct desc *desc = &port->rx_desc_tab[i];

> +			struct sk_buff *skb = port->rx_skb_tab[i];

> +			if (skb) {

> +				if (desc->data)

> +					dma_unmap_single(&port->netdev->dev,

> +							 desc->data, MAX_MRU,

> +							 DMA_FROM_DEVICE);

> +				dev_kfree_skb(skb);

> +			}

> +		}

> +		dma_pool_free(dma_pool, port->rx_desc_tab,

> +			      port->rx_desc_tab_phys);

> +		port->rx_desc_tab = NULL;

> +	}

> +

> +	if (!ports_open && tx_desc_tab) {

> +		for (i = 0; i < PKT_DESCS; i++) {

> +			struct desc *desc = &tx_desc_tab[i];

> +			struct sk_buff *skb = tx_skb_tab[i];

> +			if (skb) {

> +				if (desc->data)

> +					dma_unmap_single(&port->netdev->dev,

> +							 desc->data,

> +							 desc->buf_len,

> +							 DMA_TO_DEVICE);

> +				dev_kfree_skb(skb);

> +			}

> +		}

> +		dma_pool_free(dma_pool, tx_desc_tab, tx_desc_tab_phys);

> +		tx_desc_tab = NULL;

> +	}

> +	if (!ports_open && dma_pool) {

> +		dma_pool_destroy(dma_pool);

> +		dma_pool = NULL;

> +	}

> +}

> +

> +static int eth_load_firmware(struct net_device *dev, struct npe *npe)

> +{

> +	struct msg msg;

> +	int err;

> +

> +	if ((err = npe_load_firmware(npe, npe_name(npe), &dev->dev)) != 0)

> +		return err;

> +

> +	if ((err = npe_recv_message(npe, &msg, "ETH_GET_STATUS")) != 0) {

> +		printk(KERN_ERR "%s: %s not responding\n", dev->name,

> +		       npe_name(npe));

> +		return err;

> +	}

> +	return 0;

> +}

> +

> +static int eth_open(struct net_device *dev)

> +{

> +	struct port *port = netdev_priv(dev);

> +	struct npe *npe = port->npe;

> +	struct msg msg;

> +	int i, err;

> +

> +	if (!npe_running(npe))

> +		if (eth_load_firmware(dev, npe))

> +			return -EIO;

> +

> +	if (!npe_running(mdio_npe))

> +		if (eth_load_firmware(dev, mdio_npe))

> +			return -EIO;

> +

> +	memset(&msg, 0, sizeof(msg));

> +	msg.cmd = NPE_VLAN_SETRXQOSENTRY;

> +	msg.eth_id = port->id;

> +	msg.byte5 = port->plat->rxq | 0x80;

> +	msg.byte7 = port->plat->rxq << 4;

> +	for (i = 0; i < 8; i++) {

> +		msg.byte3 = i;

> +		if (npe_send_recv_message(port->npe, &msg, "ETH_SET_RXQ"))

> +			return -EIO;

> +	}

> +

> +	msg.cmd = NPE_EDB_SETPORTADDRESS;

> +	msg.eth_id = PHYSICAL_ID(port);

> +	memcpy(msg.mac, dev->dev_addr, ETH_ALEN);

> +	if (npe_send_recv_message(port->npe, &msg, "ETH_SET_MAC"))

> +		return -EIO;

> +

> +	memset(&msg, 0, sizeof(msg));

> +	msg.cmd = NPE_FW_SETFIREWALLMODE;

> +	msg.eth_id = port->id;

> +	if (npe_send_recv_message(port->npe, &msg, "ETH_SET_FIREWALL_MODE"))

> +		return -EIO;

> +

> +	if ((err = request_queues(port)) != 0)

> +		return err;

> +

> +	if ((err = init_queues(port)) != 0) {

> +		destroy_queues(port);

> +		release_queues(port);

> +		return err;

> +	}

> +

> +	for (i = 0; i < ETH_ALEN; i++)

> +		__raw_writel(dev->dev_addr[i], &port->regs->hw_addr[i]);

> +	__raw_writel(0x08, &port->regs->random_seed);

> +	__raw_writel(0x12, &port->regs->partial_empty_threshold);

> +	__raw_writel(0x30, &port->regs->partial_full_threshold);

> +	__raw_writel(0x08, &port->regs->tx_start_bytes);

> +	__raw_writel(0x15, &port->regs->tx_deferral);

> +	__raw_writel(0x08, &port->regs->tx_2part_deferral[0]);

> +	__raw_writel(0x07, &port->regs->tx_2part_deferral[1]);

> +	__raw_writel(0x80, &port->regs->slot_time);

> +	__raw_writel(0x01, &port->regs->int_clock_threshold);

> +	__raw_writel(TX_CNTRL1_RETRIES, &port->regs->tx_control[1]);

> +	__raw_writel(TX_CNTRL0_TX_EN | TX_CNTRL0_RETRY | TX_CNTRL0_PAD_EN |

> +		     TX_CNTRL0_APPEND_FCS | TX_CNTRL0_2DEFER,

> +		     &port->regs->tx_control[0]);

> +	__raw_writel(0, &port->regs->rx_control[1]);

> +	__raw_writel(RX_CNTRL0_RX_EN | RX_CNTRL0_PADSTRIP_EN,

> +		     &port->regs->rx_control[0]);

> +

> +	if (mii_check_media(&port->mii, 1, 1))

> +		eth_set_duplex(port);

> +	eth_set_mcast_list(dev);

> +	netif_start_queue(dev);

> +	schedule_delayed_work(&port->mdio_thread, MDIO_INTERVAL);

> +

> +	qmgr_set_irq(port->plat->rxq, QUEUE_IRQ_SRC_NOT_EMPTY,

> +		     eth_rx_irq, dev);

> +	qmgr_set_irq(TX_QUEUE(port->plat), QUEUE_IRQ_SRC_NOT_FULL,

> +		     eth_xmit_ready_irq, dev);

> +	qmgr_enable_irq(port->plat->rxq);

> +	qmgr_enable_irq(TX_QUEUE(port->plat));

> +	ports_open++;

> +	return 0;

> +}

> +

> +static int eth_close(struct net_device *dev)

> +{

> +	struct port *port = netdev_priv(dev);

> +

> +	ports_open--;

> +	qmgr_disable_irq(port->plat->rxq);

> +	qmgr_disable_irq(TX_QUEUE(port->plat));

> +	netif_stop_queue(dev);

> +

> +	clr_regbits(RX_CNTRL0_RX_EN, &port->regs->rx_control[0]);

> +	clr_regbits(TX_CNTRL0_TX_EN, &port->regs->tx_control[0]);

> +	set_regbits(CORE_RESET | CORE_RX_FIFO_FLUSH | CORE_TX_FIFO_FLUSH,

> +		    &port->regs->core_control);

> +	udelay(10);

> +	clr_regbits(CORE_RESET | CORE_RX_FIFO_FLUSH | CORE_TX_FIFO_FLUSH,

> +		    &port->regs->core_control);

> +

> +	cancel_rearming_delayed_work(&port->mdio_thread);

> +	destroy_queues(port);

> +	release_queues(port);

> +	return 0;

> +}

> +

> +static int __devinit eth_init_one(struct platform_device *pdev)

> +{

> +	struct port *port;

> +	struct net_device *dev;

> +	struct mac_plat_info *plat = pdev->dev.platform_data;

> +	u32 regs_phys;

> +	int err;

> +

> +	if (!(dev = alloc_etherdev(sizeof(struct port))))

> +		return -ENOMEM;

> +

> +	SET_MODULE_OWNER(dev);

> +	SET_NETDEV_DEV(dev, &pdev->dev);

> +	port = netdev_priv(dev);

> +	port->netdev = dev;

> +	port->id = pdev->id;

> +

> +	switch (port->id) {

> +	case IXP4XX_ETH_NPEA:

> +		port->regs = (struct eth_regs __iomem *)IXP4XX_EthA_BASE_VIRT;

> +		regs_phys  = IXP4XX_EthA_BASE_PHYS;

> +		break;

> +	case IXP4XX_ETH_NPEB:

> +		port->regs = (struct eth_regs __iomem *)IXP4XX_EthB_BASE_VIRT;

> +		regs_phys  = IXP4XX_EthB_BASE_PHYS;

> +		break;

> +	case IXP4XX_ETH_NPEC:

> +		port->regs = (struct eth_regs __iomem *)IXP4XX_EthC_BASE_VIRT;

> +		regs_phys  = IXP4XX_EthC_BASE_PHYS;

> +		break;

> +	default:

> +		err = -ENOSYS;

> +		goto err_free;

> +	}

> +

> +	dev->open = eth_open;

> +	dev->hard_start_xmit = eth_xmit;

> +	dev->poll = eth_poll;

> +	dev->stop = eth_close;

> +	dev->get_stats = eth_stats;

> +	dev->do_ioctl = eth_ioctl;

> +	dev->set_multicast_list = eth_set_mcast_list;

> +	dev->weight = 16;

> +	dev->tx_queue_len = 100;

> +

> +	if (!(port->npe = npe_request(NPE_ID(port)))) {

> +		err = -EIO;

> +		goto err_free;

> +	}

> +

> +	if (register_netdev(dev)) {

> +		err = -EIO;

> +		goto err_npe_rel;

> +	}

> +

> +	port->mem_res = request_mem_region(regs_phys, REGS_SIZE, dev->name);

> +	if (!port->mem_res) {

> +		err = -EBUSY;

> +		goto err_unreg;

> +	}

> +

> +	port->plat = plat;

> +	memcpy(dev->dev_addr, plat->hwaddr, ETH_ALEN);

I think my comment about adding randomised MAC addresses in case of

no hwaddr stands - it's really not that complex.
Christian's driver did this:
	/* The place of the MAC address is very system dependent.

	 * Here we use a random one to be replaced by one of the

	 * following commands:

	 * "ip link set address 02:03:04:04:04:01 dev eth0"

	 * "ifconfig eth0 hw ether 02:03:04:04:04:07"

	*/
	if (is_zero_ether_addr(plat->hwaddr)) {

		random_ether_addr(dev->dev_addr);

		dev->dev_addr[5] = plat->phy_id;

	}

	else

		memcpy(dev->dev_addr, plat->hwaddr, 6);
> +

quoted text
> +	platform_set_drvdata(pdev, dev);

> +

> +	__raw_writel(CORE_RESET, &port->regs->core_control);

> +	udelay(50);

> +	__raw_writel(CORE_MDC_EN, &port->regs->core_control);

> +	udelay(50);

> +

> +	port->mii.dev = dev;

> +	port->mii.mdio_read = mdio_read;

> +	port->mii.mdio_write = mdio_write;

> +	port->mii.phy_id = plat->phy;

> +	port->mii.phy_id_mask = 0x1F;

> +	port->mii.reg_num_mask = 0x1F;

> +

> +	INIT_DELAYED_WORK(&port->mdio_thread, mdio_thread);

> +

> +	printk(KERN_INFO "%s: MII PHY %i on %s\n", dev->name, plat->phy,

> +	       npe_name(port->npe));

> +	return 0;

> +

> +err_unreg:

> +	unregister_netdev(dev);

> +err_npe_rel:

> +	npe_release(port->npe);

> +err_free:

> +	free_netdev(dev);

> +	return err;

> +}

> +

> +static int __devexit eth_remove_one(struct platform_device *pdev)

> +{

> +	struct net_device *dev = platform_get_drvdata(pdev);

> +	struct port *port = netdev_priv(dev);

> +

> +	unregister_netdev(dev);

> +	platform_set_drvdata(pdev, NULL);

> +	npe_release(port->npe);

> +	release_resource(port->mem_res);

> +	free_netdev(dev);

> +	return 0;

> +}

> +

> +static struct platform_driver drv = {

> +	.driver.name	= DRV_NAME,

> +	.probe		= eth_init_one,

> +	.remove		= eth_remove_one,

> +};

> +

> +static int __init eth_init_module(void)

> +{

> +	if (!(ixp4xx_read_fuses() & IXP4XX_FUSE_NPEB_ETH0))

> +		return -ENOSYS;

> +

> +	/* All MII PHY accesses use NPE-B Ethernet registers */

> +	if (!(mdio_npe = npe_request(1)))

> +		return -EIO;

> +	spin_lock_init(&mdio_lock);

> +	mdio_regs = (struct eth_regs __iomem *)IXP4XX_EthB_BASE_VIRT;

> +

> +	return platform_driver_register(&drv);

> +}

> +

> +static void __exit eth_cleanup_module(void)

> +{

> +	platform_driver_unregister(&drv);

> +	npe_release(mdio_npe);

> +}

> +

> +MODULE_AUTHOR("Krzysztof Halasa");

> +MODULE_DESCRIPTION("Intel IXP4xx Ethernet driver");

> +MODULE_LICENSE("GPL v2");

> +module_init(eth_init_module);

For our flash and eeprom notifiers to work, we need this converted to

a late_initcall:

http://trac.nslu2-linux.org/kernel/browser/trunk/patches/2.6.21/37-

ixp4xx-net-driver-fix-mac-handling.patch
akpm suggested this fix, but we don't absolutely know if it's

upstream acceptable.
> +module_exit(eth_cleanup_module);

quoted text
> diff --git a/drivers/net/wan/Kconfig b/drivers/net/wan/Kconfig

> index 5f79622..b891e10 100644

> --- a/drivers/net/wan/Kconfig

> +++ b/drivers/net/wan/Kconfig

> @@ -342,6 +342,16 @@ config DSCC4_PCI_RST

>

>  	  Say Y if your card supports this feature.

>

> +config IXP4XX_HSS

> +	tristate "IXP4xx HSS (synchronous serial port) support"

> +	depends on ARM && ARCH_IXP4XX

> +	select IXP4XX_NPE

> +	select IXP4XX_QMGR

> +	select HDLC

> +	help

> +	  Say Y here if you want to use built-in HSS ports

> +	  on IXP4xx processor.

> +

>  config DLCI

>  	tristate "Frame Relay DLCI support"

>  	---help---

> diff --git a/drivers/net/wan/Makefile b/drivers/net/wan/Makefile

> index d61fef3..1b1d116 100644

> --- a/drivers/net/wan/Makefile

> +++ b/drivers/net/wan/Makefile

> @@ -42,6 +42,7 @@ obj-$(CONFIG_C101)		+= c101.o

>  obj-$(CONFIG_WANXL)		+= wanxl.o

>  obj-$(CONFIG_PCI200SYN)		+= pci200syn.o

>  obj-$(CONFIG_PC300TOO)		+= pc300too.o

> +obj-$(CONFIG_IXP4XX_HSS)	+= ixp4xx_hss.o

>

>  clean-files := wanxlfw.inc

>  $(obj)/wanxl.o:	$(obj)/wanxlfw.inc

> diff --git a/drivers/net/wan/ixp4xx_hss.c b/drivers/net/wan/

> ixp4xx_hss.c

> new file mode 100644

> index 0000000..ed56ed8

> --- /dev/null

> +++ b/drivers/net/wan/ixp4xx_hss.c

> @@ -0,0 +1,1048 @@

> +/*

> + * Intel IXP4xx HSS (synchronous serial port) driver for Linux

> + *

> + * Copyright (C) 2007 Krzysztof Halasa

> + *

> + * This program is free software; you can redistribute it and/or

> modify it

> + * under the terms of version 2 of the GNU General Public License

> + * as published by the Free Software Foundation.

> + */

> +

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +#include

> +

> +#ifndef __ARMEB__

> +#warning Little endian mode not supported

> +#endif

Personally I'm less fussed about WAN / LE support. Anyone with any

sense will run ixp4xx boards doing such a specialised network

operation as BE. Also, NSLU2-Linux can't test this functionality with

our LE setup as we don't have this hardware on-board. You may just

want to declare a depends on ARMEB in Kconfig (with or without OR

(ARM || BROKEN) ) and have done with it - it's up to you.
> +

quoted text
> +#define DEBUG_QUEUES		0

> +#define DEBUG_RX		0

> +#define DEBUG_TX		0

> +

> +#define DRV_NAME		"ixp4xx_hss"

> +#define DRV_VERSION		"0.03"

> +

> +#define PKT_EXTRA_FLAGS		0 /* orig 1 */

> +#define FRAME_SYNC_OFFSET	0 /* unused, channelized only */

> +#define FRAME_SYNC_SIZE		1024

> +#define PKT_NUM_PIPES		1 /* 1, 2 or 4 */

> +#define PKT_PIPE_FIFO_SIZEW	4 /* total 4 dwords per HSS */

> +

> +#define RX_DESCS		16 /* also length of queues: RX-ready, RX */

> +#define TX_DESCS		16 /* also length of queues: TX-done, TX */

> +

> +#define POOL_ALLOC_SIZE		(sizeof(struct desc) * (RX_DESCS +

> TX_DESCS))

> +#define RX_SIZE			(HDLC_MAX_MRU + 4) /* NPE needs more space */

> +

> +/* Queue IDs */

> +#define HSS0_CHL_RXTRIG_QUEUE	12	/* orig size = 32 dwords */

> +#define HSS0_PKT_RX_QUEUE	13	/* orig size = 32 dwords */

> +#define HSS0_PKT_TX0_QUEUE	14	/* orig size = 16 dwords */

> +#define HSS0_PKT_TX1_QUEUE	15

> +#define HSS0_PKT_TX2_QUEUE	16

> +#define HSS0_PKT_TX3_QUEUE	17

> +#define HSS0_PKT_RXFREE0_QUEUE	18	/* orig size = 16 dwords */

> +#define HSS0_PKT_RXFREE1_QUEUE	19

> +#define HSS0_PKT_RXFREE2_QUEUE	20

> +#define HSS0_PKT_RXFREE3_QUEUE	21

> +#define HSS0_PKT_TXDONE_QUEUE	22	/* orig size = 64 dwords */

> +

> +#define HSS1_CHL_RXTRIG_QUEUE	10

> +#define HSS1_PKT_RX_QUEUE	0

> +#define HSS1_PKT_TX0_QUEUE	5

> +#define HSS1_PKT_TX1_QUEUE	6

> +#define HSS1_PKT_TX2_QUEUE	7

> +#define HSS1_PKT_TX3_QUEUE	8

> +#define HSS1_PKT_RXFREE0_QUEUE	1

> +#define HSS1_PKT_RXFREE1_QUEUE	2

> +#define HSS1_PKT_RXFREE2_QUEUE	3

> +#define HSS1_PKT_RXFREE3_QUEUE	4

> +#define HSS1_PKT_TXDONE_QUEUE	9

> +

> +#define NPE_PKT_MODE_HDLC		0

> +#define NPE_PKT_MODE_RAW		1

> +#define NPE_PKT_MODE_56KMODE		2

> +#define NPE_PKT_MODE_56KENDIAN_MSB	4

> +

> +/* PKT_PIPE_HDLC_CFG_WRITE flags */

> +#define PKT_HDLC_IDLE_ONES		0x1 /* default = flags */

> +#define PKT_HDLC_CRC_32			0x2 /* default = CRC-16 */

> +#define PKT_HDLC_MSB_ENDIAN		0x4 /* default = LE */

> +

> +

> +/* hss_config, PCRs */

> +/* Frame sync sampling, default = active low */

> +#define PCR_FRM_SYNC_ACTIVE_HIGH	0x40000000

> +#define PCR_FRM_SYNC_FALLINGEDGE	0x80000000

> +#define PCR_FRM_SYNC_RISINGEDGE		0xC0000000

> +

> +/* Frame sync pin: input (default) or output generated off a given

> clk edge */

> +#define PCR_FRM_SYNC_OUTPUT_FALLING	0x20000000

> +#define PCR_FRM_SYNC_OUTPUT_RISING	0x30000000

> +

> +/* Frame and data clock sampling on edge, default = falling */

> +#define PCR_FCLK_EDGE_RISING		0x08000000

> +#define PCR_DCLK_EDGE_RISING		0x04000000

> +

> +/* Clock direction, default = input */

> +#define PCR_SYNC_CLK_DIR_OUTPUT		0x02000000

> +

> +/* Generate/Receive frame pulses, default = enabled */

> +#define PCR_FRM_PULSE_DISABLED		0x01000000

> +

> + /* Data rate is full (default) or half the configured clk speed */

> +#define PCR_HALF_CLK_RATE		0x00200000

> +

> +/* Invert data between NPE and HSS FIFOs? (default = no) */

> +#define PCR_DATA_POLARITY_INVERT	0x00100000

> +

> +/* TX/RX endianness, default = LSB */

> +#define PCR_MSB_ENDIAN			0x00080000

> +

> +/* Normal (default) / open drain mode (TX only) */

> +#define PCR_TX_PINS_OPEN_DRAIN		0x00040000

> +

> +/* No framing bit transmitted and expected on RX? (default =

> framing bit) */

> +#define PCR_SOF_NO_FBIT			0x00020000

> +

> +/* Drive data pins? */

> +#define PCR_TX_DATA_ENABLE		0x00010000

> +

> +/* Voice 56k type: drive the data pins low (default), high, high Z */

> +#define PCR_TX_V56K_HIGH		0x00002000

> +#define PCR_TX_V56K_HIGH_IMP		0x00004000

> +

> +/* Unassigned type: drive the data pins low (default), high, high

> Z */

> +#define PCR_TX_UNASS_HIGH		0x00000800

> +#define PCR_TX_UNASS_HIGH_IMP		0x00001000

> +

> +/* T1 @ 1.544MHz only: Fbit dictated in FIFO (default) or high Z */

> +#define PCR_TX_FB_HIGH_IMP		0x00000400

> +

> +/* 56k data endiannes - which bit unused: high (default) or low */

> +#define PCR_TX_56KE_BIT_0_UNUSED	0x00000200

> +

> +/* 56k data transmission type: 32/8 bit data (default) or 56K data */

> +#define PCR_TX_56KS_56K_DATA		0x00000100

> +

> +/* hss_config, cCR */

> +/* Number of packetized clients, default = 1 */

> +#define CCR_NPE_HFIFO_2_HDLC		0x04000000

> +#define CCR_NPE_HFIFO_3_OR_4HDLC	0x08000000

> +

> +/* default = no loopback */

> +#define CCR_LOOPBACK			0x02000000

> +

> +/* HSS number, default = 0 (first) */

> +#define CCR_SECOND_HSS			0x01000000

> +

> +

> +/* hss_config, clkCR: main:10, num:10, denom:12 */

> +#define CLK42X_SPEED_EXP	((0x3FF << 22) | (  2 << 12) |   15) /*65

> KHz*/

> +

> +#define CLK42X_SPEED_512KHZ	((  130 << 22) | (  2 << 12) |   15)

> +#define CLK42X_SPEED_1536KHZ	((   43 << 22) | ( 18 << 12) |   47)

> +#define CLK42X_SPEED_1544KHZ	((   43 << 22) | ( 33 << 12) |  192)

> +#define CLK42X_SPEED_2048KHZ	((   32 << 22) | ( 34 << 12) |   63)

> +#define CLK42X_SPEED_4096KHZ	((   16 << 22) | ( 34 << 12) |  127)

> +#define CLK42X_SPEED_8192KHZ	((    8 << 22) | ( 34 << 12) |  255)

> +

> +#define CLK46X_SPEED_512KHZ	((  130 << 22) | ( 24 << 12) |  127)

> +#define CLK46X_SPEED_1536KHZ	((   43 << 22) | (152 << 12) |  383)

> +#define CLK46X_SPEED_1544KHZ	((   43 << 22) | ( 66 << 12) |  385)

> +#define CLK46X_SPEED_2048KHZ	((   32 << 22) | (280 << 12) |  511)

> +#define CLK46X_SPEED_4096KHZ	((   16 << 22) | (280 << 12) | 1023)

> +#define CLK46X_SPEED_8192KHZ	((    8 << 22) | (280 << 12) | 2047)

> +

> +

> +/* hss_config, LUTs: default = unassigned */

> +#define TDMMAP_HDLC		1	/* HDLC - packetised */

> +#define TDMMAP_VOICE56K		2	/* Voice56K - channelised */

> +#define TDMMAP_VOICE64K		3	/* Voice64K - channelised */

> +

> +

> +/* NPE command codes */

> +/* writes the ConfigWord value to the location specified by offset */

> +#define PORT_CONFIG_WRITE			0x40

> +

> +/* triggers the NPE to load the contents of the configuration

> table */

> +#define PORT_CONFIG_LOAD			0x41

> +

> +/* triggers the NPE to return an HssErrorReadResponse message */

> +#define PORT_ERROR_READ				0x42

> +

> +/* reset NPE internal status and enable the HssChannelized

> operation */

> +#define CHAN_FLOW_ENABLE			0x43

> +#define CHAN_FLOW_DISABLE			0x44

> +#define CHAN_IDLE_PATTERN_WRITE			0x45

> +#define CHAN_NUM_CHANS_WRITE			0x46

> +#define CHAN_RX_BUF_ADDR_WRITE			0x47

> +#define CHAN_RX_BUF_CFG_WRITE			0x48

> +#define CHAN_TX_BLK_CFG_WRITE			0x49

> +#define CHAN_TX_BUF_ADDR_WRITE			0x4A

> +#define CHAN_TX_BUF_SIZE_WRITE			0x4B

> +#define CHAN_TSLOTSWITCH_ENABLE			0x4C

> +#define CHAN_TSLOTSWITCH_DISABLE		0x4D

> +

> +/* downloads the gainWord value for a timeslot switching channel

> associated

> +   with bypassNum */

> +#define CHAN_TSLOTSWITCH_GCT_DOWNLOAD		0x4E

> +

> +/* triggers the NPE to reset internal status and enable the

> HssPacketized

> +   operation for the flow specified by pPipe */

Greater-than-one-line comments not conforming to Kernel coding style

- someone much more angry than me will jump on that.
> +#define PKT_PIPE_FLOW_ENABLE			0x50

quoted text
> +#define PKT_PIPE_FLOW_DISABLE			0x51

> +#define PKT_NUM_PIPES_WRITE			0x52

> +#define PKT_PIPE_FIFO_SIZEW_WRITE		0x53

> +#define PKT_PIPE_HDLC_CFG_WRITE			0x54

> +#define PKT_PIPE_IDLE_PATTERN_WRITE		0x55

> +#define PKT_PIPE_RX_SIZE_WRITE			0x56

> +#define PKT_PIPE_MODE_WRITE			0x57

> +

> +

Lots of double returns.
> +#define HSS_TIMESLOTS		128

quoted text
> +#define HSS_LUT_BITS		2

> +

> +/* HDLC packet status values - desc->status */

> +#define ERR_SHUTDOWN		1 /* stop or shutdown occurrance */

> +#define ERR_HDLC_ALIGN		2 /* HDLC alignment error */

> +#define ERR_HDLC_FCS		3 /* HDLC Frame Check Sum error */

> +#define ERR_RXFREE_Q_EMPTY	4 /* RX-free queue became empty while

> receiving

> +				     this packet (if buf_len < pkt_len) */

> +#define ERR_HDLC_TOO_LONG	5 /* HDLC frame size too long */

> +#define ERR_HDLC_ABORT		6 /* abort sequence received */

> +#define ERR_DISCONNECTING	7 /* disconnect is in progress */

> +

> +

> +struct port {

> +	struct npe *npe;

> +	struct net_device *netdev;

> +	struct hss_plat_info *plat;

> +	struct sk_buff *rx_skb_tab[RX_DESCS], *tx_skb_tab[TX_DESCS];

> +	struct desc *desc_tab;	/* coherent */

> +	u32 desc_tab_phys;

> +	sync_serial_settings settings;

> +	int id;

> +	u8 hdlc_cfg;

> +};

> +

[snip]
Again, looking good.
Michael-Luke Jones
-

unsubscribe notice
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in

the body of a message to majordomo@vger.kernel.org

More majordomo info at  http://vger.kernel.org/majordomo-info.html

Please read the FAQ at  http://www.tux.org/lkml/