[PATCH] pwm: lpss: Only include <linux/pwm.h> where needed

Uwe Kleine-König posted 1 patch 11 months ago
drivers/pwm/pwm-lpss.c | 1 +
drivers/pwm/pwm-lpss.h | 1 -
2 files changed, 1 insertion(+), 1 deletion(-)
[PATCH] pwm: lpss: Only include <linux/pwm.h> where needed
Posted by Uwe Kleine-König 11 months ago
Among the three files that include pwm-lpss.h only pwm-lpss.c actually
needs <linux/pwm.h>. So move the #include statement from the former to
the latter.

Signed-off-by: Uwe Kleine-König <u.kleine-koenig@baylibre.com>
---
 drivers/pwm/pwm-lpss.c | 1 +
 drivers/pwm/pwm-lpss.h | 1 -
 2 files changed, 1 insertion(+), 1 deletion(-)

diff --git a/drivers/pwm/pwm-lpss.c b/drivers/pwm/pwm-lpss.c
index e3c72ed7fff1..c976ff1c8ed9 100644
--- a/drivers/pwm/pwm-lpss.c
+++ b/drivers/pwm/pwm-lpss.c
@@ -19,6 +19,7 @@
 #include <linux/kernel.h>
 #include <linux/module.h>
 #include <linux/pm_runtime.h>
+#include <linux/pwm.h>
 #include <linux/time.h>
 
 #include "pwm-lpss.h"
diff --git a/drivers/pwm/pwm-lpss.h b/drivers/pwm/pwm-lpss.h
index b5267ab5193b..60792181401e 100644
--- a/drivers/pwm/pwm-lpss.h
+++ b/drivers/pwm/pwm-lpss.h
@@ -10,7 +10,6 @@
 #ifndef __PWM_LPSS_H
 #define __PWM_LPSS_H
 
-#include <linux/pwm.h>
 #include <linux/types.h>
 
 #include <linux/platform_data/x86/pwm-lpss.h>

base-commit: 232f121837ad8b1c21cc80f2c8842a4090c5a2a0
-- 
2.47.1

Re: [PATCH] pwm: lpss: Only include <linux/pwm.h> where needed
Posted by Andy Shevchenko 11 months ago
On Thu, Jan 23, 2025 at 12:39 PM Uwe Kleine-König
<u.kleine-koenig@baylibre.com> wrote:
>
> Among the three files that include pwm-lpss.h only pwm-lpss.c actually
> needs <linux/pwm.h>. So move the #include statement from the former to
> the latter.

Reviewed-by: Andy Shevchenko <andy@kernel.org>

-- 
With Best Regards,
Andy Shevchenko
Re: [PATCH] pwm: lpss: Only include <linux/pwm.h> where needed
Posted by Uwe Kleine-König 11 months ago
Hello Andy,

On Thu, Jan 23, 2025 at 01:31:03PM +0200, Andy Shevchenko wrote:
> On Thu, Jan 23, 2025 at 12:39 PM Uwe Kleine-König
> <u.kleine-koenig@baylibre.com> wrote:
> >
> > Among the three files that include pwm-lpss.h only pwm-lpss.c actually
> > needs <linux/pwm.h>. So move the #include statement from the former to
> > the latter.
> 
> Reviewed-by: Andy Shevchenko <andy@kernel.org>

Thanks! Applied to

	https://git.kernel.org/pub/scm/linux/kernel/git/ukleinek/linux.git pwm/for-nexxt

where it will propagate to pwm/for-next after the merge window.

Best regards
Uwe