On 10/04/2013 09:10 AM, Benoit Cousson wrote: > On 04/10/2013 04:00, Joel Fernandes wrote: >> On 10/03/2013 08:25 AM, Benoit Cousson wrote: >>> + DT list and DT maintainers. >>> >>> Hi Joel, >>> >>> Thanks for the update. It looks good to me. >>> >>> For the new bindings added below; >>> >>>> .../devicetree/bindings/crypto/omap-aes.txt | 34 ++++++++++ >>>> .../devicetree/bindings/crypto/omap-sham.txt | 31 +++++++++ >>> >>> I will need the acked-by from one of the DT maintainers. >> >> Sure. To help with giving Ack for this, I'd like to also mention these patches >> were due for long time and reposted. The supporting code is already in the >> kernel. >> >> Also bindings were reviewed by Mark Rutland comments in the following thread >> were addressed: >> http://www.spinics.net/lists/arm-kernel/msg269006.html > > OK, that's good news, at least it was already reviewed by Mark. > > I'm fixing and pulling the series, and will add Mark's Acked-by as soon as he > will send it. > > Meanwhile some nits, I'm fixing myself this time :-) > > None of your subjects is following the same guideline and the repertory > guideline... > > omap4: dts: Add node for AES > omap4: dts: Add node for DES3DES module > am33xx: dts: Fix AES interrupt number > ARM: am437x: dts: Add AES node for am437x > ARM: am437x: dts: Add DES node for am437x > > It should be ARM: dts: OMAPXXX or AMXXX... > > Regards, > Benoit > On 10/04/2013 09:10 AM, Benoit Cousson wrote:> On 04/10/2013 04:00, Joel Fernandes wrote: >> On 10/03/2013 08:25 AM, Benoit Cousson wrote: >>> + DT list and DT maintainers. >>> >>> Hi Joel, >>> >>> Thanks for the update. It looks good to me. >>> >>> For the new bindings added below; >>> >>>> .../devicetree/bindings/crypto/omap-aes.txt | 34 ++++++++++ >>>> .../devicetree/bindings/crypto/omap-sham.txt | 31 +++++++++ >>> >>> I will need the acked-by from one of the DT maintainers. >> >> Sure. To help with giving Ack for this, I'd like to also mention these patches >> were due for long time and reposted. The supporting code is already in the >> kernel. >> >> Also bindings were reviewed by Mark Rutland comments in the following thread >> were addressed: >> http://www.spinics.net/lists/arm-kernel/msg269006.html > > OK, that's good news, at least it was already reviewed by Mark. > > I'm fixing and pulling the series, and will add Mark's Acked-by as soon as he > will send it. > > Meanwhile some nits, I'm fixing myself this time :-) > > None of your subjects is following the same guideline and the repertory > guideline... > > omap4: dts: Add node for AES > omap4: dts: Add node for DES3DES module > am33xx: dts: Fix AES interrupt number > ARM: am437x: dts: Add AES node for am437x > ARM: am437x: dts: Add DES node for am437x > > It should be ARM: dts: OMAPXXX or AMXXX... Thanks & sorry about that. I think Mark came back with some more comments on 2 of those patches. I will work on those and repost. -Joel -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html