i2c: efm32: correct namespacing of location property
authorUwe Kleine-König <u.kleine-koenig@pengutronix.de>
Fri, 11 Jul 2014 08:50:14 +0000 (10:50 +0200)
committerWolfram Sang <wsa@the-dreams.de>
Thu, 17 Jul 2014 13:38:49 +0000 (15:38 +0200)
Olof Johansson pointed out that usually the company name is picked as
namespace prefix to specific properties. So expect "energymicro,location"
but fall back to the previously introduced name "efm32,location".

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Wolfram Sang <wsa@the-dreams.de>
Documentation/devicetree/bindings/i2c/i2c-efm32.txt
drivers/i2c/busses/i2c-efm32.c

index fc15ac5..50b25c3 100644 (file)
@@ -10,7 +10,7 @@ Required properties :
 Recommended properties :
 
  - clock-frequency : maximal I2C bus clock frequency in Hz.
- - efm32,location : Decides the location of the USART I/O pins.
+ - energymicro,location : Decides the location of the USART I/O pins.
    Allowed range : [0 .. 6]
 
 Example:
@@ -23,7 +23,7 @@ Example:
                clocks = <&cmu clk_HFPERCLKI2C0>;
                clock-frequency = <100000>;
                status = "ok";
-               efm32,location = <3>;
+               energymicro,location = <3>;
 
                eeprom@50 {
                        compatible = "microchip,24c02";
index f7eccd6..10b8323 100644 (file)
@@ -370,7 +370,13 @@ static int efm32_i2c_probe(struct platform_device *pdev)
                return ret;
        }
 
-       ret = of_property_read_u32(np, "efm32,location", &location);
+
+       ret = of_property_read_u32(np, "energymicro,location", &location);
+
+       if (ret)
+               /* fall back to wrongly namespaced property */
+               ret = of_property_read_u32(np, "efm32,location", &location);
+
        if (!ret) {
                dev_dbg(&pdev->dev, "using location %u\n", location);
        } else {