From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 17371C197A0 for ; Fri, 17 Nov 2023 12:33:10 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1345765AbjKQMdL (ORCPT ); Fri, 17 Nov 2023 07:33:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44200 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229436AbjKQMdJ (ORCPT ); Fri, 17 Nov 2023 07:33:09 -0500 Received: from smtp.kernel.org (relay.kernel.org [52.25.139.140]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id A9744120 for ; Fri, 17 Nov 2023 04:33:06 -0800 (PST) Received: by smtp.kernel.org (Postfix) with ESMTPSA id 8AD36C433C8; Fri, 17 Nov 2023 12:33:03 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=k20201202; t=1700224386; bh=6GwD/+0RW2AAON4W/qJdWK7zK0D+kuihmnQOP4ByGns=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=Q8/AkM6qMsYZXczTXA7cHWvewZuvlhCf7ZVVeo5iIVmwEZrVSyLiGZZJG5PFlIfYS adUik5JSfkw38cS68Ei5jwSAKcMIXgU6HTEe/kTg+RVSUKl0awhTx6+WDooxzfZzyk 424/QbYjoMaoLyOc17nb3t2UD0sRpNCVrDWl9fVdkC5ijyALU9eHK36KrP8mvm/OVU vQDxefo/ZyS/AMjRUeIWVxgemBcQhL6iT2txRHYymjrfhcScw69JFVqUN8H4HBEn1x 3lrtRc+JPxWI2XkV0UDJ4XY2uZlW8Ykt2+qstKv00MyDsKbJp2rIs8W3RwjU0SzZ5T /D4yh+oSa3FCg== Date: Fri, 17 Nov 2023 12:32:59 +0000 From: Lee Jones To: Shiji Yang , Florian Fainelli , Andrew Lunn Cc: linux-leds@vger.kernel.org, devicetree@vger.kernel.org, Pavel Machek , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Jisheng Zhang , linux-kernel@vger.kernel.org Subject: Re: [PATCH v3] dt-bindings: leds: add 'internet' and 'signal' function definitions Message-ID: <20231117123259.GA153049@google.com> References: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org [Intentional top-post] Andrew, Florian, Does this fall into your area of expertise? On Fri, 03 Nov 2023, Shiji Yang wrote: > These two types of LEDs are widely used in routers and NICs. > > The 'signal' LED is used to display the wireless signal strength. > Usually, there are 3~4 LEDs in one group to indicate the signal > strength, similar to the signal icon on a mobile phone. > > The 'internet' LED can indicate whether the device can access a > specific server. It's different from 'wan'. 'wan' usually indicates > whether the WAN port is connected to the modem (internet services > may still be unavailable). But the 'internet' shows if the device > can successfully ping servers such as 8.8.8.8 to detect the internet > connection status. When the router is running in AP only mode, we > can even connect LAN port to the AC/modem to connect to the internet. > In this case, the 'internet' LED should be on. On some routers, both > 'internet' and 'wan' are available and can be controlled separately. > > Signed-off-by: Shiji Yang > --- > > Changes in v2: > * Remove the LED name sorting patch as it changes the ABI. > * Add "devicetree@vger.kernel.org" to '--to' list. > Thanks to Rob Herring and Krzysztof Kozlowski for letting me know I > can send patch to multiple mailing list at once. > > Changes in v3: > * Add more information about the new added LEDs. > * Remove the missing LED fix as Jisheng Zhang has already sent a > similar one. I should search the mailing list first... > > v1: > https://lore.kernel.org/all/TYAP286MB0315FE921FF113BF76F7B700BCA0A@TYAP286MB0315.JPNP286.PROD.OUTLOOK.COM/ > > v2: > https://lore.kernel.org/all/TYAP286MB03159A83A77E6FD59F271D9BBCA0A@TYAP286MB0315.JPNP286.PROD.OUTLOOK.COM/ > > include/dt-bindings/leds/common.h | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/include/dt-bindings/leds/common.h b/include/dt-bindings/leds/common.h > index 9a0d33d02..55a426e39 100644 > --- a/include/dt-bindings/leds/common.h > +++ b/include/dt-bindings/leds/common.h > @@ -88,11 +88,13 @@ > #define LED_FUNCTION_FLASH "flash" > #define LED_FUNCTION_HEARTBEAT "heartbeat" > #define LED_FUNCTION_INDICATOR "indicator" > +#define LED_FUNCTION_INTERNET "internet" > #define LED_FUNCTION_LAN "lan" > #define LED_FUNCTION_MAIL "mail" > #define LED_FUNCTION_MTD "mtd" > #define LED_FUNCTION_PANIC "panic" > #define LED_FUNCTION_PROGRAMMING "programming" > +#define LED_FUNCTION_RSSI "rssi" > #define LED_FUNCTION_RX "rx" > #define LED_FUNCTION_SD "sd" > #define LED_FUNCTION_STANDBY "standby" > -- > 2.39.2 > -- Lee Jones [李琼斯]