linux-kernel - RE: [PATCH v5 1/3] dt-bindings: hwmon: Add Microchip emc2305 support

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID:
 <DU7PR04MB111631E2E5B074A306033D001FFD92@DU7PR04MB11163.eurprd04.prod.outlook.com>
Date: Wed, 19 Mar 2025 11:54:57 +0000
From: "Florin Leotescu (OSS)" <florin.leotescu@....nxp.com>
To: Krzysztof Kozlowski <krzk@...nel.org>, "Florin Leotescu (OSS)"
	<florin.leotescu@....nxp.com>
CC: Jean Delvare <jdelvare@...e.com>, Guenter Roeck <linux@...ck-us.net>, Rob
 Herring <robh@...nel.org>, Krzysztof Kozlowski <krzk+dt@...nel.org>, Conor
 Dooley <conor+dt@...nel.org>, Michael Shych <michaelsh@...dia.com>,
	"linux-hwmon@...r.kernel.org" <linux-hwmon@...r.kernel.org>,
	"devicetree@...r.kernel.org" <devicetree@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>, Viorel Suman
	<viorel.suman@....com>, Carlos Song <carlos.song@....com>,
	"linux-arm-kernel@...ts.infradead.org"
	<linux-arm-kernel@...ts.infradead.org>, "imx@...ts.linux.dev"
	<imx@...ts.linux.dev>, "festevam@...il.com" <festevam@...il.com>, Florin
 Leotescu <florin.leotescu@....com>, Frank Li <frank.li@....com>
Subject: RE: [PATCH v5 1/3] dt-bindings: hwmon: Add Microchip emc2305 support

>>On Tue, Mar 18, 2025 at 10:54:42AM +0200, florin.leotescu@....nxp.com wrote:
>> From: Florin Leotescu <florin.leotescu@....com>
>> 
>> Introduce yaml schema for Microchip emc2305 pwm fan controller.
>> 
>> Signed-off-by: Florin Leotescu <florin.leotescu@....com>
>> Reviewed-by: Frank Li <Frank.Li@....com>
>> ---
>>  .../bindings/hwmon/microchip,emc2305.yaml     | 113 ++++++++++++++++++
>>  1 file changed, 113 insertions(+)
>>  create mode 100644 
>> Documentation/devicetree/bindings/hwmon/microchip,emc2305.yaml
>> 
>> diff --git 
>> a/Documentation/devicetree/bindings/hwmon/microchip,emc2305.yaml 
>> b/Documentation/devicetree/bindings/hwmon/microchip,emc2305.yaml
>> new file mode 100644
>> index 000000000000..e61ef97e63af
>> --- /dev/null
>> +++ b/Documentation/devicetree/bindings/hwmon/microchip,emc2305.yaml
>> @@ -0,0 +1,113 @@
>> +# SPDX-License-Identifier: (GPL-2.0-only OR BSD-2-Clause) %YAML 1.2
>> +---
>> +

> No blank line here. Use existing code as template. If you find such code, share so we can fix it.

Thanks!   I will remove it.

Identified similar blank line on the following dt-binding files:

1 adi,ad741x.yaml
2 adi,adm1275.yaml
3 adi,ltc2991.yaml
4 maxim,max20730.yaml
5 maxim,max6639.yaml
6 maxim,max6650.yaml
7 nuvoton,nct6775.yaml
8 nuvoton,nct7363.yaml
9 nuvoton,nct7802.yaml
10 ti,adc128d818.yaml
11 ti,ads7828.yaml
12 ti,ina2xx.yaml
13 ti,lm87.yaml
14 ti,tmp513.yaml
15 ti,tps23861.yaml
16 winbond,w83781d.yaml

>> +$id: http://devicetree.org/schemas/hwmon/microchip,emc2305.yaml#
>> +$schema: http://devicetree.org/meta-schemas/core.yaml#
>> +
>> +title: Microchip EMC2305 SMBus compliant PWM fan controller
>> +
>> +maintainers:
>> +  - Michael Shych <michaelsh@...dia.com>
>> +
>> +description:
>> +  Microchip EMC2301/2/3/5 pwm controller which supports
>> +  up to five programmable fan control circuits.
>> +
>> +properties:
>> +  compatible:
>> +    oneOf:
>> +      - enum:
>> +          - microchip,emc2305
>> +      - items:
>> +          - enum:
>> +              - microchip,emc2303
>> +              - microchip,emc2302
>> +              - microchip,emc2301
>> +          - const: microchip,emc2305
>> +
>> +  reg:
>> +    maxItems: 1
>> +
>> +  '#address-cells':
>> +    const: 1
>> +
>> +  '#size-cells':
>> +    const: 0
>> +
>> +  '#pwm-cells':
>> +    const: 3
>> +    description: |
>> +      Number of cells in a PWM specifier.
>> +      - cell 0: The PWM frequency
>> +      - cell 1: The PWM polarity: 0 or PWM_POLARITY_INVERTED
>> +      - cell 2: The PWM output config:
>> +           - 0 (Open-Drain)
>> +           - 1 (Push-Pull)
>> +
>> +

> Just one blank line

Ok, I'll remove it. Thank you!

> Reviewed-by: Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>

Best regards,
Florin