From mboxrd@z Thu Jan 1 00:00:00 1970 From: Florian Fainelli Subject: Re: [PATCH] net: dsa: set parent of hwmon device Date: Tue, 20 Jan 2015 21:58:02 -0800 Message-ID: References: <1421799212-2028-1-git-send-email-vivien.didelot@savoirfairelinux.com> <54BF221E.8000409@roeck-us.net> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Cc: Vivien Didelot , netdev , "David S . Miller" , "linux-kernel@vger.kernel.org" , kernel@savoirfairelinux.com To: Guenter Roeck Return-path: In-Reply-To: <54BF221E.8000409@roeck-us.net> Sender: linux-kernel-owner@vger.kernel.org List-Id: netdev.vger.kernel.org 2015-01-20 19:50 GMT-08:00 Guenter Roeck : > On 01/20/2015 04:13 PM, Vivien Didelot wrote: >> >> Set the dsa device as the parent of the hwmon device, in order to link >> the hwmon subsystem under the corresponding /sys/devices/platform/dsa.X/ >> sysfs directory. >> >> Signed-off-by: Vivien Didelot >> --- >> net/dsa/dsa.c | 2 +- >> 1 file changed, 1 insertion(+), 1 deletion(-) >> >> diff --git a/net/dsa/dsa.c b/net/dsa/dsa.c >> index 3731714..363102a 100644 >> --- a/net/dsa/dsa.c >> +++ b/net/dsa/dsa.c >> @@ -347,7 +347,7 @@ dsa_switch_setup(struct dsa_switch_tree *dst, int >> index, >> hname[j] = '\0'; >> scnprintf(ds->hwmon_name, sizeof(ds->hwmon_name), >> "%s_dsa%d", >> hname, index); >> - ds->hwmon_dev = hwmon_device_register_with_groups(NULL, >> + ds->hwmon_dev = hwmon_device_register_with_groups(parent, >> ds->hwmon_name, ds, >> dsa_hwmon_groups); >> if (IS_ERR(ds->hwmon_dev)) >> ds->hwmon_dev = NULL; >> > > Looking into my old e-mail, turns out we did not add the parent device > because > it affected the output of the "sensors" command, and we wanted the device > to be handled as 'virtual device' (which implies no parent). That was an > explicit > part of the patch set (v2 of 'net: dsa: Add support for reporting switch > chip > temperatures'), compared to v1, which did set the parent device. > > I would suggest to keep the code as is. Maybe follow-up with a comment adding that above the call to hwmon_device_register_with_groups()? I suspect the intent is clear if you are deep into hwmon devices, but not necessarily for the reader ;) > > Thanks, > Guenter > > > -- > To unsubscribe from this list: send the line "unsubscribe linux-kernel" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > Please read the FAQ at http://www.tux.org/lkml/ -- Florian