linux-kernel - Re: [PATCH v3 0/2] shtc1: add support for device tree bindings

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]

Date:   Tue, 7 Jul 2020 16:03:47 +0800
From:   Chris Ruehl <chris.ruehl@...ys.com.hk>
To:     unlisted-recipients:; (no To-header on input)
Cc:     Jack Lo <jack.lo@...ys.com.hk>, devicetree@...r.kernel.org,
        Jean Delvare <jdelvare@...e.com>,
        Guenter Roeck <linux@...ck-us.net>,
        linux-hwmon@...r.kernel.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v3 0/2] shtc1: add support for device tree bindings


On 7/7/2020 4:01 pm, Chris Ruehl wrote:
> Add support for DTS bindings to the shtc driver
> The patches add the compatible table and of_property_read_bool to the
> shtc1.c. Newly created Yaml document has been released to the
> Documentation/devicetree/hwmon/sensirion,shtc1.yaml
> 
> Signed-off-by: Chris Ruehl <chris.ruehl@...ys.com.hk>
> ---
>   Version 3
> 	Fix errors report with checkpatch.pl
> 	Correct logic, add (!) when check for sensirion,low_precision
>   Version 2
> 	remove the #ifdef CONFIG_OF
> 	ignore platform data if dev->of_node is valid
> 	use boolean only therefor use sensirion,low_precise to fit the logic
> 	add missing driver.of_match_table entry
>   Version 1
> 	initial version
> 

I didn't fix the WARNINGS coming up with checkpatch, they are initial commit 
errors when added to the kernel.
But I have a patch pending to fix the format issue , once this is accepted.