Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S932936AbbENJxk (ORCPT ); Thu, 14 May 2015 05:53:40 -0400 Received: from mx4.wp.pl ([212.77.101.12]:35247 "EHLO mx4.wp.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S932301AbbENJxh (ORCPT ); Thu, 14 May 2015 05:53:37 -0400 Date: Thu, 14 May 2015 11:51:06 +0200 From: Jakub =?UTF-8?B?S2ljacWEc2tp?= To: ram kiran Cc: "linux-serial@vger.kernel.org" , "linux-kernel@vger.kernel.org" , "jslaby@suse.cz" , "gregkh@linuxfoundation.org" , "indrakanti.ram@gmail.com" Subject: Re: [PATCH v2] sc16is7xx: spi interface is added Message-ID: <20150514115106.41bfdbca@north> In-Reply-To: References: <20150513142112.32fb6781@north> <20150514100316.75e23c2f@north> X-Mailer: Claws Mail 3.11.1-85-ga87522 (GTK+ 2.24.27; x86_64-unknown-linux-gnu) MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-WP-AV: skaner antywirusowy poczty Wirtualnej Polski S. A. X-WP-SPAM: NO 0000000 [QXNU] Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1540 Lines: 32 On Thu, 14 May 2015 14:45:47 +0530, ram kiran wrote: > > I know little about kbuild but I'm worried that someone doing oldconfig > > can still get SERIAL_SC16IS7XX selected while saying no to all the > > others. > > > > Other option would be to swap the names between SERIAL_SC16IS7XX and > > SERIAL_SC16IS7XX_SELECT, oldconfig would run smoother. > I think, with the above, there would need a configuration change for sure. > It should be okay, as I2C is default Y. Exactly, but with what you proposed we need a configuration change as well, no? SERIAL_SC16IS7XX_SELECT is new so users would have to know that it's what SERIAL_SC16IS7XX used to be. > Swap names would need Makefile changes, i was just thinking to avoid this. > obj-$(CONFIG_SERIAL_SC16IS7XX) += sc16is7xx.o > would be > obj-$(CONFIG_SERIAL_SC16IS7XX_SELECT) += sc16is7xx.o > > I think its some that need not be there. Do suggest.. Perhaps *_SELECT is not the best name then but we could use something like *_CORE or *_BASE. Changes to the Makefile are not user-visible so no worries. It would be nice if people who run oldconfig by default got the same behaviour as they did so far (i2c if SC16IS7XX was enabled in previous config). I think with names swapped and modification of Makefile we would get exactly that. -- 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/