From mboxrd@z Thu Jan 1 00:00:00 1970 From: Benoit Cousson Subject: Re: [PATCH v2 0/9] ARM: dts: DT data for OMAP platforms for v3.13 Date: Fri, 04 Oct 2013 16:10:43 +0200 Message-ID: <524ECC63.3050303@baylibre.com> References: <1380553986-15910-1-git-send-email-joelf@ti.com> <524D7038.9070303@baylibre.com> <524E213B.6030306@ti.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1; format=flowed Content-Transfer-Encoding: 7bit Return-path: Received: from mail-wg0-f48.google.com ([74.125.82.48]:34147 "EHLO mail-wg0-f48.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754120Ab3JDOKr (ORCPT ); Fri, 4 Oct 2013 10:10:47 -0400 Received: by mail-wg0-f48.google.com with SMTP id n12so4181768wgh.15 for ; Fri, 04 Oct 2013 07:10:46 -0700 (PDT) In-Reply-To: <524E213B.6030306@ti.com> Sender: linux-mmc-owner@vger.kernel.org List-Id: linux-mmc@vger.kernel.org To: Joel Fernandes Cc: Linux OMAP List , Linux ARM Kernel List , Linux MMC List , Mark Rutland , "swarren@wwwdotorg.org" , "rob.herring@calxeda.com" , "devicetree@vger.kernel.org" , Pawel Moll 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