From mboxrd@z Thu Jan 1 00:00:00 1970 From: Antti Palosaari Date: Sun, 20 Jan 2013 18:07:55 +0000 Subject: Re: [PATCH] [media] use IS_ENABLED() macro Message-Id: <50FC327B.6050903@iki.fi> List-Id: References: <1358659976-8767-1-git-send-email-peter.senna@gmail.com> In-Reply-To: <1358659976-8767-1-git-send-email-peter.senna@gmail.com> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Peter Senna Tschudin Cc: mchehab@redhat.com, mkrufky@linuxtv.org, patricechotard@free.fr, kosio.dimitrov@gmail.com, liplianin@me.by, danny.kukawka@bisect.de, s.nawrocki@samsung.com, laurent.pinchart@ideasonboard.com, hans.verkuil@cisco.com, linux-media@vger.kernel.org, kernel-janitors@vger.kernel.org On 01/20/2013 07:32 AM, Peter Senna Tschudin wrote: > This patch introduces the use of IS_ENABLED() macro. For example, > replacing: > #if defined(CONFIG_I2C) || (defined(CONFIG_I2C_MODULE) && defined(MODULE)) > > with: > #if IS_ENABLED(CONFIG_I2C) > > All changes made by this patch respect the same replacement pattern. > > Reported-by: Mauro Carvalho Chehab > Signed-off-by: Peter Senna Tschudin For my drivers > drivers/media/tuners/qt1010.h | 2 +- > drivers/media/tuners/xc4000.h | 2 +- Acked-by: Antti Palosaari Why you didn't changed all those drivers as once? I was planning to do just similar change to all dvb drivers (I did it for DVB USB V2 remote controller already). I have also changed some of my new driver printk() calls from these headers to new pr/dev _foo() style. Please, could you fix it also :) regard Antti -- http://palosaari.fi/