| Message ID | 20230613201231.2826352-2-robh@kernel.org |
|---|---|
| State | Accepted |
| Commit | 0778a2e9b35c00197db1df957dcb46853eead95f |
| Headers | show |
| Series | None | expand |
On Tue, Jun 13, 2023 at 3:11 PM Conor Dooley <conor@kernel.org> wrote: > > On Tue, Jun 13, 2023 at 02:12:30PM -0600, Rob Herring wrote: > > The "linux,keycode" property is non-standard. Add the common property > > "linux,keycodes" and mark "linux,keycode" deprecated so that the mistake > > is not propagated. > > This is actually used in the driver for this device, should the driver > not also be updated to use the corrected property? Yes, but that doesn't have to be in sync with the binding change. I mainly want to add this so it doesn't get propagated to new users rather than move this case off of linux,keycode. Also, the input subsystem should probably have a common function to read linux,keycodes as right now every driver does it. Rob
On Wed, Jun 14, 2023 at 08:53:20AM -0600, Rob Herring wrote: > On Tue, Jun 13, 2023 at 3:11 PM Conor Dooley <conor@kernel.org> wrote: > > > > On Tue, Jun 13, 2023 at 02:12:30PM -0600, Rob Herring wrote: > > > The "linux,keycode" property is non-standard. Add the common property > > > "linux,keycodes" and mark "linux,keycode" deprecated so that the mistake > > > is not propagated. > > > > This is actually used in the driver for this device, should the driver > > not also be updated to use the corrected property? > > Yes, but that doesn't have to be in sync with the binding change. I > mainly want to add this so it doesn't get propagated to new users > rather than move this case off of linux,keycode. Also, the input > subsystem should probably have a common function to read > linux,keycodes as right now every driver does it. Perhaps one of the nxp folk in the audience should have a crack at that so. Acked-by: Conor Dooley <conor.dooley@microchip.com> Cheers, Conor.
diff --git a/Documentation/devicetree/bindings/crypto/fsl,sec-v4.0-mon.yaml b/Documentation/devicetree/bindings/crypto/fsl,sec-v4.0-mon.yaml index 6052129bf852..e879bc0be8e2 100644 --- a/Documentation/devicetree/bindings/crypto/fsl,sec-v4.0-mon.yaml +++ b/Documentation/devicetree/bindings/crypto/fsl,sec-v4.0-mon.yaml @@ -105,6 +105,11 @@ properties: linux,keycode: $ref: /schemas/types.yaml#/definitions/uint32 default: 116 + deprecated: true + + linux,keycodes: + maxItems: 1 + default: 116 required: - compatible
The "linux,keycode" property is non-standard. Add the common property "linux,keycodes" and mark "linux,keycode" deprecated so that the mistake is not propagated. Signed-off-by: Rob Herring <robh@kernel.org> --- .../devicetree/bindings/crypto/fsl,sec-v4.0-mon.yaml | 5 +++++ 1 file changed, 5 insertions(+)