2014-01-09 10:54:01

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

On Friday 29 November 2013 01:44 PM, Sekhar Nori wrote:
> Wim,
>
> On Thursday 28 November 2013 06:29 PM, Guenter Roeck wrote:
>> On 11/27/2013 10:04 PM, Sekhar Nori wrote:
>>> On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote:
>>>> On 11/27/2013 06:00 AM, Sekhar Nori wrote:
>>>>> On Wednesday 27 November 2013 07:01 PM, Ivan Khoronzhuk wrote:
>>>>>> As we switch to use the watchdog core which permits more than one
>>>>>> active watchdog in the system, rename platform driver to
>>>>>> "davinci-wdt" to be identifiable.
>>>>>>
>>>>>> Signed-off-by: Ivan Khoronzhuk <[email protected]>
>>>>>
>>>>> Looks good to me. Since bulk of this patch touches mach-davinci, I
>>>>> would
>>>>> like to take this through my tree to avoid conflicts with other
>>>>> mach-davinci patches I accept.
>>>>>
>>>>
>>>> Good idea, and makes sense.
>>>
>>> Added to v3.14/soc with your Reviewed-by
>>>
>>
>> You should get an Ack from Wim.
>
> Can you please provide your ack? I will add it to the existing commit.

I never received an ack from Wim, but I am going to send this for
merging through ARM-SoC anyway. I will keep Wim copied on the pull
request so he knows.

Thanks,
Sekhar


2014-01-09 11:09:13

by Wim Van Sebroeck

[permalink] [raw]
Subject: Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

Hi Sekhar,

> On Friday 29 November 2013 01:44 PM, Sekhar Nori wrote:
> > Wim,
> >
> > On Thursday 28 November 2013 06:29 PM, Guenter Roeck wrote:
> >> On 11/27/2013 10:04 PM, Sekhar Nori wrote:
> >>> On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote:
> >>>> On 11/27/2013 06:00 AM, Sekhar Nori wrote:
> >>>>> On Wednesday 27 November 2013 07:01 PM, Ivan Khoronzhuk wrote:
> >>>>>> As we switch to use the watchdog core which permits more than one
> >>>>>> active watchdog in the system, rename platform driver to
> >>>>>> "davinci-wdt" to be identifiable.
> >>>>>>
> >>>>>> Signed-off-by: Ivan Khoronzhuk <[email protected]>
> >>>>>
> >>>>> Looks good to me. Since bulk of this patch touches mach-davinci, I
> >>>>> would
> >>>>> like to take this through my tree to avoid conflicts with other
> >>>>> mach-davinci patches I accept.
> >>>>>
> >>>>
> >>>> Good idea, and makes sense.
> >>>
> >>> Added to v3.14/soc with your Reviewed-by
> >>>
> >>
> >> You should get an Ack from Wim.
> >
> > Can you please provide your ack? I will add it to the existing commit.
>
> I never received an ack from Wim, but I am going to send this for
> merging through ARM-SoC anyway. I will keep Wim copied on the pull
> request so he knows.

This has my Ack. No issue with getting this through the ARM-SoC tree but bear in mind that there are allready other davinci pateches in the linux-watchdog-next tree+.

Kind regards,
Wim.

2014-01-09 11:15:12

by Sekhar Nori

[permalink] [raw]
Subject: Re: [PATCH] watchdog: davinci: rename platform driver to davinci-wdt

On Thursday 09 January 2014 04:38 PM, Wim Van Sebroeck wrote:
> Hi Sekhar,
>
>> On Friday 29 November 2013 01:44 PM, Sekhar Nori wrote:
>>> Wim,
>>>
>>> On Thursday 28 November 2013 06:29 PM, Guenter Roeck wrote:
>>>> On 11/27/2013 10:04 PM, Sekhar Nori wrote:
>>>>> On Wednesday 27 November 2013 09:27 PM, Guenter Roeck wrote:
>>>>>> On 11/27/2013 06:00 AM, Sekhar Nori wrote:
>>>>>>> On Wednesday 27 November 2013 07:01 PM, Ivan Khoronzhuk wrote:
>>>>>>>> As we switch to use the watchdog core which permits more than one
>>>>>>>> active watchdog in the system, rename platform driver to
>>>>>>>> "davinci-wdt" to be identifiable.
>>>>>>>>
>>>>>>>> Signed-off-by: Ivan Khoronzhuk <[email protected]>
>>>>>>>
>>>>>>> Looks good to me. Since bulk of this patch touches mach-davinci, I
>>>>>>> would
>>>>>>> like to take this through my tree to avoid conflicts with other
>>>>>>> mach-davinci patches I accept.
>>>>>>>
>>>>>>
>>>>>> Good idea, and makes sense.
>>>>>
>>>>> Added to v3.14/soc with your Reviewed-by
>>>>>
>>>>
>>>> You should get an Ack from Wim.
>>>
>>> Can you please provide your ack? I will add it to the existing commit.
>>
>> I never received an ack from Wim, but I am going to send this for
>> merging through ARM-SoC anyway. I will keep Wim copied on the pull
>> request so he knows.
>
> This has my Ack. No issue with getting this through the ARM-SoC tree but bear in mind that there are allready other davinci pateches in the linux-watchdog-next tree+.

Okay. I will add you ack (I just applied my signed tag, grr).
I did a test merge of this patch with today's linux-next and see no
conflicts.

Thanks,
Sekhar