From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
        id S933267AbcH2NKS (ORCPT <rfc822;w@1wt.eu>);
        Mon, 29 Aug 2016 09:10:18 -0400
Received: from mailout2.w1.samsung.com ([210.118.77.12]:53648 "EHLO
        mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
        with ESMTP id S932900AbcH2NKQ (ORCPT
        <rfc822;linux-kernel@vger.kernel.org>);
        Mon, 29 Aug 2016 09:10:16 -0400
MIME-version: 1.0
Content-type: text/plain; charset=utf-8; format=flowed
X-AuditID: cbfec7f4-f79cb6d000001359-0a-57c43435d54e
Subject: Re: [PATCH V2] Documentation: move oneshot trigger attributes
 documentation to ABI
To: =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= <zajec5@gmail.com>,
        Richard Purdie <rpurdie@rpsys.net>
References: <20160825094134.13811-1-zajec5@gmail.com>
 <20160826142021.10229-1-zajec5@gmail.com>
Cc: Greg KH <gregkh@linuxfoundation.org>,
        =?UTF-8?B?UmFmYcWCIE1pxYJlY2tp?= <rafal@milecki.pl>,
        Jonathan Corbet <corbet@lwn.net>,
        open list <linux-kernel@vger.kernel.org>,
        "open list:DOCUMENTATION" <linux-doc@vger.kernel.org>
From: Jacek Anaszewski <j.anaszewski@samsung.com>
Message-id: <f303e6a6-9a71-dcce-0562-bb1b768b2df9@samsung.com>
Date: Mon, 29 Aug 2016 15:10:11 +0200
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101
 Thunderbird/45.2.0
In-reply-to: <20160826142021.10229-1-zajec5@gmail.com>
Content-transfer-encoding: 8bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFrrHLMWRmVeSWpSXmKPExsVy+t/xK7qmJkfCDa58Y7Z4cqCd0aJ58Xo2
        i4VtS1gsLu+aw2axe+0iJovdu56yWqw5merA7rFz1l12j/1z17B7LO6bzOpxfqqnx575P1g9
        Pm+SC2CL4rJJSc3JLEst0rdL4Mr4Mf0qY8Fu5YoVs1rYGhh7ZboYOTkkBEwk5q5ZxQJhi0lc
        uLeerYuRi0NIYCmjxPJtaxlBErwCghI/Jt8DK2IWMJP48vIwK0TRM0aJlW/eASU4OIQFYiU6
        3nmDmCJA5vEHvCDlQgKJEtO3NjCDlDODlO+f0Qc2h03AUOLni9dMEPPtJJY3/WUGsVkEVCVm
        fdrPCmKLCkRI3Fr1EewGTqC9L9s3Qt0gL3HwynOWCYwCs5CcNwvJebOQlC1gZF7FKJpamlxQ
        nJSea6hXnJhbXJqXrpecn7uJERLqX3YwLj5mdYhRgINRiYc34vmhcCHWxLLiytxDjBIczEoi
        vIuMjoQL8aYkVlalFuXHF5XmpBYfYpTmYFES5527632IkEB6YklqdmpqQWoRTJaJg1OqgbFo
        67PLWvs2beA/l7D6vvu7Kj+lTT/zhfZkCLnulD2z2Xi79+GvB04fDbp2ey+/jtm6POWb/+4v
        FmB6knZsx52AWuVnEgcZP/ucfifiG3V11bE5xye4X2E7JDhhafWliNmxlzkjE2NEplj9uyH5
        fvK+jf+5/r67/k/uEpN1/7spIXxO7TfbL0R+UGIpzkg01GIuKk4EAFRaMCJxAgAA
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

Hi Rafał,

On 08/26/2016 04:19 PM, Rafał Miłecki wrote:
> From: Rafał Miłecki <rafal@milecki.pl>
>
> Documentation of sysfs interface should be in ABI in the first place.
> This moves relevant part of documentation and mentions where to look for
> it.
> Fix trivial typos whilst we are at it.
>
> Signed-off-by: Rafał Miłecki <rafal@milecki.pl>
> ---
> V2: s/Default/Defaults/
>     s/  / /
>     s/change/changes/
> ---
>  Documentation/ABI/testing/sysfs-class-led          |  3 +-
>  .../ABI/testing/sysfs-class-led-trigger-oneshot    | 36 ++++++++++++++++++++++
>  Documentation/leds/ledtrig-oneshot.txt             | 20 ++----------
>  3 files changed, 40 insertions(+), 19 deletions(-)
>  create mode 100644 Documentation/ABI/testing/sysfs-class-led-trigger-oneshot
>
> diff --git a/Documentation/ABI/testing/sysfs-class-led b/Documentation/ABI/testing/sysfs-class-led
> index 3646ec8..86ace28 100644
> --- a/Documentation/ABI/testing/sysfs-class-led
> +++ b/Documentation/ABI/testing/sysfs-class-led
> @@ -24,7 +24,8 @@ Description:
>  		of led events.
>  		You can change triggers in a similar manner to the way an IO
>  		scheduler is chosen. Trigger specific parameters can appear in
> -		/sys/class/leds/<led> once a given trigger is selected.
> +		/sys/class/leds/<led> once a given trigger is selected. For
> +		their documentation see sysfs-class-led-trigger-*.
>
>  What:		/sys/class/leds/<led>/inverted
>  Date:		January 2011
> diff --git a/Documentation/ABI/testing/sysfs-class-led-trigger-oneshot b/Documentation/ABI/testing/sysfs-class-led-trigger-oneshot
> new file mode 100644
> index 0000000..378a3a4
> --- /dev/null
> +++ b/Documentation/ABI/testing/sysfs-class-led-trigger-oneshot
> @@ -0,0 +1,36 @@
> +What:		/sys/class/leds/<led>/delay_on
> +Date:		Jun 2012
> +KernelVersion:	3.6
> +Contact:	linux-leds@vger.kernel.org
> +Description:
> +		Specifies for how many milliseconds the LED has to stay at
> +		LED_FULL brightness after it has been armed.
> +		Defaults to 100 ms.
> +
> +What:		/sys/class/leds/<led>/delay_off
> +Date:		Jun 2012
> +KernelVersion:	3.6
> +Contact:	linux-leds@vger.kernel.org
> +Description:
> +		Specifies for how many milliseconds the LED has to stay at
> +		LED_OFF brightness after it has been armed.
> +		Defaults to 100 ms.
> +
> +What:		/sys/class/leds/<led>/invert
> +Date:		Jun 2012
> +KernelVersion:	3.6
> +Contact:	linux-leds@vger.kernel.org
> +Description:
> +		Reverse the blink logic. If set to 0 (default) blink on for
> +		delay_on ms, then blink off for delay_off ms, leaving the LED
> +		normally off. If set to 1, blink off for delay_off ms, then
> +		blink on for delay_on ms, leaving the LED normally on.
> +		Setting this value also immediately changes the LED state.
> +
> +What:		/sys/class/leds/<led>/shot
> +Date:		Jun 2012
> +KernelVersion:	3.6
> +Contact:	linux-leds@vger.kernel.org
> +Description:
> +		Write any non-empty string to signal an events, this starts a
> +		blink sequence if not already running.
> diff --git a/Documentation/leds/ledtrig-oneshot.txt b/Documentation/leds/ledtrig-oneshot.txt
> index 07cd1fa..fe57474 100644
> --- a/Documentation/leds/ledtrig-oneshot.txt
> +++ b/Documentation/leds/ledtrig-oneshot.txt
> @@ -21,24 +21,8 @@ below:
>
>    echo oneshot > trigger
>
> -This adds the following sysfs attributes to the LED:
> -
> -  delay_on - specifies for how many milliseconds the LED has to stay at
> -             LED_FULL brightness after it has been armed.
> -             Default to 100 ms.
> -
> -  delay_off - specifies for how many milliseconds the LED has to stay at
> -              LED_OFF brightness after it has been armed.
> -              Default to 100 ms.
> -
> -  invert - reverse the blink logic.  If set to 0 (default) blink on for delay_on
> -           ms, then blink off for delay_off ms, leaving the LED normally off.  If
> -           set to 1, blink off for delay_off ms, then blink on for delay_on ms,
> -           leaving the LED normally on.
> -           Setting this value also immediately change the LED state.
> -
> -  shot - write any non-empty string to signal an events, this starts a blink
> -         sequence if not already running.
> +This adds sysfs attributes to the LED that are documented in:
> +Documentation/ABI/testing/sysfs-class-led-trigger-oneshot
>
>  Example use-case: network devices, initialization:
>
>

Applied, thanks.

-- 
Best regards,
Jacek Anaszewski