From nobody Sun Oct 5 01:46:19 2025 Received: from mail-wm1-f53.google.com (mail-wm1-f53.google.com [209.85.128.53]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D5080226CF9 for ; Mon, 11 Aug 2025 22:30:58 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.128.53 ARC-Seal: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754951460; cv=none; b=do3eieMztS2ZiQ1D78/209gL4I11Sk5qeGV6eRfUUL/6P03M2jNF7onIQA8WmGZ5wnunS+nh4JbhljkmY8wO29z7A9byWyHCskelAHS0VdJqcRLfOiI5GxKJsqfUakIhjY1sYtC7CPbElPQIswYE5aOoV2r/VL7TgJCVBC/0QI4= ARC-Message-Signature: i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1754951460; c=relaxed/simple; bh=QhsYH0yhdInsawGFOOU7wcTCxTp2DjRCkw5oJ+fqeTM=; h=From:To:Cc:Subject:Date:Message-ID:In-Reply-To:References: MIME-Version; b=R4e+RUq+UGAzfMEG85fMUyYlwOlwQlxuzc0e9jE22BOGQSkQ7QUag81cbtBWV34rRzWzpphKykjdCxn74ZHF/ISB/wasg80NHxlNhegA/k7ZddbUIc22HsZa7D7uusXuw3JF+C+vm5lHnLrDO6YCI23MhdHkuoC/sntxq+QJlr4= ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org; spf=pass smtp.mailfrom=linaro.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b=M3x5+Mbx; arc=none smtp.client-ip=209.85.128.53 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=linaro.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=linaro.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=linaro.org header.i=@linaro.org header.b="M3x5+Mbx" Received: by mail-wm1-f53.google.com with SMTP id 5b1f17b1804b1-459e7ea3ebeso18376765e9.0 for ; Mon, 11 Aug 2025 15:30:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; t=1754951457; x=1755556257; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to; bh=ffT6YWZkdUEe6FghMWArz7x5VLbOw7EcrzjaESrmOVU=; b=M3x5+MbxIAZ1hYugfjac1JsqdVvTHxfYNIdfX0Gn023YlKzEMxAdRwyiMmnHYV3GlZ 802X4DBCxRqBdL1+r0POGO8rPgF2tqc7qun2TCi34mmsAcwrI/n9uDWED3sK6+BzVUyX bcc8jQGUYOKXmECXE6K5BAbOx1BpmGDsyQGi2omJjNN81C6RUbsETVydEM1XKpRt43H+ X5cqbTBsNAyrsGCVMi8449A4UkFE8S06O8f5FOET49LeRVWH0XsxUVZlazF1MT82YrVH Pg4eDeUURk3In4VyvLJnHS3BkhgOx0LYqO6eAfw21Y3z1qDBSgtMW6wV56b81il8TTWu W1AQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1754951457; x=1755556257; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-message-state:from:to:cc :subject:date:message-id:reply-to; bh=ffT6YWZkdUEe6FghMWArz7x5VLbOw7EcrzjaESrmOVU=; b=TwNPAWbT/r7xlbU4ogqCqahJIYSmAfDfjTykWPkg0ThnREtQMZy6G5hvTyFFAck806 fW2TDfvsXAGLCH76Bhudfanz5BLGV+tgJMKUXbuEfiUDGnL0GPAb38wEYcMfS5mi/tFN jTmX32WPwREmGnUrGYmno/HPvbaTSSCsa7ydOZI7ipyHhkyqUCLGng23bkPbUMoi+xlZ YCcvy2PkUMRU5uDQMQSn+Q17RCLQpGKOqxSySYexFcyrdsshPBQVrqmBZTNUOUeGvdzy vXJ/cU22nAmjDUZHAauzLiRGMx8MlvAS8naZmJkYvkMOA4aSAyjzhy3mfhqTQkbObLxk 3ARA== X-Forwarded-Encrypted: i=1; AJvYcCUhKTTOHaVWSRSxDNAwBOyolR1r6zSqxVwjLprnym7+si1cHpoouV5Gu4pwNvOoQX01TfpRyzq8ijLRsFE=@vger.kernel.org X-Gm-Message-State: AOJu0YyDJxeC165p/Pp3RRKP5DE4RmZ424Kpt4vhXs3zai5aYk0t+x3A cW+HpvFBsFo+0gxAIEBqaY9uosBj7r/rYkhVHFgj+xVLWobNxlGgDh+b0lzcrX/enGM= X-Gm-Gg: ASbGncsv3r9dWYSgg6bV+JRAR+3wo67PqeypSUhn/GfHg4iCham2j4ih53otKX1bJV5 2LMc8uCtwXWz3EZcokEvhaAM8xNj0N78BPp1NNSKrX/c9ade1ZRX45nCXsMuHNvfkO7fGKlw7qH n1kE8gB9E+q/uv/woNWnq4VgX+/KbJw+iefGQuFSBdvYv+0AstKeTWW9137ll06GpXPROedC0q/ CMbNc0HO+tyxsSVEjxzlqJ4y/Gfxd/skoD5jKsTwZxElH0Ho6aNgaahFEGeq65kLwNPp9qfxSoh kZy/iOs71Vsu5/3M/j6EwdxmN/YGqo0v3rtSQw2sBC/d29ph3g9I26qnCTLAxHGPQqMJJWGHK0k 6UDyF5MrWPI3G2WcCRTLTAy1QaSInvzwQ/UScQ4ve/wrWXN54vYsmRvDXj/0= X-Google-Smtp-Source: AGHT+IHVXO4RAN9/7Z9SLjJsYn02aSZQYS+lU4qmFjKvqvXByVPkXWX5bCNH7Uajhq9MOI+lo9gT7Q== X-Received: by 2002:a05:6000:24c4:b0:3b7:940e:6529 with SMTP id ffacd0b85a97d-3b910fdb838mr872543f8f.10.1754951457121; Mon, 11 Aug 2025 15:30:57 -0700 (PDT) Received: from mai.. (146725694.box.freepro.com. [130.180.211.218]) by smtp.gmail.com with ESMTPSA id 5b1f17b1804b1-459e5853104sm271557715e9.8.2025.08.11.15.30.56 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Mon, 11 Aug 2025 15:30:56 -0700 (PDT) From: Daniel Lezcano To: ukleinek@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, Frank.Li@nxp.com Cc: linux-pwm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Ghennadi.Procopciuc@nxp.com, s32@nxp.com Subject: [PATCH v2 2/2] pwm: Add the S32G support in the Freescale FTM driver Date: Tue, 12 Aug 2025 00:30:40 +0200 Message-ID: <20250811223044.3087090-3-daniel.lezcano@linaro.org> X-Mailer: git-send-email 2.43.0 In-Reply-To: <20250811223044.3087090-1-daniel.lezcano@linaro.org> References: <20250811223044.3087090-1-daniel.lezcano@linaro.org> Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable Content-Type: text/plain; charset="utf-8" From: Ghennadi Procopciuc The Automotive S32G2 and S32G3 platforms include two FTM timers for pwm. Each FTM has 6 PWM channels. The current Freescale FTM driver supports the iMX8 and the Vybrid Family FTM IP. The FTM IP found on the S32G platforms is almost identical except for the number of channels and the register mapping. These changes allow to deal with different number of channels and support the holes found in the register memory mapping for s32gx for suspend / resume. The fault register does not exist on the s32gx and at resume time all the mapping is wrote back leading to a kernel crash. /* restore all registers from cache */ regcache_cache_only(fpc->regmap, false); regcache_sync(fpc->regmap); The regmap callbacks 'writeable_reg()' and 'readable_reg()' will skip the address corresponding to a register which is not present. Tested on a s32g274-rdb2 J5 PWM pin output with signal visualization on oscilloscope. Signed-off-by: Ghennadi Procopciuc Co-developed-by: Daniel Lezcano Signed-off-by: Daniel Lezcano --- drivers/pwm/pwm-fsl-ftm.c | 35 +++++++++++++++++++++++++++++++++-- 1 file changed, 33 insertions(+), 2 deletions(-) diff --git a/drivers/pwm/pwm-fsl-ftm.c b/drivers/pwm/pwm-fsl-ftm.c index c45a5fca4cbb..e0069dbdb02d 100644 --- a/drivers/pwm/pwm-fsl-ftm.c +++ b/drivers/pwm/pwm-fsl-ftm.c @@ -3,6 +3,7 @@ * Freescale FlexTimer Module (FTM) PWM Driver * * Copyright 2012-2013 Freescale Semiconductor, Inc. + * Copyright 2020-2025 NXP */ =20 #include @@ -31,6 +32,8 @@ enum fsl_pwm_clk { =20 struct fsl_ftm_soc { bool has_enable_bits; + bool has_flt_reg; + unsigned int npwm; }; =20 struct fsl_pwm_periodcfg { @@ -386,6 +389,20 @@ static bool fsl_pwm_volatile_reg(struct device *dev, u= nsigned int reg) return false; } =20 +static bool fsl_pwm_is_reg(struct device *dev, unsigned int reg) +{ + struct pwm_chip *chip =3D dev_get_drvdata(dev); + struct fsl_pwm_chip *fpc =3D to_fsl_chip(chip); + + if (reg >=3D FTM_CSC(fpc->soc->npwm) && reg < FTM_CNTIN) + return false; + + if ((reg =3D=3D FTM_FLTCTRL || reg =3D=3D FTM_FLTPOL) && !fpc->soc->has_f= lt_reg) + return false; + + return true; +} + static const struct regmap_config fsl_pwm_regmap_config =3D { .reg_bits =3D 32, .reg_stride =3D 4, @@ -394,23 +411,26 @@ static const struct regmap_config fsl_pwm_regmap_conf= ig =3D { .max_register =3D FTM_PWMLOAD, .volatile_reg =3D fsl_pwm_volatile_reg, .cache_type =3D REGCACHE_FLAT, + .writeable_reg =3D fsl_pwm_is_reg, + .readable_reg =3D fsl_pwm_is_reg, }; =20 static int fsl_pwm_probe(struct platform_device *pdev) { + const struct fsl_ftm_soc *soc =3D of_device_get_match_data(&pdev->dev); struct pwm_chip *chip; struct fsl_pwm_chip *fpc; void __iomem *base; int ret; =20 - chip =3D devm_pwmchip_alloc(&pdev->dev, 8, sizeof(*fpc)); + chip =3D devm_pwmchip_alloc(&pdev->dev, soc->npwm, sizeof(*fpc)); if (IS_ERR(chip)) return PTR_ERR(chip); fpc =3D to_fsl_chip(chip); =20 mutex_init(&fpc->lock); =20 - fpc->soc =3D of_device_get_match_data(&pdev->dev); + fpc->soc =3D soc; =20 base =3D devm_platform_ioremap_resource(pdev, 0); if (IS_ERR(base)) @@ -526,15 +546,26 @@ static const struct dev_pm_ops fsl_pwm_pm_ops =3D { =20 static const struct fsl_ftm_soc vf610_ftm_pwm =3D { .has_enable_bits =3D false, + .has_flt_reg =3D true, + .npwm =3D 8, }; =20 static const struct fsl_ftm_soc imx8qm_ftm_pwm =3D { .has_enable_bits =3D true, + .has_flt_reg =3D true, + .npwm =3D 8, +}; + +static const struct fsl_ftm_soc s32g2_ftm_pwm =3D { + .has_enable_bits =3D true, + .has_flt_reg =3D false, + .npwm =3D 6, }; =20 static const struct of_device_id fsl_pwm_dt_ids[] =3D { { .compatible =3D "fsl,vf610-ftm-pwm", .data =3D &vf610_ftm_pwm }, { .compatible =3D "fsl,imx8qm-ftm-pwm", .data =3D &imx8qm_ftm_pwm }, + { .compatible =3D "nxp,s32g2-ftm-pwm", .data =3D &s32g2_ftm_pwm }, { /* sentinel */ } }; MODULE_DEVICE_TABLE(of, fsl_pwm_dt_ids); --=20 2.43.0