diff mbox series

[v2] dt-bindings: hwmon: Increase max number of io-channels

Message ID 5110a313a5ce52ce6d7b5cb6b08368d42063dc30.1701419691.git.michal.simek@amd.com
State Not Applicable
Headers show
Series [v2] dt-bindings: hwmon: Increase max number of io-channels | expand

Checks

Context Check Description
robh/checkpatch success
robh/patch-applied success
robh/dtbs-check warning build log
robh/dt-meta-schema success

Commit Message

Michal Simek Dec. 1, 2023, 8:35 a.m. UTC 
The Analog Monitoring System (AMS) defines 51 channels
(Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml)
that's why increase number to 51.

Signed-off-by: Michal Simek <michal.simek@amd.com>
Acked-by: Conor Dooley <conor.dooley@microchip.com>
---

Changes in v2:
- Add Conor's ack
- Update commit message

 Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Guenter Roeck Dec. 3, 2023, 5:15 p.m. UTC | #1 
On Fri, Dec 01, 2023 at 09:35:00AM +0100, Michal Simek wrote:
> The Analog Monitoring System (AMS) defines 51 channels
> (Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml)
> that's why increase number to 51.
> 
> Signed-off-by: Michal Simek <michal.simek@amd.com>
> Acked-by: Conor Dooley <conor.dooley@microchip.com>

Applied to hwmon-next. If there are still concerns about the commit message
please let me know and I'll drop it until resolved.

Thanks,
Guenter
Conor Dooley Dec. 4, 2023, 12:33 a.m. UTC | #2 
On Sun, Dec 03, 2023 at 09:15:40AM -0800, Guenter Roeck wrote:
> On Fri, Dec 01, 2023 at 09:35:00AM +0100, Michal Simek wrote:
> > The Analog Monitoring System (AMS) defines 51 channels
> > (Documentation/devicetree/bindings/iio/adc/xlnx,zynqmp-ams.yaml)
> > that's why increase number to 51.
> > 
> > Signed-off-by: Michal Simek <michal.simek@amd.com>
> > Acked-by: Conor Dooley <conor.dooley@microchip.com>
> 
> Applied to hwmon-next. If there are still concerns about the commit message
> please let me know and I'll drop it until resolved.

You're good. I did ack the v1 wording even if I didn't like it, didn't
feel it was worth one resend, definitely not requesting _two_ resends.
diff mbox series

Patch

diff --git a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
index e5b24782f448..be5c7d4579bb 100644
--- a/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
+++ b/Documentation/devicetree/bindings/hwmon/iio-hwmon.yaml
@@ -19,7 +19,7 @@  properties:
 
   io-channels:
     minItems: 1
-    maxItems: 8 # Should be enough
+    maxItems: 51 # Should be enough
     description: >
       List of phandles to ADC channels to read the monitoring values