Shouldn't this patch be merged into #5? Also, a binding doc should be added.
Regards, Shawn On Tue, Aug 28, 2012 at 01:48:24PM +0200, Sascha Hauer wrote: > From: Philipp Zabel <p.za...@pengutronix.de> > > Signed-off-by: Philipp Zabel <p.za...@pengutronix.de> > Signed-off-by: Sascha Hauer <s.ha...@pengutronix.de> > --- > drivers/pwm/pwm-imx.c | 25 ++++++++++++++++++++++--- > 1 file changed, 22 insertions(+), 3 deletions(-) > > diff --git a/drivers/pwm/pwm-imx.c b/drivers/pwm/pwm-imx.c > index 308ab51..dce2acb 100644 > --- a/drivers/pwm/pwm-imx.c > +++ b/drivers/pwm/pwm-imx.c > @@ -206,9 +206,25 @@ static struct pwm_ops imx_pwm_ops = { > .owner = THIS_MODULE, > }; > > +struct imx_pwm_data { > + int (*config)(struct pwm_chip *chip, > + struct pwm_device *pwm, int duty_ns, int period_ns); > + void (*set_enable)(struct pwm_chip *chip, bool enable); > +}; > + > +static struct imx_pwm_data imx_pwm_data_v1 = { > + .config = imx_pwm_config_v1, > + .set_enable = imx_pwm_set_enable_v1, > +}; > + > +static struct imx_pwm_data imx_pwm_data_v2 = { > + .config = imx_pwm_config_v2, > + .set_enable = imx_pwm_set_enable_v2, > +}; > + > static const struct of_device_id imx_pwm_dt_ids[] = { > - { .compatible = "fsl,imx1-pwm", .data = &imx_pwm_config_v1, }, > - { .compatible = "fsl,imx27-pwm", .data = &imx_pwm_config_v2, }, > + { .compatible = "fsl,imx1-pwm", .data = &imx_pwm_data_v1, }, > + { .compatible = "fsl,imx27-pwm", .data = &imx_pwm_data_v2, }, > { /* sentinel */ } > }; > MODULE_DEVICE_TABLE(of, imx_pwm_of_match); > @@ -257,7 +273,9 @@ static int __devinit imx_pwm_probe(struct platform_device > *pdev) > return -EADDRNOTAVAIL; > > if (of_id) { > - imx->config = of_id->data; > + struct imx_pwm_data *data = of_id->data; > + imx->config = data->config; > + imx->set_enable = data->set_enable; > } else { > if (cpu_is_mx1() || cpu_is_mx21()) { > imx->config = imx_pwm_config_v1; > @@ -290,6 +308,7 @@ static int __devexit imx_pwm_remove(struct > platform_device *pdev) > static struct platform_driver imx_pwm_driver = { > .driver = { > .name = "mxc_pwm", > + .of_match_table = of_match_ptr(imx_pwm_dt_ids), > }, > .probe = imx_pwm_probe, > .remove = __devexit_p(imx_pwm_remove), > -- > 1.7.10.4 > -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/