Skip to content

Commit

Permalink
[POWERPC] netdevices: Constify & voidify get_property()
Browse files Browse the repository at this point in the history
Now that get_property() returns a void *, there's no need to cast its
return value. Also, treat the return value as const, so we can
constify get_property later.

powerpc-specific network device driver changes.

Signed-off-by: Jeremy Kerr <jk@ozlabs.org>
Signed-off-by: Paul Mackerras <paulus@samba.org>
  • Loading branch information
Jeremy Kerr authored and Paul Mackerras committed Jul 31, 2006
1 parent 294ef16 commit 1a2509c
Show file tree
Hide file tree
Showing 4 changed files with 16 additions and 13 deletions.
13 changes: 8 additions & 5 deletions drivers/net/bmac.c
Original file line number Diff line number Diff line change
Expand Up @@ -1264,22 +1264,25 @@ static int __devinit bmac_probe(struct macio_dev *mdev, const struct of_device_i
{
int j, rev, ret;
struct bmac_data *bp;
unsigned char *addr;
const unsigned char *prop_addr;
unsigned char addr[6];
struct net_device *dev;
int is_bmac_plus = ((int)match->data) != 0;

if (macio_resource_count(mdev) != 3 || macio_irq_count(mdev) != 3) {
printk(KERN_ERR "BMAC: can't use, need 3 addrs and 3 intrs\n");
return -ENODEV;
}
addr = get_property(macio_get_of_node(mdev), "mac-address", NULL);
if (addr == NULL) {
addr = get_property(macio_get_of_node(mdev), "local-mac-address", NULL);
if (addr == NULL) {
prop_addr = get_property(macio_get_of_node(mdev), "mac-address", NULL);
if (prop_addr == NULL) {
prop_addr = get_property(macio_get_of_node(mdev),
"local-mac-address", NULL);
if (prop_addr == NULL) {
printk(KERN_ERR "BMAC: Can't get mac-address\n");
return -ENODEV;
}
}
memcpy(addr, prop_addr, sizeof(addr));

dev = alloc_etherdev(PRIV_BYTES);
if (!dev) {
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/mace.c
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,7 @@ static int __devinit mace_probe(struct macio_dev *mdev, const struct of_device_i
struct device_node *mace = macio_get_of_node(mdev);
struct net_device *dev;
struct mace_data *mp;
unsigned char *addr;
const unsigned char *addr;
int j, rev, rc = -EBUSY;

if (macio_resource_count(mdev) != 3 || macio_irq_count(mdev) != 3) {
Expand Down
12 changes: 6 additions & 6 deletions drivers/net/spider_net.c
Original file line number Diff line number Diff line change
Expand Up @@ -1812,10 +1812,10 @@ spider_net_setup_phy(struct spider_net_card *card)
*/
static int
spider_net_download_firmware(struct spider_net_card *card,
u8 *firmware_ptr)
const void *firmware_ptr)
{
int sequencer, i;
u32 *fw_ptr = (u32 *)firmware_ptr;
const u32 *fw_ptr = firmware_ptr;

/* stop sequencers */
spider_net_write_reg(card, SPIDER_NET_GSINIT,
Expand Down Expand Up @@ -1872,7 +1872,7 @@ spider_net_init_firmware(struct spider_net_card *card)
{
struct firmware *firmware = NULL;
struct device_node *dn;
u8 *fw_prop = NULL;
const u8 *fw_prop = NULL;
int err = -ENOENT;
int fw_size;

Expand All @@ -1898,7 +1898,7 @@ spider_net_init_firmware(struct spider_net_card *card)
if (!dn)
goto out_err;

fw_prop = (u8 *)get_property(dn, "firmware", &fw_size);
fw_prop = get_property(dn, "firmware", &fw_size);
if (!fw_prop)
goto out_err;

Expand Down Expand Up @@ -2058,7 +2058,7 @@ spider_net_setup_netdev(struct spider_net_card *card)
struct net_device *netdev = card->netdev;
struct device_node *dn;
struct sockaddr addr;
u8 *mac;
const u8 *mac;

SET_MODULE_OWNER(netdev);
SET_NETDEV_DEV(netdev, &card->pdev->dev);
Expand Down Expand Up @@ -2089,7 +2089,7 @@ spider_net_setup_netdev(struct spider_net_card *card)
if (!dn)
return -EIO;

mac = (u8 *)get_property(dn, "local-mac-address", NULL);
mac = get_property(dn, "local-mac-address", NULL);
if (!mac)
return -EIO;
memcpy(addr.sa_data, mac, ETH_ALEN);
Expand Down
2 changes: 1 addition & 1 deletion drivers/net/sungem.c
Original file line number Diff line number Diff line change
Expand Up @@ -2896,7 +2896,7 @@ static int __devinit gem_get_device_address(struct gem *gp)
if (use_idprom)
memcpy(dev->dev_addr, idprom->id_ethaddr, 6);
#elif defined(CONFIG_PPC_PMAC)
unsigned char *addr;
const unsigned char *addr;

addr = get_property(gp->of_node, "local-mac-address", NULL);
if (addr == NULL) {
Expand Down

0 comments on commit 1a2509c

Please sign in to comment.