EXYNOS5: FDT Support for I2C

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73 ++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

Hello Rajeshwari

Quoted 5 lines · Rajeshwari Shinde

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

Quoted 8 lines

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73
++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

I tried to apply this patch to current mainline head commit 59852d03867108217fe88e3bfc3e1e9cedfe63c5
Merge: 22b6fcb b3e5cd1
Author: Tom Rini <[email protected]>
Date: Fri Nov 9 08:47:25 2012 -0700

Merge branch 'master' of git://www.denx.de/git/u-boot-microblaze

but I got this error:

[hs at pollux u-boot-i2c]$ git am index.html
Applying: EXYNOS5: FDT: Add I2C device node data
error: arch/arm/dts/exynos5250.dtsi: does not exist in index Patch failed at 0001 EXYNOS5: FDT: Add I2C device node data When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". [hs at pollux u-boot-i2c]$

Against which tree is this patchset? Should it go through the samsung tree from Minkyu Kang found here:

http://git.denx.de/u-boot-samsung.git

? If so, I send my Acked-by for this patchset ...

bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Hi Heiko,

Thank you for Acking the patchset.
It is based on u-boot-samsung, where the basic FDT support patches for Exynos are submitted.

Regards
Rajeshwari Shinde.

Quoted 45 lines · Heiko Schocher

Hello Rajeshwari

Quoted 5 lines · Rajeshwari Shinde

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

Quoted 8 lines

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73
++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

I tried to apply this patch to current mainline head commit 59852d03867108217fe88e3bfc3e1e9cedfe63c5
Merge: 22b6fcb b3e5cd1
Author: Tom Rini <[email protected]>
Date: Fri Nov 9 08:47:25 2012 -0700

Merge branch 'master' of git://www.denx.de/git/u-boot-microblaze

but I got this error:

[hs at pollux u-boot-i2c]$ git am index.html
Applying: EXYNOS5: FDT: Add I2C device node data
error: arch/arm/dts/exynos5250.dtsi: does not exist in index Patch failed at 0001 EXYNOS5: FDT: Add I2C device node data When you have resolved this problem run "git am --resolved". If you would prefer to skip this patch, instead run "git am --skip". To restore the original branch and stop patching run "git am --abort". [hs at pollux u-boot-i2c]$

Against which tree is this patchset? Should it go through the samsung tree from Minkyu Kang found here:

http://git.denx.de/u-boot-samsung.git

? If so, I send my Acked-by for this patchset ...

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73 ++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

Hello Rajeshwari,

Quoted 5 lines · Rajeshwari Shinde

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

Quoted 8 lines

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73
++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

Acked-by Heiko Schocher <[email protected]>

bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

Hi,

Quoted 19 lines · Heiko Schocher

Hello Rajeshwari,

Quoted 13 lines · Rajeshwari Shinde

Add I2C device node data for exynos

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
Added Periph id to the I2C device node

  arch/arm/dts/exynos-periph-id.dtsi |   35 +++++++++++++++++
  arch/arm/dts/exynos5250.dtsi       |   73
++++++++++++++++++++++++++++++++++++
  2 files changed, 108 insertions(+), 0 deletions(-)

create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

Quoted 1 line

Acked-by Heiko Schocher <[email protected]>

I'm sorry to say that there is one problem with this. It is using a non-standard dtc feature (Stephen Warren's symbolic work), so I think we should wait until Rajeshwari updates it to avoid that. I believe he will do that soon.

This patch adds aliases for I2C.

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
- None.

  board/samsung/dts/exynos5250-smdk5250.dts |   11 +++++++++++
  1 files changed, 11 insertions(+), 0 deletions(-)

Add required compatible information for I2C driver.

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
- None

  include/fdtdec.h |    1 +
  lib/fdtdec.c     |    1 +
  2 files changed, 2 insertions(+), 0 deletions(-)

This patch adds api to find compatible id for a given FDT node

Signed-off-by Rajeshwari Shinde<[email protected]>

Acked-by Simon Glass<[email protected]>

---
Changes in V2:
- None

  include/fdtdec.h |   14 ++++++++++++++
  lib/fdtdec.c     |   12 ++++++++++++
  2 files changed, 26 insertions(+), 0 deletions(-)

