Re: [U-Boot] [PATCH v4 02/19] dm: timer: uclass: add timer init in uclass driver to add timer device

2016-01-17 Thread Simon Glass 
Hi Mugunthan,

On 16 January 2016 at 09:51, Mugunthan V N  wrote:
> On Saturday 16 January 2016 10:11 PM, Simon Glass wrote:
>> Hi Mugunthan,
>>
>> On 16 January 2016 at 09:08, Mugunthan V N  wrote:
>>> On Saturday 16 January 2016 09:33 PM, Mugunthan V N wrote:
 Adding timer init function in timer-uclass driver to create and
 initialize the timer device on platforms where u-boot,dm-pre-reloc
 is not used. Since there will be multiple timer devices in the
 system, adding a tick-timer node in chosen node to know which
 timer device to be used as tick timer in u-boot.

 Signed-off-by: Mugunthan V N 
>>>
>>> Please drop this patch, will send a fixup patch as the series is already
>>> applied to u-boot-dm tree.
>>

>> You can send a new patch if you like and I can replace it. Either works for 
>> me.
>>
>
> Just sent a follow-up patch. Its good to not change the pushed history.

I actually do this all the time as I rebase against master before
testing and sending pull requests. It helps to avoid conflicts for Tom
to deal with.

So since you have given me two options, I've pick up this patch and
replaced the v3 patch with it. I checked that the effect is the same
as applying your fix-up. The patches were sitting around since
Christmas so I wanted to get them in to permit the maximum possible
testing window.

Applied to u-boot-dm, thanks!

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v4 02/19] dm: timer: uclass: add timer init in uclass driver to add timer device

2016-01-16 Thread Mugunthan V N 
On Saturday 16 January 2016 09:33 PM, Mugunthan V N wrote:
> Adding timer init function in timer-uclass driver to create and
> initialize the timer device on platforms where u-boot,dm-pre-reloc
> is not used. Since there will be multiple timer devices in the
> system, adding a tick-timer node in chosen node to know which
> timer device to be used as tick timer in u-boot.
> 
> Signed-off-by: Mugunthan V N 

Please drop this patch, will send a fixup patch as the series is already
applied to u-boot-dm tree.

Regards
Mugunthan V N
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v4 02/19] dm: timer: uclass: add timer init in uclass driver to add timer device

2016-01-16 Thread Simon Glass 
Hi Mugunthan,

On 16 January 2016 at 09:08, Mugunthan V N  wrote:
> On Saturday 16 January 2016 09:33 PM, Mugunthan V N wrote:
>> Adding timer init function in timer-uclass driver to create and
>> initialize the timer device on platforms where u-boot,dm-pre-reloc
>> is not used. Since there will be multiple timer devices in the
>> system, adding a tick-timer node in chosen node to know which
>> timer device to be used as tick timer in u-boot.
>>
>> Signed-off-by: Mugunthan V N 
>
> Please drop this patch, will send a fixup patch as the series is already
> applied to u-boot-dm tree.

You can send a new patch if you like and I can replace it. Either works for me.

Regards,
Simon
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH v4 02/19] dm: timer: uclass: add timer init in uclass driver to add timer device

2016-01-16 Thread Mugunthan V N 
On Saturday 16 January 2016 10:11 PM, Simon Glass wrote:
> Hi Mugunthan,
> 
> On 16 January 2016 at 09:08, Mugunthan V N  wrote:
>> On Saturday 16 January 2016 09:33 PM, Mugunthan V N wrote:
>>> Adding timer init function in timer-uclass driver to create and
>>> initialize the timer device on platforms where u-boot,dm-pre-reloc
>>> is not used. Since there will be multiple timer devices in the
>>> system, adding a tick-timer node in chosen node to know which
>>> timer device to be used as tick timer in u-boot.
>>>
>>> Signed-off-by: Mugunthan V N 
>>
>> Please drop this patch, will send a fixup patch as the series is already
>> applied to u-boot-dm tree.
> 
> You can send a new patch if you like and I can replace it. Either works for 
> me.
> 

Just sent a follow-up patch. Its good to not change the pushed history.

Regards
Mugunthan V N

___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot