Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755658Ab3FAU5R (ORCPT ); Sat, 1 Jun 2013 16:57:17 -0400 Received: from moutng.kundenserver.de ([212.227.17.9]:59693 "EHLO moutng.kundenserver.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753530Ab3FAU5O (ORCPT ); Sat, 1 Jun 2013 16:57:14 -0400 From: Arnd Bergmann To: Rob Herring Subject: Re: [PATCH 15/15] OF: remove #ifdef from linux/of_platform.h Date: Sat, 1 Jun 2013 22:57:02 +0200 User-Agent: KMail/1.12.2 (Linux/3.8.0-22-generic; KDE/4.3.2; x86_64; ; ) Cc: "linux-kernel@vger.kernel.org" , Grant Likely , patches@lists.linaro.org, Rob Herring , "linux-arm-kernel@lists.infradead.org" , Benjamin Herrenschmidt References: <1370038972-2318779-1-git-send-email-arnd@arndb.de> <201306012203.39133.arnd@arndb.de> <51AA59D0.7020805@gmail.com> In-Reply-To: <51AA59D0.7020805@gmail.com> MIME-Version: 1.0 Content-Type: Text/Plain; charset="iso-8859-1" Content-Transfer-Encoding: 7bit Message-Id: <201306012257.02327.arnd@arndb.de> X-Provags-ID: V02:K0:AMDm9ZfDZ9br7X1mrkqzNailbvEQPin1RrGRKfH/qRc vcJuE47z/Zd+1MFCc3kkJtubFQaATyswxd5Vtw+HYvJi4sfV4b MTPWuNAqVrL8do1a7T+tVehGIT9XkwuNWXAOKiYTQsagz2WbQH TiAo8NnqgrTT6B/4Fs3xVlvnafUzc1dYfkCW28H5RAZUiZebF/ +0AJ8vE7D21dZu3igPQ8Hf4CToh4+azk+UXDDEtQ1nuvIc+FXU mt4/rcu9aRfd3tM1Yc0hJNT8sdu4CVOY9HyxJjJb/0NkYkGCWO Bvg9FyM3uL3f6+29E7aiv2z+gPjWqsTUvUQ/Wk/VCwFfeicc/+ jKej35V0Hvw9gpWR7K3w= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 687 Lines: 20 On Saturday 01 June 2013, Rob Herring wrote: > No, we still need empty functions. Here is what of_device.h looks like: > > http://tinyurl.com/l2azz5m > > BTW, it has your ack. > Could you add a patch on top that only puts the function declarations inside of #ifdef that don't have an inline wrapper? It's really annoying to have to change the header file every time one needs to call a function from a driver in the DT-only case. Arnd -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/