lkml.org 
[lkml]   [2016]   [Jun]   [10]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
Patch in this message
/
Date
From
SubjectRe: [PATCH v8 6/9] pwms: pwm-ti*: Get the clock from the PWMSS parent when using old bindings
On Tue, May 03, 2016 at 10:56:52AM -0500, Franklin S Cooper Jr wrote:
> When using the old eCAP and ePWM bindings for AM335x and AM437x the clock
> can be retrieved from the PWMSS parent. Newer bindings will insure that
> this clock is provided via device tree.
>
> Therefore, update this driver to support the newer and older bindings. In
> the case of the older binding being used give a warning.
>
> Signed-off-by: Franklin S Cooper Jr <fcooper@ti.com>
> ---
> drivers/pwm/pwm-tiecap.c | 7 +++++++
> drivers/pwm/pwm-tiehrpwm.c | 7 +++++++
> 2 files changed, 14 insertions(+)
>
> diff --git a/drivers/pwm/pwm-tiecap.c b/drivers/pwm/pwm-tiecap.c
> index 616af76..a0833c8 100644
> --- a/drivers/pwm/pwm-tiecap.c
> +++ b/drivers/pwm/pwm-tiecap.c
> @@ -195,6 +195,7 @@ static const struct pwm_ops ecap_pwm_ops = {
> };
>
> static const struct of_device_id ecap_of_match[] = {
> + { .compatible = "ti,am3352-ecap" },
> { .compatible = "ti,am33xx-ecap" },
> {},
> };
> @@ -213,6 +214,12 @@ static int ecap_pwm_probe(struct platform_device *pdev)
> return -ENOMEM;
>
> clk = devm_clk_get(&pdev->dev, "fck");
> +
> + if (IS_ERR(clk) && of_device_is_compatible(pdev->dev.of_node, "ti,am33xx-ecap")) {
> + dev_warn(&pdev->dev, "Binding is obsolete.\n");
> + clk = devm_clk_get(pdev->dev.parent, "fck");
> + }

I rewrote this slightly for readability:

struct device_node *np = pdev->dev.of_node;

if (IS_ERR(clk)) {
if (of_device_is_compatible(np, "ti,am33xx-ecap")) {
...
}
}

> if (IS_ERR(clk)) {
> dev_err(&pdev->dev, "failed to get clock\n");
> return PTR_ERR(clk);
> diff --git a/drivers/pwm/pwm-tiehrpwm.c b/drivers/pwm/pwm-tiehrpwm.c
> index 6a41e66..cee29a2 100644
> --- a/drivers/pwm/pwm-tiehrpwm.c
> +++ b/drivers/pwm/pwm-tiehrpwm.c
> @@ -426,6 +426,7 @@ static const struct pwm_ops ehrpwm_pwm_ops = {
> };
>
> static const struct of_device_id ehrpwm_of_match[] = {
> + { .compatible = "ti,am3352-ehrpwm" },
> { .compatible = "ti,am33xx-ehrpwm" },
> {},
> };
> @@ -444,6 +445,12 @@ static int ehrpwm_pwm_probe(struct platform_device *pdev)
> return -ENOMEM;
>
> clk = devm_clk_get(&pdev->dev, "fck");
> +
> + if (IS_ERR(clk) && of_device_is_compatible(pdev->dev.of_node, "ti,am33xx-ecap")) {
> + dev_warn(&pdev->dev, "Binding is obsolete.\n");
> + clk = devm_clk_get(pdev->dev.parent, "fck");
> + }
> +

Same here.

Thierry
[unhandled content-type:application/pgp-signature]
\
 
 \ /
  Last update: 2016-06-10 15:21    [W:0.132 / U:0.692 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site