Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752866AbcDUOUs (ORCPT ); Thu, 21 Apr 2016 10:20:48 -0400 Received: from down.free-electrons.com ([37.187.137.238]:36906 "EHLO mail.free-electrons.com" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1751790AbcDUOUq (ORCPT ); Thu, 21 Apr 2016 10:20:46 -0400 Date: Thu, 21 Apr 2016 16:20:33 +0200 From: Alexandre Belloni To: Wadim Egorov Cc: linux-kernel@vger.kernel.org, linux-clk@vger.kernel.org, rtc-linux@googlegroups.com, devicetree@vger.kernel.org, linux-rockchip@lists.infradead.org, robh+dt@kernel.org, pawel.moll@arm.com, mark.rutland@arm.com, ijc+devicetree@hellion.org.uk, galak@codeaurora.org, mturquette@baylibre.com, sboyd@codeaurora.org, lee.jones@linaro.org, lgirdwood@gmail.com, broonie@kernel.org, a.zummo@towertech.it, dianders@chromium.org, zyw@rock-chips.com Subject: Re: [PATCH v2 1/7] drivers: mfd: rk808: rename rk808 struct to rk8xx Message-ID: <20160421142033.GY29844@piout.net> References: <1461244361-45686-1-git-send-email-w.egorov@phytec.de> <1461244361-45686-2-git-send-email-w.egorov@phytec.de> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <1461244361-45686-2-git-send-email-w.egorov@phytec.de> User-Agent: Mutt/1.5.24 (2015-08-30) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 16324 Lines: 440 On 21/04/2016 at 15:12:35 +0200, Wadim Egorov wrote : > This patch renames the rk808 struct. So it is more clear that this > struct can be shared between all RK8XX related PMIC drivers. > I'm still thinking this is unnecessary and that the rk8xx will become more confusing that rx808 in the future. If rx8xx needed to be different from rk808, I'd have a different opinion but that is not the case here. > Signed-off-by: Wadim Egorov > --- > drivers/clk/clk-rk808.c | 20 +++++++-------- > drivers/mfd/rk808.c | 36 +++++++++++++------------- > drivers/regulator/rk808-regulator.c | 8 +++--- > drivers/rtc/rtc-rk808.c | 50 ++++++++++++++++++------------------- > include/linux/mfd/rk808.h | 2 +- > 5 files changed, 58 insertions(+), 58 deletions(-) > > diff --git a/drivers/clk/clk-rk808.c b/drivers/clk/clk-rk808.c > index 0fee2f4..6df2e72 100644 > --- a/drivers/clk/clk-rk808.c > +++ b/drivers/clk/clk-rk808.c > @@ -25,7 +25,7 @@ > #define RK808_NR_OUTPUT 2 > > struct rk808_clkout { > - struct rk808 *rk808; > + struct rk8xx *rk8xx; > struct clk_onecell_data clk_data; > struct clk_hw clkout1_hw; > struct clk_hw clkout2_hw; > @@ -42,9 +42,9 @@ static int rk808_clkout2_enable(struct clk_hw *hw, bool enable) > struct rk808_clkout *rk808_clkout = container_of(hw, > struct rk808_clkout, > clkout2_hw); > - struct rk808 *rk808 = rk808_clkout->rk808; > + struct rk8xx *rk8xx = rk808_clkout->rk8xx; > > - return regmap_update_bits(rk808->regmap, RK808_CLK32OUT_REG, > + return regmap_update_bits(rk8xx->regmap, RK808_CLK32OUT_REG, > CLK32KOUT2_EN, enable ? CLK32KOUT2_EN : 0); > } > > @@ -63,10 +63,10 @@ static int rk808_clkout2_is_prepared(struct clk_hw *hw) > struct rk808_clkout *rk808_clkout = container_of(hw, > struct rk808_clkout, > clkout2_hw); > - struct rk808 *rk808 = rk808_clkout->rk808; > + struct rk8xx *rk8xx = rk808_clkout->rk8xx; > uint32_t val; > > - int ret = regmap_read(rk808->regmap, RK808_CLK32OUT_REG, &val); > + int ret = regmap_read(rk8xx->regmap, RK808_CLK32OUT_REG, &val); > > if (ret < 0) > return ret; > @@ -87,8 +87,8 @@ static const struct clk_ops rk808_clkout2_ops = { > > static int rk808_clkout_probe(struct platform_device *pdev) > { > - struct rk808 *rk808 = dev_get_drvdata(pdev->dev.parent); > - struct i2c_client *client = rk808->i2c; > + struct rk8xx *rk8xx = dev_get_drvdata(pdev->dev.parent); > + struct i2c_client *client = rk8xx->i2c; > struct device_node *node = client->dev.of_node; > struct clk_init_data init = {}; > struct clk **clk_table; > @@ -99,7 +99,7 @@ static int rk808_clkout_probe(struct platform_device *pdev) > if (!rk808_clkout) > return -ENOMEM; > > - rk808_clkout->rk808 = rk808; > + rk808_clkout->rk8xx = rk8xx; > > clk_table = devm_kcalloc(&client->dev, RK808_NR_OUTPUT, > sizeof(struct clk *), GFP_KERNEL); > @@ -144,8 +144,8 @@ static int rk808_clkout_probe(struct platform_device *pdev) > > static int rk808_clkout_remove(struct platform_device *pdev) > { > - struct rk808 *rk808 = dev_get_drvdata(pdev->dev.parent); > - struct i2c_client *client = rk808->i2c; > + struct rk8xx *rk8xx = dev_get_drvdata(pdev->dev.parent); > + struct i2c_client *client = rk8xx->i2c; > struct device_node *node = client->dev.of_node; > > of_clk_del_provider(node); > diff --git a/drivers/mfd/rk808.c b/drivers/mfd/rk808.c > index 4b1e439..0386251 100644 > --- a/drivers/mfd/rk808.c > +++ b/drivers/mfd/rk808.c > @@ -152,15 +152,15 @@ static struct i2c_client *rk808_i2c_client; > static void rk808_device_shutdown(void) > { > int ret; > - struct rk808 *rk808 = i2c_get_clientdata(rk808_i2c_client); > + struct rk8xx *rk8xx = i2c_get_clientdata(rk808_i2c_client); > > - if (!rk808) { > + if (!rk8xx) { > dev_warn(&rk808_i2c_client->dev, > "have no rk808, so do nothing here\n"); > return; > } > > - ret = regmap_update_bits(rk808->regmap, > + ret = regmap_update_bits(rk8xx->regmap, > RK808_DEVCTRL_REG, > DEV_OFF_RST, DEV_OFF_RST); > if (ret) > @@ -171,7 +171,7 @@ static int rk808_probe(struct i2c_client *client, > const struct i2c_device_id *id) > { > struct device_node *np = client->dev.of_node; > - struct rk808 *rk808; > + struct rk8xx *rk8xx; > int pm_off = 0; > int ret; > int i; > @@ -181,18 +181,18 @@ static int rk808_probe(struct i2c_client *client, > return -EINVAL; > } > > - rk808 = devm_kzalloc(&client->dev, sizeof(*rk808), GFP_KERNEL); > - if (!rk808) > + rk8xx = devm_kzalloc(&client->dev, sizeof(*rk8xx), GFP_KERNEL); > + if (!rk8xx) > return -ENOMEM; > > - rk808->regmap = devm_regmap_init_i2c(client, &rk808_regmap_config); > - if (IS_ERR(rk808->regmap)) { > + rk8xx->regmap = devm_regmap_init_i2c(client, &rk808_regmap_config); > + if (IS_ERR(rk8xx->regmap)) { > dev_err(&client->dev, "regmap initialization failed\n"); > - return PTR_ERR(rk808->regmap); > + return PTR_ERR(rk8xx->regmap); > } > > for (i = 0; i < ARRAY_SIZE(pre_init_reg); i++) { > - ret = regmap_update_bits(rk808->regmap, pre_init_reg[i].addr, > + ret = regmap_update_bits(rk8xx->regmap, pre_init_reg[i].addr, > pre_init_reg[i].mask, > pre_init_reg[i].value); > if (ret) { > @@ -202,20 +202,20 @@ static int rk808_probe(struct i2c_client *client, > } > } > > - ret = regmap_add_irq_chip(rk808->regmap, client->irq, > + ret = regmap_add_irq_chip(rk8xx->regmap, client->irq, > IRQF_ONESHOT, -1, > - &rk808_irq_chip, &rk808->irq_data); > + &rk808_irq_chip, &rk8xx->irq_data); > if (ret) { > dev_err(&client->dev, "Failed to add irq_chip %d\n", ret); > return ret; > } > > - rk808->i2c = client; > - i2c_set_clientdata(client, rk808); > + rk8xx->i2c = client; > + i2c_set_clientdata(client, rk8xx); > > ret = mfd_add_devices(&client->dev, -1, > rk808s, ARRAY_SIZE(rk808s), > - NULL, 0, regmap_irq_get_domain(rk808->irq_data)); > + NULL, 0, regmap_irq_get_domain(rk8xx->irq_data)); > if (ret) { > dev_err(&client->dev, "failed to add MFD devices %d\n", ret); > goto err_irq; > @@ -231,15 +231,15 @@ static int rk808_probe(struct i2c_client *client, > return 0; > > err_irq: > - regmap_del_irq_chip(client->irq, rk808->irq_data); > + regmap_del_irq_chip(client->irq, rk8xx->irq_data); > return ret; > } > > static int rk808_remove(struct i2c_client *client) > { > - struct rk808 *rk808 = i2c_get_clientdata(client); > + struct rk8xx *rk8xx = i2c_get_clientdata(client); > > - regmap_del_irq_chip(client->irq, rk808->irq_data); > + regmap_del_irq_chip(client->irq, rk8xx->irq_data); > mfd_remove_devices(&client->dev); > pm_power_off = NULL; > > diff --git a/drivers/regulator/rk808-regulator.c b/drivers/regulator/rk808-regulator.c > index d86a3dc..37aaba9 100644 > --- a/drivers/regulator/rk808-regulator.c > +++ b/drivers/regulator/rk808-regulator.c > @@ -568,8 +568,8 @@ dt_parse_end: > > static int rk808_regulator_probe(struct platform_device *pdev) > { > - struct rk808 *rk808 = dev_get_drvdata(pdev->dev.parent); > - struct i2c_client *client = rk808->i2c; > + struct rk8xx *rk8xx = dev_get_drvdata(pdev->dev.parent); > + struct i2c_client *client = rk8xx->i2c; > struct regulator_config config = {}; > struct regulator_dev *rk808_rdev; > struct rk808_regulator_data *pdata; > @@ -580,7 +580,7 @@ static int rk808_regulator_probe(struct platform_device *pdev) > return -ENOMEM; > > ret = rk808_regulator_dt_parse_pdata(&pdev->dev, &client->dev, > - rk808->regmap, pdata); > + rk8xx->regmap, pdata); > if (ret < 0) > return ret; > > @@ -594,7 +594,7 @@ static int rk808_regulator_probe(struct platform_device *pdev) > > config.dev = &client->dev; > config.driver_data = pdata; > - config.regmap = rk808->regmap; > + config.regmap = rk8xx->regmap; > config.of_node = rk808_reg_matches[i].of_node; > config.init_data = rk808_reg_matches[i].init_data; > > diff --git a/drivers/rtc/rtc-rk808.c b/drivers/rtc/rtc-rk808.c > index 35c9aad..b3fe5a8 100644 > --- a/drivers/rtc/rtc-rk808.c > +++ b/drivers/rtc/rtc-rk808.c > @@ -51,7 +51,7 @@ > #define NUM_ALARM_REGS (RK808_ALARM_YEARS_REG - RK808_ALARM_SECONDS_REG + 1) > > struct rk808_rtc { > - struct rk808 *rk808; > + struct rk8xx *rk8xx; > struct rtc_device *rtc; > int irq; > }; > @@ -96,12 +96,12 @@ static void gregorian_to_rockchip(struct rtc_time *tm) > static int rk808_rtc_readtime(struct device *dev, struct rtc_time *tm) > { > struct rk808_rtc *rk808_rtc = dev_get_drvdata(dev); > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > u8 rtc_data[NUM_TIME_REGS]; > int ret; > > /* Force an update of the shadowed registers right now */ > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_CTRL_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_CTRL_REG, > BIT_RTC_CTRL_REG_RTC_GET_TIME, > BIT_RTC_CTRL_REG_RTC_GET_TIME); > if (ret) { > @@ -115,7 +115,7 @@ static int rk808_rtc_readtime(struct device *dev, struct rtc_time *tm) > * 32khz. If we clear the GET_TIME bit here, the time of i2c transfer > * certainly more than 31.25us: 16 * 2.5us at 400kHz bus frequency. > */ > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_CTRL_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_CTRL_REG, > BIT_RTC_CTRL_REG_RTC_GET_TIME, > 0); > if (ret) { > @@ -123,7 +123,7 @@ static int rk808_rtc_readtime(struct device *dev, struct rtc_time *tm) > return ret; > } > > - ret = regmap_bulk_read(rk808->regmap, RK808_SECONDS_REG, > + ret = regmap_bulk_read(rk8xx->regmap, RK808_SECONDS_REG, > rtc_data, NUM_TIME_REGS); > if (ret) { > dev_err(dev, "Failed to bulk read rtc_data: %d\n", ret); > @@ -149,7 +149,7 @@ static int rk808_rtc_readtime(struct device *dev, struct rtc_time *tm) > static int rk808_rtc_set_time(struct device *dev, struct rtc_time *tm) > { > struct rk808_rtc *rk808_rtc = dev_get_drvdata(dev); > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > u8 rtc_data[NUM_TIME_REGS]; > int ret; > > @@ -166,7 +166,7 @@ static int rk808_rtc_set_time(struct device *dev, struct rtc_time *tm) > rtc_data[6] = bin2bcd(tm->tm_wday); > > /* Stop RTC while updating the RTC registers */ > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_CTRL_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_CTRL_REG, > BIT_RTC_CTRL_REG_STOP_RTC_M, > BIT_RTC_CTRL_REG_STOP_RTC_M); > if (ret) { > @@ -174,14 +174,14 @@ static int rk808_rtc_set_time(struct device *dev, struct rtc_time *tm) > return ret; > } > > - ret = regmap_bulk_write(rk808->regmap, RK808_SECONDS_REG, > + ret = regmap_bulk_write(rk8xx->regmap, RK808_SECONDS_REG, > rtc_data, NUM_TIME_REGS); > if (ret) { > dev_err(dev, "Failed to bull write rtc_data: %d\n", ret); > return ret; > } > /* Start RTC again */ > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_CTRL_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_CTRL_REG, > BIT_RTC_CTRL_REG_STOP_RTC_M, 0); > if (ret) { > dev_err(dev, "Failed to update RTC control: %d\n", ret); > @@ -194,12 +194,12 @@ static int rk808_rtc_set_time(struct device *dev, struct rtc_time *tm) > static int rk808_rtc_readalarm(struct device *dev, struct rtc_wkalrm *alrm) > { > struct rk808_rtc *rk808_rtc = dev_get_drvdata(dev); > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > u8 alrm_data[NUM_ALARM_REGS]; > uint32_t int_reg; > int ret; > > - ret = regmap_bulk_read(rk808->regmap, RK808_ALARM_SECONDS_REG, > + ret = regmap_bulk_read(rk8xx->regmap, RK808_ALARM_SECONDS_REG, > alrm_data, NUM_ALARM_REGS); > > alrm->time.tm_sec = bcd2bin(alrm_data[0] & SECONDS_REG_MSK); > @@ -210,7 +210,7 @@ static int rk808_rtc_readalarm(struct device *dev, struct rtc_wkalrm *alrm) > alrm->time.tm_year = (bcd2bin(alrm_data[5] & YEARS_REG_MSK)) + 100; > rockchip_to_gregorian(&alrm->time); > > - ret = regmap_read(rk808->regmap, RK808_RTC_INT_REG, &int_reg); > + ret = regmap_read(rk8xx->regmap, RK808_RTC_INT_REG, &int_reg); > if (ret) { > dev_err(dev, "Failed to read RTC INT REG: %d\n", ret); > return ret; > @@ -228,10 +228,10 @@ static int rk808_rtc_readalarm(struct device *dev, struct rtc_wkalrm *alrm) > > static int rk808_rtc_stop_alarm(struct rk808_rtc *rk808_rtc) > { > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > int ret; > > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_INT_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_INT_REG, > BIT_RTC_INTERRUPTS_REG_IT_ALARM_M, 0); > > return ret; > @@ -239,10 +239,10 @@ static int rk808_rtc_stop_alarm(struct rk808_rtc *rk808_rtc) > > static int rk808_rtc_start_alarm(struct rk808_rtc *rk808_rtc) > { > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > int ret; > > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_INT_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_INT_REG, > BIT_RTC_INTERRUPTS_REG_IT_ALARM_M, > BIT_RTC_INTERRUPTS_REG_IT_ALARM_M); > > @@ -252,7 +252,7 @@ static int rk808_rtc_start_alarm(struct rk808_rtc *rk808_rtc) > static int rk808_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm) > { > struct rk808_rtc *rk808_rtc = dev_get_drvdata(dev); > - struct rk808 *rk808 = rk808_rtc->rk808; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > u8 alrm_data[NUM_ALARM_REGS]; > int ret; > > @@ -274,7 +274,7 @@ static int rk808_rtc_setalarm(struct device *dev, struct rtc_wkalrm *alrm) > alrm_data[4] = bin2bcd(alrm->time.tm_mon + 1); > alrm_data[5] = bin2bcd(alrm->time.tm_year - 100); > > - ret = regmap_bulk_write(rk808->regmap, RK808_ALARM_SECONDS_REG, > + ret = regmap_bulk_write(rk8xx->regmap, RK808_ALARM_SECONDS_REG, > alrm_data, NUM_ALARM_REGS); > if (ret) { > dev_err(dev, "Failed to bulk write: %d\n", ret); > @@ -314,11 +314,11 @@ static int rk808_rtc_alarm_irq_enable(struct device *dev, > static irqreturn_t rk808_alarm_irq(int irq, void *data) > { > struct rk808_rtc *rk808_rtc = data; > - struct rk808 *rk808 = rk808_rtc->rk808; > - struct i2c_client *client = rk808->i2c; > + struct rk8xx *rk8xx = rk808_rtc->rk8xx; > + struct i2c_client *client = rk8xx->i2c; > int ret; > > - ret = regmap_write(rk808->regmap, RK808_RTC_STATUS_REG, > + ret = regmap_write(rk8xx->regmap, RK808_RTC_STATUS_REG, > RTC_STATUS_MASK); > if (ret) { > dev_err(&client->dev, > @@ -373,7 +373,7 @@ static SIMPLE_DEV_PM_OPS(rk808_rtc_pm_ops, > > static int rk808_rtc_probe(struct platform_device *pdev) > { > - struct rk808 *rk808 = dev_get_drvdata(pdev->dev.parent); > + struct rk8xx *rk8xx = dev_get_drvdata(pdev->dev.parent); > struct rk808_rtc *rk808_rtc; > struct rtc_time tm; > int ret; > @@ -383,10 +383,10 @@ static int rk808_rtc_probe(struct platform_device *pdev) > return -ENOMEM; > > platform_set_drvdata(pdev, rk808_rtc); > - rk808_rtc->rk808 = rk808; > + rk808_rtc->rk8xx = rk8xx; > > /* start rtc running by default, and use shadowed timer. */ > - ret = regmap_update_bits(rk808->regmap, RK808_RTC_CTRL_REG, > + ret = regmap_update_bits(rk8xx->regmap, RK808_RTC_CTRL_REG, > BIT_RTC_CTRL_REG_STOP_RTC_M | > BIT_RTC_CTRL_REG_RTC_READSEL_M, > BIT_RTC_CTRL_REG_RTC_READSEL_M); > @@ -396,7 +396,7 @@ static int rk808_rtc_probe(struct platform_device *pdev) > return ret; > } > > - ret = regmap_write(rk808->regmap, RK808_RTC_STATUS_REG, > + ret = regmap_write(rk8xx->regmap, RK808_RTC_STATUS_REG, > RTC_STATUS_MASK); > if (ret) { > dev_err(&pdev->dev, > diff --git a/include/linux/mfd/rk808.h b/include/linux/mfd/rk808.h > index 441b6ee..117d79f 100644 > --- a/include/linux/mfd/rk808.h > +++ b/include/linux/mfd/rk808.h > @@ -191,7 +191,7 @@ enum { > BOOST_ILMIN_250MA, > }; > > -struct rk808 { > +struct rk8xx { > struct i2c_client *i2c; > struct regmap_irq_chip_data *irq_data; > struct regmap *regmap; > -- > 1.9.1 > -- Alexandre Belloni, Free Electrons Embedded Linux, Kernel and Android engineering http://free-electrons.com