[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-Id: <1173253167.28531.41.camel@roc-desktop>
Date: Wed, 07 Mar 2007 15:39:27 +0800
From: "Wu, Bryan" <bryan.wu@...log.com>
To: Andrew Morton <akpm@...ux-foundation.org>
Cc: Jean Delvare <khali@...ux-fr.org>, Bryan Wu <bryan.wu@...log.com>,
Alexey Dobriyan <adobriyan@...il.com>,
Sonic Zhang <sonic.adi@...il.com>,
Mike Frysinger <vapier.adi@...il.com>,
linux-kernel@...r.kernel.org, mhfan@...c.edu
Subject: Re: [PATCH -mm] Blackfin: blackfin i2c driver
On Tue, 2007-03-06 at 23:14 -0800, Andrew Morton wrote:
> On Wed, 7 Mar 2007 07:58:22 +0100 Jean Delvare <khali@...ux-fr.org> wrote:
>
> > > +config BFIN_SDA
> >
> > I2C_BLACKFIN_SDA
>
> The blackfin architecture uses "bfin" pretty much universally, so this
> usage is consistent.
>
> box:/usr/src/25> grep -i blackfin patches/blackfin*|wc -l
> 1608
> box:/usr/src/25> grep -i bfin patches/blackfin*|wc -l
> 6198
>
Thanks for you understanding, but now we want to move to use
CONFIG_BLACKFIN options. There is a new task in our development plan to
change things to CONFIG_BLACKFIN.
At this moment, we both provide CONFIG_BFIN and CONFIG_BLACKFIN. When
all the things relied on CONFIG_BFIN/bfin are changed to
CONFIG_BLACKFIN/blackfin, the CONFIG_BFIN will be removed.
So here I will follow Jean's comments.
> Let's just hope nobody makes a bluefin.
So it is ok for both blackfin and bluefin. But I think Black is cooler
than Blue. -:)
>
> > > + range 0 15 if (BF533 || BF532 || BF531)
> >
> > Trailing whitespace.
>
> I always remove that when merging a patch.
Thanks a lot, could you please give me a script just to kill this
whitespace? So I can do it before sending you patches.
Thanks Jean and Andrew.
-Bryan
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists