On Sat, 2014-03-22 at 20:22 +0100, Hans de Goede wrote:
> One thing that stands out when doing a diff against sunxi-merge-v2014.04-rc2
> is this:
> 
> --- u-boot/drivers/net/designware.c     2014-03-21 16:26:42.229522420 +0100
> +++ u-boot-sunxi/drivers/net/designware.c       2014-03-22 17:21:51.907879720 
> +
> @@ -249,10 +249,10 @@
>         rx_descs_init(dev);
>         tx_descs_init(dev);
> 
> -       writel(FIXEDBURST | PRIORXTX_41 | BURST_16, &dma_p->busmode);
> +       writel(FIXEDBURST | PRIORXTX_41 | BURST_8, &dma_p->busmode);
> 
> -       writel(readl(&dma_p->opmode) | FLUSHTXFIFO | STOREFORWARD,
> -              &dma_p->opmode);
> +       writel(readl(&dma_p->opmode) | FLUSHTXFIFO | STOREFORWARD |
> +               TXSECONDFRAME, &dma_p->opmode);
> 
>         writel(readl(&dma_p->opmode) | RXSTART | TXSTART, &dma_p->opmode);
> 
> 
> Which we should probably fix given that you've enabled gmac support.

Things work for me without. I was under the impression that Wens (or
someone else) had this under control WRT upstreaming so I was leaving it
for now, it can easily go in as an independent fix AFAICT.

> I also noticed that in include/configs/sunxi-common.h you've a formatting
> fix for the CONFIG_PHY_ADDR #define which is not yet in u-boot-sunxi.

CONFIG_PHY_ADDR arrived as part of the merge of v2014.04-rc2, so the fix
got folded into the merge when I redid it.

I might manage a v3, which will be a couple of cleanups and a lot of
clarifications about licensing, today, or I might watch game of thrones
and do v3 later in the week, lets see how it goes...

Ian.

-- 
You received this message because you are subscribed to the Google Groups 
"linux-sunxi" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to linux-sunxi+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to