From: Linus Walleij Subject: Re: [PATCH 5/6] crypto: ux500/cryp - Enable DT probing of the driver Date: Fri, 24 May 2013 13:16:42 +0200 Message-ID: References: <1368703646-26418-1-git-send-email-lee.jones@linaro.org> <1368703646-26418-6-git-send-email-lee.jones@linaro.org> <20130524083231.GQ22683@gmail.com> Mime-Version: 1.0 Content-Type: text/plain; charset=ISO-8859-1 Cc: Herbert Xu , "linux-arm-kernel@lists.infradead.org" , "linux-kernel@vger.kernel.org" , Arnd Bergmann , Linus WALLEIJ , Srinidhi KASAGAR , "linux-crypto@vger.kernel.org" To: Lee Jones , "devicetree-discuss@lists.ozlabs.org" , Rob Herring Return-path: Received: from mail-ie0-f177.google.com ([209.85.223.177]:41081 "EHLO mail-ie0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751514Ab3EXLQm (ORCPT ); Fri, 24 May 2013 07:16:42 -0400 Received: by mail-ie0-f177.google.com with SMTP id 9so11718740iec.22 for ; Fri, 24 May 2013 04:16:42 -0700 (PDT) In-Reply-To: <20130524083231.GQ22683@gmail.com> Sender: linux-crypto-owner@vger.kernel.org List-ID: On Fri, May 24, 2013 at 10:32 AM, Lee Jones wrote: > On Fri, 24 May 2013, Linus Walleij wrote: >> On Thu, May 16, 2013 at 1:27 PM, Lee Jones wrote: >> >> > By providing an OF match table with a suitable compatible string, we >> > can ensure the ux500-crypt driver is probed by supplying an associated >> > DT node in a given platform's Device Tree. >> > >> > Cc: Herbert Xu >> > Cc: linux-crypto@vger.kernel.org >> > Signed-off-by: Lee Jones >> >> Herbert, can I have your ACK on patch 5 & 6 in this series to take it >> through the ARM SoC tree? > > Right, there are no bindings, so they don't need a document. Surely the requirement that the node must have , and for the driver to even probe is a binding? The fact that the drivers/of/* core on Linux will auto-populate platform devices with those properties is a pure Linux pecularity, and the bindings are there for OS independence. Whether we need to document it for such standard things is another thing, hopefully Rob or someone can answer that? Yours, Linus Walleij