mirror of
https://github.com/torvalds/linux.git
synced 2024-11-13 23:51:39 +00:00
dd6c6d57ab
The following commitabf6569d64
("pwm: imx-tpm: Make use of devm_pwmchip_alloc() function") introduced an issue that accessing registers without clock which results in the following boot crash on MX7ULP platform. Fixed it by enabling clock properly. Unhandled fault: external abort on non-linefetch (0x1008) at 0xf0978004 [f0978004] *pgd=64009811, *pte=40250653, *ppte=40250453 Internal error: : 1008 [#1] SMP ARM Modules linked in: CPU: 0 PID: 1 Comm: swapper/0 Not tainted 6.8.0-rc6-next-20240301 #18 Hardware name: Freescale i.MX7ULP (Device Tree) PC is at pwm_imx_tpm_probe+0x1c/0xd8 LR is at __devm_ioremap_resource+0xf8/0x1dc pc : [<c0629e58>] lr : [<c0562d4c>] psr: 80000053 sp : f0825e10 ip : 00000000 fp : 00000000 r10: c148f8c0 r9 : c41fc338 r8 : c164b000 r7 : 00000000 r6 : c406b400 r5 : c406b410 r4 : f0978000 r3 : 00000005 r2 : 00000000 r1 : a0000053 r0 : f0978000 Flags: Nzcv IRQs on FIQs off Mode SVC_32 ISA ARM Segment none Control: 10c5387d Table: 6000406a DAC: 00000051 ... Call trace: pwm_imx_tpm_probe from platform_probe+0x58/0xb0 platform_probe from really_probe+0xc4/0x2e0 really_probe from __driver_probe_device+0x84/0x19c __driver_probe_device from driver_probe_device+0x2c/0x104 driver_probe_device from __driver_attach+0x90/0x170 __driver_attach from bus_for_each_dev+0x7c/0xd0 bus_for_each_dev from bus_add_driver+0xc4/0x1cc bus_add_driver from driver_register+0x7c/0x114 driver_register from do_one_initcall+0x58/0x270 do_one_initcall from kernel_init_freeable+0x170/0x218 kernel_init_freeable from kernel_init+0x14/0x140 kernel_init from ret_from_fork+0x14/0x20 Fixes:abf6569d64
("pwm: imx-tpm: Make use of devm_pwmchip_alloc() function") Signed-off-by: Dong Aisheng <aisheng.dong@nxp.com> Link: https://lore.kernel.org/r/20240304102929.893542-1-aisheng.dong@nxp.com Signed-off-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de>
433 lines
11 KiB
C
433 lines
11 KiB
C
// SPDX-License-Identifier: GPL-2.0
|
|
/*
|
|
* Copyright 2018-2019 NXP.
|
|
*
|
|
* Limitations:
|
|
* - The TPM counter and period counter are shared between
|
|
* multiple channels, so all channels should use same period
|
|
* settings.
|
|
* - Changes to polarity cannot be latched at the time of the
|
|
* next period start.
|
|
* - Changing period and duty cycle together isn't atomic,
|
|
* with the wrong timing it might happen that a period is
|
|
* produced with old duty cycle but new period settings.
|
|
*/
|
|
|
|
#include <linux/bitfield.h>
|
|
#include <linux/bitops.h>
|
|
#include <linux/clk.h>
|
|
#include <linux/err.h>
|
|
#include <linux/io.h>
|
|
#include <linux/module.h>
|
|
#include <linux/of.h>
|
|
#include <linux/platform_device.h>
|
|
#include <linux/pwm.h>
|
|
#include <linux/slab.h>
|
|
|
|
#define PWM_IMX_TPM_PARAM 0x4
|
|
#define PWM_IMX_TPM_GLOBAL 0x8
|
|
#define PWM_IMX_TPM_SC 0x10
|
|
#define PWM_IMX_TPM_CNT 0x14
|
|
#define PWM_IMX_TPM_MOD 0x18
|
|
#define PWM_IMX_TPM_CnSC(n) (0x20 + (n) * 0x8)
|
|
#define PWM_IMX_TPM_CnV(n) (0x24 + (n) * 0x8)
|
|
|
|
#define PWM_IMX_TPM_PARAM_CHAN GENMASK(7, 0)
|
|
|
|
#define PWM_IMX_TPM_SC_PS GENMASK(2, 0)
|
|
#define PWM_IMX_TPM_SC_CMOD GENMASK(4, 3)
|
|
#define PWM_IMX_TPM_SC_CMOD_INC_EVERY_CLK FIELD_PREP(PWM_IMX_TPM_SC_CMOD, 1)
|
|
#define PWM_IMX_TPM_SC_CPWMS BIT(5)
|
|
|
|
#define PWM_IMX_TPM_CnSC_CHF BIT(7)
|
|
#define PWM_IMX_TPM_CnSC_MSB BIT(5)
|
|
#define PWM_IMX_TPM_CnSC_MSA BIT(4)
|
|
|
|
/*
|
|
* The reference manual describes this field as two separate bits. The
|
|
* semantic of the two bits isn't orthogonal though, so they are treated
|
|
* together as a 2-bit field here.
|
|
*/
|
|
#define PWM_IMX_TPM_CnSC_ELS GENMASK(3, 2)
|
|
#define PWM_IMX_TPM_CnSC_ELS_INVERSED FIELD_PREP(PWM_IMX_TPM_CnSC_ELS, 1)
|
|
#define PWM_IMX_TPM_CnSC_ELS_NORMAL FIELD_PREP(PWM_IMX_TPM_CnSC_ELS, 2)
|
|
|
|
|
|
#define PWM_IMX_TPM_MOD_WIDTH 16
|
|
#define PWM_IMX_TPM_MOD_MOD GENMASK(PWM_IMX_TPM_MOD_WIDTH - 1, 0)
|
|
|
|
struct imx_tpm_pwm_chip {
|
|
struct clk *clk;
|
|
void __iomem *base;
|
|
struct mutex lock;
|
|
u32 user_count;
|
|
u32 enable_count;
|
|
u32 real_period;
|
|
};
|
|
|
|
struct imx_tpm_pwm_param {
|
|
u8 prescale;
|
|
u32 mod;
|
|
u32 val;
|
|
};
|
|
|
|
static inline struct imx_tpm_pwm_chip *
|
|
to_imx_tpm_pwm_chip(struct pwm_chip *chip)
|
|
{
|
|
return pwmchip_get_drvdata(chip);
|
|
}
|
|
|
|
/*
|
|
* This function determines for a given pwm_state *state that a consumer
|
|
* might request the pwm_state *real_state that eventually is implemented
|
|
* by the hardware and the necessary register values (in *p) to achieve
|
|
* this.
|
|
*/
|
|
static int pwm_imx_tpm_round_state(struct pwm_chip *chip,
|
|
struct imx_tpm_pwm_param *p,
|
|
struct pwm_state *real_state,
|
|
const struct pwm_state *state)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
u32 rate, prescale, period_count, clock_unit;
|
|
u64 tmp;
|
|
|
|
rate = clk_get_rate(tpm->clk);
|
|
tmp = (u64)state->period * rate;
|
|
clock_unit = DIV_ROUND_CLOSEST_ULL(tmp, NSEC_PER_SEC);
|
|
if (clock_unit <= PWM_IMX_TPM_MOD_MOD)
|
|
prescale = 0;
|
|
else
|
|
prescale = ilog2(clock_unit) + 1 - PWM_IMX_TPM_MOD_WIDTH;
|
|
|
|
if ((!FIELD_FIT(PWM_IMX_TPM_SC_PS, prescale)))
|
|
return -ERANGE;
|
|
p->prescale = prescale;
|
|
|
|
period_count = (clock_unit + ((1 << prescale) >> 1)) >> prescale;
|
|
p->mod = period_count;
|
|
|
|
/* calculate real period HW can support */
|
|
tmp = (u64)period_count << prescale;
|
|
tmp *= NSEC_PER_SEC;
|
|
real_state->period = DIV_ROUND_CLOSEST_ULL(tmp, rate);
|
|
|
|
/*
|
|
* if eventually the PWM output is inactive, either
|
|
* duty cycle is 0 or status is disabled, need to
|
|
* make sure the output pin is inactive.
|
|
*/
|
|
if (!state->enabled)
|
|
real_state->duty_cycle = 0;
|
|
else
|
|
real_state->duty_cycle = state->duty_cycle;
|
|
|
|
tmp = (u64)p->mod * real_state->duty_cycle;
|
|
p->val = DIV64_U64_ROUND_CLOSEST(tmp, real_state->period);
|
|
|
|
real_state->polarity = state->polarity;
|
|
real_state->enabled = state->enabled;
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int pwm_imx_tpm_get_state(struct pwm_chip *chip,
|
|
struct pwm_device *pwm,
|
|
struct pwm_state *state)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
u32 rate, val, prescale;
|
|
u64 tmp;
|
|
|
|
/* get period */
|
|
state->period = tpm->real_period;
|
|
|
|
/* get duty cycle */
|
|
rate = clk_get_rate(tpm->clk);
|
|
val = readl(tpm->base + PWM_IMX_TPM_SC);
|
|
prescale = FIELD_GET(PWM_IMX_TPM_SC_PS, val);
|
|
tmp = readl(tpm->base + PWM_IMX_TPM_CnV(pwm->hwpwm));
|
|
tmp = (tmp << prescale) * NSEC_PER_SEC;
|
|
state->duty_cycle = DIV_ROUND_CLOSEST_ULL(tmp, rate);
|
|
|
|
/* get polarity */
|
|
val = readl(tpm->base + PWM_IMX_TPM_CnSC(pwm->hwpwm));
|
|
if ((val & PWM_IMX_TPM_CnSC_ELS) == PWM_IMX_TPM_CnSC_ELS_INVERSED)
|
|
state->polarity = PWM_POLARITY_INVERSED;
|
|
else
|
|
/*
|
|
* Assume reserved values (2b00 and 2b11) to yield
|
|
* normal polarity.
|
|
*/
|
|
state->polarity = PWM_POLARITY_NORMAL;
|
|
|
|
/* get channel status */
|
|
state->enabled = FIELD_GET(PWM_IMX_TPM_CnSC_ELS, val) ? true : false;
|
|
|
|
return 0;
|
|
}
|
|
|
|
/* this function is supposed to be called with mutex hold */
|
|
static int pwm_imx_tpm_apply_hw(struct pwm_chip *chip,
|
|
struct imx_tpm_pwm_param *p,
|
|
struct pwm_state *state,
|
|
struct pwm_device *pwm)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
bool period_update = false;
|
|
bool duty_update = false;
|
|
u32 val, cmod, cur_prescale;
|
|
unsigned long timeout;
|
|
struct pwm_state c;
|
|
|
|
if (state->period != tpm->real_period) {
|
|
/*
|
|
* TPM counter is shared by multiple channels, so
|
|
* prescale and period can NOT be modified when
|
|
* there are multiple channels in use with different
|
|
* period settings.
|
|
*/
|
|
if (tpm->user_count > 1)
|
|
return -EBUSY;
|
|
|
|
val = readl(tpm->base + PWM_IMX_TPM_SC);
|
|
cmod = FIELD_GET(PWM_IMX_TPM_SC_CMOD, val);
|
|
cur_prescale = FIELD_GET(PWM_IMX_TPM_SC_PS, val);
|
|
if (cmod && cur_prescale != p->prescale)
|
|
return -EBUSY;
|
|
|
|
/* set TPM counter prescale */
|
|
val &= ~PWM_IMX_TPM_SC_PS;
|
|
val |= FIELD_PREP(PWM_IMX_TPM_SC_PS, p->prescale);
|
|
writel(val, tpm->base + PWM_IMX_TPM_SC);
|
|
|
|
/*
|
|
* set period count:
|
|
* if the PWM is disabled (CMOD[1:0] = 2b00), then MOD register
|
|
* is updated when MOD register is written.
|
|
*
|
|
* if the PWM is enabled (CMOD[1:0] ≠ 2b00), the period length
|
|
* is latched into hardware when the next period starts.
|
|
*/
|
|
writel(p->mod, tpm->base + PWM_IMX_TPM_MOD);
|
|
tpm->real_period = state->period;
|
|
period_update = true;
|
|
}
|
|
|
|
pwm_imx_tpm_get_state(chip, pwm, &c);
|
|
|
|
/* polarity is NOT allowed to be changed if PWM is active */
|
|
if (c.enabled && c.polarity != state->polarity)
|
|
return -EBUSY;
|
|
|
|
if (state->duty_cycle != c.duty_cycle) {
|
|
/*
|
|
* set channel value:
|
|
* if the PWM is disabled (CMOD[1:0] = 2b00), then CnV register
|
|
* is updated when CnV register is written.
|
|
*
|
|
* if the PWM is enabled (CMOD[1:0] ≠ 2b00), the duty length
|
|
* is latched into hardware when the next period starts.
|
|
*/
|
|
writel(p->val, tpm->base + PWM_IMX_TPM_CnV(pwm->hwpwm));
|
|
duty_update = true;
|
|
}
|
|
|
|
/* make sure MOD & CnV registers are updated */
|
|
if (period_update || duty_update) {
|
|
timeout = jiffies + msecs_to_jiffies(tpm->real_period /
|
|
NSEC_PER_MSEC + 1);
|
|
while (readl(tpm->base + PWM_IMX_TPM_MOD) != p->mod
|
|
|| readl(tpm->base + PWM_IMX_TPM_CnV(pwm->hwpwm))
|
|
!= p->val) {
|
|
if (time_after(jiffies, timeout))
|
|
return -ETIME;
|
|
cpu_relax();
|
|
}
|
|
}
|
|
|
|
/*
|
|
* polarity settings will enabled/disable output status
|
|
* immediately, so if the channel is disabled, need to
|
|
* make sure MSA/MSB/ELS are set to 0 which means channel
|
|
* disabled.
|
|
*/
|
|
val = readl(tpm->base + PWM_IMX_TPM_CnSC(pwm->hwpwm));
|
|
val &= ~(PWM_IMX_TPM_CnSC_ELS | PWM_IMX_TPM_CnSC_MSA |
|
|
PWM_IMX_TPM_CnSC_MSB);
|
|
if (state->enabled) {
|
|
/*
|
|
* set polarity (for edge-aligned PWM modes)
|
|
*
|
|
* ELS[1:0] = 2b10 yields normal polarity behaviour,
|
|
* ELS[1:0] = 2b01 yields inversed polarity.
|
|
* The other values are reserved.
|
|
*/
|
|
val |= PWM_IMX_TPM_CnSC_MSB;
|
|
val |= (state->polarity == PWM_POLARITY_NORMAL) ?
|
|
PWM_IMX_TPM_CnSC_ELS_NORMAL :
|
|
PWM_IMX_TPM_CnSC_ELS_INVERSED;
|
|
}
|
|
writel(val, tpm->base + PWM_IMX_TPM_CnSC(pwm->hwpwm));
|
|
|
|
/* control the counter status */
|
|
if (state->enabled != c.enabled) {
|
|
val = readl(tpm->base + PWM_IMX_TPM_SC);
|
|
if (state->enabled) {
|
|
if (++tpm->enable_count == 1)
|
|
val |= PWM_IMX_TPM_SC_CMOD_INC_EVERY_CLK;
|
|
} else {
|
|
if (--tpm->enable_count == 0)
|
|
val &= ~PWM_IMX_TPM_SC_CMOD;
|
|
}
|
|
writel(val, tpm->base + PWM_IMX_TPM_SC);
|
|
}
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int pwm_imx_tpm_apply(struct pwm_chip *chip,
|
|
struct pwm_device *pwm,
|
|
const struct pwm_state *state)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
struct imx_tpm_pwm_param param;
|
|
struct pwm_state real_state;
|
|
int ret;
|
|
|
|
ret = pwm_imx_tpm_round_state(chip, ¶m, &real_state, state);
|
|
if (ret)
|
|
return ret;
|
|
|
|
mutex_lock(&tpm->lock);
|
|
ret = pwm_imx_tpm_apply_hw(chip, ¶m, &real_state, pwm);
|
|
mutex_unlock(&tpm->lock);
|
|
|
|
return ret;
|
|
}
|
|
|
|
static int pwm_imx_tpm_request(struct pwm_chip *chip, struct pwm_device *pwm)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
|
|
mutex_lock(&tpm->lock);
|
|
tpm->user_count++;
|
|
mutex_unlock(&tpm->lock);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static void pwm_imx_tpm_free(struct pwm_chip *chip, struct pwm_device *pwm)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = to_imx_tpm_pwm_chip(chip);
|
|
|
|
mutex_lock(&tpm->lock);
|
|
tpm->user_count--;
|
|
mutex_unlock(&tpm->lock);
|
|
}
|
|
|
|
static const struct pwm_ops imx_tpm_pwm_ops = {
|
|
.request = pwm_imx_tpm_request,
|
|
.free = pwm_imx_tpm_free,
|
|
.get_state = pwm_imx_tpm_get_state,
|
|
.apply = pwm_imx_tpm_apply,
|
|
};
|
|
|
|
static int pwm_imx_tpm_probe(struct platform_device *pdev)
|
|
{
|
|
struct pwm_chip *chip;
|
|
struct imx_tpm_pwm_chip *tpm;
|
|
struct clk *clk;
|
|
void __iomem *base;
|
|
int ret;
|
|
unsigned int npwm;
|
|
u32 val;
|
|
|
|
base = devm_platform_ioremap_resource(pdev, 0);
|
|
if (IS_ERR(base))
|
|
return PTR_ERR(base);
|
|
|
|
clk = devm_clk_get_enabled(&pdev->dev, NULL);
|
|
if (IS_ERR(clk))
|
|
return dev_err_probe(&pdev->dev, PTR_ERR(clk),
|
|
"failed to get PWM clock\n");
|
|
|
|
/* get number of channels */
|
|
val = readl(base + PWM_IMX_TPM_PARAM);
|
|
npwm = FIELD_GET(PWM_IMX_TPM_PARAM_CHAN, val);
|
|
|
|
chip = devm_pwmchip_alloc(&pdev->dev, npwm, sizeof(*tpm));
|
|
if (IS_ERR(chip))
|
|
return PTR_ERR(chip);
|
|
tpm = to_imx_tpm_pwm_chip(chip);
|
|
|
|
platform_set_drvdata(pdev, tpm);
|
|
|
|
tpm->base = base;
|
|
tpm->clk = clk;
|
|
|
|
chip->ops = &imx_tpm_pwm_ops;
|
|
|
|
mutex_init(&tpm->lock);
|
|
|
|
ret = devm_pwmchip_add(&pdev->dev, chip);
|
|
if (ret)
|
|
return dev_err_probe(&pdev->dev, ret, "failed to add PWM chip\n");
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int pwm_imx_tpm_suspend(struct device *dev)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = dev_get_drvdata(dev);
|
|
|
|
if (tpm->enable_count > 0)
|
|
return -EBUSY;
|
|
|
|
/*
|
|
* Force 'real_period' to be zero to force period update code
|
|
* can be executed after system resume back, since suspend causes
|
|
* the period related registers to become their reset values.
|
|
*/
|
|
tpm->real_period = 0;
|
|
|
|
clk_disable_unprepare(tpm->clk);
|
|
|
|
return 0;
|
|
}
|
|
|
|
static int pwm_imx_tpm_resume(struct device *dev)
|
|
{
|
|
struct imx_tpm_pwm_chip *tpm = dev_get_drvdata(dev);
|
|
int ret = 0;
|
|
|
|
ret = clk_prepare_enable(tpm->clk);
|
|
if (ret)
|
|
dev_err(dev, "failed to prepare or enable clock: %d\n", ret);
|
|
|
|
return ret;
|
|
}
|
|
|
|
static DEFINE_SIMPLE_DEV_PM_OPS(imx_tpm_pwm_pm,
|
|
pwm_imx_tpm_suspend, pwm_imx_tpm_resume);
|
|
|
|
static const struct of_device_id imx_tpm_pwm_dt_ids[] = {
|
|
{ .compatible = "fsl,imx7ulp-pwm", },
|
|
{ /* sentinel */ }
|
|
};
|
|
MODULE_DEVICE_TABLE(of, imx_tpm_pwm_dt_ids);
|
|
|
|
static struct platform_driver imx_tpm_pwm_driver = {
|
|
.driver = {
|
|
.name = "imx7ulp-tpm-pwm",
|
|
.of_match_table = imx_tpm_pwm_dt_ids,
|
|
.pm = pm_ptr(&imx_tpm_pwm_pm),
|
|
},
|
|
.probe = pwm_imx_tpm_probe,
|
|
};
|
|
module_platform_driver(imx_tpm_pwm_driver);
|
|
|
|
MODULE_AUTHOR("Anson Huang <Anson.Huang@nxp.com>");
|
|
MODULE_DESCRIPTION("i.MX TPM PWM Driver");
|
|
MODULE_LICENSE("GPL v2");
|