| Message ID | 20190805125848.15751-2-fparent@baylibre.com |
|---|---|
| State | Accepted |
| Headers | show |
| Series | [1/2] dt-bindings: pwm: pwm-mediatek: Add documentation for MT8516 | expand |
On 05/08/2019 14:58, Fabien Parent wrote: > Add the compatible and the platform data to support PWM on the MT8516 > SoC. > > Signed-off-by: Fabien Parent <fparent@baylibre.com> Reviewed-by: Matthias Brugger <matthias.bgg@gmail.com> > --- > drivers/pwm/pwm-mediatek.c | 7 +++++++ > 1 file changed, 7 insertions(+) > > diff --git a/drivers/pwm/pwm-mediatek.c b/drivers/pwm/pwm-mediatek.c > index eb6674ce995f..6697e30811e7 100644 > --- a/drivers/pwm/pwm-mediatek.c > +++ b/drivers/pwm/pwm-mediatek.c > @@ -302,11 +302,18 @@ static const struct mtk_pwm_platform_data mt7628_pwm_data = { > .has_clks = false, > }; > > +static const struct mtk_pwm_platform_data mt8516_pwm_data = { > + .num_pwms = 5, > + .pwm45_fixup = false, > + .has_clks = true, > +}; > + > static const struct of_device_id mtk_pwm_of_match[] = { > { .compatible = "mediatek,mt2712-pwm", .data = &mt2712_pwm_data }, > { .compatible = "mediatek,mt7622-pwm", .data = &mt7622_pwm_data }, > { .compatible = "mediatek,mt7623-pwm", .data = &mt7623_pwm_data }, > { .compatible = "mediatek,mt7628-pwm", .data = &mt7628_pwm_data }, > + { .compatible = "mediatek,mt8516-pwm", .data = &mt8516_pwm_data }, > { }, > }; > MODULE_DEVICE_TABLE(of, mtk_pwm_of_match); >
On Mon, Aug 05, 2019 at 02:58:48PM +0200, Fabien Parent wrote: > Add the compatible and the platform data to support PWM on the MT8516 > SoC. > > Signed-off-by: Fabien Parent <fparent@baylibre.com> Acked-by: Uwe Kleine-König <u.kleine-koenig@pengutronix.de> The change looks fine, there is however another series currently waiting for application for this driver that conflicts with this one (I think). Maybe it would be sensible to join your forces and produce a single series without conflicts? Best regards Uwe
diff --git a/drivers/pwm/pwm-mediatek.c b/drivers/pwm/pwm-mediatek.c index eb6674ce995f..6697e30811e7 100644 --- a/drivers/pwm/pwm-mediatek.c +++ b/drivers/pwm/pwm-mediatek.c @@ -302,11 +302,18 @@ static const struct mtk_pwm_platform_data mt7628_pwm_data = { .has_clks = false, }; +static const struct mtk_pwm_platform_data mt8516_pwm_data = { + .num_pwms = 5, + .pwm45_fixup = false, + .has_clks = true, +}; + static const struct of_device_id mtk_pwm_of_match[] = { { .compatible = "mediatek,mt2712-pwm", .data = &mt2712_pwm_data }, { .compatible = "mediatek,mt7622-pwm", .data = &mt7622_pwm_data }, { .compatible = "mediatek,mt7623-pwm", .data = &mt7623_pwm_data }, { .compatible = "mediatek,mt7628-pwm", .data = &mt7628_pwm_data }, + { .compatible = "mediatek,mt8516-pwm", .data = &mt8516_pwm_data }, { }, }; MODULE_DEVICE_TABLE(of, mtk_pwm_of_match);
Add the compatible and the platform data to support PWM on the MT8516 SoC. Signed-off-by: Fabien Parent <fparent@baylibre.com> --- drivers/pwm/pwm-mediatek.c | 7 +++++++ 1 file changed, 7 insertions(+)