[PATCH v3 1/3] can: c_can: Add device tree support to Bosch C_CAN/D_CAN controller

AnilKumar, Chimata anilkumar at ti.com
Thu Aug 2 22:01:37 EST 2012


Hi Arnd,

Thanks for the review

On Thu, Aug 02, 2012 at 17:03:59, Arnd Bergmann wrote:
> On Thursday 02 August 2012 16:32:17 AnilKumar Ch wrote:
> > +- interrupt-parent     : The parent interrupt controller
> > +
> > +Optional properties:
> > +- ti,hwmods            : Must be "d_can<n>" or "c_can<n>", n being the
> > +                         instance number
> 
> interrupt-parent should be optional, not mandatory. It's usually implied
> by the parent's interrupt-parent.
> 

In case of AM33XX specific we need to connect interrupt to interrupt
controller so I added to mandatory list. Yes I agree with you, in generic
case this should be optional I will change.

Thanks
AnilKumar


More information about the devicetree-discuss mailing list