Re: [PATCH] blackfin: bf537: fix typo "CONFIG_SND_SOC_ADV80X_MODULE"

2014-03-04 Thread Steven Miao
Hi Paul, On Sat, Mar 1, 2014 at 5:13 AM, Paul Bolle wrote: > Steven, > > On Wed, 2014-02-26 at 18:35 +0800, Steven Miao wrote: >> On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle wrote: >> >> 1) There are many lines that might be converted to IS_ENABLED() in this >> >> file. I'm not sure if and how

Re: [PATCH] blackfin: bf537: fix typo CONFIG_SND_SOC_ADV80X_MODULE

2014-03-04 Thread Steven Miao
Hi Paul, On Sat, Mar 1, 2014 at 5:13 AM, Paul Bolle pebo...@tiscali.nl wrote: Steven, On Wed, 2014-02-26 at 18:35 +0800, Steven Miao wrote: On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle pebo...@tiscali.nl wrote: 1) There are many lines that might be converted to IS_ENABLED() in this file.

Re: [PATCH] blackfin: bf537: fix typo "CONFIG_SND_SOC_ADV80X_MODULE"

2014-02-28 Thread Paul Bolle
Steven, On Wed, 2014-02-26 at 18:35 +0800, Steven Miao wrote: > On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle wrote: > >> 1) There are many lines that might be converted to IS_ENABLED() in this > >> file. I'm not sure if and how that should be done. > Sorry for the late reply. There are many

Re: [PATCH] blackfin: bf537: fix typo CONFIG_SND_SOC_ADV80X_MODULE

2014-02-28 Thread Paul Bolle
Steven, On Wed, 2014-02-26 at 18:35 +0800, Steven Miao wrote: On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle pebo...@tiscali.nl wrote: 1) There are many lines that might be converted to IS_ENABLED() in this file. I'm not sure if and how that should be done. Sorry for the late reply. There are

Re: [PATCH] blackfin: bf537: fix typo "CONFIG_SND_SOC_ADV80X_MODULE"

2014-02-26 Thread Steven Miao
Hi Paul, On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle wrote: > [Replaced previous maintainer and list with current maintainer and > list.] > > On Fri, 2013-03-08 at 13:06 +0100, Paul Bolle wrote: >> There's a (rather subtle) typo in "CONFIG_SND_SOC_ADV80X_MODULE". Fix it >> once and for all by

Re: [PATCH] blackfin: bf537: fix typo CONFIG_SND_SOC_ADV80X_MODULE

2014-02-26 Thread Steven Miao
Hi Paul, On Thu, Feb 13, 2014 at 5:40 PM, Paul Bolle pebo...@tiscali.nl wrote: [Replaced previous maintainer and list with current maintainer and list.] On Fri, 2013-03-08 at 13:06 +0100, Paul Bolle wrote: There's a (rather subtle) typo in CONFIG_SND_SOC_ADV80X_MODULE. Fix it once and for

Re: [PATCH] blackfin: bf537: fix typo "CONFIG_SND_SOC_ADV80X_MODULE"

2014-02-13 Thread Paul Bolle
[Replaced previous maintainer and list with current maintainer and list.] On Fri, 2013-03-08 at 13:06 +0100, Paul Bolle wrote: > There's a (rather subtle) typo in "CONFIG_SND_SOC_ADV80X_MODULE". Fix it > once and for all by using IS_ENABLED(), which is designed to avoid > issues like this. > >

Re: [PATCH] blackfin: bf537: fix typo CONFIG_SND_SOC_ADV80X_MODULE

2014-02-13 Thread Paul Bolle
[Replaced previous maintainer and list with current maintainer and list.] On Fri, 2013-03-08 at 13:06 +0100, Paul Bolle wrote: There's a (rather subtle) typo in CONFIG_SND_SOC_ADV80X_MODULE. Fix it once and for all by using IS_ENABLED(), which is designed to avoid issues like this.

[PATCH] blackfin: bf537: fix typo "CONFIG_SND_SOC_ADV80X_MODULE"

2013-03-08 Thread Paul Bolle
There's a (rather subtle) typo in "CONFIG_SND_SOC_ADV80X_MODULE". Fix it once and for all by using IS_ENABLED(), which is designed to avoid issues like this. Signed-off-by: Paul Bolle --- 0) Untested! And this needs build- and runtime testing, especially for the MODULE case! 1) There are many

[PATCH] blackfin: bf537: fix typo CONFIG_SND_SOC_ADV80X_MODULE

2013-03-08 Thread Paul Bolle
There's a (rather subtle) typo in CONFIG_SND_SOC_ADV80X_MODULE. Fix it once and for all by using IS_ENABLED(), which is designed to avoid issues like this. Signed-off-by: Paul Bolle pebo...@tiscali.nl --- 0) Untested! And this needs build- and runtime testing, especially for the MODULE case! 1)