Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Sun, 7 Nov 2021 23:32:56 +0100
From:   Ansuel Smith <ansuelsmth@gmail.com>
To:     Marek =?iso-8859-1?Q?Beh=FAn?= <kabel@kernel.org>
Cc:     Andrew Lunn <andrew@lunn.ch>,
        Vivien Didelot <vivien.didelot@gmail.com>,
        Florian Fainelli <f.fainelli@gmail.com>,
        Vladimir Oltean <olteanv@gmail.com>,
        "David S. Miller" <davem@davemloft.net>,
        Jakub Kicinski <kuba@kernel.org>,
        Rob Herring <robh+dt@kernel.org>,
        Jonathan Corbet <corbet@lwn.net>, Pavel Machek <pavel@ucw.cz>,
        John Crispin <john@phrozen.org>, netdev@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
        linux-doc@vger.kernel.org, linux-leds@vger.kernel.org
Subject: Re: [RFC PATCH 2/6] leds: permit to declare supported offload
 triggers
Message-ID: <YYhUGNs1I0RWriln@Ansuel-xps.localdomain>
References: <20211107175718.9151-1-ansuelsmth@gmail.com>
 <20211107175718.9151-3-ansuelsmth@gmail.com>
 <20211107230624.5251eccb@thinkpad>
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <20211107230624.5251eccb@thinkpad>
Precedence: bulk

On Sun, Nov 07, 2021 at 11:06:24PM +0100, Marek Beh?n wrote:
> On Sun,  7 Nov 2021 18:57:14 +0100
> Ansuel Smith <ansuelsmth@gmail.com> wrote:
> 
> > With LEDs that can be offload driven, permit to declare supported triggers
> > in the dts and add them to the cled struct to be used by the related
> > offload trigger. This is particurally useful for phy that have support
> > for HW blinking on tx/rx traffic or based on the speed link.
> > 
> > Signed-off-by: Ansuel Smith <ansuelsmth@gmail.com>
> 
> NAK. The device-tree shouldn't define this, only the LED's function as
> designated by the manufacturer of the device.
> 
> Marek

Sure I will add a way to ask the led driver if the trigger is supported
and report it.

-- 
	Ansuel