Re: [U-Boot] [PATCH 11/12] sh_eth: Fix return value of sh_eth_initialize()

On Sunday 24 January 2010 05:07:59 Matthias Kaehlcke wrote:

--- a/drivers/net/sh_eth.c +++ b/drivers/net/sh_eth.c eth = (struct sh_eth_dev *)malloc(sizeof(struct sh_eth_dev)); if (!eth) { printf(SHETHER_NAME ": %s: malloc failed\n", __func__);

• ret = -ENOMEM;
  

• ret = 0;
  

  goto err; }

  dev = (struct eth_device *)malloc(sizeof(struct eth_device)); if (!dev) { printf(SHETHER_NAME ": %s: malloc failed\n", __func__);

• ret = -ENOMEM;
  

• ret = 0;
  

  goto err; }

these are error paths and thus should be -1 -mike