diff options
author | Joe Perches <joe@perches.com> | 2010-01-28 15:59:21 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-02-01 02:26:54 -0500 |
commit | e9cd1cbcda5e21706b03b543b1b73600b07f323f (patch) | |
tree | 91d4b820848ccd781b213109fa4ec810ae0484fc /drivers/net/tulip | |
parent | dde7c8ef167996689bc464eb65d2b09ef05263c2 (diff) |
tulip/eeprom.c: Use dev_<level> and pr_<level>
Convert printks to dev_<level> where a dev is available
Convert printks to pr_<level> where not
Coalesce format strings
Change print formats with %d.dx to %0dx
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/tulip')
-rw-r--r-- | drivers/net/tulip/eeprom.c | 47 |
1 files changed, 24 insertions, 23 deletions
diff --git a/drivers/net/tulip/eeprom.c b/drivers/net/tulip/eeprom.c index 889f57aae89b..93f4e8309f81 100644 --- a/drivers/net/tulip/eeprom.c +++ b/drivers/net/tulip/eeprom.c | |||
@@ -161,15 +161,15 @@ void __devinit tulip_parse_eeprom(struct net_device *dev) | |||
161 | if (ee_data[0] == 0xff) { | 161 | if (ee_data[0] == 0xff) { |
162 | if (last_mediatable) { | 162 | if (last_mediatable) { |
163 | controller_index++; | 163 | controller_index++; |
164 | printk(KERN_INFO "%s: Controller %d of multiport board.\n", | 164 | dev_info(&dev->dev, |
165 | dev->name, controller_index); | 165 | "Controller %d of multiport board\n", |
166 | controller_index); | ||
166 | tp->mtable = last_mediatable; | 167 | tp->mtable = last_mediatable; |
167 | ee_data = last_ee_data; | 168 | ee_data = last_ee_data; |
168 | goto subsequent_board; | 169 | goto subsequent_board; |
169 | } else | 170 | } else |
170 | printk(KERN_INFO "%s: Missing EEPROM, this interface may " | 171 | dev_info(&dev->dev, |
171 | "not work correctly!\n", | 172 | "Missing EEPROM, this interface may not work correctly!\n"); |
172 | dev->name); | ||
173 | return; | 173 | return; |
174 | } | 174 | } |
175 | /* Do a fix-up based on the vendor half of the station address prefix. */ | 175 | /* Do a fix-up based on the vendor half of the station address prefix. */ |
@@ -181,16 +181,15 @@ void __devinit tulip_parse_eeprom(struct net_device *dev) | |||
181 | i++; /* An Accton EN1207, not an outlaw Maxtech. */ | 181 | i++; /* An Accton EN1207, not an outlaw Maxtech. */ |
182 | memcpy(ee_data + 26, eeprom_fixups[i].newtable, | 182 | memcpy(ee_data + 26, eeprom_fixups[i].newtable, |
183 | sizeof(eeprom_fixups[i].newtable)); | 183 | sizeof(eeprom_fixups[i].newtable)); |
184 | printk(KERN_INFO "%s: Old format EEPROM on '%s' board. Using" | 184 | dev_info(&dev->dev, |
185 | " substitute media control info.\n", | 185 | "Old format EEPROM on '%s' board. Using substitute media control info\n", |
186 | dev->name, eeprom_fixups[i].name); | 186 | eeprom_fixups[i].name); |
187 | break; | 187 | break; |
188 | } | 188 | } |
189 | } | 189 | } |
190 | if (eeprom_fixups[i].name == NULL) { /* No fixup found. */ | 190 | if (eeprom_fixups[i].name == NULL) { /* No fixup found. */ |
191 | printk(KERN_INFO "%s: Old style EEPROM with no media selection " | 191 | dev_info(&dev->dev, |
192 | "information.\n", | 192 | "Old style EEPROM with no media selection information\n"); |
193 | dev->name); | ||
194 | return; | 193 | return; |
195 | } | 194 | } |
196 | } | 195 | } |
@@ -218,7 +217,8 @@ subsequent_board: | |||
218 | /* there is no phy information, don't even try to build mtable */ | 217 | /* there is no phy information, don't even try to build mtable */ |
219 | if (count == 0) { | 218 | if (count == 0) { |
220 | if (tulip_debug > 0) | 219 | if (tulip_debug > 0) |
221 | printk(KERN_WARNING "%s: no phy info, aborting mtable build\n", dev->name); | 220 | dev_warn(&dev->dev, |
221 | "no phy info, aborting mtable build\n"); | ||
222 | return; | 222 | return; |
223 | } | 223 | } |
224 | 224 | ||
@@ -234,8 +234,8 @@ subsequent_board: | |||
234 | mtable->has_nonmii = mtable->has_mii = mtable->has_reset = 0; | 234 | mtable->has_nonmii = mtable->has_mii = mtable->has_reset = 0; |
235 | mtable->csr15dir = mtable->csr15val = 0; | 235 | mtable->csr15dir = mtable->csr15val = 0; |
236 | 236 | ||
237 | printk(KERN_INFO "%s: EEPROM default media type %s.\n", dev->name, | 237 | dev_info(&dev->dev, "EEPROM default media type %s\n", |
238 | media & 0x0800 ? "Autosense" : medianame[media & MEDIA_MASK]); | 238 | media & 0x0800 ? "Autosense" : medianame[media & MEDIA_MASK]); |
239 | for (i = 0; i < count; i++) { | 239 | for (i = 0; i < count; i++) { |
240 | struct medialeaf *leaf = &mtable->mleaf[i]; | 240 | struct medialeaf *leaf = &mtable->mleaf[i]; |
241 | 241 | ||
@@ -298,16 +298,17 @@ subsequent_board: | |||
298 | } | 298 | } |
299 | if (tulip_debug > 1 && leaf->media == 11) { | 299 | if (tulip_debug > 1 && leaf->media == 11) { |
300 | unsigned char *bp = leaf->leafdata; | 300 | unsigned char *bp = leaf->leafdata; |
301 | printk(KERN_INFO "%s: MII interface PHY %d, setup/reset " | 301 | dev_info(&dev->dev, |
302 | "sequences %d/%d long, capabilities %2.2x %2.2x.\n", | 302 | "MII interface PHY %d, setup/reset sequences %d/%d long, capabilities %02x %02x\n", |
303 | dev->name, bp[0], bp[1], bp[2 + bp[1]*2], | 303 | bp[0], bp[1], bp[2 + bp[1]*2], |
304 | bp[5 + bp[2 + bp[1]*2]*2], bp[4 + bp[2 + bp[1]*2]*2]); | 304 | bp[5 + bp[2 + bp[1]*2]*2], |
305 | bp[4 + bp[2 + bp[1]*2]*2]); | ||
305 | } | 306 | } |
306 | printk(KERN_INFO "%s: Index #%d - Media %s (#%d) described " | 307 | dev_info(&dev->dev, |
307 | "by a %s (%d) block.\n", | 308 | "Index #%d - Media %s (#%d) described by a %s (%d) block\n", |
308 | dev->name, i, medianame[leaf->media & 15], leaf->media, | 309 | i, medianame[leaf->media & 15], leaf->media, |
309 | leaf->type < ARRAY_SIZE(block_name) ? block_name[leaf->type] : "<unknown>", | 310 | leaf->type < ARRAY_SIZE(block_name) ? block_name[leaf->type] : "<unknown>", |
310 | leaf->type); | 311 | leaf->type); |
311 | } | 312 | } |
312 | if (new_advertise) | 313 | if (new_advertise) |
313 | tp->sym_advertise = new_advertise; | 314 | tp->sym_advertise = new_advertise; |