nagai/kernel
3
0
Fork 0
forked from mirrors/linux
Code Pull requests Activity

pwm: img: Rename variable pointing to driver private data

Browse source 
Status quo is that variables of type struct img_pwm_chip * are named
"pwm_chip", "pwm" or "chip" which are all not optimal because there is a
struct pwm_chip in the core, usually only struct pwm_device * variables are
named "pwm" and "chip" is usually used for variabled of type struct
pwm_chip *.

So consistently use the same and non-conflicting name "imgchip".

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
Signed-off-by: Thierry Reding <thierry.reding@gmail.com>
This commit is contained in:
Uwe Kleine-König 2021-11-23 10:29:35 +01:00 committed by Thierry Reding
parent b23fd25ec8
commit 22e8e19a46
1 changed files with 70 additions and 71 deletions
repo.diff.show_diff_stats Download patch file Download diff file Expand all files Collapse all files

141
drivers/pwm/pwm-img.c
View file

@ -77,16 +77,15 @@ static inline struct img_pwm_chip *to_img_pwm_chip(struct pwm_chip *chip)
return container_of(chip, struct img_pwm_chip, chip); return container_of(chip, struct img_pwm_chip, chip);
} }
static inline void img_pwm_writel(struct img_pwm_chip *chip, static inline void img_pwm_writel(struct img_pwm_chip *imgchip,
u32 reg, u32 val) u32 reg, u32 val)
{ {
writel(val, chip->base + reg); writel(val, imgchip->base + reg);
} }
static inline u32 img_pwm_readl(struct img_pwm_chip *chip, static inline u32 img_pwm_readl(struct img_pwm_chip *imgchip, u32 reg)
u32 reg)
{ {
return readl(chip->base + reg); return readl(imgchip->base + reg);
} }
static int img_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm, static int img_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm,
@ -94,17 +93,17 @@ static int img_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm,
{ {
u32 val, div, duty, timebase; u32 val, div, duty, timebase;
unsigned long mul, output_clk_hz, input_clk_hz; unsigned long mul, output_clk_hz, input_clk_hz;
struct img_pwm_chip *pwm_chip = to_img_pwm_chip(chip); struct img_pwm_chip *imgchip = to_img_pwm_chip(chip);
unsigned int max_timebase = pwm_chip->data->max_timebase; unsigned int max_timebase = imgchip->data->max_timebase;
int ret; int ret;
if (period_ns < pwm_chip->min_period_ns || if (period_ns < imgchip->min_period_ns ||
period_ns > pwm_chip->max_period_ns) { period_ns > imgchip->max_period_ns) {
dev_err(chip->dev, "configured period not in range\n"); dev_err(chip->dev, "configured period not in range\n");
return -ERANGE; return -ERANGE;
} }
input_clk_hz = clk_get_rate(pwm_chip->pwm_clk); input_clk_hz = clk_get_rate(imgchip->pwm_clk);
output_clk_hz = DIV_ROUND_UP(NSEC_PER_SEC, period_ns); output_clk_hz = DIV_ROUND_UP(NSEC_PER_SEC, period_ns);
mul = DIV_ROUND_UP(input_clk_hz, output_clk_hz); mul = DIV_ROUND_UP(input_clk_hz, output_clk_hz);
@ -132,15 +131,15 @@ static int img_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm,
if (ret < 0) if (ret < 0)
return ret; return ret;
val = img_pwm_readl(pwm_chip, PWM_CTRL_CFG); val = img_pwm_readl(imgchip, PWM_CTRL_CFG);
val &= ~(PWM_CTRL_CFG_DIV_MASK << PWM_CTRL_CFG_DIV_SHIFT(pwm->hwpwm)); val &= ~(PWM_CTRL_CFG_DIV_MASK << PWM_CTRL_CFG_DIV_SHIFT(pwm->hwpwm));
val |= (div & PWM_CTRL_CFG_DIV_MASK) << val |= (div & PWM_CTRL_CFG_DIV_MASK) <<
PWM_CTRL_CFG_DIV_SHIFT(pwm->hwpwm); PWM_CTRL_CFG_DIV_SHIFT(pwm->hwpwm);
img_pwm_writel(pwm_chip, PWM_CTRL_CFG, val); img_pwm_writel(imgchip, PWM_CTRL_CFG, val);
val = (duty << PWM_CH_CFG_DUTY_SHIFT) | val = (duty << PWM_CH_CFG_DUTY_SHIFT) |
(timebase << PWM_CH_CFG_TMBASE_SHIFT); (timebase << PWM_CH_CFG_TMBASE_SHIFT);
img_pwm_writel(pwm_chip, PWM_CH_CFG(pwm->hwpwm), val); img_pwm_writel(imgchip, PWM_CH_CFG(pwm->hwpwm), val);
pm_runtime_mark_last_busy(chip->dev); pm_runtime_mark_last_busy(chip->dev);
pm_runtime_put_autosuspend(chip->dev); pm_runtime_put_autosuspend(chip->dev);
@ -151,18 +150,18 @@ static int img_pwm_config(struct pwm_chip *chip, struct pwm_device *pwm,
static int img_pwm_enable(struct pwm_chip *chip, struct pwm_device *pwm) static int img_pwm_enable(struct pwm_chip *chip, struct pwm_device *pwm)
{ {
u32 val; u32 val;
struct img_pwm_chip *pwm_chip = to_img_pwm_chip(chip); struct img_pwm_chip *imgchip = to_img_pwm_chip(chip);
int ret; int ret;
ret = pm_runtime_resume_and_get(chip->dev); ret = pm_runtime_resume_and_get(chip->dev);
if (ret < 0) if (ret < 0)
return ret; return ret;
val = img_pwm_readl(pwm_chip, PWM_CTRL_CFG); val = img_pwm_readl(imgchip, PWM_CTRL_CFG);
val |= BIT(pwm->hwpwm); val |= BIT(pwm->hwpwm);
img_pwm_writel(pwm_chip, PWM_CTRL_CFG, val); img_pwm_writel(imgchip, PWM_CTRL_CFG, val);
regmap_update_bits(pwm_chip->periph_regs, PERIP_PWM_PDM_CONTROL, regmap_update_bits(imgchip->periph_regs, PERIP_PWM_PDM_CONTROL,
PERIP_PWM_PDM_CONTROL_CH_MASK << PERIP_PWM_PDM_CONTROL_CH_MASK <<
PERIP_PWM_PDM_CONTROL_CH_SHIFT(pwm->hwpwm), 0); PERIP_PWM_PDM_CONTROL_CH_SHIFT(pwm->hwpwm), 0);
@ -172,11 +171,11 @@ static int img_pwm_enable(struct pwm_chip *chip, struct pwm_device *pwm)
static void img_pwm_disable(struct pwm_chip *chip, struct pwm_device *pwm) static void img_pwm_disable(struct pwm_chip *chip, struct pwm_device *pwm)
{ {
u32 val; u32 val;
struct img_pwm_chip *pwm_chip = to_img_pwm_chip(chip); struct img_pwm_chip *imgchip = to_img_pwm_chip(chip);
val = img_pwm_readl(pwm_chip, PWM_CTRL_CFG); val = img_pwm_readl(imgchip, PWM_CTRL_CFG);
val &= ~BIT(pwm->hwpwm); val &= ~BIT(pwm->hwpwm);
img_pwm_writel(pwm_chip, PWM_CTRL_CFG, val); img_pwm_writel(imgchip, PWM_CTRL_CFG, val);
pm_runtime_mark_last_busy(chip->dev); pm_runtime_mark_last_busy(chip->dev);
pm_runtime_put_autosuspend(chip->dev); pm_runtime_put_autosuspend(chip->dev);
@ -227,29 +226,29 @@ MODULE_DEVICE_TABLE(of, img_pwm_of_match);
static int img_pwm_runtime_suspend(struct device *dev) static int img_pwm_runtime_suspend(struct device *dev)
{ {
struct img_pwm_chip *pwm_chip = dev_get_drvdata(dev); struct img_pwm_chip *imgchip = dev_get_drvdata(dev);
clk_disable_unprepare(pwm_chip->pwm_clk); clk_disable_unprepare(imgchip->pwm_clk);
clk_disable_unprepare(pwm_chip->sys_clk); clk_disable_unprepare(imgchip->sys_clk);
return 0; return 0;
} }
static int img_pwm_runtime_resume(struct device *dev) static int img_pwm_runtime_resume(struct device *dev)
{ {
struct img_pwm_chip *pwm_chip = dev_get_drvdata(dev); struct img_pwm_chip *imgchip = dev_get_drvdata(dev);
int ret; int ret;
ret = clk_prepare_enable(pwm_chip->sys_clk); ret = clk_prepare_enable(imgchip->sys_clk);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "could not prepare or enable sys clock\n"); dev_err(dev, "could not prepare or enable sys clock\n");
return ret; return ret;
} }
ret = clk_prepare_enable(pwm_chip->pwm_clk); ret = clk_prepare_enable(imgchip->pwm_clk);
if (ret < 0) { if (ret < 0) {
dev_err(dev, "could not prepare or enable pwm clock\n"); dev_err(dev, "could not prepare or enable pwm clock\n");
clk_disable_unprepare(pwm_chip->sys_clk); clk_disable_unprepare(imgchip->sys_clk);
return ret; return ret;
} }
@ -261,42 +260,42 @@ static int img_pwm_probe(struct platform_device *pdev)
int ret; int ret;
u64 val; u64 val;
unsigned long clk_rate; unsigned long clk_rate;
struct img_pwm_chip *pwm; struct img_pwm_chip *imgchip;
const struct of_device_id *of_dev_id; const struct of_device_id *of_dev_id;
pwm = devm_kzalloc(&pdev->dev, sizeof(*pwm), GFP_KERNEL); imgchip = devm_kzalloc(&pdev->dev, sizeof(*imgchip), GFP_KERNEL);
if (!pwm) if (!imgchip)
return -ENOMEM; return -ENOMEM;
pwm->dev = &pdev->dev; imgchip->dev = &pdev->dev;
pwm->base = devm_platform_ioremap_resource(pdev, 0); imgchip->base = devm_platform_ioremap_resource(pdev, 0);
if (IS_ERR(pwm->base)) if (IS_ERR(imgchip->base))
return PTR_ERR(pwm->base); return PTR_ERR(imgchip->base);
of_dev_id = of_match_device(img_pwm_of_match, &pdev->dev); of_dev_id = of_match_device(img_pwm_of_match, &pdev->dev);
if (!of_dev_id) if (!of_dev_id)
return -ENODEV; return -ENODEV;
pwm->data = of_dev_id->data; imgchip->data = of_dev_id->data;
pwm->periph_regs = syscon_regmap_lookup_by_phandle(pdev->dev.of_node, imgchip->periph_regs = syscon_regmap_lookup_by_phandle(pdev->dev.of_node,
"img,cr-periph"); "img,cr-periph");
if (IS_ERR(pwm->periph_regs)) if (IS_ERR(imgchip->periph_regs))
return PTR_ERR(pwm->periph_regs); return PTR_ERR(imgchip->periph_regs);
pwm->sys_clk = devm_clk_get(&pdev->dev, "sys"); imgchip->sys_clk = devm_clk_get(&pdev->dev, "sys");
if (IS_ERR(pwm->sys_clk)) { if (IS_ERR(imgchip->sys_clk)) {
dev_err(&pdev->dev, "failed to get system clock\n"); dev_err(&pdev->dev, "failed to get system clock\n");
return PTR_ERR(pwm->sys_clk); return PTR_ERR(imgchip->sys_clk);
} }
pwm->pwm_clk = devm_clk_get(&pdev->dev, "pwm"); imgchip->pwm_clk = devm_clk_get(&pdev->dev, "imgchip");
if (IS_ERR(pwm->pwm_clk)) { if (IS_ERR(imgchip->pwm_clk)) {
dev_err(&pdev->dev, "failed to get pwm clock\n"); dev_err(&pdev->dev, "failed to get imgchip clock\n");
return PTR_ERR(pwm->pwm_clk); return PTR_ERR(imgchip->pwm_clk);
} }
platform_set_drvdata(pdev, pwm); platform_set_drvdata(pdev, imgchip);
pm_runtime_set_autosuspend_delay(&pdev->dev, IMG_PWM_PM_TIMEOUT); pm_runtime_set_autosuspend_delay(&pdev->dev, IMG_PWM_PM_TIMEOUT);
pm_runtime_use_autosuspend(&pdev->dev); pm_runtime_use_autosuspend(&pdev->dev);
@ -307,27 +306,27 @@ static int img_pwm_probe(struct platform_device *pdev)
goto err_pm_disable; goto err_pm_disable;
} }
clk_rate = clk_get_rate(pwm->pwm_clk); clk_rate = clk_get_rate(imgchip->pwm_clk);
if (!clk_rate) { if (!clk_rate) {
dev_err(&pdev->dev, "pwm clock has no frequency\n"); dev_err(&pdev->dev, "imgchip clock has no frequency\n");
ret = -EINVAL; ret = -EINVAL;
goto err_suspend; goto err_suspend;
} }
/* The maximum input clock divider is 512 */ /* The maximum input clock divider is 512 */
val = (u64)NSEC_PER_SEC * 512 * pwm->data->max_timebase; val = (u64)NSEC_PER_SEC * 512 * imgchip->data->max_timebase;
do_div(val, clk_rate); do_div(val, clk_rate);
pwm->max_period_ns = val; imgchip->max_period_ns = val;
val = (u64)NSEC_PER_SEC * MIN_TMBASE_STEPS; val = (u64)NSEC_PER_SEC * MIN_TMBASE_STEPS;
do_div(val, clk_rate); do_div(val, clk_rate);
pwm->min_period_ns = val; imgchip->min_period_ns = val;
pwm->chip.dev = &pdev->dev; imgchip->chip.dev = &pdev->dev;
pwm->chip.ops = &img_pwm_ops; imgchip->chip.ops = &img_pwm_ops;
pwm->chip.npwm = IMG_PWM_NPWM; imgchip->chip.npwm = IMG_PWM_NPWM;
ret = pwmchip_add(&pwm->chip); ret = pwmchip_add(&imgchip->chip);
if (ret < 0) { if (ret < 0) {
dev_err(&pdev->dev, "pwmchip_add failed: %d\n", ret); dev_err(&pdev->dev, "pwmchip_add failed: %d\n", ret);
goto err_suspend; goto err_suspend;
@ -346,13 +345,13 @@ static int img_pwm_probe(struct platform_device *pdev)
static int img_pwm_remove(struct platform_device *pdev) static int img_pwm_remove(struct platform_device *pdev)
{ {
struct img_pwm_chip *pwm_chip = platform_get_drvdata(pdev); struct img_pwm_chip *imgchip = platform_get_drvdata(pdev);
pm_runtime_disable(&pdev->dev); pm_runtime_disable(&pdev->dev);
if (!pm_runtime_status_suspended(&pdev->dev)) if (!pm_runtime_status_suspended(&pdev->dev))
img_pwm_runtime_suspend(&pdev->dev); img_pwm_runtime_suspend(&pdev->dev);
pwmchip_remove(&pwm_chip->chip); pwmchip_remove(&imgchip->chip);
return 0; return 0;
} }
@ -360,7 +359,7 @@ static int img_pwm_remove(struct platform_device *pdev)
#ifdef CONFIG_PM_SLEEP #ifdef CONFIG_PM_SLEEP
static int img_pwm_suspend(struct device *dev) static int img_pwm_suspend(struct device *dev)
{ {
struct img_pwm_chip *pwm_chip = dev_get_drvdata(dev); struct img_pwm_chip *imgchip = dev_get_drvdata(dev);
int i, ret; int i, ret;
if (pm_runtime_status_suspended(dev)) { if (pm_runtime_status_suspended(dev)) {
@ -369,11 +368,11 @@ static int img_pwm_suspend(struct device *dev)
return ret; return ret;
} }
for (i = 0; i < pwm_chip->chip.npwm; i++) for (i = 0; i < imgchip->chip.npwm; i++)
pwm_chip->suspend_ch_cfg[i] = img_pwm_readl(pwm_chip, imgchip->suspend_ch_cfg[i] = img_pwm_readl(imgchip,
PWM_CH_CFG(i)); PWM_CH_CFG(i));
pwm_chip->suspend_ctrl_cfg = img_pwm_readl(pwm_chip, PWM_CTRL_CFG); imgchip->suspend_ctrl_cfg = img_pwm_readl(imgchip, PWM_CTRL_CFG);
img_pwm_runtime_suspend(dev); img_pwm_runtime_suspend(dev);
@ -382,7 +381,7 @@ static int img_pwm_suspend(struct device *dev)
static int img_pwm_resume(struct device *dev) static int img_pwm_resume(struct device *dev)
{ {
struct img_pwm_chip *pwm_chip = dev_get_drvdata(dev); struct img_pwm_chip *imgchip = dev_get_drvdata(dev);
int ret; int ret;
int i; int i;
@ -390,15 +389,15 @@ static int img_pwm_resume(struct device *dev)
if (ret) if (ret)
return ret; return ret;
for (i = 0; i < pwm_chip->chip.npwm; i++) for (i = 0; i < imgchip->chip.npwm; i++)
img_pwm_writel(pwm_chip, PWM_CH_CFG(i), img_pwm_writel(imgchip, PWM_CH_CFG(i),
pwm_chip->suspend_ch_cfg[i]); imgchip->suspend_ch_cfg[i]);
img_pwm_writel(pwm_chip, PWM_CTRL_CFG, pwm_chip->suspend_ctrl_cfg); img_pwm_writel(imgchip, PWM_CTRL_CFG, imgchip->suspend_ctrl_cfg);
for (i = 0; i < pwm_chip->chip.npwm; i++) for (i = 0; i < imgchip->chip.npwm; i++)
if (pwm_chip->suspend_ctrl_cfg & BIT(i)) if (imgchip->suspend_ctrl_cfg & BIT(i))
regmap_update_bits(pwm_chip->periph_regs, regmap_update_bits(imgchip->periph_regs,
PERIP_PWM_PDM_CONTROL, PERIP_PWM_PDM_CONTROL,
PERIP_PWM_PDM_CONTROL_CH_MASK << PERIP_PWM_PDM_CONTROL_CH_MASK <<
PERIP_PWM_PDM_CONTROL_CH_SHIFT(i), PERIP_PWM_PDM_CONTROL_CH_SHIFT(i),