Commit 46d7031e authored by Al Viro's avatar Al Viro Committed by Linus Torvalds

[PATCH] cassini annotations and fixes

 - __user annotations
 - NULL noise removal
 - C99 initializers
 - s/u32/pm_message_t/ in ->suspend()
 - removal of bogus casts in iounmap() arguments
 - if_mii() instead of open-coded variant

Remains to be done: ethtool conversion.
Signed-off-by: default avatarAl Viro <viro@zeniv.linux.org.uk>
Signed-off-by: default avatarLinus Torvalds <torvalds@osdl.org>
parent 74ba1fd9
...@@ -3244,7 +3244,7 @@ static int cas_get_vpd_info(struct cas *cp, unsigned char *dev_addr, ...@@ -3244,7 +3244,7 @@ static int cas_get_vpd_info(struct cas *cp, unsigned char *dev_addr,
goto use_random_mac_addr; goto use_random_mac_addr;
/* search for beginning of vpd */ /* search for beginning of vpd */
base = 0; base = NULL;
for (i = 2; i < EXPANSION_ROM_SIZE; i++) { for (i = 2; i < EXPANSION_ROM_SIZE; i++) {
/* check for PCIR */ /* check for PCIR */
if ((readb(p + i + 0) == 0x50) && if ((readb(p + i + 0) == 0x50) &&
...@@ -4564,7 +4564,7 @@ static void cas_set_multicast(struct net_device *dev) ...@@ -4564,7 +4564,7 @@ static void cas_set_multicast(struct net_device *dev)
/* Eventually add support for changing the advertisement /* Eventually add support for changing the advertisement
* on autoneg. * on autoneg.
*/ */
static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user) static int cas_ethtool_ioctl(struct net_device *dev, void __user *ep_user)
{ {
struct cas *cp = netdev_priv(dev); struct cas *cp = netdev_priv(dev);
u16 bmcr; u16 bmcr;
...@@ -4578,7 +4578,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user) ...@@ -4578,7 +4578,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user)
switch(ecmd.cmd) { switch(ecmd.cmd) {
case ETHTOOL_GDRVINFO: { case ETHTOOL_GDRVINFO: {
struct ethtool_drvinfo info = { cmd: ETHTOOL_GDRVINFO }; struct ethtool_drvinfo info = { .cmd = ETHTOOL_GDRVINFO };
strncpy(info.driver, DRV_MODULE_NAME, strncpy(info.driver, DRV_MODULE_NAME,
ETHTOOL_BUSINFO_LEN); ETHTOOL_BUSINFO_LEN);
...@@ -4738,7 +4738,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user) ...@@ -4738,7 +4738,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user)
/* get link status */ /* get link status */
case ETHTOOL_GLINK: { case ETHTOOL_GLINK: {
struct ethtool_value edata = { cmd: ETHTOOL_GLINK }; struct ethtool_value edata = { .cmd = ETHTOOL_GLINK };
edata.data = (cp->lstate == link_up); edata.data = (cp->lstate == link_up);
if (copy_to_user(ep_user, &edata, sizeof(edata))) if (copy_to_user(ep_user, &edata, sizeof(edata)))
...@@ -4748,7 +4748,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user) ...@@ -4748,7 +4748,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user)
/* get message-level */ /* get message-level */
case ETHTOOL_GMSGLVL: { case ETHTOOL_GMSGLVL: {
struct ethtool_value edata = { cmd: ETHTOOL_GMSGLVL }; struct ethtool_value edata = { .cmd = ETHTOOL_GMSGLVL };
edata.data = cp->msg_enable; edata.data = cp->msg_enable;
if (copy_to_user(ep_user, &edata, sizeof(edata))) if (copy_to_user(ep_user, &edata, sizeof(edata)))
...@@ -4874,7 +4874,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user) ...@@ -4874,7 +4874,7 @@ static int cas_ethtool_ioctl(struct net_device *dev, void *ep_user)
static int cas_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd) static int cas_ioctl(struct net_device *dev, struct ifreq *ifr, int cmd)
{ {
struct cas *cp = netdev_priv(dev); struct cas *cp = netdev_priv(dev);
struct mii_ioctl_data *data = (struct mii_ioctl_data *)&ifr->ifr_data; struct mii_ioctl_data *data = if_mii(ifr);
unsigned long flags; unsigned long flags;
int rc = -EOPNOTSUPP; int rc = -EOPNOTSUPP;
...@@ -5168,7 +5168,7 @@ static int __devinit cas_init_one(struct pci_dev *pdev, ...@@ -5168,7 +5168,7 @@ static int __devinit cas_init_one(struct pci_dev *pdev,
cas_shutdown(cp); cas_shutdown(cp);
up(&cp->pm_sem); up(&cp->pm_sem);
iounmap((void *) cp->regs); iounmap(cp->regs);
err_out_free_res: err_out_free_res:
...@@ -5216,7 +5216,7 @@ static void __devexit cas_remove_one(struct pci_dev *pdev) ...@@ -5216,7 +5216,7 @@ static void __devexit cas_remove_one(struct pci_dev *pdev)
#endif #endif
pci_free_consistent(pdev, sizeof(struct cas_init_block), pci_free_consistent(pdev, sizeof(struct cas_init_block),
cp->init_block, cp->block_dvma); cp->init_block, cp->block_dvma);
iounmap((void *) cp->regs); iounmap(cp->regs);
free_netdev(dev); free_netdev(dev);
pci_release_regions(pdev); pci_release_regions(pdev);
pci_disable_device(pdev); pci_disable_device(pdev);
...@@ -5224,7 +5224,7 @@ static void __devexit cas_remove_one(struct pci_dev *pdev) ...@@ -5224,7 +5224,7 @@ static void __devexit cas_remove_one(struct pci_dev *pdev)
} }
#ifdef CONFIG_PM #ifdef CONFIG_PM
static int cas_suspend(struct pci_dev *pdev, u32 state) static int cas_suspend(struct pci_dev *pdev, pm_message_t state)
{ {
struct net_device *dev = pci_get_drvdata(pdev); struct net_device *dev = pci_get_drvdata(pdev);
struct cas *cp = netdev_priv(dev); struct cas *cp = netdev_priv(dev);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment