Re: [RFC] [PATCH v3 1/2] introduce ALS sysfs class

From: Zhang Rui
Date: Mon Aug 24 2009 - 23:38:24 EST


On Fri, 2009-08-21 at 23:08 +0800, Greg KH wrote:
> On Fri, Aug 21, 2009 at 09:54:07AM +0800, Zhang Rui wrote:
> > ---
> > Documentation/ABI/testing/sysfs-class-als | 47 +++++
> > drivers/Kconfig | 2
> > drivers/Makefile | 1
> > drivers/als/Kconfig | 10 +
> > drivers/als/Makefile | 5
> > drivers/als/als_sys.c | 256 ++++++++++++++++++++++++++++++
> > include/linux/als_sys.h | 57 ++++++
>
> Does this file really need to be in include/linux/ ?
>
> Or could you just put it into drivers/als/ and it would be fine?
>
I don't think so, because all the other native ALS drivers like ACPI ALS
driver should depend on this header file.

thanks,
rui

--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/