Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S934307AbbEMMmY (ORCPT ); Wed, 13 May 2015 08:42:24 -0400 Received: from mail-wi0-f181.google.com ([209.85.212.181]:38466 "EHLO mail-wi0-f181.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S934251AbbEMMmV convert rfc822-to-8bit (ORCPT ); Wed, 13 May 2015 08:42:21 -0400 Subject: Re: [PATCH 4/5] doc: ABI: bone_capemgr sysfs API Mime-Version: 1.0 (Mac OS X Mail 8.2 \(2098\)) Content-Type: text/plain; charset=utf-8 From: Pantelis Antoniou In-Reply-To: <20150513120854.GA1516@kroah.com> Date: Wed, 13 May 2015 15:42:12 +0300 Cc: Matt Porter , Koen Kooi , Robert Nelson , Rob Herring , Grant Likely , Jonathan Corbet , Srinivas Kandagatla , Guenter Roeck , Geert Uytterhoeven , =?utf-8?Q?Beno=C3=AEt_Cousso?= , linux-api@vger.kernel.org, linux-doc@vger.kernel.org, devicetree , linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8BIT Message-Id: References: <1431503985-31853-1-git-send-email-pantelis.antoniou@konsulko.com> <1431503985-31853-5-git-send-email-pantelis.antoniou@konsulko.com> <20150513115259.GA1021@kroah.com> <28C3D47C-8D31-4DD9-8751-F72A7F09DF6F@konsulko.com> <20150513120854.GA1516@kroah.com> To: Greg Kroah-Hartman X-Mailer: Apple Mail (2.2098) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 4906 Lines: 142 Hi Greg, > On May 13, 2015, at 15:08 , Greg Kroah-Hartman wrote: > > On Wed, May 13, 2015 at 02:56:49PM +0300, Pantelis Antoniou wrote: >> Hi Greg, >> >>> On May 13, 2015, at 14:52 , Greg Kroah-Hartman wrote: >>> >>> On Wed, May 13, 2015 at 10:59:44AM +0300, Pantelis Antoniou wrote: >>>> Document the beaglebone's capemgr sysfs API >>>> >>>> Signed-off-by: Pantelis Antoniou >>>> --- >>>> .../testing/sysfs-devices-platform-bone_capemgr | 63 ++++++++++++++++++++++ >>>> 1 file changed, 63 insertions(+) >>>> create mode 100644 Documentation/ABI/testing/sysfs-devices-platform-bone_capemgr >>>> >>>> diff --git a/Documentation/ABI/testing/sysfs-devices-platform-bone_capemgr b/Documentation/ABI/testing/sysfs-devices-platform-bone_capemgr >>>> new file mode 100644 >>>> index 0000000..e2df613 >>>> --- /dev/null >>>> +++ b/Documentation/ABI/testing/sysfs-devices-platform-bone_capemgr >>>> @@ -0,0 +1,63 @@ >>>> +What: /sys/devices/platform/bone_capemgr/slots >>>> +Date: May 2015 >>>> +KernelVersion: 4.0 >>> >>> I don't think that version is correct :) >>> >> >> Bah, ++ >> >>>> +Contact: Pantelis Antoniou >>>> +Description: >>>> + READ: >>>> + Describe the state of all the slots of the beaglebone capemgr. >>>> + Each line of the output describes a slot: >>> >>> sysfs files are "one value per file", so a sysfs file that displays >>> multiple lines like this is not allowed at all, sorry. >>> >>> Please either make it a debugfs file (if this is only for debugging, or >>> split it out into individual files, one per slot (hint, one per slot is >>> probably best.) >>> >> >> Well, it’s a status file. And it’s been used as is for a couple of years >> so it was worth a shot for backward compatibility. > > There is not "backwards compatiblity" for when you do things wrong in > the first place, you can't claim that here, sorry. > > And don't "try" to introduce things you know is wrong, that just makes > maintainers _very_ suspicious of everything else you are doing here… > OK >>>> + The slot format is as following: >>>> + : [P-][F-][O-][l-][L-][D-] \ >>>> + ,, >>>> + , >>>> + >>>> + Where the flags are: >>>> + P: Slot has been probed >>>> + F: Slot has failed probing (i.e. no EEPROM detected) >>>> + O: Slot has been overridden by the user >>>> + l: Slot is current loading >>>> + L: Slot has completed loading and is ready >>>> + D: Slot has been disabled >>>> + >>>> + Example: >>>> + 0: P---L- -1 BeagleBone RS232 CAPE,00A1,Beagleboardtoys,BB-BONE-SERL-03 >>>> + 1: PF---- -1 >>>> + 2: PF---- -1 >>>> + 3: PF---- -1 >>>> + >>>> + WRITE: >>>> + Writing a string of the form [:version] issues a request to >>>> + load a firmware blob containing an overlay. The name of the firmware blob >>>> + is -[version|00A0].dtbo. This act is defined as a slot override. >>>> + >>>> + Writing a negative slot id removes the slot if it was an overridden one, or >>>> + unloads a slot that was probed. >>>> + >>>> +What: /sys/devices/platform/bone_capemgr/baseboard/ >>>> +Date: May 2015 >>>> +KernelVersion: 4.0 >>>> +Contact: Pantelis Antoniou >>>> +Description: Contains the probed base board EEPROM field; one of: >>>> + board-name - board-name as stored in cape EEPROM >>>> + dc-supplied - whether the cape draws or supplies DC >>>> + eeprom-format-revision - EEPROM format rev, only 00A0 supported >>>> + header - header; should be 'aa 55 33 ee' >>> >>> If it's always this value, why have the file? >>> >> >> These are the contents of the EEPROM. If the format of the EEPROM changes then the >> header information will change. > > Then don't say "should be", because what happens in the future if it is > not. > OK >>>> + manufacturer - manufacturer string >>>> + part-number - part-number of the cape >>>> + serial-number - serial number of the cape >>>> + version - version of the cape, i.e. 00A0 >>>> + number-of-pins - displayed but ignored >>>> + pin-usage - displayed but ignored >>>> + sys-5v - displayed but ignored >>>> + vdd-3v3exp - displayed but ignored >>>> + vdd-5v - displayed but ignored >>> >>> Are these all individual different files? >>> >> >> Yes > > Then write out the individual files please as different entries. > > Also, the "displayed but ignored" doesn't make sense, please fix that > up. > OK > greg k-h Regards — Pantelis -- 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/