From patchwork Wed Apr 22 16:13:54 2020 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Simon Glass X-Patchwork-Id: 238333 List-Id: U-Boot discussion From: sjg at chromium.org (Simon Glass) Date: Wed, 22 Apr 2020 10:13:54 -0600 Subject: [PATCH 2/2] i2c: designware_i2c: Check if the device is powered In-Reply-To: <20200422161354.187417-1-sjg@chromium.org> References: <20200422161354.187417-1-sjg@chromium.org> Message-ID: <20200422101349.2.If5527972508e0f4b35cc9ecdb1491b1ce85ff3af@changeid> From: Raul E Rangel If the device doesn't return a version that means the device is non-functional. The dw_i2c_regs had invalid offsets for the version field. I got the correct value from the DesignWare databook. It also matches what the Picasso PPR says. Signed-off-by: Raul E Rangel Reviewed-by: Simon Glass Reviewed-by: Furquan Shaikh Tested on chromebook_coral: Signed-off-by: Simon Glass --- drivers/i2c/designware_i2c.c | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/drivers/i2c/designware_i2c.c b/drivers/i2c/designware_i2c.c index ac170769f4..f7a48f6225 100644 --- a/drivers/i2c/designware_i2c.c +++ b/drivers/i2c/designware_i2c.c @@ -16,6 +16,12 @@ #include #include +/* + * This assigned unique hex value is constant and is derived from the two ASCII + * letters 'DW' followed by a 16-bit unsigned number + */ +#define DW_I2C_COMP_TYPE 0x44570140 + #ifdef CONFIG_SYS_I2C_DW_ENABLE_STATUS_UNSUPPORTED static int dw_i2c_enable(struct i2c_regs *i2c_base, bool enable) { @@ -764,6 +770,17 @@ int designware_i2c_ofdata_to_platdata(struct udevice *bus) int designware_i2c_probe(struct udevice *bus) { struct dw_i2c *priv = dev_get_priv(bus); + uint comp_type; + + comp_type = readl(&priv->regs->comp_type); + if (comp_type != DW_I2C_COMP_TYPE) { + log_err("I2C bus %s has unknown type %#x\n", bus->name, + comp_type); + return -ENXIO; + } + + log_info("I2C bus %s version %#x\n", bus->name, + readl(&priv->regs->comp_version)); return __dw_i2c_init(priv->regs, 0, 0); }