| From | andy.shevchenko@gmail ... | Date | Sat, 3 Jun 2023 23:12:06 +0300 | Subject | Re: [PATCH v1 16/43] pwm: ep93xx: add DT support for Cirrus EP93xx |
| |
Thu, Jun 01, 2023 at 08:34:07AM +0300, Nikita Shubin kirjoitti: > - find register range from the device tree
> - provide clock access via of
Not sure what does this supose to mean. Copy'n'paste mistake?
Please, revisit commit messages in *all* your patches. With this addressed, for this patch Reviewed-by: Andy Shevchenko <andy.shevchenko@gmail.com>
> Signed-off-by: Nikita Shubin <nikita.shubin@maquefel.me> > --- > > Notes: > v0 -> v1: > > - fixed headers > - fixed id table > > drivers/pwm/pwm-ep93xx.c | 8 ++++++++ > 1 file changed, 8 insertions(+) > > diff --git a/drivers/pwm/pwm-ep93xx.c b/drivers/pwm/pwm-ep93xx.c > index c45a75e65c86..361984ef4c0b 100644 > --- a/drivers/pwm/pwm-ep93xx.c > +++ b/drivers/pwm/pwm-ep93xx.c > @@ -17,6 +17,7 @@ > */ > > #include <linux/module.h> > +#include <linux/mod_devicetable.h> > #include <linux/platform_device.h> > #include <linux/slab.h> > #include <linux/clk.h> > @@ -190,9 +191,16 @@ static int ep93xx_pwm_probe(struct platform_device *pdev) > return 0; > } > > +static const struct of_device_id ep93xx_pwm_of_ids[] = { > + { .compatible = "cirrus,ep9301-pwm" }, > + { /* sentinel */} > +}; > +MODULE_DEVICE_TABLE(of, ep93xx_pwm_of_ids); > + > static struct platform_driver ep93xx_pwm_driver = { > .driver = { > .name = "ep93xx-pwm", > + .of_match_table = ep93xx_pwm_of_ids, > }, > .probe = ep93xx_pwm_probe, > }; > -- > 2.37.4 >
-- With Best Regards, Andy Shevchenko
|