On 25 April 2015 at 05:51, Hans de Goede <hdego...@redhat.com> wrote:
> Besides being spelled wron, the DMA_CPU_TRRESHOLD define actually has
:-)
> nothing to do with DMA as we only use mmio fifo access. Rename it to
> EMAC_RX_BUFSIZE to properly reflect what it does.
>
> Signed-off-by: Hans de Goede <hdego...@redhat.com>
> ---
> drivers/net/sunxi_emac.c | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
Reviewed-by: Simon Glass <s...@chromium.org>
>
> diff --git a/drivers/net/sunxi_emac.c b/drivers/net/sunxi_emac.c
> index 038f474..e43b1e7 100644
> --- a/drivers/net/sunxi_emac.c
> +++ b/drivers/net/sunxi_emac.c
> @@ -153,7 +153,7 @@ struct sunxi_sramc_regs {
> #define EMAC_CRCERR (0x1 << 4)
> #define EMAC_LENERR (0x3 << 5)
>
> -#define DMA_CPU_TRRESHOLD 2000
> +#define EMAC_RX_BUFSIZE 2000
>
> struct emac_eth_dev {
> struct emac_regs *regs;
> @@ -455,7 +455,7 @@ static int _sunxi_emac_eth_recv(struct emac_eth_dev
> *priv, void *packet)
>
> /* Move data from EMAC */
> if (good_packet) {
> - if (rx_len > DMA_CPU_TRRESHOLD) {
> + if (rx_len > EMAC_RX_BUFSIZE) {
> printf("Received packet is too big (len=%d)\n",
> rx_len);
> return -EMSGSIZE;
> }
> --
> 2.3.5
>
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
