* [PATCH] rt2x00: increase led's name buffer length
@ 2012-03-17 23:16 Jakub Kicinski
2012-03-19 11:03 ` [rt2x00-users] " Stanislaw Gruszka
2012-03-19 20:21 ` Gertjan van Wingerde
0 siblings, 2 replies; 3+ messages in thread
From: Jakub Kicinski @ 2012-03-17 23:16 UTC (permalink / raw)
To: linville
Cc: IvDoorn, gwingerde, helmut.schaa, linux-wireless, users,
Jakub Kicinski
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>
---
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 ca585e3..8679d78 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,
void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
{
- char dev_name[16];
- char name[32];
+ char name[36];
int retval;
unsigned long on_period;
unsigned long off_period;
-
- snprintf(dev_name, sizeof(dev_name), "%s-%s",
- rt2x00dev->ops->name, wiphy_name(rt2x00dev->hw->wiphy));
+ const char *phy_name = wiphy_name(rt2x00dev->hw->wiphy);
if (rt2x00dev->led_radio.flags & LED_INITIALIZED) {
- snprintf(name, sizeof(name), "%s::radio", dev_name);
+ snprintf(name, sizeof(name), "%s-%s::radio",
+ rt2x00dev->ops->name, phy_name);
retval = rt2x00leds_register_led(rt2x00dev,
&rt2x00dev->led_radio,
@@ -144,7 +142,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
}
if (rt2x00dev->led_assoc.flags & LED_INITIALIZED) {
- snprintf(name, sizeof(name), "%s::assoc", dev_name);
+ snprintf(name, sizeof(name), "%s-%s::assoc",
+ rt2x00dev->ops->name, phy_name);
retval = rt2x00leds_register_led(rt2x00dev,
&rt2x00dev->led_assoc,
@@ -154,7 +153,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
}
if (rt2x00dev->led_qual.flags & LED_INITIALIZED) {
- snprintf(name, sizeof(name), "%s::quality", dev_name);
+ snprintf(name, sizeof(name), "%s-%s::quality",
+ rt2x00dev->ops->name, phy_name);
retval = rt2x00leds_register_led(rt2x00dev,
&rt2x00dev->led_qual,
--
1.7.7.6
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [rt2x00-users] [PATCH] rt2x00: increase led's name buffer length
2012-03-17 23:16 [PATCH] rt2x00: increase led's name buffer length Jakub Kicinski
@ 2012-03-19 11:03 ` Stanislaw Gruszka
2012-03-19 20:21 ` Gertjan van Wingerde
1 sibling, 0 replies; 3+ messages in thread
From: Stanislaw Gruszka @ 2012-03-19 11:03 UTC (permalink / raw)
To: Jakub Kicinski; +Cc: linville, linux-wireless, users
On Sun, Mar 18, 2012 at 12:16:52AM +0100, Jakub Kicinski wrote:
> 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>
> ---
> 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 ca585e3..8679d78 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,
>
> void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
> {
> - char dev_name[16];
> - char name[32];
> + char name[36];
> int retval;
> unsigned long on_period;
> unsigned long off_period;
> -
> - snprintf(dev_name, sizeof(dev_name), "%s-%s",
> - rt2x00dev->ops->name, wiphy_name(rt2x00dev->hw->wiphy));
> + const char *phy_name = wiphy_name(rt2x00dev->hw->wiphy);
>
> if (rt2x00dev->led_radio.flags & LED_INITIALIZED) {
> - snprintf(name, sizeof(name), "%s::radio", dev_name);
> + snprintf(name, sizeof(name), "%s-%s::radio",
> + rt2x00dev->ops->name, phy_name);
I guess simpler would be just increase dev_name length, but patch looks
correct to me anyway.
Reviewed-by: Stanislaw Gruszka <sgruszka@redhat.com>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: [PATCH] rt2x00: increase led's name buffer length
2012-03-17 23:16 [PATCH] rt2x00: increase led's name buffer length Jakub Kicinski
2012-03-19 11:03 ` [rt2x00-users] " Stanislaw Gruszka
@ 2012-03-19 20:21 ` Gertjan van Wingerde
1 sibling, 0 replies; 3+ messages in thread
From: Gertjan van Wingerde @ 2012-03-19 20:21 UTC (permalink / raw)
To: Jakub Kicinski
Cc: linville@tuxdriver.com, IvDoorn@gmail.com,
helmut.schaa@googlemail.com, linux-wireless@vger.kernel.org,
users@rt2x00.serialmonkey.com, Jakub Kicinski
On 18 mrt. 2012, at 00:16, Jakub Kicinski <kubakici@wp.pl> wrote:
> 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>
Acked-by: Gertjan van Wingerde <gwingerde@gmail.com>
> ---
> 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 ca585e3..8679d78 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,
>
> void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
> {
> - char dev_name[16];
> - char name[32];
> + char name[36];
> int retval;
> unsigned long on_period;
> unsigned long off_period;
> -
> - snprintf(dev_name, sizeof(dev_name), "%s-%s",
> - rt2x00dev->ops->name, wiphy_name(rt2x00dev->hw->wiphy));
> + const char *phy_name = wiphy_name(rt2x00dev->hw->wiphy);
>
> if (rt2x00dev->led_radio.flags & LED_INITIALIZED) {
> - snprintf(name, sizeof(name), "%s::radio", dev_name);
> + snprintf(name, sizeof(name), "%s-%s::radio",
> + rt2x00dev->ops->name, phy_name);
>
> retval = rt2x00leds_register_led(rt2x00dev,
> &rt2x00dev->led_radio,
> @@ -144,7 +142,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
> }
>
> if (rt2x00dev->led_assoc.flags & LED_INITIALIZED) {
> - snprintf(name, sizeof(name), "%s::assoc", dev_name);
> + snprintf(name, sizeof(name), "%s-%s::assoc",
> + rt2x00dev->ops->name, phy_name);
>
> retval = rt2x00leds_register_led(rt2x00dev,
> &rt2x00dev->led_assoc,
> @@ -154,7 +153,8 @@ void rt2x00leds_register(struct rt2x00_dev *rt2x00dev)
> }
>
> if (rt2x00dev->led_qual.flags & LED_INITIALIZED) {
> - snprintf(name, sizeof(name), "%s::quality", dev_name);
> + snprintf(name, sizeof(name), "%s-%s::quality",
> + rt2x00dev->ops->name, phy_name);
>
> retval = rt2x00leds_register_led(rt2x00dev,
> &rt2x00dev->led_qual,
> --
> 1.7.7.6
>
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2012-03-19 20:21 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-17 23:16 [PATCH] rt2x00: increase led's name buffer length Jakub Kicinski
2012-03-19 11:03 ` [rt2x00-users] " Stanislaw Gruszka
2012-03-19 20:21 ` Gertjan van Wingerde
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).