lkml.org 
[lkml]   [2023]   [Apr]   [24]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
SubjectRe: [PATCH v3 2/2] pwm: mediatek: Add support for MT7981
From


On 21/04/2023 01:23, Daniel Golle wrote:
> The PWM unit on MT7981 uses different register offsets than previous
> MediaTek PWM units. Add support for these new offsets and add support
> for PWM on MT7981 which has 3 PWM channels, one of them is typically
> used for a temperature controlled fan.
> While at it, also reorder pwm_mediatek_of_data entries to restore
> alphabetic order.
>
> Signed-off-by: Daniel Golle <daniel@makrotopia.org>
> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>

Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com>

> ---
> drivers/pwm/pwm-mediatek.c | 39 ++++++++++++++++++++++++++++++--------
> 1 file changed, 31 insertions(+), 8 deletions(-)
>
> diff --git a/drivers/pwm/pwm-mediatek.c b/drivers/pwm/pwm-mediatek.c
> index 5b5eeaff35da6..7a51d210a8778 100644
> --- a/drivers/pwm/pwm-mediatek.c
> +++ b/drivers/pwm/pwm-mediatek.c
> @@ -38,6 +38,7 @@ struct pwm_mediatek_of_data {
> unsigned int num_pwms;
> bool pwm45_fixup;
> bool has_ck_26m_sel;
> + const unsigned int *reg_offset;
> };
>
> /**
> @@ -59,10 +60,14 @@ struct pwm_mediatek_chip {
> const struct pwm_mediatek_of_data *soc;
> };
>
> -static const unsigned int pwm_mediatek_reg_offset[] = {
> +static const unsigned int mtk_pwm_reg_offset_v1[] = {
> 0x0010, 0x0050, 0x0090, 0x00d0, 0x0110, 0x0150, 0x0190, 0x0220
> };
>
> +static const unsigned int mtk_pwm_reg_offset_v2[] = {
> + 0x0080, 0x00c0, 0x0100, 0x0140, 0x0180, 0x01c0, 0x0200, 0x0240
> +};
> +
> static inline struct pwm_mediatek_chip *
> to_pwm_mediatek_chip(struct pwm_chip *chip)
> {
> @@ -111,7 +116,7 @@ static inline void pwm_mediatek_writel(struct pwm_mediatek_chip *chip,
> unsigned int num, unsigned int offset,
> u32 value)
> {
> - writel(value, chip->regs + pwm_mediatek_reg_offset[num] + offset);
> + writel(value, chip->regs + chip->soc->reg_offset[num] + offset);
> }
>
> static int pwm_mediatek_config(struct pwm_chip *chip, struct pwm_device *pwm,
> @@ -285,60 +290,77 @@ static const struct pwm_mediatek_of_data mt2712_pwm_data = {
> .num_pwms = 8,
> .pwm45_fixup = false,
> .has_ck_26m_sel = false,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt6795_pwm_data = {
> .num_pwms = 7,
> .pwm45_fixup = false,
> .has_ck_26m_sel = false,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt7622_pwm_data = {
> .num_pwms = 6,
> .pwm45_fixup = false,
> .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt7623_pwm_data = {
> .num_pwms = 5,
> .pwm45_fixup = true,
> .has_ck_26m_sel = false,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt7628_pwm_data = {
> .num_pwms = 4,
> .pwm45_fixup = true,
> .has_ck_26m_sel = false,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt7629_pwm_data = {
> .num_pwms = 1,
> .pwm45_fixup = false,
> .has_ck_26m_sel = false,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> -static const struct pwm_mediatek_of_data mt8183_pwm_data = {
> - .num_pwms = 4,
> +static const struct pwm_mediatek_of_data mt7981_pwm_data = {
> + .num_pwms = 3,
> .pwm45_fixup = false,
> .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v2,
> };
>
> -static const struct pwm_mediatek_of_data mt8365_pwm_data = {
> - .num_pwms = 3,
> +static const struct pwm_mediatek_of_data mt7986_pwm_data = {
> + .num_pwms = 2,
> .pwm45_fixup = false,
> .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> -static const struct pwm_mediatek_of_data mt7986_pwm_data = {
> - .num_pwms = 2,
> +static const struct pwm_mediatek_of_data mt8183_pwm_data = {
> + .num_pwms = 4,
> + .pwm45_fixup = false,
> + .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> +};
> +
> +static const struct pwm_mediatek_of_data mt8365_pwm_data = {
> + .num_pwms = 3,
> .pwm45_fixup = false,
> .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct pwm_mediatek_of_data mt8516_pwm_data = {
> .num_pwms = 5,
> .pwm45_fixup = false,
> .has_ck_26m_sel = true,
> + .reg_offset = mtk_pwm_reg_offset_v1,
> };
>
> static const struct of_device_id pwm_mediatek_of_match[] = {
> @@ -348,6 +370,7 @@ static const struct of_device_id pwm_mediatek_of_match[] = {
> { .compatible = "mediatek,mt7623-pwm", .data = &mt7623_pwm_data },
> { .compatible = "mediatek,mt7628-pwm", .data = &mt7628_pwm_data },
> { .compatible = "mediatek,mt7629-pwm", .data = &mt7629_pwm_data },
> + { .compatible = "mediatek,mt7981-pwm", .data = &mt7981_pwm_data },
> { .compatible = "mediatek,mt7986-pwm", .data = &mt7986_pwm_data },
> { .compatible = "mediatek,mt8183-pwm", .data = &mt8183_pwm_data },
> { .compatible = "mediatek,mt8365-pwm", .data = &mt8365_pwm_data },

\
 
 \ /
  Last update: 2023-04-24 14:30    [W:0.062 / U:0.372 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site