Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1757473Ab0AOM1i (ORCPT ); Fri, 15 Jan 2010 07:27:38 -0500 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1757091Ab0AOM1h (ORCPT ); Fri, 15 Jan 2010 07:27:37 -0500 Received: from faui40.informatik.uni-erlangen.de ([131.188.34.40]:60006 "EHLO faui40.informatik.uni-erlangen.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756884Ab0AOM1g (ORCPT ); Fri, 15 Jan 2010 07:27:36 -0500 Date: Fri, 15 Jan 2010 13:27:56 +0100 From: Christoph Egger To: linux-kernel@vger.kernel.org, linux-media@vger.kernel.org Cc: siccegge@stud.informatik.uni-erlangen.de, Reinhard.Tartler@informatik.uni-erlangen.de Subject: [PATCH] obsolete config in kernel source (FB_SOFT_CURSOR) Message-ID: <20100115122755.GC3321@faui49.informatik.uni-erlangen.de> MIME-Version: 1.0 Content-Type: multipart/mixed; boundary="adJ1OR3c6QgCpb/j" Content-Disposition: inline User-Agent: Mutt/1.5.20 (2009-06-14) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org --adJ1OR3c6QgCpb/j Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Hi all! As part of the VAMOS[0] research project at the University of Erlangen we're checking referential integrity between kernel KConfig options and in-code Conditional blocks. While probably not strictly a integrity violation (as FB_SOFT_CURSOR can still be set as it is once mentioned in a KConfig select statement this looks like a left-over of c465e05a03209651078b95686158648fd7ed84c5 Please keep me informed of this patch getting confirmed / merged so we can keep track of it. Regards Christoph Egger [0] http://vamos1.informatik.uni-erlangen.de/ --adJ1OR3c6QgCpb/j Content-Type: text/x-diff; charset=us-ascii Content-Disposition: attachment; filename="0001-Clean-missed-bit-of-FB_SOFT_CURSOR.patch" >From 5461c8d1ebc54e9f5c86233aa831cefc7c06a140 Mon Sep 17 00:00:00 2001 From: Christoph Egger Date: Fri, 15 Jan 2010 11:03:50 +0100 Subject: [PATCH 1/4] Clean missed bit of FB_SOFT_CURSOR While the config Option FB_SOFT_BUFFER was removed in c465e05a03209651078b95686158648fd7ed84c5 while moving to fbcon this single driver has it left as a select in KConfig / #ifdef in source. This last occurence is removed here so the option is really gone Signed-off-by: Christoph Egger --- drivers/video/Kconfig | 1 - drivers/video/sis/sis_main.c | 3 --- 2 files changed, 0 insertions(+), 4 deletions(-) diff --git a/drivers/video/Kconfig b/drivers/video/Kconfig index 5a5c303..7fe1839 100644 --- a/drivers/video/Kconfig +++ b/drivers/video/Kconfig @@ -1494,7 +1494,6 @@ config FB_VIA select FB_CFB_FILLRECT select FB_CFB_COPYAREA select FB_CFB_IMAGEBLIT - select FB_SOFT_CURSOR select I2C_ALGOBIT select I2C help diff --git a/drivers/video/sis/sis_main.c b/drivers/video/sis/sis_main.c index 9d2b6bc..a531a0f 100644 --- a/drivers/video/sis/sis_main.c +++ b/drivers/video/sis/sis_main.c @@ -1891,9 +1891,6 @@ static struct fb_ops sisfb_ops = { .fb_fillrect = fbcon_sis_fillrect, .fb_copyarea = fbcon_sis_copyarea, .fb_imageblit = cfb_imageblit, -#ifdef CONFIG_FB_SOFT_CURSOR - .fb_cursor = soft_cursor, -#endif .fb_sync = fbcon_sis_sync, #ifdef SIS_NEW_CONFIG_COMPAT .fb_compat_ioctl= sisfb_ioctl, -- 1.6.3.3 --adJ1OR3c6QgCpb/j-- -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/