Functions added to get the I2C bus number and reset I2C bus using FDT node.

Signed-off-by Simon Glass <[email protected]>

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes in V2:
- Added periph id to I2C bus structure.
- Modified i2c_get_bus_num_fdt function to compare with node.
- Board i2c init moved to driver in case of FDT.

 drivers/i2c/s3c24x0_i2c.c |   90 ++++++++++++++++++++++++++++++++++++++++++++-
 drivers/i2c/s3c24x0_i2c.h |    8 ++++
 include/i2c.h             |   26 +++++++++++++
 3 files changed, 123 insertions(+), 1 deletions(-)

drivers/i2c/s3c24x0_i2c.c +47 -1

index 9bc4c7f..840585d 100644
@@ -27,9 +27,11 @@
  */
 
 #include <common.h>
+#include <fdtdec.h>
 #ifdef CONFIG_EXYNOS5
 #include <asm/arch/clk.h>
 #include <asm/arch/cpu.h>
+#include <asm/arch/pinmux.h>
 #else
 #include <asm/arch/s3c24x0_cpu.h>
 #endif

@@ -60,7 +62,14 @@
 #define I2C_TIMEOUT 1          /* 1 second */
 
 
-static unsigned int g_current_bus;     /* Stores Current I2C Bus */
+/*
+ * For SPL boot some boards need i2c before SDRAM is initialised so force
+ * variables to live in SRAM
+ */
+static unsigned int g_current_bus __attribute__((section(".data")));
+static struct s3c24x0_i2c_bus i2c_bus[CONFIG_MAX_I2C_NUM]
+                       __attribute__((section(".data")));
+static int i2c_busses __attribute__((section(".data")));
 
 #ifndef CONFIG_EXYNOS5
 static int GetI2CSDA(void)

@@ -507,4 +516,83 @@ int i2c_write(uchar chip, uint addr, int alen, uchar *buffer, int len)
                (i2c, I2C_WRITE, chip << 1, &xaddr[4 - alen], alen, buffer,
                 len) != 0);
 }
+
+#ifdef CONFIG_OF_CONTROL
+void board_i2c_init(const void *blob)
+{
+
+       int node_list[CONFIG_MAX_I2C_NUM];
+       int count, i;
+
+       count = fdtdec_find_aliases_for_id(blob, "i2c",
+               COMPAT_SAMSUNG_S3C2440_I2C, node_list,
+               CONFIG_MAX_I2C_NUM);
+       for (i = 0; i < count; i++) {
+               struct s3c24x0_i2c_bus *bus;
+               int node = node_list[i];
+
+               if (node < 0)
+                       continue;
+               bus = &i2c_bus[i];
+               bus->regs = (struct s3c24x0_i2c *)
+                       fdtdec_get_addr(blob, node, "reg");
+               bus->id = (enum periph_id)
+                       fdtdec_get_int(blob, node, "samsung,periph-id", -1);
+               bus->node = node;
+               bus->bus_num = i2c_busses++;
+               exynos_pinmux_config(bus->id, 0);
+       }
+
+}
+
+static struct s3c24x0_i2c_bus *get_bus(unsigned int bus_idx)
+{
+       if (bus_idx < i2c_busses)
+               return &i2c_bus[bus_idx];
+       debug("Undefined bus: %d\n", bus_idx);
+       return NULL;
+}
+

drivers/i2c/s3c24x0_i2c.c +32 -0

