diff options
author | Jakub Kicinski <kubakici@wp.pl> | 2012-03-17 19:16:52 -0400 |
---|---|---|
committer | John W. Linville <linville@tuxdriver.com> | 2012-04-09 16:09:34 -0400 |
commit | 4272a27f2f5773f4e4cb5285feba655450a720a8 (patch) | |
tree | 7810c6bdeeffd415a0e25823017125ae5c77b69b /drivers/net/wireless/rt2x00 | |
parent | d6b6fc14f01e4c940e8acb29c611cfaef4d16917 (diff) |
rt2x00: increase led's name buffer length
With 9-letter driver names phy's number was truncated
to two characters, which caused warnings when creating
sysfs entries for leds on systems with multiple devices.
Signed-off-by: Jakub Kicinski <kubakici@wp.pl>
Reviewed-by: Stanislaw Gruszka <sgruszka@redhat.com>
Acked-by: Gertjan van Wingerde <gwingerde@gmail.com>
Signed-off-by: John W. Linville <linville@tuxdriver.com>
Diffstat (limited to 'drivers/net/wireless/rt2x00')
-rw-r--r-- | drivers/net/wireless/rt2x00/rt2x00leds.c | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/drivers/net/wireless/rt2x00/rt2x00leds.c b/drivers/net/wireless/rt2x00/rt2x00leds.c index ca585e34d00e..8679d781a264 100644 --- a/drivers/net/wireless/rt2x00/rt2x00leds.c +++ b/drivers/net/wireless/rt2x00/rt2x00leds.c | |||
@@ -124,17 +124,15 @@ static int rt2x00leds_register_led(struct rt2x00_dev *rt2x00dev, | |||
124 | 124 | ||
125 | void rt2x00leds_register(struct rt2x00_dev *rt2x00dev) | 125 | void rt2x00leds_register(struct rt2x00_dev *rt2x00dev) |
126 | { | 126 | { |
127 | char dev_name[16]; | 127 | char name[36]; |
128 | char name[32]; | ||
129 | int retval; | 128 | int retval; |
130 | unsigned long on_period; | 129 | unsigned long on_period; |
131 | unsigned long off_period; | 130 | unsigned long off_period; |
132 | 131 | const char *phy_name = wiphy_name(rt2x00dev->hw->wiphy); | |
133 | snprintf(dev_name, sizeof(dev_name), "%s-%s", | ||
134 | rt2x00dev->ops->name, wiphy_name(rt2x00dev->hw->wiphy)); | ||
135 | 132 | ||
136 | if (rt2x00dev->led_radio.flags & LED_INITIALIZED) { | 133 | if (rt2x00dev->led_radio.flags & LED_INITIALIZED) { |
137 | snprintf(name, sizeof(name), "%s::radio", dev_name); | 134 | snprintf(name, sizeof(name), "%s-%s::radio", |
135 | rt2x00dev->ops->name, phy_name); | ||
138 | 136 | ||
139 | retval = rt2x00leds_register_led(rt2x00dev, | 137 | retval = rt2x00leds_register_led(rt2x00dev, |
140 | &rt2x00dev->led_radio, | 138 | &rt2x00dev->led_radio, |
@@ -144,7 +142,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev) | |||
144 | } | 142 | } |
145 | 143 | ||
146 | if (rt2x00dev->led_assoc.flags & LED_INITIALIZED) { | 144 | if (rt2x00dev->led_assoc.flags & LED_INITIALIZED) { |
147 | snprintf(name, sizeof(name), "%s::assoc", dev_name); | 145 | snprintf(name, sizeof(name), "%s-%s::assoc", |
146 | rt2x00dev->ops->name, phy_name); | ||
148 | 147 | ||
149 | retval = rt2x00leds_register_led(rt2x00dev, | 148 | retval = rt2x00leds_register_led(rt2x00dev, |
150 | &rt2x00dev->led_assoc, | 149 | &rt2x00dev->led_assoc, |
@@ -154,7 +153,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev) | |||
154 | } | 153 | } |
155 | 154 | ||
156 | if (rt2x00dev->led_qual.flags & LED_INITIALIZED) { | 155 | if (rt2x00dev->led_qual.flags & LED_INITIALIZED) { |
157 | snprintf(name, sizeof(name), "%s::quality", dev_name); | 156 | snprintf(name, sizeof(name), "%s-%s::quality", |
157 | rt2x00dev->ops->name, phy_name); | ||
158 | 158 | ||
159 | retval = rt2x00leds_register_led(rt2x00dev, | 159 | retval = rt2x00leds_register_led(rt2x00dev, |
160 | &rt2x00dev->led_qual, | 160 | &rt2x00dev->led_qual, |