[Top] [All Lists]

Re: [PATCH] MIPS: Alchemy: move MMC driver registration to board

To: Ralf Baechle <>
Subject: Re: [PATCH] MIPS: Alchemy: move MMC driver registration to board code.
From: Manuel Lauss <>
Date: Wed, 10 Mar 2010 19:34:00 +0100
Cc: Sergei Shtylyov <>, Linux-MIPS <>, Manuel Lauss <>
Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed;; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:cc:content-type :content-transfer-encoding; bh=1+NTcxS3mtuZBtS5Q7frQkxrLTLE2HzmfAEvFOu4jKI=; b=lK11ahXweQmABnK+aIyhWRxditsNPt8A1LB1louXyYTxZlRvVhqVKyD/0b8Ttv40ER txSt3L/NxRr1afJDRaEmvV0AXWHO/FLoI1AtFnEFne/imvP/OvzTYVHFCHZ5kbn1ObvA Grw5Say6HjD5FPU1/Io47gwbAp5pt3/Ouoc5w=
Domainkey-signature: a=rsa-sha1; c=nofws;; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :cc:content-type:content-transfer-encoding; b=S5piBG/+BwGFOEH+GFTtaHLwIQMeHhzmZwYQsRHJO4h0LyMYl67gEYQKDk+IXgljsj UYcMcKEiHdRKm/ZR3DzhRAqvFGISaryRvUFDozMZIFcs22WvDXZmFwkjhMcHVryVbn+n xnGioH74YO7yhMe7lV0zexNQUqj98ghEEW5aQ=
In-reply-to: <>
Original-recipient: rfc822;
References: <> <> <> <> <> <> <>
On Wed, Mar 10, 2010 at 5:48 PM, Ralf Baechle <> wrote:
> 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 -

For things like fixed internal interrupts, sure.  But for pieces that depend
on where the (and which) chip is used, not so much.

By this logic there should be tons of ifdefs in the interrupt tables for every
in-tree board.

> 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.

I prefer a simpler solution.


<Prev in Thread] Current Thread [Next in Thread>