[PATCH 1/3] wdt: sunxi: Introduce a new compatible for the A10 and A31

[Maxime Ripard]

Hi Hans,

On Thu, Feb 06, 2014 at 12:14:21PM +0100, Hans de Goede wrote:
> On 02/02/2014 02:55 PM, Maxime Ripard wrote:
> >For historical reasons, the Allwinner A10 compatibles are not following the
> >patterns used for this other Allwinner SoCs.
> >
> >Introduce a new compatible following the usual pattern, and deprecate the olders.
> >
> >Signed-off-by: Maxime Ripard <maxime.ripard at free-electrons.com>
> >---
> >  Documentation/devicetree/bindings/watchdog/sunxi-wdt.txt | 7 ++++---
> >  drivers/watchdog/sunxi_wdt.c                             | 1 +
> >  2 files changed, 5 insertions(+), 3 deletions(-)
> >
> >diff --git a/Documentation/devicetree/bindings/watchdog/sunxi-wdt.txt b/Documentation/devicetree/bindings/watchdog/sunxi-wdt.txt
> >index e39cb26..6e8c937 100644
> >--- a/Documentation/devicetree/bindings/watchdog/sunxi-wdt.txt
> >+++ b/Documentation/devicetree/bindings/watchdog/sunxi-wdt.txt
> >@@ -2,13 +2,14 @@ Allwinner SoCs Watchdog timer
> >
> >  Required properties:
> >
> >-- compatible : should be "allwinner,<soc-family>-wdt", the currently supported
> >-  SoC families being sun4i and sun6i
> >+- compatible : should be either "allwinner,sun4i-a10-wdt" or
> >+               "allwinner,sun6i-a31-wdt" (deprecated:
> >+               "allwinner,sun4i-wdt", "allwinner,sun6i-wdt")
> >  - reg : Specifies base physical address and size of the registers.
> >
> >  Example:
> >
> >  wdt: watchdog at 01c20c90 {
> >-	compatible = "allwinner,sun4i-wdt";
> >+	compatible = "allwinner,sun4i-a10-wdt";
> >  	reg = <0x01c20c90 0x10>;
> >  };
> 
> You talk about deprecating the old compat strings in the commit message, but
> here you outright replace them, which will break things with old dtb files ?

I'm replacing them only in the Documentation here.

I only add it to the driver after that, which means that the driver
will be probed with both the new and the old compatibles.

However, the DT maintainers said it was ok to remove them entirely, so
I'll send a v2.

Maxime

> 
> Other 2 patches in the series look good and are:
> 
> Reviewed-by: Hans de Goede <hdegoede at redhat.com>
> 
> 
> >diff --git a/drivers/watchdog/sunxi_wdt.c b/drivers/watchdog/sunxi_wdt.c
> >index 76332d8..7c8923d 100644
> >--- a/drivers/watchdog/sunxi_wdt.c
> >+++ b/drivers/watchdog/sunxi_wdt.c
> >@@ -206,6 +206,7 @@ static void sunxi_wdt_shutdown(struct platform_device *pdev)
> >
> >  static const struct of_device_id sunxi_wdt_dt_ids[] = {
> >  	{ .compatible = "allwinner,sun4i-wdt" },
> >+	{ .compatible = "allwinner,sun4i-a10-wdt" },
> >  	{ /* sentinel */ }
> >  };
> >  MODULE_DEVICE_TABLE(of, sunxi_wdt_dt_ids);
> >
> 
> Regards,
> 
> Hans

-- 
Maxime Ripard, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20140207/f1d6c888/attachment.sig>