Re: [U-Boot] [PATCH] i2c: designware_i2c: Optionally check enable status register

[Heiko Schocher]

Hello Stefan,

Am 28.04.2016 um 08:10 schrieb Stefan Roese:

Hi Heiko,

On 28.04.2016 08:06, Heiko Schocher wrote:

Am 27.04.2016 um 09:02 schrieb Stefan Roese:

Some platforms don't implement the enable status register at offset 0x9c.
The SPEAr600 platform is one of them. The recently added check to this
status register can't be performend on these platforms.

This patch introduces a new config option that can be enabled on such
platforms not supporting this register.

Signed-off-by: Stefan Roese 
Cc: Heiko Schocher 
---
  drivers/i2c/designware_i2c.c | 9 +
  1 file changed, 9 insertions(+)


No objections, but I miss an entry in drivers/i2c/Kconfig or at least an
entry in README. Please add this, thanks!


Yes, I am aware that this is not available via Kconfig. I hesitate
to add it to the README, as this is a bit outdated. And we should
move to Kconfig anyways.


Yes!


I'll work on a patch to move this driver and this config option to
Kconfig instead.


Ok, thanks for the info, so I am fine with this patch:

Reviewed-by: Heiko Schocher 

bye,
Heiko
--
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] i2c: designware_i2c: Optionally check enable status register

[Stefan Roese]

Hi Heiko,

On 28.04.2016 08:06, Heiko Schocher wrote:

Am 27.04.2016 um 09:02 schrieb Stefan Roese:

Some platforms don't implement the enable status register at offset 0x9c.
The SPEAr600 platform is one of them. The recently added check to this
status register can't be performend on these platforms.

This patch introduces a new config option that can be enabled on such
platforms not supporting this register.

Signed-off-by: Stefan Roese 
Cc: Heiko Schocher 
---
  drivers/i2c/designware_i2c.c | 9 +
  1 file changed, 9 insertions(+)


No objections, but I miss an entry in drivers/i2c/Kconfig or at least an
entry in README. Please add this, thanks!


Yes, I am aware that this is not available via Kconfig. I hesitate
to add it to the README, as this is a bit outdated. And we should
move to Kconfig anyways.

I'll work on a patch to move this driver and this config option to
Kconfig instead.

Thanks,
Stefan
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] i2c: designware_i2c: Optionally check enable status register

[Heiko Schocher]

Hello Stefan,

Am 27.04.2016 um 09:02 schrieb Stefan Roese:

Some platforms don't implement the enable status register at offset 0x9c.
The SPEAr600 platform is one of them. The recently added check to this
status register can't be performend on these platforms.

This patch introduces a new config option that can be enabled on such
platforms not supporting this register.

Signed-off-by: Stefan Roese 
Cc: Heiko Schocher 
---
  drivers/i2c/designware_i2c.c | 9 +
  1 file changed, 9 insertions(+)


No objections, but I miss an entry in drivers/i2c/Kconfig or at least an
entry in README. Please add this, thanks!

bye,
Heiko


diff --git a/drivers/i2c/designware_i2c.c b/drivers/i2c/designware_i2c.c
index 0c7cd0b..e60fd0a 100644
--- a/drivers/i2c/designware_i2c.c
+++ b/drivers/i2c/designware_i2c.c
@@ -36,6 +36,14 @@ struct dw_i2c {
struct dw_scl_sda_cfg *scl_sda_cfg;
  };

+#ifdef CONFIG_SYS_I2C_DW_ENABLE_STATUS_UNSUPPORTED
+static void dw_i2c_enable(struct i2c_regs *i2c_base, bool enable)
+{
+   u32 ena = enable ? IC_ENABLE_0B : 0;
+
+   writel(ena, _base->ic_enable);
+}
+#else
  static void dw_i2c_enable(struct i2c_regs *i2c_base, bool enable)
  {
u32 ena = enable ? IC_ENABLE_0B : 0;
@@ -56,6 +64,7 @@ static void dw_i2c_enable(struct i2c_regs *i2c_base, bool 
enable)

printf("timeout in %sabling I2C adapter\n", enable ? "en" : "dis");
  }
+#endif

  /*
   * i2c_set_bus_speed - Set the i2c speed



--
DENX Software Engineering GmbH,  Managing Director: Wolfgang Denk
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot