Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751462AbaBKJu7 (ORCPT ); Tue, 11 Feb 2014 04:50:59 -0500 Received: from mail-pb0-f51.google.com ([209.85.160.51]:48479 "EHLO mail-pb0-f51.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751096AbaBKJux convert rfc822-to-8bit (ORCPT ); Tue, 11 Feb 2014 04:50:53 -0500 MIME-Version: 1.0 In-Reply-To: <20140211094726.GH32042@lee--X1> References: <1392045953-26596-1-git-send-email-lpapp@kde.org> <20140210160842.GB26997@lee--X1> <20140210173811.04ba5964@endymion.delvare> <20140211085014.4ef32b56@endymion.delvare> <20140211085053.GD32042@lee--X1> <20140211094726.GH32042@lee--X1> Date: Tue, 11 Feb 2014 09:50:52 +0000 X-Google-Sender-Auth: u1GVrogxS9OVHzuroCh_O_AnQ-Q Message-ID: Subject: Re: [lm-sensors] [PATCH] hwmon: (max6650) Rename the device ids to contain the hwmon suffix From: Laszlo Papp To: Lee Jones Cc: Jean Delvare , LKML , lm-sensors@lm-sensors.org Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Feb 11, 2014 at 9:47 AM, Lee Jones wrote: >> >> >> On Mon, Feb 10, 2014 at 4:38 PM, Jean Delvare wrote: >> >> >> > Additionally, dashes are explicitly forbidden in hwmon >> >> >> > device names. >> >> >> >> >> >> Also, where is that documented? >> >> > >> >> > In Documentation/hwmon/sysfs-interface: >> >> > >> >> > ********************* >> >> > * Global attributes * >> >> > ********************* >> >> > >> >> > name The chip name. >> >> > This should be a short, lowercase string, not containing >> >> > spaces nor dashes, representing the chip name. This is >> >> > the only mandatory attribute. >> >> > I2C devices get this attribute created automatically. >> >> > RO >> >> >> >> Time to revisit this decision.... >> >> >> >> So, based on the fact that children device names usually contain >> >> dashes, I do not understand why hwmon would be any special in this >> >> regard. It is possible that the hwmon developers have not faced much >> >> MFD situation before, and so, this was not considered to be handled >> >> like in other subsystems. >> >> >> >> I am proposing to change this "rule"... Any objection? >> > >> > Prior to proposing such an invasive change which is highly likely to >> > come up against heavy opposition, >> >> It is possible that someone does not understand why you think it may >> be invasive, right? Could you please explain the reason for that? > > The reason is a good one. In the kernel we make every attempt not to > break userspace. By that I mean _any_ userspace application. Userspace > applications which interface with the kernel can do so via a variety of > methods. One of those is Sysfs, where this name you are attempting to > change appears. The userspace applications already mentioned parse for > these devices, regex:ing for '-' separators. If you add an additional > '-' separator there is a chance that these applications will get > confused and break without warning. Only for new devices that they have not supported before, right? Provided, we apply the logic only for new drivers. Is it unacceptable to have an interim "obsolete" period for a while and the recommendation is the dash for the long future? Also, how about applications that would expect hwmon to behave the same way like all the rest? They also break with the current hwmon schema, right? > > -- > Lee Jones > Linaro STMicroelectronics Landing Team Lead > Linaro.org │ Open source software for ARM SoCs > Follow Linaro: Facebook | Twitter | Blog -- 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/