Received: by 2002:a05:6358:d09b:b0:dc:cd0c:909e with SMTP id jc27csp1039767rwb; Thu, 1 Dec 2022 11:29:49 -0800 (PST) X-Google-Smtp-Source: AA0mqf5TNC44cmrxkipgB7Prkz0fCXfS/0wdbHKngVKhgn0bXIwo9k+fFdvzmLoYSTWDOsvLdvgM X-Received: by 2002:a17:906:3510:b0:781:b7f2:bce9 with SMTP id r16-20020a170906351000b00781b7f2bce9mr58486264eja.269.1669922989063; Thu, 01 Dec 2022 11:29:49 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1669922989; cv=none; d=google.com; s=arc-20160816; b=nubm9oSFIZ8OE6bdk0TTLjWk4dHNdfYqdZOTChiumPX/wHdjk8DHirY6YehopaOxzQ Ustq7iMoNOe3/o2WudQK+5fxOMSCbKxAWM7I3X7AXVrJr6W5O5oYu/vrzkedk3MpF4D6 Dda7mwH/jCwy2bmxvdHB7c9ZPzbGCB60WHGUwJC4SlxLSm4k6s0AWbdalXjRKJt49xC3 zjU5Q8zoqTQlvuAHIx/0jjgzCFZA346c1+TR9hWkrAZeWdt54EH+f+Zv3FuSPdCH0fvc rAe1Bi2cPpmZtLmN+I85w3jGLnFABdqC8TFWpNqIv/OfB0IfvHDPBZpbFMorvwaH0vj4 zmJg== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:cc:to:subject:message-id:date:from:in-reply-to :references:mime-version:dkim-signature; bh=i63ILiYKNdFdyIi+lkwKMcE7oZYw8auZyDHNIq81dXA=; b=s9ly1fWKJKQRkb0hN4Y5xd9EIiof31//50BBZsT0f/+dBlfdUtENBp6V/pJlJf94S9 iNLI7k3G+Qgs5fR1t64bVwnsnXRdSqUVepS3hPq8lnW0yPL6QCY80Te4ehDEhDtz9KoY 6VPZoZqiMfpjNZwkfpUIlhGO79XXLUf34AvzOZTeK+jGRglIpTHaJuKCCITITDP7Wp93 b8xEAs7TftHogVnEQ3xHhtcZz48wAxNAwJTv5hm/v1/vhJw/tJx+VG1pczRvoG/aE7b2 zIVi1Vy8oa5dIUkxojZIPbBh37GihZgRNidrOXa9PCblXI30wEfil3oHJ4M+CTjceUyb rM1w== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@gateworks-com.20210112.gappssmtp.com header.s=20210112 header.b=dREByz5g; 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 Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id l22-20020a056402231600b00468fd5712a3si4020888eda.351.2022.12.01.11.29.29; Thu, 01 Dec 2022 11:29:49 -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=@gateworks-com.20210112.gappssmtp.com header.s=20210112 header.b=dREByz5g; 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 Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229447AbiLASgB (ORCPT + 82 others); Thu, 1 Dec 2022 13:36:01 -0500 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:33256 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229810AbiLASf7 (ORCPT ); Thu, 1 Dec 2022 13:35:59 -0500 Received: from mail-pf1-x42a.google.com (mail-pf1-x42a.google.com [IPv6:2607:f8b0:4864:20::42a]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 2338AAC6EA for ; Thu, 1 Dec 2022 10:35:58 -0800 (PST) Received: by mail-pf1-x42a.google.com with SMTP id x66so2705953pfx.3 for ; Thu, 01 Dec 2022 10:35:58 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gateworks-com.20210112.gappssmtp.com; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:from:to:cc:subject:date:message-id:reply-to; bh=i63ILiYKNdFdyIi+lkwKMcE7oZYw8auZyDHNIq81dXA=; b=dREByz5gq5l+QODuRzOM/db1b5cUlqK+44d9FNtc4uNKR7VTcWUcPAFzTNBX4ZZ2Sl 3+9kMTz+85EPk9258asao9s2h5yOKFOc8Ym2CCBwvQqxMWcmBt3bJC18w3mVOQUqNuML p258ChiwJAdIxYyXp4ikct9qEYtTboETexXQ/Ijynq7HyVB07gi9LxZXNSPOqFS7C5iv QFWbgvE081x9HjgKryHgRSCKoJLBLuIY1ylCrDNglH7Lr1Bl7F/Klt6zAtzZBhn8a6Fv AdK93/L/LgyQveNPXK5iIsQQmEh+33LCq/V2gGQ+MwP0T4900PS7cZVrky82ZDhL8lYs MBrw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=cc:to:subject:message-id:date:from:in-reply-to:references :mime-version:x-gm-message-state:from:to:cc:subject:date:message-id :reply-to; bh=i63ILiYKNdFdyIi+lkwKMcE7oZYw8auZyDHNIq81dXA=; b=oeDnfd92zXqrQA3UukOonJzAQyzQEcv/EQEHlcnViUn/6FXVi2fj37tIFltvycKVwf BBcn3lPaPddAW0E4GzhX1HPakWT1Nayh4dDe5rKdxzGXRwXp7xHv0r7i1BvYA3AxpGCM lCQAUx9zlGcakSwS6G4LDoOUXX5nGKXdtnVZR1HDSNpcgBYkN02Nlr9dRawyejSQrjBG IxrKBBs40/ixqq8BC1tFEHE2fJ9hlP6hnvYEgtsWexbiJen0vTyO5d+jOL0xBqeWcRaN ijXmHkkougl84XLFHnDsIojx/0iGIGjZP9Ilxy2SE7l8TCs6wheH2ByqyMCUTiBnbEl6 MMBg== X-Gm-Message-State: ANoB5plC28Dsb/3qj+ybSMowQSbloXGs4Whm+Jlnr/W+uCvMoub32VOJ wSvZ83NqIVDtzsYGk338c3imcgGZi45WjJRrwG9Jtw== X-Received: by 2002:a63:f214:0:b0:477:f449:3644 with SMTP id v20-20020a63f214000000b00477f4493644mr25277863pgh.484.1669919757574; Thu, 01 Dec 2022 10:35:57 -0800 (PST) MIME-Version: 1.0 References: <20221118001548.635752-1-tharvey@gateworks.com> <6388f310.050a0220.532be.7cd5@mx.google.com> In-Reply-To: <6388f310.050a0220.532be.7cd5@mx.google.com> From: Tim Harvey Date: Thu, 1 Dec 2022 10:35:46 -0800 Message-ID: Subject: Re: [PATCH 0/3] add dt configuration for dp83867 led modes To: Christian Marangi Cc: 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 , Andrew Lunn Content-Type: text/plain; charset="UTF-8" X-Spam-Status: No, score=-1.9 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,RCVD_IN_DNSWL_NONE,SPF_HELO_NONE,SPF_PASS 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 Thu, Dec 1, 2022 at 10:31 AM Christian Marangi wrote: > > On Thu, Dec 01, 2022 at 10:26:09AM -0800, Tim Harvey wrote: > > On Sun, Nov 20, 2022 at 3:35 PM Andrew Lunn wrote: > > > > > > 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 > > > > Ansuel, > > > > Are you planning on posting a v7 of 'Adds support for PHY LEDs with > > offload triggers' [1]? > > > > Best Regards, > > > > Tim > > [1] https://patches.linaro.org/project/linux-leds/list/?series=174704 > > I can consider that only if there is a real interest for it and would > love some help by the netdev team to actually have a review from the > leds team... > > I tried multiple time to propose it but I never got a review... only a > review from an external guy that wanted to follow his idea in every way > possible with the only intention of applying his code (sorry to be rude > about that but i'm more than happy to recover the work and search for a > common solution) > > So yes this is still in my TODO list but it would help if others can > tell me that they want to actually review it. That would put that > project on priority and I would recover and push a v7. > > -- > Ansuel Ansuel, Considering Andrew is nak'ing any phy code to configure LED's until a solution using via /sys/class/leds is provided I would say there is real interest. It seems to me that you got very positive feedback for this last series. I would think if you submitted without RFC it would catch more eyes as well. Best Regards, Tim