Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-11-01 Thread tnt
> + while (bcom_buffer_done(priv->tx_dmatsk)) { > + struct sk_buff *skb; > + skb = bcom_retrieve_buffer(priv->tx_dmatsk, NULL, NULL); > + /* Here (and in rx routines) would be a good place for > + * dma_unmap_single(), but bcom doesn't return bco

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Stephen Hemminger
On Thu, 25 Oct 2007 13:29:08 -0700 "Dale Farnsworth" <[EMAIL PROTECTED]> wrote: > On Thu, Oct 25, 2007 at 09:41:14PM +0200, Domen Puncer wrote: > > On 25/10/07 11:57 -0700, Dale Farnsworth wrote: > > > Domen wrote: > > > > > use your platform's dma mapping functions, rather than virt_to_phys() > >

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Jeff Garzik
Dale Farnsworth wrote: > IMO, it's still a requirement that we call dma_unmap_single() for > each call to dma_map_single(). Yep... ___ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Dale Farnsworth
On Thu, Oct 25, 2007 at 09:41:14PM +0200, Domen Puncer wrote: > On 25/10/07 11:57 -0700, Dale Farnsworth wrote: > > Domen wrote: > > > > use your platform's dma mapping functions, rather than virt_to_phys() > > > > > > > > it might be the exact same implementation, inside the platform > > > > int

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Domen Puncer
On 25/10/07 11:57 -0700, Dale Farnsworth wrote: > Domen wrote: > > > use your platform's dma mapping functions, rather than virt_to_phys() > > > > > > it might be the exact same implementation, inside the platform > > > internals, but drivers should not be using this directly. > > > > I've repla

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Dale Farnsworth
Domen wrote: > > use your platform's dma mapping functions, rather than virt_to_phys() > > > > it might be the exact same implementation, inside the platform > > internals, but drivers should not be using this directly. > > I've replaced this with dma_map_single(), unmatched with > dma_unmap_sin

Re: [PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-25 Thread Jeff Garzik
Domen Puncer wrote: > +static int mpc52xx_fec_alloc_rx_buffers(struct bcom_task *rxtsk) > +{ > + while (!bcom_queue_full(rxtsk)) { > + struct sk_buff *skb; > + struct bcom_fec_bd *bd; > + > + skb = dev_alloc_skb(FEC_RX_BUFFER_SIZE); > + if (skb ==

[PATCH v4] FEC - fast ethernet controller for mpc52xx

2007-10-19 Thread Domen Puncer
Driver for ethernet on mpc5200/mpc5200b SoCs (FEC). Signed-off-by: Domen Puncer <[EMAIL PROTECTED]> --- On 18/10/07 15:14 -0400, Jeff Garzik wrote: > > except a resend combining patches 3 and 4 as requested :) OK, here it goes. Sorry for the delay. diffstat: drivers/net/Kconfig |