diff options
author | Sascha Hauer <s.hauer@pengutronix.de> | 2016-06-27 09:12:53 +0200 |
---|---|---|
committer | Sascha Hauer <s.hauer@pengutronix.de> | 2016-06-28 07:27:08 +0200 |
commit | 1ebb663db9acc9e011fe2dbc071b0ef5ca02c7d8 (patch) | |
tree | 7a9853149774f597c9b92848cd8e5263d346d762 /drivers/net/dm9k.c | |
parent | 8ef4478c3751c14a35cc99e619184baa11a641c8 (diff) | |
download | barebox-1ebb663db9acc9e011fe2dbc071b0ef5ca02c7d8.tar.gz barebox-1ebb663db9acc9e011fe2dbc071b0ef5ca02c7d8.tar.xz |
net: dm9k: Fix buswidth setting for platform data probe
priv->buswidth expects IORESOURCE_MEM_* macros which are not
identical to the integer byte bus width, so calling
dm9000_setup_buswidth() for the platform_data case is wrong.
fixes: d818f02 net: dm9k: add device tree support
Signed-off-by: Sascha Hauer <s.hauer@pengutronix.de>
Reported by: 张忠山 <zzs213@126.com>
Diffstat (limited to 'drivers/net/dm9k.c')
-rw-r--r-- | drivers/net/dm9k.c | 42 |
1 files changed, 18 insertions, 24 deletions
diff --git a/drivers/net/dm9k.c b/drivers/net/dm9k.c index acc0abbc48..25f08602f6 100644 --- a/drivers/net/dm9k.c +++ b/drivers/net/dm9k.c @@ -729,26 +729,6 @@ static int dm9k_init_dev(struct eth_device *edev) return 0; } -static int dm9000_setup_buswidth(struct device_d *dev, struct dm9k *priv, uint32_t width) -{ - switch (width) { - case 1: - priv->buswidth = IORESOURCE_MEM_8BIT; - break; - case 2: - priv->buswidth = IORESOURCE_MEM_16BIT; - break; - case 4: - priv->buswidth = IORESOURCE_MEM_32BIT; - break; - default: - dev_err(dev, "Wrong io resource size\n"); - return -EINVAL; - } - - return 0; -} - static int dm9000_parse_dt(struct device_d *dev, struct dm9k *priv) { struct device_node *np = dev->device_node; @@ -768,19 +748,33 @@ static int dm9000_parse_dt(struct device_d *dev, struct dm9k *priv) prop = 1; } - return dm9000_setup_buswidth(dev, priv, prop); + switch (prop) { + case 1: + priv->buswidth = IORESOURCE_MEM_8BIT; + break; + case 2: + priv->buswidth = IORESOURCE_MEM_16BIT; + break; + case 4: + priv->buswidth = IORESOURCE_MEM_32BIT; + break; + default: + dev_err(dev, "Wrong io resource size\n"); + return -EINVAL; + } + + return 0; } static int dm9000_parse_pdata(struct device_d *dev, struct dm9k *priv) { struct dm9000_platform_data *pdata = dev->platform_data; - uint32_t width; priv->srom = pdata->srom; - width = dev->resource[0].flags & IORESOURCE_MEM_TYPE_MASK; + priv->buswidth = dev->resource[0].flags & IORESOURCE_MEM_TYPE_MASK; - return dm9000_setup_buswidth(dev, priv, width); + return 0; } static int dm9k_probe(struct device_d *dev) |