+       for (i = 0; i < i2c_busses; i++) {
+               if (node == i2c_bus[i].node)
+                       return i;
+       }
+
+       debug("%s: Can't find any matched I2C bus\n", __func__);
+       return -1;
+}
+
+int i2c_reset_port_fdt(const void *blob, int node)
+{
+       struct s3c24x0_i2c_bus *i2c;
+
+       int bus;
+
+       bus = i2c_get_bus_num_fdt(blob, node);
+       if (bus < 0) {
+               debug("could not get bus for node %d\n", node);
+               return -1;
+       }
+       i2c = get_bus(bus);
+       if (!i2c) {
+               debug("get_bus() failed for node node %d\n", node);
+               return -1;
+       }
+
+       i2c_ch_init(i2c->regs, CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
+
+       return 0;
+}
+#endif
+
 #endif /* CONFIG_HARD_I2C */

drivers/i2c/s3c24x0_i2c.h +8 -0

index 2dd4b06..1243bf1 100644
@@ -30,4 +30,12 @@ struct s3c24x0_i2c {
        u32     iicds;
        u32     iiclc;
 };
+
+struct s3c24x0_i2c_bus {
+       int node;       /* device tree node */
+       int bus_num;    /* i2c bus number */
+       struct s3c24x0_i2c *regs;
+       enum periph_id id;
+};
+
 #endif /* _S3C24X0_I2C_H */

include/i2c.h +26 -1

index 16f099d..efb097f 100644
@@ -262,4 +262,30 @@ extern int get_multi_scl_pin(void);
 extern int get_multi_sda_pin(void);
 extern int multi_i2c_init(void);
 #endif
+
+/**
+ * Get FDT values for i2c bus.
+ *
+ * @param blob  Device tree blbo
+ * @return the number of I2C bus
+ */
+void board_i2c_init(const void *blob);
+
+/**
+ * Find the I2C bus number by given a FDT I2C node.
+ *
+ * @param blob  Device tree blbo
+ * @param node  FDT I2C node to find
+ * @return the number of I2C bus (zero based), or -1 on error
+ */
+int i2c_get_bus_num_fdt(const void *blob, int node);
+
+/**
+ * Reset the I2C bus represented by the given a FDT I2C node.
+ *
+ * @param blob  Device tree blbo
+ * @param node  FDT I2C node to find
+ * @return 0 if port was reset, -1 if not found
+ */
+int i2c_reset_port_fdt(const void *blob, int node);
 #endif /* _I2C_H_ */
--

1.7.4.4

Functions added to get the I2C bus number and reset I2C bus using FDT node.

Signed-off-by Simon Glass<[email protected]>

Signed-off-by Rajeshwari Shinde<[email protected]>

---
Changes in V2:
- Added periph id to I2C bus structure.
- Modified i2c_get_bus_num_fdt function to compare with node.
- Board i2c init moved to driver in case of FDT.

  drivers/i2c/s3c24x0_i2c.c |   90 ++++++++++++++++++++++++++++++++++++++++++++-
  drivers/i2c/s3c24x0_i2c.h |    8 ++++
  include/i2c.h             |   26 +++++++++++++
  3 files changed, 123 insertions(+), 1 deletions(-)

drivers/i2c/s3c24x0_i2c.c +0 -0

index 9bc4c7f..840585d 100644

drivers/i2c/s3c24x0_i2c.c +8 -0

@@ -507,4 +516,83 @@ int i2c_write(uchar chip, uint addr, int alen, uchar *buffer, int len)
  		(i2c, I2C_WRITE, chip<<  1,&xaddr[4 - alen], alen, buffer,
  		 len) != 0);
  }
+
+#ifdef CONFIG_OF_CONTROL
+void board_i2c_init(const void *blob)
+{
+
+	int node_list[CONFIG_MAX_I2C_NUM];
+	int count, i;
+

drivers/i2c/s3c24x0_i2c.c +4 -0

+		bus->regs = (struct s3c24x0_i2c *)
+			fdtdec_get_addr(blob, node, "reg");
+		bus->id = (enum periph_id)
+			fdtdec_get_int(blob, node, "samsung,periph-id", -1);

drivers/i2c/s3c24x0_i2c.c +21 -0

+	debug("Undefined bus: %d\n", bus_idx);
+	return NULL;
+}
+
+int i2c_get_bus_num_fdt(const void *blob, int node)
+{
+	enum fdt_compat_id compat;
+	int i;
+
+	compat = fdtdec_lookup(blob, node);
+	if (compat != COMPAT_SAMSUNG_S3C2440_I2C) {
+		debug("%s: Not a supported I2C node\n", __func__);
+		return -1;
+	}
+	for (i = 0; i<  i2c_busses; i++) {
+		if (node == i2c_bus[i].node)
+			return i;
+	}
+
+	debug("%s: Can't find any matched I2C bus\n", __func__);
+	return -1;

drivers/i2c/s3c24x0_i2c.c +15 -0

+		debug("could not get bus for node %d\n", node);
+		return -1;
+	}
+	i2c = get_bus(bus);
+	if (!i2c) {
+		debug("get_bus() failed for node node %d\n", node);
+		return -1;
+	}
+
+	i2c_ch_init(i2c->regs, CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
+
+	return 0;
+}
+#endif
+
  #endif /* CONFIG_HARD_I2C */

