* Re: [PATCH RESEND 0/8] Resend LED patches [not found] <20221226123630.6515-1-pali@kernel.org> @ 2023-01-20 16:41 ` Arnd Bergmann 2023-01-20 17:15 ` Lee Jones 0 siblings, 1 reply; 5+ messages in thread From: Arnd Bergmann @ 2023-01-20 16:41 UTC (permalink / raw) To: Pali Rohár, Linus Walleij Cc: Lee Jones, linux-kernel, Nicholas Piggin, soc, Pavel Machek, linuxppc-dev, linux-leds On Mon, Dec 26, 2022, at 13:36, Pali Rohár wrote: > Linus Walleij suggested me to send these patches to SoC tree [1] > instead. So I'm doing it. > > This patch series contains LED patches which are on the linux-leds > mailing list for a long time without any future movement. Could you > please handle them here via SoC tree? Thanks. > > [1] - > https://lore.kernel.org/linux-leds/CACRpkdad6WDo7rGfa4MW8zz0mLXmcPHo+SEC-yLQnRz_kdrryA@mail.gmail.com/ I'm going through the backlog of patches sent to soc@kernel.org and came across this series. While I don't mind taking these patches through the soc tree in principle, it is important that this is only done as an exception, and with all the relevant parties on Cc. In particular, the original series that you got no feedback for did not include the arch/powerpc/ changes, and I would assume those should go through the powerpc tree anyway. We have recently decided to take risc-v and loongarch dts changes through the soc tree, and I don't mind doing it for powerpc as well if the powerpc maintainers prefer that, but this is not something we have even discussed so far. I've added everyone to Cc on this mail, but please resend the series once more so everyone has the patches, and then we can decide who will pick up what. Arnd > > Marek Behún (3): > leds: turris-omnia: support HW controlled mode via private trigger > leds: turris-omnia: initialize multi-intensity to full > leds: turris-omnia: change max brightness from 255 to 1 > > Pali Rohár (5): > dt-bindings: leds: register-bit-led: Add active-low property > leds: syscon: Implement support for active-low property > powerpc/85xx: DTS: Add CPLD definitions for P1021RDB Combo Board CPL > Design > dt-bindings: leds: Add cznic,turris1x-leds.yaml binding > leds: Add support for Turris 1.x LEDs > > .../testing/sysfs-class-led-driver-turris1x | 31 ++ > .../bindings/leds/cznic,turris1x-leds.yaml | 118 +++++ > .../bindings/leds/register-bit-led.yaml | 5 + > arch/powerpc/boot/dts/fsl/p1020mbg-pc.dtsi | 92 ++++ > arch/powerpc/boot/dts/fsl/p1020mbg-pc_32b.dts | 6 +- > arch/powerpc/boot/dts/fsl/p1020mbg-pc_36b.dts | 6 +- > arch/powerpc/boot/dts/fsl/p1020rdb-pd.dts | 44 +- > arch/powerpc/boot/dts/fsl/p1020utm-pc.dtsi | 37 ++ > arch/powerpc/boot/dts/fsl/p1020utm-pc_32b.dts | 4 +- > arch/powerpc/boot/dts/fsl/p1020utm-pc_36b.dts | 4 +- > arch/powerpc/boot/dts/fsl/p1021rdb-pc.dtsi | 37 ++ > arch/powerpc/boot/dts/fsl/p1021rdb-pc_32b.dts | 5 +- > arch/powerpc/boot/dts/fsl/p1021rdb-pc_36b.dts | 5 +- > arch/powerpc/boot/dts/fsl/p2020rdb-pc.dtsi | 33 +- > drivers/leds/Kconfig | 10 + > drivers/leds/Makefile | 1 + > drivers/leds/leds-syscon.c | 14 +- > drivers/leds/leds-turris-1x.c | 474 ++++++++++++++++++ > drivers/leds/leds-turris-omnia.c | 46 +- > 19 files changed, 945 insertions(+), 27 deletions(-) > create mode 100644 > Documentation/ABI/testing/sysfs-class-led-driver-turris1x > create mode 100644 > Documentation/devicetree/bindings/leds/cznic,turris1x-leds.yaml > create mode 100644 drivers/leds/leds-turris-1x.c > > -- > 2.20.1 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND 0/8] Resend LED patches 2023-01-20 16:41 ` [PATCH RESEND 0/8] Resend LED patches Arnd Bergmann @ 2023-01-20 17:15 ` Lee Jones 2023-01-20 17:47 ` Arnd Bergmann 2023-01-26 20:07 ` Linus Walleij 0 siblings, 2 replies; 5+ messages in thread From: Lee Jones @ 2023-01-20 17:15 UTC (permalink / raw) To: Arnd Bergmann Cc: Linus Walleij, linuxppc-dev, linux-kernel, Nicholas Piggin, soc, Pavel Machek, Pali Rohár, linux-leds On Fri, 20 Jan 2023, Arnd Bergmann wrote: > On Mon, Dec 26, 2022, at 13:36, Pali Rohár wrote: > > Linus Walleij suggested me to send these patches to SoC tree [1] > > instead. So I'm doing it. > > > > This patch series contains LED patches which are on the linux-leds > > mailing list for a long time without any future movement. Could you > > please handle them here via SoC tree? Thanks. > > > > [1] - > > https://lore.kernel.org/linux-leds/CACRpkdad6WDo7rGfa4MW8zz0mLXmcPHo+SEC-yLQnRz_kdrryA@mail.gmail.com/ > > I'm going through the backlog of patches sent to soc@kernel.org > and came across this series. While I don't mind taking these > patches through the soc tree in principle, it is important > that this is only done as an exception, and with all the > relevant parties on Cc. > > In particular, the original series that you got no > feedback for did not include the arch/powerpc/ changes, > and I would assume those should go through the powerpc > tree anyway. We have recently decided to take > risc-v and loongarch dts changes through the soc > tree, and I don't mind doing it for powerpc as well > if the powerpc maintainers prefer that, but this is > not something we have even discussed so far. > > I've added everyone to Cc on this mail, but please > resend the series once more so everyone has the patches, > and then we can decide who will pick up what. Thanks Arnd (PSB). > > Marek Behún (3): > > leds: turris-omnia: support HW controlled mode via private trigger > > leds: turris-omnia: initialize multi-intensity to full > > leds: turris-omnia: change max brightness from 255 to 1 > > > > Pali Rohár (5): > > dt-bindings: leds: register-bit-led: Add active-low property > > leds: syscon: Implement support for active-low property > > powerpc/85xx: DTS: Add CPLD definitions for P1021RDB Combo Board CPL > > Design > > dt-bindings: leds: Add cznic,turris1x-leds.yaml binding > > leds: Add support for Turris 1.x LEDs > > > > .../testing/sysfs-class-led-driver-turris1x | 31 ++ > > .../bindings/leds/cznic,turris1x-leds.yaml | 118 +++++ > > .../bindings/leds/register-bit-led.yaml | 5 + > > arch/powerpc/boot/dts/fsl/p1020mbg-pc.dtsi | 92 ++++ > > arch/powerpc/boot/dts/fsl/p1020mbg-pc_32b.dts | 6 +- > > arch/powerpc/boot/dts/fsl/p1020mbg-pc_36b.dts | 6 +- > > arch/powerpc/boot/dts/fsl/p1020rdb-pd.dts | 44 +- > > arch/powerpc/boot/dts/fsl/p1020utm-pc.dtsi | 37 ++ > > arch/powerpc/boot/dts/fsl/p1020utm-pc_32b.dts | 4 +- > > arch/powerpc/boot/dts/fsl/p1020utm-pc_36b.dts | 4 +- > > arch/powerpc/boot/dts/fsl/p1021rdb-pc.dtsi | 37 ++ > > arch/powerpc/boot/dts/fsl/p1021rdb-pc_32b.dts | 5 +- > > arch/powerpc/boot/dts/fsl/p1021rdb-pc_36b.dts | 5 +- > > arch/powerpc/boot/dts/fsl/p2020rdb-pc.dtsi | 33 +- > > drivers/leds/Kconfig | 10 + > > drivers/leds/Makefile | 1 + > > drivers/leds/leds-syscon.c | 14 +- > > drivers/leds/leds-turris-1x.c | 474 ++++++++++++++++++ > > drivers/leds/leds-turris-omnia.c | 46 +- If everyone is convinced that applying these drivers is the correct thing to do, I'd be happy to (rather) take them via LEDs. -- Lee Jones [李琼斯] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND 0/8] Resend LED patches 2023-01-20 17:15 ` Lee Jones @ 2023-01-20 17:47 ` Arnd Bergmann 2023-01-20 20:02 ` Lee Jones 2023-01-26 20:07 ` Linus Walleij 1 sibling, 1 reply; 5+ messages in thread From: Arnd Bergmann @ 2023-01-20 17:47 UTC (permalink / raw) To: Lee Jones Cc: Linus Walleij, linuxppc-dev, linux-kernel, Nicholas Piggin, soc, Pavel Machek, Pali Rohár, linux-leds On Fri, Jan 20, 2023, at 18:15, Lee Jones wrote: > On Fri, 20 Jan 2023, Arnd Bergmann wrote: >> > Marek Behún (3): >> > leds: turris-omnia: support HW controlled mode via private trigger >> > leds: turris-omnia: initialize multi-intensity to full >> > leds: turris-omnia: change max brightness from 255 to 1 >> > >> > Pali Rohár (5): >> > dt-bindings: leds: register-bit-led: Add active-low property >> > leds: syscon: Implement support for active-low property >> > powerpc/85xx: DTS: Add CPLD definitions for P1021RDB Combo Board CPL >> > Design >> > dt-bindings: leds: Add cznic,turris1x-leds.yaml binding >> > leds: Add support for Turris 1.x LEDs >> > >> > .../testing/sysfs-class-led-driver-turris1x | 31 ++ >> > .../bindings/leds/cznic,turris1x-leds.yaml | 118 +++++ >> > .../bindings/leds/register-bit-led.yaml | 5 + >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc.dtsi | 92 ++++ >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc_32b.dts | 6 +- >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc_36b.dts | 6 +- >> > arch/powerpc/boot/dts/fsl/p1020rdb-pd.dts | 44 +- >> > arch/powerpc/boot/dts/fsl/p1020utm-pc.dtsi | 37 ++ >> > arch/powerpc/boot/dts/fsl/p1020utm-pc_32b.dts | 4 +- >> > arch/powerpc/boot/dts/fsl/p1020utm-pc_36b.dts | 4 +- >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc.dtsi | 37 ++ >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc_32b.dts | 5 +- >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc_36b.dts | 5 +- >> > arch/powerpc/boot/dts/fsl/p2020rdb-pc.dtsi | 33 +- > >> > drivers/leds/Kconfig | 10 + >> > drivers/leds/Makefile | 1 + >> > drivers/leds/leds-syscon.c | 14 +- >> > drivers/leds/leds-turris-1x.c | 474 ++++++++++++++++++ >> > drivers/leds/leds-turris-omnia.c | 46 +- > > If everyone is convinced that applying these drivers is the correct > thing to do, I'd be happy to (rather) take them via LEDs. Ok, thanks. I had not actually looked at the patches until today. They were in the soc tree backlog but appeared to be misplaced there until I read the 0/10 message text. Looking at it now, I see: - patches 1 and 2 seem obvious and have been reviewed by others already - patch 3 is for arch/powerpc and should get merged through there if there are no objections to the binding in patch 4. - patch 5 is the big driver patch, with a Reviewed-by tag from Marek Behún, who is the author of the last three patches. An earlier version of this patch was sent in June and got a few Acks and detailed feedback from Andy [1], but he's also not on Cc, and I don't know if his comments are all resolved in this version. - Patches 6, 7 and 8 all seem simple LED subsystem patches, they just need review from you in order to get applied. These are also missing a Signed-off-by from the submitter in addition to the author in order to be applied. Arnd [1] https://lore.kernel.org/all/CAHp75Vcr6o2rm+T6Tr8sS4VXCLVHtmLPWy-njOKAvO4AcZoW=A@mail.gmail.com/ ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND 0/8] Resend LED patches 2023-01-20 17:47 ` Arnd Bergmann @ 2023-01-20 20:02 ` Lee Jones 0 siblings, 0 replies; 5+ messages in thread From: Lee Jones @ 2023-01-20 20:02 UTC (permalink / raw) To: Arnd Bergmann Cc: Linus Walleij, linuxppc-dev, linux-kernel, Nicholas Piggin, soc, Pavel Machek, Pali Rohár, linux-leds On Fri, 20 Jan 2023, Arnd Bergmann wrote: > On Fri, Jan 20, 2023, at 18:15, Lee Jones wrote: > > On Fri, 20 Jan 2023, Arnd Bergmann wrote: > > >> > Marek Behún (3): > >> > leds: turris-omnia: support HW controlled mode via private trigger > >> > leds: turris-omnia: initialize multi-intensity to full > >> > leds: turris-omnia: change max brightness from 255 to 1 > >> > > >> > Pali Rohár (5): > >> > dt-bindings: leds: register-bit-led: Add active-low property > >> > leds: syscon: Implement support for active-low property > >> > powerpc/85xx: DTS: Add CPLD definitions for P1021RDB Combo Board CPL > >> > Design > >> > dt-bindings: leds: Add cznic,turris1x-leds.yaml binding > >> > leds: Add support for Turris 1.x LEDs > >> > > >> > .../testing/sysfs-class-led-driver-turris1x | 31 ++ > >> > .../bindings/leds/cznic,turris1x-leds.yaml | 118 +++++ > >> > .../bindings/leds/register-bit-led.yaml | 5 + > >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc.dtsi | 92 ++++ > >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc_32b.dts | 6 +- > >> > arch/powerpc/boot/dts/fsl/p1020mbg-pc_36b.dts | 6 +- > >> > arch/powerpc/boot/dts/fsl/p1020rdb-pd.dts | 44 +- > >> > arch/powerpc/boot/dts/fsl/p1020utm-pc.dtsi | 37 ++ > >> > arch/powerpc/boot/dts/fsl/p1020utm-pc_32b.dts | 4 +- > >> > arch/powerpc/boot/dts/fsl/p1020utm-pc_36b.dts | 4 +- > >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc.dtsi | 37 ++ > >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc_32b.dts | 5 +- > >> > arch/powerpc/boot/dts/fsl/p1021rdb-pc_36b.dts | 5 +- > >> > arch/powerpc/boot/dts/fsl/p2020rdb-pc.dtsi | 33 +- > > > >> > drivers/leds/Kconfig | 10 + > >> > drivers/leds/Makefile | 1 + > >> > drivers/leds/leds-syscon.c | 14 +- > >> > drivers/leds/leds-turris-1x.c | 474 ++++++++++++++++++ > >> > drivers/leds/leds-turris-omnia.c | 46 +- > > > > If everyone is convinced that applying these drivers is the correct > > thing to do, I'd be happy to (rather) take them via LEDs. > > Ok, thanks. I had not actually looked at the patches until today. > They were in the soc tree backlog but appeared to be misplaced > there until I read the 0/10 message text. > > Looking at it now, I see: > > - patches 1 and 2 seem obvious and have been reviewed by > others already > > - patch 3 is for arch/powerpc and should get merged through > there if there are no objections to the binding in patch 4. > > - patch 5 is the big driver patch, with a Reviewed-by tag > from Marek Behún, who is the author of the last three patches. > An earlier version of this patch was sent in June and got > a few Acks and detailed feedback from Andy [1], but he's also > not on Cc, and I don't know if his comments are all resolved > in this version. > > - Patches 6, 7 and 8 all seem simple LED subsystem patches, > they just need review from you in order to get applied. > These are also missing a Signed-off-by from the submitter > in addition to the author in order to be applied. Very well. Let's have them resent then please (with past reviewers on Cc:) and we'll go from there. -- Lee Jones [李琼斯] ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [PATCH RESEND 0/8] Resend LED patches 2023-01-20 17:15 ` Lee Jones 2023-01-20 17:47 ` Arnd Bergmann @ 2023-01-26 20:07 ` Linus Walleij 1 sibling, 0 replies; 5+ messages in thread From: Linus Walleij @ 2023-01-26 20:07 UTC (permalink / raw) To: Lee Jones Cc: Arnd Bergmann, linuxppc-dev, linux-kernel, Nicholas Piggin, soc, Pavel Machek, Pali Rohár, linux-leds On Fri, Jan 20, 2023 at 6:15 PM Lee Jones <lee@kernel.org> wrote: > If everyone is convinced that applying these drivers is the correct > thing to do, I'd be happy to (rather) take them via LEDs. Oh you are co-maintainer of the LED subsystem since a month! Sadly this series stalled way before that, so that's why we didn't notice. By all means, pick it up! Yours, Linus Walleij ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-01-26 20:08 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
[not found] <20221226123630.6515-1-pali@kernel.org>
2023-01-20 16:41 ` [PATCH RESEND 0/8] Resend LED patches Arnd Bergmann
2023-01-20 17:15 ` Lee Jones
2023-01-20 17:47 ` Arnd Bergmann
2023-01-20 20:02 ` Lee Jones
2023-01-26 20:07 ` Linus Walleij
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).