linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [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).