Thread (6 messages) 6 messages, 5 authors, 2024-08-14

Re: [PATCH] dt-bindings: watchdog: fsl-imx-wdt: Add missing 'big-endian' property

From: Frank Li <Frank.li@nxp.com>
Date: 2024-08-06 15:49:55
Also in: imx, linux-arm-kernel, linux-watchdog, lkml

On Tue, Aug 06, 2024 at 12:43:07PM +0200, Krzysztof Kozlowski wrote:
On 06/08/2024 12:38, Animesh Agarwal wrote:
quoted
Add missing big-endian property in watchdog/fsl-imx-wdt.yaml schema.
This fixes dtbs_check errors.

Cc: Daniel Baluta <daniel.baluta@nxp.com>
Signed-off-by: Animesh Agarwal <redacted>
---
There are 12 similar errors related to this missing property from
different blobs.

./arch/arm64/boot/dts/freescale/fsl-ls1012a-frdm.dtb: watchdog@2ad0000:
Unevaluated properties are not allowed ('big-endian' was unexpected)from
schema $id: http://devicetree.org/schemas/watchdog/fsl-imx-wdt.yaml#
---
Not sure if this is correct. I mean, technically it is, but Frank Li was
removing big-endian properties so please choose consistent approach.
drivers/watchdog/imx2_wdt.c never parser big-endian. I suggest remove
big-endian in dts file. I suggest keep big-endian only if it really used.

Frank
Best regards,
Krzysztof
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help