From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jacek Anaszewski Subject: Re: [PATCH v3 2/7] leds: documentation: 'ide-disk' to 'disk-activity' Date: Wed, 22 Jun 2016 09:55:43 +0200 Message-ID: <576A447F.2020002@samsung.com> References: <20160608223000.433-1-linz@li-pro.net> <20160608223000.433-2-linz@li-pro.net> <20160621150532.GG3302@leverpostej> Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: QUOTED-PRINTABLE Return-path: In-reply-to: <20160621150532.GG3302@leverpostej> Sender: linux-kernel-owner@vger.kernel.org To: Mark Rutland , Stephan Linz Cc: linux-leds@vger.kernel.org, linux-ide@vger.kernel.org, Joseph Jezak , Nico Macrionitis , =?UTF-8?B?SsO2cmcgU29tbWVy?= , Rob Herring , Pawel Moll , Ian Campbell , Kumar Gala , Jonathan Corbet , devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-doc@vger.kernel.org List-Id: devicetree@vger.kernel.org On 06/21/2016 05:05 PM, Mark Rutland wrote: > On Thu, Jun 09, 2016 at 12:29:37AM +0200, Stephan Linz wrote: >> Cc: Joseph Jezak >> Cc: Nico Macrionitis >> Cc: J=C3=B6rg Sommer >> Signed-off-by: Stephan Linz >> --- >> Documentation/devicetree/bindings/leds/common.txt | 2 +- >> Documentation/devicetree/bindings/leds/leds-gpio.txt | 2 +- >> Documentation/laptops/asus-laptop.txt | 2 +- >> Documentation/leds/leds-class.txt | 2 +- >> 4 files changed, 4 insertions(+), 4 deletions(-) >> >> diff --git a/Documentation/devicetree/bindings/leds/common.txt b/Doc= umentation/devicetree/bindings/leds/common.txt >> index af10678..1e97169 100644 >> --- a/Documentation/devicetree/bindings/leds/common.txt >> +++ b/Documentation/devicetree/bindings/leds/common.txt >> @@ -26,7 +26,7 @@ Optional properties for child nodes: >> "default-on" - LED will turn on (but for leds-gpio see "defau= lt-state" >> property in Documentation/devicetree/bindings/gpio/led.txt) >> "heartbeat" - LED "double" flashes at a load average based ra= te >> - "ide-disk" - LED indicates disk activity >> + "disk-activity" - LED indicates disk activity >> "timer" - LED flashes at a fixed, configurable rate > > We should not break the binding. > > Code must continue to support "ide-disk", though we can mark it > deprecated in the binding documentation, and update the in-kernel dts > files to use "disk-activity". The code in the version 4 of the patchset supports also "ide-disk". Stephan, could you send a new version of this patch, with preserved "ide-disk" property, marked as deprecated? --=20 Best regards, Jacek Anaszewski