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 PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Sun, 17 Oct 2021 18:18:05 +0200 From: David Heidelberg <david@...t.cz> To: Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com> Cc: Rob Herring <robh@...nel.org>, netdev@...r.kernel.org, linux-kernel@...r.kernel.org, Rob Herring <robh+dt@...nel.org>, Jakub Kicinski <kuba@...nel.org>, "David S. Miller" <davem@...emloft.net>, devicetree@...r.kernel.org, ~okias/devicetree@...ts.sr.ht Subject: Re: [PATCH v3] dt-bindings: net: nfc: nxp,pn544: Convert txt bindings to yaml On Sun, Oct 10 2021 at 23:01:24 +0200, Krzysztof Kozlowski <krzysztof.kozlowski@...onical.com> wrote: > On 10/10/2021 21:31, Rob Herring wrote: >> On Sat, 09 Oct 2021 18:19:42 +0200, David Heidelberg wrote: >>> Convert bindings for NXP PN544 NFC driver to YAML syntax. >>> >>> Signed-off-by: David Heidelberg <david@...t.cz> >>> --- >>> v2 >>> - Krzysztof is a maintainer >>> - pintctrl dropped >>> - 4 space indent for example >>> - nfc node name >>> v3 >>> - remove whole pinctrl >>> .../bindings/net/nfc/nxp,pn544.yaml | 61 >>> +++++++++++++++++++ >>> .../devicetree/bindings/net/nfc/pn544.txt | 33 ---------- >>> 2 files changed, 61 insertions(+), 33 deletions(-) >>> create mode 100644 >>> Documentation/devicetree/bindings/net/nfc/nxp,pn544.yaml >>> delete mode 100644 >>> Documentation/devicetree/bindings/net/nfc/pn544.txt >>> >> >> Running 'make dtbs_check' with the schema in this patch gives the >> following warnings. Consider if they are expected or the schema is >> incorrect. These may not be new warnings. >> >> Note that it is not yet a requirement to have 0 warnings for >> dtbs_check. >> This will change in the future. >> >> Full log is available here: >> https://patchwork.ozlabs.org/patch/1538804 >> >> >> pn547@28: 'clock-frequency' is a required property >> arch/arm64/boot/dts/qcom/msm8992-msft-lumia-octagon-talkman.dt.yaml >> arch/arm64/boot/dts/qcom/msm8994-msft-lumia-octagon-cityman.dt.yaml >> > > I think clock-frequency should be dropped from I2C slave device. > Similarly to this one: > https://lore.kernel.org/linux-nfc/f955726a-eb2d-7b3e-9c5f-978358710eb6@canonical.com/T/#u > You have right, it isn't parsed by driver and values match parent i2c bus. I dropped it in next revision. David > > Best regards, > Krzysztof
Powered by blists - more mailing lists