From mboxrd@z Thu Jan 1 00:00:00 1970 From: Johan Jonker Subject: Re: [PATCH v2 2/3] arm64: dts: rockchip: rk3399-roc-pc: Fix MMC numbering for LED triggers Date: Mon, 27 Apr 2020 16:12:51 +0200 Message-ID: References: <20200427073132.29997-3-wens@kernel.org> <684132b8-4a84-8295-474b-38ccb992bba7@gmail.com> <65d15254-08da-895c-1a0c-ef6ce231b620@gmail.com> <74a984fc-ce57-211b-936c-2d77e2e642bb@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <74a984fc-ce57-211b-936c-2d77e2e642bb@gmail.com> Content-Language: en-US Sender: linux-leds-owner@vger.kernel.org To: Chen-Yu Tsai Cc: devicetree , dmurphy@ti.com, =?UTF-8?Q?Heiko_St=c3=bcbner?= , jacek.anaszewski@gmail.com, linux-arm-kernel , linux-kernel , linux-leds@vger.kernel.org, "open list:ARM/Rockchip SoC..." , Pavel Machek , Rob Herring List-Id: linux-rockchip.vger.kernel.org Hi, >> So for fixing up the LED node names, we'd probably want the following: >> >> diy_led: led-0 >> yellow_led: led-1 >> work_led: led-2 Change proposal for led nodes to comply with preexisting dts. Does this work? diy_led: led_0: led-0 yellow_led: led_1: led-1 work_led: led_2: led-2 blue: led_0: led-0 A check does not give any warnings. make -k ARCH=arm dtbs_check DT_SCHEMA_FILES=Documentation/devicetree/bindings/leds/leds-gpio.yaml > > That doesn't look pretty either. > Would like to hear the maintainers view on how to handle other cases > without 'led' like for example 'blue' for mk808. >