Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S933700Ab3CVO7n (ORCPT ); Fri, 22 Mar 2013 10:59:43 -0400 Received: from slimlogic.co.uk ([89.16.172.20]:47344 "EHLO slimlogic.co.uk" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S933673Ab3CVOzS (ORCPT ); Fri, 22 Mar 2013 10:55:18 -0400 From: Ian Lartey To: linux-kernel@vger.kernel.org Cc: linux-doc@vger.kernel.org, linux-arm-kernel@lists.infradead.org, linux-leds@vger.kernel.org, linux-watchdog@vger.kernel.org, devicetree-discuss@lists.ozlabs.org, swarren@wwwdotorg.org, grant.likely@secretlab.ca, broonie@opensource.wolfsonmicro.com, rob.herring@calxeda.com, rob@landley.net, mturquette@linaro.org, linus.walleij@linaro.org, cooloney@gmail.com, sfr@canb.auug.org.au, rpurdie@rpsys.net, akpm@linux-foundation.org, sameo@linux.intel.com, wim@iguana.be, lgirdwood@gmail.com, gg@slimlogic.co.uk, j-keerthy@ti.com, ldewangan@nvidia.com, t-kristo@ti.com Subject: [PATCH v10 03/12] mfd: palmas add variant and OTP detection Date: Fri, 22 Mar 2013 14:55:13 +0000 Message-Id: <1363964122-19201-4-git-send-email-ian@slimlogic.co.uk> X-Mailer: git-send-email 1.7.0.4 In-Reply-To: <1363964122-19201-1-git-send-email-ian@slimlogic.co.uk> References: <1363964122-19201-1-git-send-email-ian@slimlogic.co.uk> Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 9626 Lines: 308 From: Graeme Gregory Read the chip varient and the OTP information from the chip and display this on probe to aid in debugging of issues. Older palmas chips do not have the USB_ID programmed and will therefore return 0x0000 for this field. palmas register read/write/update API is now used palmas_read_product_id_and_revs used by palmas_i2c_probe to get the device id, design and software revisions id field of pamas struct renamed to device_id Updated the DT parsing to agree with the bindings document, just some minor value name changes. Signed-off-by: Graeme Gregory Signed-off-by: Ian Lartey --- drivers/mfd/palmas.c | 102 ++++++++++++++++++++++++++++++++++---------- include/linux/mfd/palmas.h | 35 +++++++++++++++- 2 files changed, 113 insertions(+), 24 deletions(-) diff --git a/drivers/mfd/palmas.c b/drivers/mfd/palmas.c index 73bf76d..913416f 100644 --- a/drivers/mfd/palmas.c +++ b/drivers/mfd/palmas.c @@ -1,9 +1,10 @@ /* * TI Palmas MFD Driver * - * Copyright 2011-2012 Texas Instruments Inc. + * Copyright 2011-2013 Texas Instruments Inc. * * Author: Graeme Gregory + * Author: Ian Lartey * * This program is free software; you can redistribute it and/or modify it * under the terms of the GNU General Public License as published by the @@ -257,6 +258,56 @@ static struct regmap_irq_chip palmas_irq_chip = { PALMAS_INT1_MASK), }; +static int palmas_read_product_id_and_revs(struct palmas *palmas) +{ + int ret; + unsigned int reg; + + /* Read variant info from the device */ + ret = palmas_read(palmas, PALMAS_ID_BASE, PALMAS_PRODUCT_ID_LSB, ®); + if (ret < 0) { + dev_err(palmas->dev, "Unable to read ID err: %d\n", ret); + return ret; + } + + palmas->product_id = reg; + + ret = palmas_read(palmas, PALMAS_ID_BASE, PALMAS_PRODUCT_ID_MSB, ®); + if (ret < 0) { + dev_err(palmas->dev, "Unable to read ID err: %d\n", ret); + return ret; + } + + palmas->product_id |= reg << 8; + + dev_info(palmas->dev, "Product ID %x\n", palmas->product_id); + + ret = palmas_read(palmas, PALMAS_DESIGNREV_BASE, + PALMAS_DESIGNREV, ®); + if (ret < 0) { + dev_err(palmas->dev, "Unable to read DESIGNREV err: %d\n", ret); + return ret; + } + + palmas->designrev = reg & PALMAS_DESIGNREV_DESIGNREV_MASK; + + dev_info(palmas->dev, "Product Design Rev %x\n", palmas->designrev); + + ret = palmas_read(palmas, PALMAS_PMU_CONTROL_BASE, PALMAS_SW_REVISION, + ®); + if (ret < 0) { + dev_err(palmas->dev, "Unable to read SW_REVISION err: %d\n", + ret); + return ret; + } + + palmas->sw_revision = reg; + + dev_info(palmas->dev, "Product SW Rev %x\n", palmas->sw_revision); + + return 0; +} + static int palmas_set_pdata_irq_flag(struct i2c_client *i2c, struct palmas_platform_data *pdata) { @@ -278,20 +329,20 @@ static void palmas_dt_to_pdata(struct i2c_client *i2c, int ret; u32 prop; - ret = of_property_read_u32(node, "ti,mux_pad1", &prop); + ret = of_property_read_u32(node, "ti,mux-pad1", &prop); if (!ret) { pdata->mux_from_pdata = 1; pdata->pad1 = prop; } - ret = of_property_read_u32(node, "ti,mux_pad2", &prop); + ret = of_property_read_u32(node, "ti,mux-pad2", &prop); if (!ret) { pdata->mux_from_pdata = 1; pdata->pad2 = prop; } /* The default for this register is all masked */ - ret = of_property_read_u32(node, "ti,power_ctrl", &prop); + ret = of_property_read_u32(node, "ti,power-ctrl", &prop); if (!ret) pdata->power_ctrl = prop; else @@ -309,7 +360,7 @@ static int palmas_i2c_probe(struct i2c_client *i2c, struct palmas_platform_data *pdata; struct device_node *node = i2c->dev.of_node; int ret = 0, i; - unsigned int reg, addr; + unsigned int reg; int slave; struct mfd_cell *children; @@ -333,7 +384,7 @@ static int palmas_i2c_probe(struct i2c_client *i2c, i2c_set_clientdata(i2c, palmas); palmas->dev = &i2c->dev; - palmas->id = id->driver_data; + palmas->product_id = id->driver_data; palmas->irq = i2c->irq; for (i = 0; i < PALMAS_NUM_CLIENTS; i++) { @@ -374,12 +425,16 @@ static int palmas_i2c_probe(struct i2c_client *i2c, goto err; } + /* Read variant info from the device */ + ret = palmas_read_product_id_and_revs(palmas); + if (ret < 0) + goto err; + /* Change IRQ into clear on read mode for efficiency */ slave = PALMAS_BASE_TO_SLAVE(PALMAS_INTERRUPT_BASE); - addr = PALMAS_BASE_TO_REG(PALMAS_INTERRUPT_BASE, PALMAS_INT_CTRL); reg = PALMAS_INT_CTRL_INT_CLEAR; - regmap_write(palmas->regmap[slave], addr, reg); + ret = palmas_write(palmas, PALMAS_INTERRUPT_BASE, PALMAS_INT_CTRL, reg); ret = regmap_add_irq_chip(palmas->regmap[slave], palmas->irq, IRQF_ONESHOT | pdata->irq_flags, 0, &palmas_irq_chip, @@ -387,17 +442,15 @@ static int palmas_i2c_probe(struct i2c_client *i2c, if (ret < 0) goto err; - slave = PALMAS_BASE_TO_SLAVE(PALMAS_PU_PD_OD_BASE); - addr = PALMAS_BASE_TO_REG(PALMAS_PU_PD_OD_BASE, - PALMAS_PRIMARY_SECONDARY_PAD1); - if (pdata->mux_from_pdata) { reg = pdata->pad1; - ret = regmap_write(palmas->regmap[slave], addr, reg); + ret = palmas_write(palmas, PALMAS_PU_PD_OD_BASE, + PALMAS_PRIMARY_SECONDARY_PAD1, reg); if (ret) goto err_irq; } else { - ret = regmap_read(palmas->regmap[slave], addr, ®); + ret = palmas_read(palmas, PALMAS_PU_PD_OD_BASE, + PALMAS_PRIMARY_SECONDARY_PAD1, ®); if (ret) goto err_irq; } @@ -423,16 +476,15 @@ static int palmas_i2c_probe(struct i2c_client *i2c, if (!(reg & PALMAS_PRIMARY_SECONDARY_PAD1_GPIO_3)) palmas->gpio_muxed |= PALMAS_GPIO_3_MUXED; - addr = PALMAS_BASE_TO_REG(PALMAS_PU_PD_OD_BASE, - PALMAS_PRIMARY_SECONDARY_PAD2); - if (pdata->mux_from_pdata) { reg = pdata->pad2; - ret = regmap_write(palmas->regmap[slave], addr, reg); + ret = palmas_write(palmas, PALMAS_PU_PD_OD_BASE, + PALMAS_PRIMARY_SECONDARY_PAD2, reg); if (ret) goto err_irq; } else { - ret = regmap_read(palmas->regmap[slave], addr, ®); + ret = palmas_read(palmas, PALMAS_PU_PD_OD_BASE, + PALMAS_PRIMARY_SECONDARY_PAD2, ®); if (ret) goto err_irq; } @@ -452,10 +504,8 @@ static int palmas_i2c_probe(struct i2c_client *i2c, reg = pdata->power_ctrl; - slave = PALMAS_BASE_TO_SLAVE(PALMAS_PMU_CONTROL_BASE); - addr = PALMAS_BASE_TO_REG(PALMAS_PMU_CONTROL_BASE, PALMAS_POWER_CTRL); - - ret = regmap_write(palmas->regmap[slave], addr, reg); + ret = palmas_write(palmas, PALMAS_PU_PD_OD_BASE, + PALMAS_PRIMARY_SECONDARY_PAD2, reg); if (ret) goto err_irq; @@ -534,6 +584,12 @@ MODULE_DEVICE_TABLE(i2c, palmas_i2c_id); static struct of_device_id of_palmas_match_tbl[] = { { .compatible = "ti,palmas", }, + { .compatible = "ti,twl6035", }, + { .compatible = "ti,twl6036", }, + { .compatible = "ti,twl6037", }, + { .compatible = "ti,tps65913", }, + { .compatible = "ti,tps65914", }, + { .compatible = "ti,tps80036", }, { /* end */ } }; diff --git a/include/linux/mfd/palmas.h b/include/linux/mfd/palmas.h index 4a066d0..4cffe31 100644 --- a/include/linux/mfd/palmas.h +++ b/include/linux/mfd/palmas.h @@ -44,7 +44,9 @@ struct palmas { struct regmap *regmap[PALMAS_NUM_CLIENTS]; /* Stored chip id */ - int id; + int product_id; + int designrev; + int sw_revision; /* IRQ Data */ int irq; @@ -438,11 +440,13 @@ enum usb_irq_events { #define PALMAS_PU_PD_OD_BASE 0x1F4 #define PALMAS_LED_BASE 0x200 #define PALMAS_INTERRUPT_BASE 0x210 +#define PALMAS_ID_BASE 0x24F #define PALMAS_USB_OTG_BASE 0x250 #define PALMAS_VIBRATOR_BASE 0x270 #define PALMAS_GPIO_BASE 0x280 #define PALMAS_USB_BASE 0x290 #define PALMAS_GPADC_BASE 0x2C0 +#define PALMAS_DESIGNREV_BASE 0x357 #define PALMAS_TRIM_GPADC_BASE 0x3CD /* Registers for function RTC */ @@ -2160,6 +2164,28 @@ enum usb_irq_events { #define PALMAS_INT_CTRL_INT_CLEAR 0x01 #define PALMAS_INT_CTRL_INT_CLEAR_SHIFT 0 +/* Registers for function ID */ +#define PALMAS_VENDOR_ID_LSB 0x0 +#define PALMAS_VENDOR_ID_MSB 0x1 +#define PALMAS_PRODUCT_ID_LSB 0x2 +#define PALMAS_PRODUCT_ID_MSB 0x3 + +/* Bit definitions for VENDOR_ID_LSB */ +#define PALMAS_VENDOR_ID_LSB_VENDOR_ID_MASK 0xff +#define PALMAS_VENDOR_ID_LSB_VENDOR_ID_SHIFT 0 + +/* Bit definitions for VENDOR_ID_MSB */ +#define PALMAS_VENDOR_ID_MSB_VENDOR_ID_MASK 0xff +#define PALMAS_VENDOR_ID_MSB_VENDOR_ID_SHIFT 0 + +/* Bit definitions for PRODUCT_ID_LSB */ +#define PALMAS_PRODUCT_ID_LSB_PRODUCT_ID_MASK 0xff +#define PALMAS_PRODUCT_ID_LSB_PRODUCT_ID_SHIFT 0 + +/* Bit definitions for PRODUCT_ID_MSB */ +#define PALMAS_PRODUCT_ID_MSB_PRODUCT_ID_MASK 0xff +#define PALMAS_PRODUCT_ID_MSB_PRODUCT_ID_SHIFT 0 + /* Registers for function USB_OTG */ #define PALMAS_USB_WAKEUP 0x3 #define PALMAS_USB_VBUS_CTRL_SET 0x4 @@ -2782,6 +2808,13 @@ enum usb_irq_events { #define PALMAS_GPADC_SMPS_VSEL_MONITORING_SMPS_VSEL_MONITORING_MASK 0x7f #define PALMAS_GPADC_SMPS_VSEL_MONITORING_SMPS_VSEL_MONITORING_SHIFT 0 +/* Registers for function DESIGNREV */ +#define PALMAS_DESIGNREV 0x0 + +/* Bit definitions for DESIGNREV */ +#define PALMAS_DESIGNREV_DESIGNREV_MASK 0x0f +#define PALMAS_DESIGNREV_DESIGNREV_SHIFT 0 + /* Registers for function GPADC */ #define PALMAS_GPADC_TRIM1 0x0 #define PALMAS_GPADC_TRIM2 0x1 -- 1.7.0.4 -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/