* [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address
@ 2025-07-03 14:46 Fabio Estevam
2025-07-08 17:06 ` Rob Herring (Arm)
2025-07-23 9:32 ` (subset) " Lee Jones
0 siblings, 2 replies; 3+ messages in thread
From: Fabio Estevam @ 2025-07-03 14:46 UTC (permalink / raw)
To: pavel
Cc: lee, robh, krzk+dt, conor+dt, alkuor, linux-leds, devicetree,
Fabio Estevam
The NCP5623C variant has the I2C address at 0x39 according its datasheet:
https://www.mouser.com/datasheet/2/308/NCP5623C-D-64591.pdf
Make 0x39 a valid I2C address in the dt-binding.
Signed-off-by: Fabio Estevam <festevam@gmail.com>
---
Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git a/Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml b/Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml
index 9c9f3a682ba2..11d45c7f741d 100644
--- a/Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml
+++ b/Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml
@@ -19,7 +19,9 @@ properties:
- onnn,ncp5623
reg:
- const: 0x38
+ enum:
+ - 0x38
+ - 0x39
multi-led:
type: object
--
2.34.1
^ permalink raw reply related [flat|nested] 3+ messages in thread
* Re: [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address
2025-07-03 14:46 [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address Fabio Estevam
@ 2025-07-08 17:06 ` Rob Herring (Arm)
2025-07-23 9:32 ` (subset) " Lee Jones
1 sibling, 0 replies; 3+ messages in thread
From: Rob Herring (Arm) @ 2025-07-08 17:06 UTC (permalink / raw)
To: Fabio Estevam
Cc: lee, krzk+dt, pavel, linux-leds, devicetree, alkuor, conor+dt
On Thu, 03 Jul 2025 11:46:44 -0300, Fabio Estevam wrote:
> The NCP5623C variant has the I2C address at 0x39 according its datasheet:
>
> https://www.mouser.com/datasheet/2/308/NCP5623C-D-64591.pdf
>
> Make 0x39 a valid I2C address in the dt-binding.
>
> Signed-off-by: Fabio Estevam <festevam@gmail.com>
> ---
> Documentation/devicetree/bindings/leds/onnn,ncp5623.yaml | 4 +++-
> 1 file changed, 3 insertions(+), 1 deletion(-)
>
Acked-by: Rob Herring (Arm) <robh@kernel.org>
^ permalink raw reply [flat|nested] 3+ messages in thread
* Re: (subset) [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address
2025-07-03 14:46 [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address Fabio Estevam
2025-07-08 17:06 ` Rob Herring (Arm)
@ 2025-07-23 9:32 ` Lee Jones
1 sibling, 0 replies; 3+ messages in thread
From: Lee Jones @ 2025-07-23 9:32 UTC (permalink / raw)
To: pavel, Fabio Estevam
Cc: lee, robh, krzk+dt, conor+dt, alkuor, linux-leds, devicetree
On Thu, 03 Jul 2025 11:46:44 -0300, Fabio Estevam wrote:
> The NCP5623C variant has the I2C address at 0x39 according its datasheet:
>
> https://www.mouser.com/datasheet/2/308/NCP5623C-D-64591.pdf
>
> Make 0x39 a valid I2C address in the dt-binding.
>
>
> [...]
Applied, thanks!
[1/1] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address
commit: 4903924ac7ef31fbbe48b3261b1bc86ce6cd7e97
--
Lee Jones [李琼斯]
^ permalink raw reply [flat|nested] 3+ messages in thread
end of thread, other threads:[~2025-07-23 9:32 UTC | newest]
Thread overview: 3+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2025-07-03 14:46 [PATCH] dt-bindings: leds: ncp5623: Add 0x39 as a valid I2C address Fabio Estevam
2025-07-08 17:06 ` Rob Herring (Arm)
2025-07-23 9:32 ` (subset) " Lee Jones
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).