drivers/i2c/s3c24x0_i2c.h +8 -0

index 2dd4b06..1243bf1 100644
@@ -30,4 +30,12 @@ struct s3c24x0_i2c {
  	u32	iicds;
  	u32	iiclc;
  };
+
+struct s3c24x0_i2c_bus {
+	int node;	/* device tree node */
+	int bus_num;	/* i2c bus number */
+	struct s3c24x0_i2c *regs;
+	enum periph_id id;
+};
+
  #endif /* _S3C24X0_I2C_H */

include/i2c.h +0 -0

index 16f099d..efb097f 100644

include/i2c.h +4 -0

+ * @param node  FDT I2C node to find
+ * @return the number of I2C bus (zero based), or -1 on error
+ */
+int i2c_get_bus_num_fdt(const void *blob, int node);

Hello Rajeshwari,

Quoted 11 lines · Rajeshwari Shinde

Functions added to get the I2C bus number and reset I2C bus using FDT node.

Signed-off-by Simon Glass<[email protected]>

Signed-off-by Rajeshwari Shinde<[email protected]>

---
Changes in V2:
- Added periph id to I2C bus structure.
- Modified i2c_get_bus_num_fdt function to compare with node.
- Board i2c init moved to driver in case of FDT.

Quoted 5 lines

  drivers/i2c/s3c24x0_i2c.c |   90
++++++++++++++++++++++++++++++++++++++++++++-
  drivers/i2c/s3c24x0_i2c.h |    8 ++++
  include/i2c.h             |   26 +++++++++++++
  3 files changed, 123 insertions(+), 1 deletions(-)

patch looks good, just some nitpicking comments ...

Quoted 4 lines

drivers/i2c/s3c24x0_i2c.c +0 -0

index 9bc4c7f..840585d 100644

[...]

Quoted 15 lines

drivers/i2c/s3c24x0_i2c.c +8 -0

@@ -507,4 +516,83 @@ int i2c_write(uchar chip, uint addr, int alen, uchar
 *buffer, int len)
                (i2c, I2C_WRITE, chip<<  1,&xaddr[4 - alen], alen, buffer,
 
                 len) != 0);
  }
