* [PATCH] mfd: mc13xxx: Move definitions out of structures
@ 2014-03-15 7:08 Alexander Shiyan
2014-03-18 8:46 ` Lee Jones
2014-03-18 9:02 ` Lee Jones
0 siblings, 2 replies; 10+ messages in thread
From: Alexander Shiyan @ 2014-03-15 7:08 UTC (permalink / raw)
To: linux-kernel; +Cc: Samuel Ortiz, Lee Jones, Alexander Shiyan
Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
---
include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
1 file changed, 12 insertions(+), 10 deletions(-)
diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
index a326c85..d63b1d3 100644
--- a/include/linux/mfd/mc13xxx.h
+++ b/include/linux/mfd/mc13xxx.h
@@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
#define MAX_LED_CONTROL_REGS 6
-struct mc13xxx_leds_platform_data {
- struct mc13xxx_led_platform_data *led;
- int num_leds;
-
/* MC13783 LED Control 0 */
#define MC13783_LED_C0_ENABLE (1 << 0)
#define MC13783_LED_C0_TRIODE_MD (1 << 7)
@@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
/* MC34708 LED Control 0 */
#define MC34708_LED_C0_CURRENT_R(x) (((x) & 0x3) << 9)
#define MC34708_LED_C0_CURRENT_G(x) (((x) & 0x3) << 21)
+
+struct mc13xxx_leds_platform_data {
+ struct mc13xxx_led_platform_data *led;
+ int num_leds;
u32 led_control[MAX_LED_CONTROL_REGS];
};
-struct mc13xxx_buttons_platform_data {
#define MC13783_BUTTON_DBNC_0MS 0
#define MC13783_BUTTON_DBNC_30MS 1
#define MC13783_BUTTON_DBNC_150MS 2
@@ -180,6 +179,8 @@ struct mc13xxx_buttons_platform_data {
#define MC13783_BUTTON_ENABLE (1 << 2)
#define MC13783_BUTTON_POL_INVERT (1 << 3)
#define MC13783_BUTTON_RESET_EN (1 << 4)
+
+struct mc13xxx_buttons_platform_data {
int b1on_flags;
unsigned short b1on_key;
int b2on_flags;
@@ -188,14 +189,14 @@ struct mc13xxx_buttons_platform_data {
unsigned short b3on_key;
};
+#define MC13783_TS_ATO_FIRST false
+#define MC13783_TS_ATO_EACH true
+
struct mc13xxx_ts_platform_data {
/* Delay between Touchscreen polarization and ADC Conversion.
* Given in clock ticks of a 32 kHz clock which gives a granularity of
* about 30.5ms */
u8 ato;
-
-#define MC13783_TS_ATO_FIRST false
-#define MC13783_TS_ATO_EACH true
/* Use the ATO delay only for the first conversion or for each one */
bool atox;
};
@@ -210,11 +211,12 @@ struct mc13xxx_codec_platform_data {
enum mc13783_ssi_port dac_ssi_port;
};
-struct mc13xxx_platform_data {
-#define MC13XXX_USE_TOUCHSCREEN (1 << 0)
+#define MC13XXX_USE_TOUCHSCREEN (1 << 0)
#define MC13XXX_USE_CODEC (1 << 1)
#define MC13XXX_USE_ADC (1 << 2)
#define MC13XXX_USE_RTC (1 << 3)
+
+struct mc13xxx_platform_data {
unsigned int flags;
struct mc13xxx_regulator_platform_data regulators;
--
1.8.3.2
^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-15 7:08 [PATCH] mfd: mc13xxx: Move definitions out of structures Alexander Shiyan
@ 2014-03-18 8:46 ` Lee Jones
2014-03-18 9:02 ` Lee Jones
1 sibling, 0 replies; 10+ messages in thread
From: Lee Jones @ 2014-03-18 8:46 UTC (permalink / raw)
To: Alexander Shiyan; +Cc: linux-kernel, Samuel Ortiz
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
> include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> 1 file changed, 12 insertions(+), 10 deletions(-)
Applied, thanks.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-15 7:08 [PATCH] mfd: mc13xxx: Move definitions out of structures Alexander Shiyan
2014-03-18 8:46 ` Lee Jones
@ 2014-03-18 9:02 ` Lee Jones
2014-03-18 9:14 ` Alexander Shiyan
1 sibling, 1 reply; 10+ messages in thread
From: Lee Jones @ 2014-03-18 9:02 UTC (permalink / raw)
To: Alexander Shiyan; +Cc: linux-kernel, Samuel Ortiz
> Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> ---
> include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> 1 file changed, 12 insertions(+), 10 deletions(-)
>
> diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> index a326c85..d63b1d3 100644
> --- a/include/linux/mfd/mc13xxx.h
> +++ b/include/linux/mfd/mc13xxx.h
> @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
>
> #define MAX_LED_CONTROL_REGS 6
>
> -struct mc13xxx_leds_platform_data {
> - struct mc13xxx_led_platform_data *led;
> - int num_leds;
> -
> /* MC13783 LED Control 0 */
> #define MC13783_LED_C0_ENABLE (1 << 0)
> #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> /* MC34708 LED Control 0 */
I don't see the line above in the MFD tree?
Which commit/branch/repo is this patch based on?
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 9:02 ` Lee Jones
@ 2014-03-18 9:14 ` Alexander Shiyan
2014-03-18 9:27 ` Lee Jones
0 siblings, 1 reply; 10+ messages in thread
From: Alexander Shiyan @ 2014-03-18 9:14 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Samuel Ortiz
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=utf-8, Size: 1376 bytes --]
Tue, 18 Mar 2014 09:02:37 +0000 Ð¾Ñ Lee Jones <lee.jones@linaro.org>:
> > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > ---
> > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> > 1 file changed, 12 insertions(+), 10 deletions(-)
> >
> > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> > index a326c85..d63b1d3 100644
> > --- a/include/linux/mfd/mc13xxx.h
> > +++ b/include/linux/mfd/mc13xxx.h
> > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> >
> > #define MAX_LED_CONTROL_REGS 6
> >
> > -struct mc13xxx_leds_platform_data {
> > - struct mc13xxx_led_platform_data *led;
> > - int num_leds;
> > -
> > /* MC13783 LED Control 0 */
> > #define MC13783_LED_C0_ENABLE (1 << 0)
> > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> > /* MC34708 LED Control 0 */
>
> I don't see the line above in the MFD tree?
>
> Which commit/branch/repo is this patch based on?
Dependency:
linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
---
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 9:14 ` Alexander Shiyan
@ 2014-03-18 9:27 ` Lee Jones
2014-03-18 9:33 ` Alexander Shiyan
0 siblings, 1 reply; 10+ messages in thread
From: Lee Jones @ 2014-03-18 9:27 UTC (permalink / raw)
To: Alexander Shiyan, cooloney; +Cc: linux-kernel, Samuel Ortiz
> > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > ---
> > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> > > 1 file changed, 12 insertions(+), 10 deletions(-)
> > >
> > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> > > index a326c85..d63b1d3 100644
> > > --- a/include/linux/mfd/mc13xxx.h
> > > +++ b/include/linux/mfd/mc13xxx.h
> > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> > >
> > > #define MAX_LED_CONTROL_REGS 6
> > >
> > > -struct mc13xxx_leds_platform_data {
> > > - struct mc13xxx_led_platform_data *led;
> > > - int num_leds;
> > > -
> > > /* MC13783 LED Control 0 */
> > > #define MC13783_LED_C0_ENABLE (1 << 0)
> > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> > > /* MC34708 LED Control 0 */
> >
> > I don't see the line above in the MFD tree?
> >
> > Which commit/branch/repo is this patch based on?
>
> Dependency:
> linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
>
> https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
Ah, you can't do that, as it will cause conflicts and Linus will
likely get upset.
Bryan you naughty boy! Why are you applying patches to files in MFD?
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 9:27 ` Lee Jones
@ 2014-03-18 9:33 ` Alexander Shiyan
2014-03-18 9:40 ` Lee Jones
0 siblings, 1 reply; 10+ messages in thread
From: Alexander Shiyan @ 2014-03-18 9:33 UTC (permalink / raw)
To: Lee Jones; +Cc: linux-kernel, Samuel Ortiz, cooloney
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=utf-8, Size: 1635 bytes --]
Tue, 18 Mar 2014 09:27:35 +0000 Ð¾Ñ Lee Jones <lee.jones@linaro.org>:
> > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > > ---
> > > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> > > > 1 file changed, 12 insertions(+), 10 deletions(-)
> > > >
> > > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> > > > index a326c85..d63b1d3 100644
> > > > --- a/include/linux/mfd/mc13xxx.h
> > > > +++ b/include/linux/mfd/mc13xxx.h
> > > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> > > >
> > > > #define MAX_LED_CONTROL_REGS 6
> > > >
> > > > -struct mc13xxx_leds_platform_data {
> > > > - struct mc13xxx_led_platform_data *led;
> > > > - int num_leds;
> > > > -
> > > > /* MC13783 LED Control 0 */
> > > > #define MC13783_LED_C0_ENABLE (1 << 0)
> > > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> > > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> > > > /* MC34708 LED Control 0 */
> > >
> > > I don't see the line above in the MFD tree?
> > >
> > > Which commit/branch/repo is this patch based on?
> >
> > Dependency:
> > linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
> >
> > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
>
> Ah, you can't do that, as it will cause conflicts and Linus will
> likely get upset.
So what do we do now? What is the solution?
---
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 9:33 ` Alexander Shiyan
@ 2014-03-18 9:40 ` Lee Jones
2014-03-18 22:31 ` Bryan Wu
0 siblings, 1 reply; 10+ messages in thread
From: Lee Jones @ 2014-03-18 9:40 UTC (permalink / raw)
To: Alexander Shiyan; +Cc: linux-kernel, Samuel Ortiz, cooloney
> > > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > > > > ---
> > > > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> > > > > 1 file changed, 12 insertions(+), 10 deletions(-)
> > > > >
> > > > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> > > > > index a326c85..d63b1d3 100644
> > > > > --- a/include/linux/mfd/mc13xxx.h
> > > > > +++ b/include/linux/mfd/mc13xxx.h
> > > > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> > > > >
> > > > > #define MAX_LED_CONTROL_REGS 6
> > > > >
> > > > > -struct mc13xxx_leds_platform_data {
> > > > > - struct mc13xxx_led_platform_data *led;
> > > > > - int num_leds;
> > > > > -
> > > > > /* MC13783 LED Control 0 */
> > > > > #define MC13783_LED_C0_ENABLE (1 << 0)
> > > > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> > > > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> > > > > /* MC34708 LED Control 0 */
> > > >
> > > > I don't see the line above in the MFD tree?
> > > >
> > > > Which commit/branch/repo is this patch based on?
> > >
> > > Dependency:
> > > linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
> > >
> > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
> >
> > Ah, you can't do that, as it will cause conflicts and Linus will
> > likely get upset.
>
> So what do we do now? What is the solution?
Don't worry about it. Bryan and I should be able to sort something out.
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 9:40 ` Lee Jones
@ 2014-03-18 22:31 ` Bryan Wu
2014-03-19 8:51 ` Lee Jones
0 siblings, 1 reply; 10+ messages in thread
From: Bryan Wu @ 2014-03-18 22:31 UTC (permalink / raw)
To: Lee Jones; +Cc: Alexander Shiyan, lkml, Samuel Ortiz
On Tue, Mar 18, 2014 at 2:40 AM, Lee Jones <lee.jones@linaro.org> wrote:
>> > > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
>> > > > > ---
>> > > > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
>> > > > > 1 file changed, 12 insertions(+), 10 deletions(-)
>> > > > >
>> > > > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
>> > > > > index a326c85..d63b1d3 100644
>> > > > > --- a/include/linux/mfd/mc13xxx.h
>> > > > > +++ b/include/linux/mfd/mc13xxx.h
>> > > > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
>> > > > >
>> > > > > #define MAX_LED_CONTROL_REGS 6
>> > > > >
>> > > > > -struct mc13xxx_leds_platform_data {
>> > > > > - struct mc13xxx_led_platform_data *led;
>> > > > > - int num_leds;
>> > > > > -
>> > > > > /* MC13783 LED Control 0 */
>> > > > > #define MC13783_LED_C0_ENABLE (1 << 0)
>> > > > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
>> > > > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
>> > > > > /* MC34708 LED Control 0 */
>> > > >
>> > > > I don't see the line above in the MFD tree?
>> > > >
>> > > > Which commit/branch/repo is this patch based on?
>> > >
>> > > Dependency:
>> > > linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
>> > >
>> > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
>> >
>> > Ah, you can't do that, as it will cause conflicts and Linus will
>> > likely get upset.
>>
>> So what do we do now? What is the solution?
>
> Don't worry about it. Bryan and I should be able to sort something out.
I lost the context here.
I merged Alexander's patch as a59ce6584d566847980f9dcad5343cd9856145c8
which modified the mc13xxx.h. But all things are related to LED driver
and only add several lines. Next time I will ask Lee to give an review
and ack.
So probably this patch has to wait for
a59ce6584d566847980f9dcad5343cd9856145c8 merged.
Thanks,
-Bryan
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-18 22:31 ` Bryan Wu
@ 2014-03-19 8:51 ` Lee Jones
2014-03-19 9:05 ` Alexander Shiyan
0 siblings, 1 reply; 10+ messages in thread
From: Lee Jones @ 2014-03-19 8:51 UTC (permalink / raw)
To: Bryan Wu; +Cc: Alexander Shiyan, lkml, Samuel Ortiz
On Tue, 18 Mar 2014, Bryan Wu wrote:
> On Tue, Mar 18, 2014 at 2:40 AM, Lee Jones <lee.jones@linaro.org> wrote:
> >> > > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> >> > > > > ---
> >> > > > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> >> > > > > 1 file changed, 12 insertions(+), 10 deletions(-)
> >> > > > >
> >> > > > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> >> > > > > index a326c85..d63b1d3 100644
> >> > > > > --- a/include/linux/mfd/mc13xxx.h
> >> > > > > +++ b/include/linux/mfd/mc13xxx.h
> >> > > > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> >> > > > >
> >> > > > > #define MAX_LED_CONTROL_REGS 6
> >> > > > >
> >> > > > > -struct mc13xxx_leds_platform_data {
> >> > > > > - struct mc13xxx_led_platform_data *led;
> >> > > > > - int num_leds;
> >> > > > > -
> >> > > > > /* MC13783 LED Control 0 */
> >> > > > > #define MC13783_LED_C0_ENABLE (1 << 0)
> >> > > > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> >> > > > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> >> > > > > /* MC34708 LED Control 0 */
> >> > > >
> >> > > > I don't see the line above in the MFD tree?
> >> > > >
> >> > > > Which commit/branch/repo is this patch based on?
> >> > >
> >> > > Dependency:
> >> > > linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
> >> > >
> >> > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
> >> >
> >> > Ah, you can't do that, as it will cause conflicts and Linus will
> >> > likely get upset.
> >>
> >> So what do we do now? What is the solution?
> >
> > Don't worry about it. Bryan and I should be able to sort something out.
>
> I lost the context here.
>
> I merged Alexander's patch as a59ce6584d566847980f9dcad5343cd9856145c8
> which modified the mc13xxx.h. But all things are related to LED driver
> and only add several lines. Next time I will ask Lee to give an review
> and ack.
>
> So probably this patch has to wait for
> a59ce6584d566847980f9dcad5343cd9856145c8 merged.
Yes, I think this is the best idea.
Alexander, can you re-send this patch with my Ack after the next merge
window please?
--
Lee Jones
Linaro STMicroelectronics Landing Team Lead
Linaro.org │ Open source software for ARM SoCs
Follow Linaro: Facebook | Twitter | Blog
^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH] mfd: mc13xxx: Move definitions out of structures
2014-03-19 8:51 ` Lee Jones
@ 2014-03-19 9:05 ` Alexander Shiyan
0 siblings, 0 replies; 10+ messages in thread
From: Alexander Shiyan @ 2014-03-19 9:05 UTC (permalink / raw)
To: Lee Jones; +Cc: lkml, Samuel Ortiz, Bryan Wu
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1: Type: text/plain; charset=utf-8, Size: 2624 bytes --]
Wed, 19 Mar 2014 08:51:44 +0000 Ð¾Ñ Lee Jones <lee.jones@linaro.org>:
> On Tue, 18 Mar 2014, Bryan Wu wrote:
>
> > On Tue, Mar 18, 2014 at 2:40 AM, Lee Jones <lee.jones@linaro.org> wrote:
> > >> > > > > Signed-off-by: Alexander Shiyan <shc_work@mail.ru>
> > >> > > > > ---
> > >> > > > > include/linux/mfd/mc13xxx.h | 22 ++++++++++++----------
> > >> > > > > 1 file changed, 12 insertions(+), 10 deletions(-)
> > >> > > > >
> > >> > > > > diff --git a/include/linux/mfd/mc13xxx.h b/include/linux/mfd/mc13xxx.h
> > >> > > > > index a326c85..d63b1d3 100644
> > >> > > > > --- a/include/linux/mfd/mc13xxx.h
> > >> > > > > +++ b/include/linux/mfd/mc13xxx.h
> > >> > > > > @@ -117,10 +117,6 @@ struct mc13xxx_led_platform_data {
> > >> > > > >
> > >> > > > > #define MAX_LED_CONTROL_REGS 6
> > >> > > > >
> > >> > > > > -struct mc13xxx_leds_platform_data {
> > >> > > > > - struct mc13xxx_led_platform_data *led;
> > >> > > > > - int num_leds;
> > >> > > > > -
> > >> > > > > /* MC13783 LED Control 0 */
> > >> > > > > #define MC13783_LED_C0_ENABLE (1 << 0)
> > >> > > > > #define MC13783_LED_C0_TRIODE_MD (1 << 7)
> > >> > > > > @@ -169,10 +165,13 @@ struct mc13xxx_leds_platform_data {
> > >> > > > > /* MC34708 LED Control 0 */
> > >> > > >
> > >> > > > I don't see the line above in the MFD tree?
> > >> > > >
> > >> > > > Which commit/branch/repo is this patch based on?
> > >> > >
> > >> > > Dependency:
> > >> > > linux-next, a59ce6584d (leds: leds-mc13783: Add MC34708 LED support)
> > >> > >
> > >> > > https://git.kernel.org/cgit/linux/kernel/git/next/linux-next.git/commit/include/linux/mfd/mc13xxx.h?id=a59ce6584d566847980f9dcad5343cd9856145c8
> > >> >
> > >> > Ah, you can't do that, as it will cause conflicts and Linus will
> > >> > likely get upset.
> > >>
> > >> So what do we do now? What is the solution?
> > >
> > > Don't worry about it. Bryan and I should be able to sort something out.
> >
> > I lost the context here.
> >
> > I merged Alexander's patch as a59ce6584d566847980f9dcad5343cd9856145c8
> > which modified the mc13xxx.h. But all things are related to LED driver
> > and only add several lines. Next time I will ask Lee to give an review
> > and ack.
> >
> > So probably this patch has to wait for
> > a59ce6584d566847980f9dcad5343cd9856145c8 merged.
>
> Yes, I think this is the best idea.
>
> Alexander, can you re-send this patch with my Ack after the next merge
> window please?
Sure.
---
ÿôèº{.nÇ+·®+%Ëÿ±éݶ\x17¥wÿº{.nÇ+·¥{±þG«éÿ{ayº\x1dÊÚë,j\a¢f£¢·hïêÿêçz_è®\x03(éÝ¢j"ú\x1a¶^[m§ÿÿ¾\a«þG«éÿ¢¸?¨èÚ&£ø§~á¶iOæ¬z·vØ^\x14\x04\x1a¶^[m§ÿÿÃ\fÿ¶ìÿ¢¸?I¥
^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2014-03-19 9:05 UTC | newest]
Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2014-03-15 7:08 [PATCH] mfd: mc13xxx: Move definitions out of structures Alexander Shiyan
2014-03-18 8:46 ` Lee Jones
2014-03-18 9:02 ` Lee Jones
2014-03-18 9:14 ` Alexander Shiyan
2014-03-18 9:27 ` Lee Jones
2014-03-18 9:33 ` Alexander Shiyan
2014-03-18 9:40 ` Lee Jones
2014-03-18 22:31 ` Bryan Wu
2014-03-19 8:51 ` Lee Jones
2014-03-19 9:05 ` Alexander Shiyan
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).