2012-10-04 13:01:24

by Fabio Estevam

[permalink] [raw]
Subject: Re: [PATCH 0/7] Add support for Freescale's mc34708 to mc13xxx driver

Hi Samuel,

On Wed, Sep 5, 2012 at 11:22 AM, Fabio Estevam <[email protected]> wrote:
> Hi Samuel,
>
> On Thu, Jul 12, 2012 at 8:02 PM, Marc Reilly <[email protected]> wrote:
>> Hi Uwe,
>>
>>> This series was tested on a Phytec pcm038 (mc13783 on spi) using
>>> traditional boot (i.e. not dt) and on a i.MX53 based machine (mc34708 on
>>> i2c) using dt boot.
>>>
>>> Philippe's patches are already in next, they are just included here for
>>> those who want to test the patches. The 'mfd/mc13xxx: drop modifying
>>> driver's id_table in probe' was already sent out yesterday and is
>>> included here because the last patch depends on it.
>>>
>>
>> For all patches (that don't already have it):
>> Acked-by: Marc Reilly <[email protected]>
>
> Can this series be applied?

Any comments on this series, please?

I want to add mc34708 support to mx53qsb and need this series to be applied.


2012-10-04 13:51:36

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH 0/7] Add support for Freescale's mc34708 to mc13xxx driver

Hi Fabio,

On Thu, Oct 04, 2012 at 10:01:21AM -0300, Fabio Estevam wrote:
> Hi Samuel,
>
> On Wed, Sep 5, 2012 at 11:22 AM, Fabio Estevam <[email protected]> wrote:
> > Hi Samuel,
> >
> > On Thu, Jul 12, 2012 at 8:02 PM, Marc Reilly <[email protected]> wrote:
> >> Hi Uwe,
> >>
> >>> This series was tested on a Phytec pcm038 (mc13783 on spi) using
> >>> traditional boot (i.e. not dt) and on a i.MX53 based machine (mc34708 on
> >>> i2c) using dt boot.
> >>>
> >>> Philippe's patches are already in next, they are just included here for
> >>> those who want to test the patches. The 'mfd/mc13xxx: drop modifying
> >>> driver's id_table in probe' was already sent out yesterday and is
> >>> included here because the last patch depends on it.
> >>>
> >>
> >> For all patches (that don't already have it):
> >> Acked-by: Marc Reilly <[email protected]>
> >
> > Can this series be applied?
>
> Any comments on this series, please?
Did you send it to my [email protected] email ? I never look at it when
going through my MFD backlog, which is why it was not applied. Which is why it
won't be part of the 3.7 merge window, sorry about that.


> I want to add mc34708 support to mx53qsb and need this series to be applied.
I understand. I'll queue it to my for-next branch as soon as the merge window
is closed.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris,
92196 Meudon Cedex, France
Registration Number: 302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.

2012-10-04 14:12:10

by Uwe Kleine-König

[permalink] [raw]
Subject: Re: [PATCH 0/7] Add support for Freescale's mc34708 to mc13xxx driver

Hi,

On Thu, Oct 04, 2012 at 03:51:30PM +0200, Samuel Ortiz wrote:
> Did you send it to my [email protected] email ? I never look at it when
> going through my MFD backlog, which is why it was not applied. Which is why it
> won't be part of the 3.7 merge window, sorry about that.
I don't know why I picked that one up, but probably it's because this
address shows up in several commits:

$ git log [email protected] | grep Samuel | sort | uniq -c
18 Acked-by: Samuel Ortiz <[email protected]>
12 Author: Samuel Ortiz <[email protected]>
27 Author: Samuel Ortiz <[email protected]>
4 Cc: Samuel Ortiz <[email protected]>
1 Signed-off-by: Samuel Ortiz <[email protected]>
50 Signed-off-by: Samuel Ortiz <[email protected]>

and one of these just happend to be the first I looked at.

hmm, *shrug*
Uwe

--
Pengutronix e.K. | Uwe Kleine-K?nig |
Industrial Linux Solutions | http://www.pengutronix.de/ |

2012-10-27 19:19:13

by Fabio Estevam

[permalink] [raw]
Subject: Re: [PATCH 0/7] Add support for Freescale's mc34708 to mc13xxx driver

Hi Samuel,

On Thu, Oct 4, 2012 at 10:51 AM, Samuel Ortiz <[email protected]> wrote:

>> I want to add mc34708 support to mx53qsb and need this series to be applied.
> I understand. I'll queue it to my for-next branch as soon as the merge window
> is closed.

Could you please queue this series? I still do not see it applied in
your 'for-next-merge' branch.

I will add mc34708 touchscreen support and need Uwe's series to be applied.

Thanks,

Fabio Estevam

2012-11-06 22:24:24

by Samuel Ortiz

[permalink] [raw]
Subject: Re: [PATCH 0/7] Add support for Freescale's mc34708 to mc13xxx driver

Hi Fabio,

On Sat, Oct 27, 2012 at 05:19:11PM -0200, Fabio Estevam wrote:
> Hi Samuel,
>
> On Thu, Oct 4, 2012 at 10:51 AM, Samuel Ortiz <[email protected]> wrote:
>
> >> I want to add mc34708 support to mx53qsb and need this series to be applied.
> > I understand. I'll queue it to my for-next branch as soon as the merge window
> > is closed.
>
> Could you please queue this series? I still do not see it applied in
> your 'for-next-merge' branch.
Patches 6 and 7 were not applied from this serie. I now applied them to my
for-next branch.

Cheers,
Samuel.

--
Intel Open Source Technology Centre
http://oss.intel.com/
---------------------------------------------------------------------
Intel Corporation SAS (French simplified joint stock company)
Registered headquarters: "Les Montalets"- 2, rue de Paris,
92196 Meudon Cedex, France
Registration Number: 302 456 199 R.C.S. NANTERRE
Capital: 4,572,000 Euros

This e-mail and any attachments may contain confidential material for
the sole use of the intended recipient(s). Any review or distribution
by others is strictly prohibited. If you are not the intended
recipient, please contact the sender and delete all copies.