diff mbox series

[v1,1/1] ASoC: madera: Replace kernel.h with the necessary inclusions

Message ID 20211222164534.67636-1-andriy.shevchenko@linux.intel.com (mailing list archive)
State New, archived
Headers show
Series [v1,1/1] ASoC: madera: Replace kernel.h with the necessary inclusions | expand

Commit Message

Andy Shevchenko Dec. 22, 2021, 4:45 p.m. UTC
When kernel.h is used in the headers it adds a lot into dependency hell,
especially when there are circular dependencies are involved.

Replace kernel.h inclusion with the list of what is really being used.

Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
---
 include/sound/madera-pdata.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Charles Keepax Dec. 29, 2021, 9:53 a.m. UTC | #1
On Wed, Dec 22, 2021 at 06:45:34PM +0200, Andy Shevchenko wrote:
> When kernel.h is used in the headers it adds a lot into dependency hell,
> especially when there are circular dependencies are involved.
> 
> Replace kernel.h inclusion with the list of what is really being used.
> 
> Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
> ---

Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>

Thanks,
Charles
Andy Shevchenko Jan. 24, 2022, 4:22 p.m. UTC | #2
On Wed, Dec 29, 2021 at 09:53:45AM +0000, Charles Keepax wrote:
> On Wed, Dec 22, 2021 at 06:45:34PM +0200, Andy Shevchenko wrote:
> > When kernel.h is used in the headers it adds a lot into dependency hell,
> > especially when there are circular dependencies are involved.
> > 
> > Replace kernel.h inclusion with the list of what is really being used.

> Acked-by: Charles Keepax <ckeepax@opensource.cirrus.com>

Thanks!

Takashi, should I resend this?
diff mbox series

Patch

diff --git a/include/sound/madera-pdata.h b/include/sound/madera-pdata.h
index e3060f48f108..58398d80c3de 100644
--- a/include/sound/madera-pdata.h
+++ b/include/sound/madera-pdata.h
@@ -9,7 +9,7 @@ 
 #ifndef MADERA_CODEC_PDATA_H
 #define MADERA_CODEC_PDATA_H
 
-#include <linux/kernel.h>
+#include <linux/types.h>
 
 #define MADERA_MAX_INPUT		6
 #define MADERA_MAX_MUXED_CHANNELS	4