From mboxrd@z Thu Jan 1 00:00:00 1970 Received: with ECARTIS (v1.0.0; list linux-mips); Wed, 10 Mar 2010 17:48:34 +0100 (CET) Received: from localhost.localdomain ([127.0.0.1]:60683 "EHLO h5.dl5rb.org.uk" rhost-flags-OK-OK-OK-FAIL) by eddie.linux-mips.org with ESMTP id S1492624Ab0CJQs3 (ORCPT ); Wed, 10 Mar 2010 17:48:29 +0100 Received: from h5.dl5rb.org.uk (localhost.localdomain [127.0.0.1]) by h5.dl5rb.org.uk (8.14.3/8.14.3) with ESMTP id o2AGmQ71018013; Wed, 10 Mar 2010 17:48:27 +0100 Received: (from ralf@localhost) by h5.dl5rb.org.uk (8.14.3/8.14.3/Submit) id o2AGmOF9018009; Wed, 10 Mar 2010 17:48:24 +0100 Date: Wed, 10 Mar 2010 17:48:24 +0100 From: Ralf Baechle To: Manuel Lauss Cc: Sergei Shtylyov , Linux-MIPS , Manuel Lauss Subject: Re: [PATCH] MIPS: Alchemy: move MMC driver registration to board code. Message-ID: <20100310164824.GC15118@linux-mips.org> References: <1268076181-29642-1-git-send-email-manuel.lauss@gmail.com> <1268076181-29642-3-git-send-email-manuel.lauss@gmail.com> <4B963210.7030906@ru.mvista.com> <4B96364E.5050202@mvista.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.20 (2009-08-17) Return-Path: X-Envelope-To: <"|/home/ecartis/ecartis -s linux-mips"> (uid 0) X-Orcpt: rfc822;linux-mips@linux-mips.org Original-Recipient: rfc822;linux-mips@linux-mips.org X-archive-position: 26176 X-ecartis-version: Ecartis v1.0.0 Sender: linux-mips-bounce@linux-mips.org Errors-to: linux-mips-bounce@linux-mips.org X-original-sender: ralf@linux-mips.org Precedence: bulk X-list: linux-mips On Tue, Mar 09, 2010 at 01:03:51PM +0100, Manuel Lauss wrote: > And on a personal note, that file just bothers me. It's messy, can > cause merge conflicts, Eye cancer. > it references structures defined inside board-specific code. In short, > it just plain annoys > my sense of aesthetics. Indeed - and I don't think Sergej disagrees with that. I agree with him that device registration code should primarily be done in the SOC code - but you'll need to somehow get that code to communicate with the platform code about what really needs to be done then register the remainder of the truely platform-specific platform devices. Something like that. Ralf