2006-08-11 07:50:51

by Johannes Berg

Subject: [PATCH] make leds.h include relevant headers

Here's something I noticed recently when writing a small led trigger...
Richard, does this look OK to you? If so, can you send it on? I don't
know through what path these things should go in.
Hopefully thunderbird doesn't mangle it...

---

This patch makes it possible to include linux/leds.h without first
including list.h and spinlock.h.


Signed-off-by: Johannes Berg <[email protected]>

--- wireless-dev.orig/include/linux/leds.h 2006-08-10 19:59:13.419652863 +0200
+++ wireless-dev/include/linux/leds.h 2006-08-10 20:02:14.979652863 +0200
@@ -12,6 +12,9 @@
#ifndef __LINUX_LEDS_H_INCLUDED
#define __LINUX_LEDS_H_INCLUDED

+#include <linux/list.h>
+#include <linux/spinlock.h>
+
struct device;
struct class_device;
/*

2006-08-11 08:32:37

by Richard Purdie

Subject: Re: [PATCH] make leds.h include relevant headers

On Fri, 2006-08-11 at 09:50 +0200, Johannes Berg wrote:
> This patch makes it possible to include linux/leds.h without first
> including list.h and spinlock.h.
>
>
> Signed-off-by: Johannes Berg <[email protected]>
Acked-by: Richard Purdie <[email protected]>

> --- wireless-dev.orig/include/linux/leds.h 2006-08-10 19:59:13.419652863 +0200
> +++ wireless-dev/include/linux/leds.h 2006-08-10 20:02:14.979652863 +0200
> @@ -12,6 +12,9 @@
> #ifndef __LINUX_LEDS_H_INCLUDED
> #define __LINUX_LEDS_H_INCLUDED
>
> +#include <linux/list.h>
> +#include <linux/spinlock.h>
> +
> struct device;
> struct class_device;
> /*
>
>