+
+#ifdef CONFIG_OF_CONTROL
+void board_i2c_init(const void *blob)
+{
+
+       int node_list[CONFIG_MAX_I2C_NUM];
+       int count, i;
+

drivers/i2c/s3c24x0_i2c.c int i2c_write(uchar chip, uint addr, int alen, uchar

Quoted 10 lines

drivers/i2c/s3c24x0_i2c.c +10 -0

+       count = fdtdec_find_aliases_for_id(blob, "i2c",
+               COMPAT_SAMSUNG_S3C2440_I2C, node_list,
+               CONFIG_MAX_I2C_NUM);
+       for (i = 0; i<  count; i++) {
+               struct s3c24x0_i2c_bus *bus;
+               int node = node_list[i];
+
+               if (node<  0)
+                       continue;
+               bus =&i2c_bus[i];

                     ^
                     space please

Quoted 5 lines

drivers/i2c/s3c24x0_i2c.c +4 -1

+               bus->regs = (struct s3c24x0_i2c *)
+                       fdtdec_get_addr(blob, node, "reg");
+               bus->id = (enum periph_id)
+                       fdtdec_get_int(blob, node, "samsung,periph-id",
-1);

drivers/i2c/s3c24x0_i2c.c int i2c_write(uchar chip, uint addr, int alen, uchar *buffer, int len)

Quoted 10 lines

drivers/i2c/s3c24x0_i2c.c +10 -0

+               bus->node = node;
+               bus->bus_num = i2c_busses++;
+               exynos_pinmux_config(bus->id, 0);
+       }
+
+}
+
+static struct s3c24x0_i2c_bus *get_bus(unsigned int bus_idx)
+{
+       if (bus_idx<  i2c_busses)

                   ^
                   space please

Quoted 2 lines

+ return&i2c_bus[bus_idx];

                      ^
                      space please

Quoted 21 lines

drivers/i2c/s3c24x0_i2c.c +21 -0

+       debug("Undefined bus: %d\n", bus_idx);
+       return NULL;
+}
+
+int i2c_get_bus_num_fdt(const void *blob, int node)
+{
+       enum fdt_compat_id compat;
+       int i;
+
+       compat = fdtdec_lookup(blob, node);
+       if (compat != COMPAT_SAMSUNG_S3C2440_I2C) {
+               debug("%s: Not a supported I2C node\n", __func__);
+               return -1;
+       }
+       for (i = 0; i<  i2c_busses; i++) {
+               if (node == i2c_bus[i].node)
+                       return i;
+       }
+
+       debug("%s: Can't find any matched I2C bus\n", __func__);
+       return -1;

drivers/i2c/s3c24x0_i2c.c int i2c_write(uchar chip, uint addr, int alen, uchar *buffer, int len)

Quoted 10 lines

drivers/i2c/s3c24x0_i2c.c +10 -0

+}
+
+int i2c_reset_port_fdt(const void *blob, int node)
+{
+       struct s3c24x0_i2c_bus *i2c;
+
+       int bus;
+
+       bus = i2c_get_bus_num_fdt(blob, node);
+       if (bus<  0) {

               ^
               space

Quoted 39 lines

drivers/i2c/s3c24x0_i2c.c +10 -0

+               debug("could not get bus for node %d\n", node);
+               return -1;
+       }
+       i2c = get_bus(bus);
+       if (!i2c) {
+               debug("get_bus() failed for node node %d\n", node);
+               return -1;
+       }
+
+       i2c_ch_init(i2c->regs, CONFIG_SYS_I2C_SPEED,

CONFIG_SYS_I2C_SLAVE);

drivers/i2c/s3c24x0_i2c.c +5 -0

+
+       return 0;
+}
+#endif
+
  #endif /* CONFIG_HARD_I2C */

drivers/i2c/s3c24x0_i2c.h +8 -0

index 2dd4b06..1243bf1 100644
@@ -30,4 +30,12 @@ struct s3c24x0_i2c {
        u32     iicds;
        u32     iiclc;
  };
+
+struct s3c24x0_i2c_bus {
+       int node;       /* device tree node */
+       int bus_num;    /* i2c bus number */
+       struct s3c24x0_i2c *regs;
+       enum periph_id id;
+};
+
  #endif /* _S3C24X0_I2C_H */

include/i2c.h +0 -0

index 16f099d..efb097f 100644

include/i2c.h extern int get_multi_scl_pin(void); · line 269

Quoted 8 lines

include/i2c.h +5 -0

@@ -262,4 +262,30 @@ extern int get_multi_scl_pin(void);
  extern int get_multi_sda_pin(void);
  extern int multi_i2c_init(void);
  #endif
+
+/**
+ * Get FDT values for i2c bus.
+ *
+ * @param blob  Device tree blbo

                               ^
                               blob

include/i2c.h extern int get_multi_scl_pin(void); · line 277

Quoted 8 lines

include/i2c.h +8 -0

+ * @return the number of I2C bus
+ */
+void board_i2c_init(const void *blob);
+
+/**
+ * Find the I2C bus number by given a FDT I2C node.
+ *
+ * @param blob  Device tree blbo

                               ^
                               blob

Quoted 4 lines

include/i2c.h +4 -0

+ * @param node  FDT I2C node to find
+ * @return the number of I2C bus (zero based), or -1 on error
+ */
+int i2c_get_bus_num_fdt(const void *blob, int node);

include/i2c.h extern int get_multi_scl_pin(void); · line 291

Quoted 10 lines

include/i2c.h +9 -0

+
+/**
+ * Reset the I2C bus represented by the given a FDT I2C node.
+ *
+ * @param blob  Device tree blbo
+ * @param node  FDT I2C node to find
+ * @return 0 if port was reset, -1 if not found
+ */
+int i2c_reset_port_fdt(const void *blob, int node);
  #endif        /* _I2C_H_ */

Thanks!

Reviewed-by Heiko Schocher <[email protected]>

bye,
Heiko
--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany

_______________________________________________
U-Boot mailing list
U-Boot at lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Functions added to get the I2C bus number and reset I2C bus using FDT node.

Signed-off-by Simon Glass<[email protected]>

Signed-off-by Rajeshwari Shinde<[email protected]>

---
Changes in V2:
- Added periph id to I2C bus structure.
- Modified i2c_get_bus_num_fdt function to compare with node.
- Board i2c init moved to driver in case of FDT.

  drivers/i2c/s3c24x0_i2c.c |   90 ++++++++++++++++++++++++++++++++++++++++++++-
  drivers/i2c/s3c24x0_i2c.h |    8 ++++
  include/i2c.h             |   26 +++++++++++++
  3 files changed, 123 insertions(+), 1 deletions(-)

This patch initialises I2C using FDT.

Signed-off-by Rajeshwari Shinde <[email protected]>

---
Changes since V2:
- board_i2c_init moved to driver in case of FDT.

 board/samsung/smdk5250/smdk5250.c |   20 +-------------------
 1 files changed, 1 insertions(+), 19 deletions(-)

board/samsung/smdk5250/smdk5250.c +1 -20

index db2457b..5ebc665 100644
@@ -272,24 +272,6 @@ static int board_uart_init(void)
        return 0;
 }
 
