Re: [PATCH v3 01/12] dt-bindings: rtc: renesas,rzn1-rtc: Add RZ/T2H and RZ/N2H support
Wolfram Sang <[email protected]>
| Newsgroups | dev.linux.lists.sashiko-reviews,org.kernel.vger.linux-devicetree,org.kernel.vger.linux-rtc |
|---|---|
| Message-ID | <aoQMyllxMOpY5sHx@ninjato> |
> [Severity: Medium] > The commit message mentions that the RZ/T2H RTC variant supports a 1Hz clock > output signal on the RTCAT1HZ pin. Since this introduces a clock provider > capability, shouldn't the #clock-cells property be documented and required > here with an appropriate const constraint? X5H has this as well but the feature has neither been implemented nor tested yet. So, I suggest to add related properties incrementally.
signature.asc
(application/pgp-signature, 833 B)
-----BEGIN PGP SIGNATURE----- iQIzBAABCgAdFiEEOZGx6rniZ1Gk92RdFA3kzBSgKbYFAmqEDMUACgkQFA3kzBSg KbYqdBAAkGVzBM7LC437y0id2WgytX+hYLCravvY636xG7hvder4rj6oyP3fnQCB sWFfmh9JW4OrIbhAX5/zK402tK3zDaEBln6mQB/OF3Dbxll/Ok4ZIzvZPFlquXBx ShDq7OHGB/b05fYuEflJ2suO6BSAChb1xpzBW2Og7ILqLuq0LYzuLTn27mtVWIoo gmoGFs7+1l2EHsjENoNEyXlE5JUalL4QhH7zT1pTzPaaMk00lEkmc4dlJMW/cJIP 1L15XKkf55HHSMTu7YsSfJbeYi24RnKmghJGIuSBzzRO2xNXpZQgdcxiG+QWpyHg g47YQD3EZnYS3Mct+35UNeVPLumRqUTGLUfHzdcYxAomRWdUvvKNVbKre2Vi6MWb mH8zQVLYBdz81rS9MTUM5hIyhc+Sx/WiyYI7yAjh9Z1nJK5wMdksnuKelSF0yIDv ysi28m65ittECWypPBm150fGGdKv+nGUrFH255J+DzbmHHv2DHiwM/8VAoJqOWkH BTQXZrHUyhBJGObJOxYm5/jIG1XySC62FBWjp1yNw4WZ8rbeTuac5XB5w7YG84jc 0BGdfe4fEegnPvnzYXBEEK3SlvXpp44MT0pRzLHuGn9kEm8BKBIXYUFvpZD6NtwE siZAJAbpxm1rNMkl4fw/a5RclQRAm24PVVXFxjlqZNtAHQ+Fv8M= =ybHS -----END PGP SIGNATURE-----