| Message ID | 20170521220953.2602-3-wsa@the-dreams.de (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
On Monday, May 22, 2017 12:09:53 AM Wolfram Sang wrote: > include/linux/i2c is not for client devices. Move the header file to a > more appropriate location. > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de> Acked-by: Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com> > --- > arch/blackfin/mach-bf537/boards/stamp.c | 2 +- > drivers/video/backlight/adp8870_bl.c | 2 +- > include/linux/{i2c => platform_data}/adp8870.h | 0 > 3 files changed, 2 insertions(+), 2 deletions(-) > rename include/linux/{i2c => platform_data}/adp8870.h (100%) Best regards, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On 21/05/17 23:09, Wolfram Sang wrote: > include/linux/i2c is not for client devices. Move the header file to a > more appropriate location. > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de> Acked-by: Daniel Thompson <daniel.thompson@linaro.org> > --- > arch/blackfin/mach-bf537/boards/stamp.c | 2 +- > drivers/video/backlight/adp8870_bl.c | 2 +- > include/linux/{i2c => platform_data}/adp8870.h | 0 > 3 files changed, 2 insertions(+), 2 deletions(-) > rename include/linux/{i2c => platform_data}/adp8870.h (100%) > > diff --git a/arch/blackfin/mach-bf537/boards/stamp.c b/arch/blackfin/mach-bf537/boards/stamp.c > index 7db90c72bd8ddc..7528148dc4925b 100644 > --- a/arch/blackfin/mach-bf537/boards/stamp.c > +++ b/arch/blackfin/mach-bf537/boards/stamp.c > @@ -1995,7 +1995,7 @@ static struct adp5588_gpio_platform_data adp5588_gpio_data = { > #endif > > #if IS_ENABLED(CONFIG_BACKLIGHT_ADP8870) > -#include <linux/i2c/adp8870.h> > +#include <linux/platform_data/adp8870.h> > static struct led_info adp8870_leds[] = { > { > .name = "adp8870-led7", > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c > index 21acac90fd773f..058d1def2d1f4b 100644 > --- a/drivers/video/backlight/adp8870_bl.c > +++ b/drivers/video/backlight/adp8870_bl.c > @@ -18,7 +18,7 @@ > #include <linux/workqueue.h> > #include <linux/slab.h> > > -#include <linux/i2c/adp8870.h> > +#include <linux/platform_data/adp8870.h> > #define ADP8870_EXT_FEATURES > #define ADP8870_USE_LEDS > > diff --git a/include/linux/i2c/adp8870.h b/include/linux/platform_data/adp8870.h > similarity index 100% > rename from include/linux/i2c/adp8870.h > rename to include/linux/platform_data/adp8870.h > -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Mon, 22 May 2017, Wolfram Sang wrote: > include/linux/i2c is not for client devices. Move the header file to a > more appropriate location. > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de> > --- > arch/blackfin/mach-bf537/boards/stamp.c | 2 +- > drivers/video/backlight/adp8870_bl.c | 2 +- > include/linux/{i2c => platform_data}/adp8870.h | 0 > 3 files changed, 2 insertions(+), 2 deletions(-) > rename include/linux/{i2c => platform_data}/adp8870.h (100%) Applied, thanks. > diff --git a/arch/blackfin/mach-bf537/boards/stamp.c b/arch/blackfin/mach-bf537/boards/stamp.c > index 7db90c72bd8ddc..7528148dc4925b 100644 > --- a/arch/blackfin/mach-bf537/boards/stamp.c > +++ b/arch/blackfin/mach-bf537/boards/stamp.c > @@ -1995,7 +1995,7 @@ static struct adp5588_gpio_platform_data adp5588_gpio_data = { > #endif > > #if IS_ENABLED(CONFIG_BACKLIGHT_ADP8870) > -#include <linux/i2c/adp8870.h> > +#include <linux/platform_data/adp8870.h> > static struct led_info adp8870_leds[] = { > { > .name = "adp8870-led7", > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c > index 21acac90fd773f..058d1def2d1f4b 100644 > --- a/drivers/video/backlight/adp8870_bl.c > +++ b/drivers/video/backlight/adp8870_bl.c > @@ -18,7 +18,7 @@ > #include <linux/workqueue.h> > #include <linux/slab.h> > > -#include <linux/i2c/adp8870.h> > +#include <linux/platform_data/adp8870.h> > #define ADP8870_EXT_FEATURES > #define ADP8870_USE_LEDS > > diff --git a/include/linux/i2c/adp8870.h b/include/linux/platform_data/adp8870.h > similarity index 100% > rename from include/linux/i2c/adp8870.h > rename to include/linux/platform_data/adp8870.h
On Fri, 09 Jun 2017, Lee Jones wrote: > On Mon, 22 May 2017, Wolfram Sang wrote: > > > include/linux/i2c is not for client devices. Move the header file to a > > more appropriate location. > > > > Signed-off-by: Wolfram Sang <wsa@the-dreams.de> > > --- > > arch/blackfin/mach-bf537/boards/stamp.c | 2 +- > > drivers/video/backlight/adp8870_bl.c | 2 +- > > include/linux/{i2c => platform_data}/adp8870.h | 0 > > 3 files changed, 2 insertions(+), 2 deletions(-) > > rename include/linux/{i2c => platform_data}/adp8870.h (100%) > > Applied, thanks. Scrap that. I can't actually apply these without Steven's Ack. > > diff --git a/arch/blackfin/mach-bf537/boards/stamp.c b/arch/blackfin/mach-bf537/boards/stamp.c > > index 7db90c72bd8ddc..7528148dc4925b 100644 > > --- a/arch/blackfin/mach-bf537/boards/stamp.c > > +++ b/arch/blackfin/mach-bf537/boards/stamp.c > > @@ -1995,7 +1995,7 @@ static struct adp5588_gpio_platform_data adp5588_gpio_data = { > > #endif > > > > #if IS_ENABLED(CONFIG_BACKLIGHT_ADP8870) > > -#include <linux/i2c/adp8870.h> > > +#include <linux/platform_data/adp8870.h> > > static struct led_info adp8870_leds[] = { > > { > > .name = "adp8870-led7", > > diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c > > index 21acac90fd773f..058d1def2d1f4b 100644 > > --- a/drivers/video/backlight/adp8870_bl.c > > +++ b/drivers/video/backlight/adp8870_bl.c > > @@ -18,7 +18,7 @@ > > #include <linux/workqueue.h> > > #include <linux/slab.h> > > > > -#include <linux/i2c/adp8870.h> > > +#include <linux/platform_data/adp8870.h> > > #define ADP8870_EXT_FEATURES > > #define ADP8870_USE_LEDS > > > > diff --git a/include/linux/i2c/adp8870.h b/include/linux/platform_data/adp8870.h > > similarity index 100% > > rename from include/linux/i2c/adp8870.h > > rename to include/linux/platform_data/adp8870.h >
diff --git a/arch/blackfin/mach-bf537/boards/stamp.c b/arch/blackfin/mach-bf537/boards/stamp.c index 7db90c72bd8ddc..7528148dc4925b 100644 --- a/arch/blackfin/mach-bf537/boards/stamp.c +++ b/arch/blackfin/mach-bf537/boards/stamp.c @@ -1995,7 +1995,7 @@ static struct adp5588_gpio_platform_data adp5588_gpio_data = { #endif #if IS_ENABLED(CONFIG_BACKLIGHT_ADP8870) -#include <linux/i2c/adp8870.h> +#include <linux/platform_data/adp8870.h> static struct led_info adp8870_leds[] = { { .name = "adp8870-led7", diff --git a/drivers/video/backlight/adp8870_bl.c b/drivers/video/backlight/adp8870_bl.c index 21acac90fd773f..058d1def2d1f4b 100644 --- a/drivers/video/backlight/adp8870_bl.c +++ b/drivers/video/backlight/adp8870_bl.c @@ -18,7 +18,7 @@ #include <linux/workqueue.h> #include <linux/slab.h> -#include <linux/i2c/adp8870.h> +#include <linux/platform_data/adp8870.h> #define ADP8870_EXT_FEATURES #define ADP8870_USE_LEDS diff --git a/include/linux/i2c/adp8870.h b/include/linux/platform_data/adp8870.h similarity index 100% rename from include/linux/i2c/adp8870.h rename to include/linux/platform_data/adp8870.h
include/linux/i2c is not for client devices. Move the header file to a more appropriate location. Signed-off-by: Wolfram Sang <wsa@the-dreams.de> --- arch/blackfin/mach-bf537/boards/stamp.c | 2 +- drivers/video/backlight/adp8870_bl.c | 2 +- include/linux/{i2c => platform_data}/adp8870.h | 0 3 files changed, 2 insertions(+), 2 deletions(-) rename include/linux/{i2c => platform_data}/adp8870.h (100%)