Re: [PATCH v3 00/10] ARM: dts: TI: Add CAN support

[Roger Quadros <rogerq@ti.com>]

On 11/05/2014 08:49 PM, Tony Lindgren wrote:
> * Roger Quadros <rogerq@ti.com> [141105 03:08]:
>> On 11/04/2014 05:55 PM, Tony Lindgren wrote:
>>> * Roger Quadros <rogerq@ti.com> [141104 02:50]:
>>>> Hi Tony,
>>>>
>>>> These patches add CAN support for am33xx, am43xx and dra7 platforms.
>>>> They must go in together with CAN driver RAMINIT Syscon support [1].
>>>
>>> Can you make this series not depend on [1] for most part without
>>> breaking existing functionality?
>>
>> I just checked that we haven't enabled DCAN on any of the boards
>> so we don't risk breaking anything even if these 2 series don't go in
>> together. 
> 
> OK 
>  
>>> Then we can do a single follow-up patch to enable things after [1]
>>> has been merged.
>>>
>>> That would remove a dependency between driver changes and .dts changes.
>>
>> Apart from patches 1, 3 and 8, that add the control module nodes the rest
>> pretty much depend on the driver DT binding changes.
>>
>> I can send them separately if needed.
> 
> Well sounds like I can already apply the .dts changes without
> affecting anything until the driver changes are merged.
> 
> Let me know if that's the case and I'll apply these assuming
> the binding is OK and not going to change.

There is one change in the binding so I'll have to post a v4.

cheers,
-roger