Re: [PATCH v1] ARM: dts: imx: add fsl, imx35-wdt compatible to all relevant watchdog nodes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




On Tue, Jan 17, 2017 at 03:23:36AM +0200, Vladimir Zapolskiy wrote:
> Watchdog device controller found on all modern SoCs from i.MX series
> and firstly introduced in i.MX35 is not one in one compatible with the
> watchdog controllers on i.MX21, i.MX27 and i.MX31, the latter
> controlles don't have WICR (and pretimeout notification support) and
> WMCR registers. To get benefit from the more advanced watchdog device
> and to avoid operations over non-existing registers on legacy SoCs add
> fsl,imx35-wdt compatible to descriptions of all i.MX35 compatible
> watchdog controllers.
> 
> Signed-off-by: Vladimir Zapolskiy <vz@xxxxxxxxx>
> ---
> RFC change is found at https://patchwork.kernel.org/patch/9350007
> 
> Changes from RFC to v1:
> * added the same change to devicetree bindings documentation, thanks
>   to Baruch Siach for review,
> * replaced a new shared compatible derived from i.MX25 with an earlier
>   from i.MX35 SoC one, thanks to Uwe Kleine-König for review.
> 
> Compatible "fsl,imx21-wdt" is preserved as a generic one, because
> interface to a watchdog controller on i.MX35 is a superset of the
> interface to a i.MX21 watchdog controller.

One could argue here, but apart from the discussion if this is correct
or not, choosing this way has the upside that teaching the driver after
this change about the i.MX35 type doesn't make the machines based on the
newer type suffer from a breakage for one commit.

>  Documentation/devicetree/bindings/watchdog/fsl-imx-wdt.txt |  2 +-

When I created my patch set for this breakage I wondered if the
documentation should point out which are the unique types.

>  arch/arm/boot/dts/imx25.dtsi                               |  3 ++-
>  arch/arm/boot/dts/imx50.dtsi                               |  3 ++-
>  arch/arm/boot/dts/imx51.dtsi                               |  6 ++++--
>  arch/arm/boot/dts/imx53.dtsi                               |  6 ++++--
>  arch/arm/boot/dts/imx6qdl.dtsi                             |  6 ++++--
>  arch/arm/boot/dts/imx6sl.dtsi                              |  6 ++++--
>  arch/arm/boot/dts/imx6sx.dtsi                              |  9 ++++++---
>  arch/arm/boot/dts/imx6ul.dtsi                              |  6 ++++--
>  arch/arm/boot/dts/imx7s.dtsi                               | 12 ++++++++----
>  arch/arm/boot/dts/ls1021a.dtsi                             |  2 +-
>  arch/arm/boot/dts/vfxxx.dtsi                               |  3 ++-

The imx21 type is also used in

	arch/arm64/boot/dts/freescale/fsl-ls1043a.dtsi
	arch/arm64/boot/dts/freescale/fsl-ls1046a.dtsi

What about these?

Best regards
Uwe

-- 
Pengutronix e.K.                           | Uwe Kleine-König            |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
--
To unsubscribe from this list: send the line "unsubscribe devicetree" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]
  Powered by Linux