From mboxrd@z Thu Jan 1 00:00:00 1970 From: Arnd Bergmann Subject: Re: [PATCH 2/2] gpio: Add Avionic Design N-bit GPIO expander support Date: Sun, 10 Jun 2012 06:43:33 +0000 Message-ID: <201206100643.33481.arnd@arndb.de> References: <1334229858-1665-1-git-send-email-thierry.reding@avionic-design.de> <20120526004805.92B373E1E4E@localhost> <20120607084015.GA27764@avionic-0098.adnet.avionic-design.de> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20120607084015.GA27764-RM9K5IK7kjIyiCvfTdI0JKcOhU4Rzj621B7CTYaBSLdn68oJJulU0Q@public.gmane.org> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: devicetree-discuss-bounces+gldd-devicetree-discuss=m.gmane.org-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Sender: "devicetree-discuss" To: devicetree-discuss-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org Cc: Linus Walleij , Rob Herring List-Id: devicetree@vger.kernel.org On Thursday 07 June 2012, Thierry Reding wrote: > > Unless you have an actual user of platform_data, don't borrow > > trouble by adding it. > > If I make the driver DT-only I should probably add a "select OF" to it's > Kconfig entry, right? Or should it rather be "depends on OF"? "depends on OF". Only architectures should select this symbol and device drivers should depend on it if they cannot work otherwise. Arnd