-#ifdef CONFIG_SYS_I2C_INIT_BOARD
-static int board_i2c_init(void)
-{
-       int i, err;
-
-       for (i = 0; i < CONFIG_MAX_I2C_NUM; i++) {
-               err = exynos_pinmux_config((PERIPH_ID_I2C0 + i),
-                                               PINMUX_FLAG_NONE);
-               if (err) {
-                       debug("I2C%d not configured\n", (PERIPH_ID_I2C0 + i));
-                       return err;
-               }
-       }
-       i2c_init(CONFIG_SYS_I2C_SPEED, CONFIG_SYS_I2C_SLAVE);
-       return 0;
-}
-#endif
-
 #ifdef CONFIG_BOARD_EARLY_INIT_F
 int board_early_init_f(void)
 {

@@ -300,7 +282,7 @@ int board_early_init_f(void)
                return err;
        }
 #ifdef CONFIG_SYS_I2C_INIT_BOARD
-       err = board_i2c_init();
+       board_i2c_init(gd->fdt_blob);
 #endif
        return err;
 }
--

1.7.4.4

This patch initialises I2C using FDT.

Signed-off-by Rajeshwari Shinde<[email protected]>

---
Changes since V2:
- board_i2c_init moved to driver in case of FDT.

  board/samsung/smdk5250/smdk5250.c |   20 +-------------------
  1 files changed, 1 insertions(+), 19 deletions(-)

This patch set adds FDT support for I2C driver and API's to acess the bus number using the fdt node and also reset the port.

Changes in V2:
- Baord i2c init moved to I2C driver in case of FDT.
- Added Periph id to device node.
- Modified i2c_get_bus_num_fdt api to compare using node.

Rajeshwari Shinde (6):
EXYNOS5: FDT: Add I2C device node data
EXYNOS5 : FDT: Add Aliases for I2C device
EXYNOS5: FDT: Add compatible string for I2C
FDT: Api to find compatible id for a given node
I2C: Driver changes for FDT support
SMDK5250: Initialise I2C using FDT

 arch/arm/dts/exynos-periph-id.dtsi        |   35 +++++++++++
 arch/arm/dts/exynos5250.dtsi              |   73 +++++++++++++++++++++++
 board/samsung/dts/exynos5250-smdk5250.dts |   11 ++++
 board/samsung/smdk5250/smdk5250.c         |   20 +------
 drivers/i2c/s3c24x0_i2c.c                 |   90 ++++++++++++++++++++++++++++-
 drivers/i2c/s3c24x0_i2c.h                 |    8 +++
 include/fdtdec.h                          |   15 +++++
 include/i2c.h                             |   26 ++++++++
 lib/fdtdec.c                              |   13 ++++
 9 files changed, 271 insertions(+), 20 deletions(-)
 create mode 100644 arch/arm/dts/exynos-periph-id.dtsi

--
1.7.4.4