Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp3498774rwb; Sun, 20 Nov 2022 15:38:53 -0800 (PST) X-Google-Smtp-Source: AA0mqf6AWIu8iiau5UdBpPPrBi4aBj/RGh7f3MyeYSfM/b2yoTKCR2mwloi5luhxmjm7qNFelS0a X-Received: by 2002:a05:6a00:798:b0:566:9f68:c0ad with SMTP id g24-20020a056a00079800b005669f68c0admr17776204pfu.57.1668987532934; Sun, 20 Nov 2022 15:38:52 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1668987532; cv=none; d=google.com; s=arc-20160816; b=sGjkOHDtPuCxiPDe+6/K0fStSx7l26/y/J4PyoVyxd4UnvkqZwrXBGwr/lHeTO5lwY YorUekdEVMMMU1ZbxdUJWPsciisOz3TqxcrE8lWfq+LOUesAaTjSRcNi97YXhG5cgfhQ PCZTcFOa44KmzCzTYGpJIfZMKNVFID6BNYTGK+Ywkk+3SRXjGH84AQdwSLjg5koFbZ0v J9kmcyfXN655VSkl/eBJi6ZMY9gsGofKe6t9TYlkmEWUW3/MDqALEJA9+JMNMnD3dmIK SEwZ9vRQ78a9j3pLOGLSNJ+Vn8RCdAugd/jT3JdikXK60rDi22+JyXH97kDTvoE76bLW Ngjg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:content-disposition:mime-version :references:message-id:subject:cc:to:from:date:dkim-signature; bh=aI2jnEgK19wCHQqoMeLWDiWUB6RdVgk21HiT4Yh9Fak=; b=HjGNXTc3wOAM4vK8bLG1V0AF2dbTj76UhAQzTFO6OQKpQBc6uTnHx7BzK1VHi4p1Uz Aodc0jRCXhJB+rUsBlqwjHluYUO6pSntWk1I+B7P63ryUTKWZ7Jnn5XpUqFGnhUz1yb4 ma9IX7R76kmPZuOtXtypT+Bs0TvHSPobMsRn0dRaMxqT8AOY11m1v742lD+5jIor6o+5 gUhfvtr67KrkBca6rBZ9N0PqdszkBeUQCVgMRrNeFMM/A9YYFjWnBK5mkMM3im8/bCC4 yJ7IKTKtM/SMpNCqmXLTZxm/lZG1z3G2tto2SnVq85RGzGS2qRBWix9pMq+ucjCPhanB reFg== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b=cYz+hH9J; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=lunn.ch Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id rj5-20020a17090b3e8500b001fe41707a2esi14743116pjb.90.2022.11.20.15.38.40; Sun, 20 Nov 2022 15:38:52 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@lunn.ch header.s=20171124 header.b=cYz+hH9J; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=lunn.ch Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229672AbiKTXgL (ORCPT + 90 others); Sun, 20 Nov 2022 18:36:11 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:44560 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229530AbiKTXgJ (ORCPT ); Sun, 20 Nov 2022 18:36:09 -0500 Received: from vps0.lunn.ch (vps0.lunn.ch [156.67.10.101]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id BBA8C2B613; Sun, 20 Nov 2022 15:36:03 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lunn.ch; s=20171124; h=In-Reply-To:Content-Disposition:Content-Type:MIME-Version: References:Message-ID:Subject:Cc:To:From:Date:From:Sender:Reply-To:Subject: Date:Message-ID:To:Cc:MIME-Version:Content-Type:Content-Transfer-Encoding: Content-ID:Content-Description:Content-Disposition:In-Reply-To:References; bh=aI2jnEgK19wCHQqoMeLWDiWUB6RdVgk21HiT4Yh9Fak=; b=cYz+hH9JTM6DDVFwQJJxm1/mW+ 4U7d5VyO+Na0o05Z9b75x0D4Kq1odfCa5rESy/6fgNMC7pA2w+B0Ja3R44WyaJy0cVsIE8AALcU0T /t9evWN/T6aj49iIwdKgvun2RCqmF8a1H5zlG/f6i0HBY6YRgZ04wfp2BiQhj/kZBN1s=; Received: from andrew by vps0.lunn.ch with local (Exim 4.94.2) (envelope-from ) id 1owtqJ-002x8q-Qr; Mon, 21 Nov 2022 00:35:19 +0100 Date: Mon, 21 Nov 2022 00:35:19 +0100 From: Andrew Lunn To: Tim Harvey Cc: Ansuel Smith , linux-kernel@vger.kernel.org, linux-arm-kernel@lists.infradead.org, devicetree@vger.kernel.org, netdev@vger.kernel.org, Heiner Kallweit , Russell King , Shawn Guo , Sascha Hauer , Pengutronix Kernel Team , Fabio Estevam , Rob Herring , Krzysztof Kozlowski Subject: Re: [PATCH 0/3] add dt configuration for dp83867 led modes Message-ID: References: <20221118001548.635752-1-tharvey@gateworks.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,T_SPF_HELO_TEMPERROR, T_SPF_TEMPERROR autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Fri, Nov 18, 2022 at 11:57:00AM -0800, Tim Harvey wrote: > On Fri, Nov 18, 2022 at 5:11 AM Andrew Lunn wrote: > > > > > Andrew, > > > > > > I completely agree with you but I haven't seen how that can be done > > > yet. What support exists for a PHY driver to expose their LED > > > configuration to be used that way? Can you point me to an example? > > > > Nobody has actually worked on this long enough to get code merged. e.g. > > https://lore.kernel.org/netdev/20201004095852.GB1104@bug/T/ > > https://lists.archive.carbon60.com/linux/kernel/3396223 > > > > This is probably the last attempt, which was not too far away from getting merged: > > https://patches.linaro.org/project/linux-leds/cover/20220503151633.18760-1-ansuelsmth@gmail.com/ > > > > I seem to NACK a patch like yours every couple of months. If all that > > wasted time was actually spent on a common framework, this would of > > been solved years ago. > > > > How important is it to you to control these LEDs? Enough to finish > > this code and get it merged? > > > > Andrew, > > Thanks for the links - the most recent attempt does look promising. > For whatever reason I don't have that series in my mail history so > it's not clear how I can respond to it. apt-get install b4 > Ansuel, are you planning on posting a v7 of 'Adds support for PHY LEDs > with offload triggers' [1]? > > I'm not all that familiar with netdev led triggers. Is there a way to > configure the default offload blink mode via dt with your series? I > didn't quite follow how the offload function/blink-mode gets set. The idea is that the PHY LEDs are just LEDs in the Linux LED framework. So read Documentation/devicetree/bindings/leds/common.yaml. The PHY should make use of these standard DT properties, including linux,default-trigger. Andrew