Received: by 2002:a25:86ce:0:0:0:0:0 with SMTP id y14csp855761ybm; Tue, 21 May 2019 04:51:37 -0700 (PDT) X-Google-Smtp-Source: APXvYqwcz97yKKJW7tqWYtrpfyZ1hiulqo8kTjEfv8lcg66QfZRVLnO1FQVCOPAKCHoXb2jb8wXB X-Received: by 2002:a17:902:4503:: with SMTP id m3mr80776781pld.97.1558439497677; Tue, 21 May 2019 04:51:37 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1558439497; cv=none; d=google.com; s=arc-20160816; b=LaG9zF5+Qqk/VphqrObUp7MIcZ7347Xmj0GJ0SR/fq8KrRaitgKhWn2EC/yjXpI8Eb 9r8dTfUoTgrGn2pQFmJH1Q1AhODzEANXUX8Al0W34bha9GK6Oa6JT6CwcnjEAJmn5y0x o4suyxRlQ8VO6TD0ozuVvez9zTkiLPQ5WFOhIVPx13MXLUIJu6R1OolLR0ov843ywmeS H3xcaeJofDjrTbhMfT2WAgAvjkPR5ob9bTDse2+HlZMl7TZcWbRV+9dbO7alXrzaT0wX cQzcqtkVfNSyLUAsfb/liaEzVQ/8IPGiDeG/GlFbcH5PkMjRclKYfglc7SPvYZuWJXvj L+nA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:sender:mime-version:message-id:date:references :in-reply-to:subject:cc:to:from; bh=kmxs9rp1vT+MNyGf3Zq2ZXZWQwj9d4xQllp3np5p5UQ=; b=uaSaei2mj2wwgEloTnhLA3iy+FgRD5QIG6w4VTtcq0PRq/QrFGInT9oB4lmORB33a8 DNqyybfQzlr71yi4/LrQ9ZFFuJMnmkb2oibVEU9+pNTaJUsuDi13SUsdjDFMO8NPHTKC MdG8p0clDntigA/ZUO+RaNpZWygw2aHD0NGPST21ZRd79m9+UZNDgMTgae0minxk0tGH pcrXw3CDjDE0tB30FOiRtgR0DKilnL+qyJgww8PFDpr4WOc9tjIowbxuICFNBCnCrGuc UPeHqWYK89dn6XMFEBOkibEc3WaWNO4CVGgsPD4h1c45bO0N0LGvSRyD0KyxxWkVwsBa 0nMg== ARC-Authentication-Results: i=1; mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Return-Path: Received: from vger.kernel.org (vger.kernel.org. [209.132.180.67]) by mx.google.com with ESMTP id f13si13861092pgs.90.2019.05.21.04.51.22; Tue, 21 May 2019 04:51:37 -0700 (PDT) Received-SPF: pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67; Authentication-Results: mx.google.com; spf=pass (google.com: best guess record for domain of linux-kernel-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1727999AbfEULto (ORCPT + 99 others); Tue, 21 May 2019 07:49:44 -0400 Received: from bilbo.ozlabs.org ([203.11.71.1]:47623 "EHLO ozlabs.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726740AbfEULto (ORCPT ); Tue, 21 May 2019 07:49:44 -0400 Received: from authenticated.ozlabs.org (localhost [127.0.0.1]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange ECDHE (P-256) server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by mail.ozlabs.org (Postfix) with ESMTPSA id 457Yw527Kzz9s7h; Tue, 21 May 2019 21:49:40 +1000 (AEST) From: Michael Ellerman To: Bartlomiej Zolnierkiewicz , Arnd Bergmann , Greg Kroah-Hartman Cc: Andrew Donnellan , Eric Piel , Frank Haverkamp , Frederic Barrat , =?utf-8?Q?Micha=C5=82_Miros?= =?utf-8?Q?=C5=82aw?= , linux-kernel@vger.kernel.org, linuxppc-dev@lists.ozlabs.org Subject: Re: [PATCH] misc: remove redundant 'default n' from Kconfig-s In-Reply-To: <1ab818ae-4d9f-d17a-f11f-7caaa5bf98bc@samsung.com> References: <1ab818ae-4d9f-d17a-f11f-7caaa5bf98bc@samsung.com> Date: Tue, 21 May 2019 21:49:38 +1000 Message-ID: <87pnoc6n8t.fsf@concordia.ellerman.id.au> MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Bartlomiej Zolnierkiewicz writes: > 'default n' is the default value for any bool or tristate Kconfig > setting so there is no need to write it explicitly. > > Also since commit f467c5640c29 ("kconfig: only write '# CONFIG_FOO > is not set' for visible symbols") the Kconfig behavior is the same > regardless of 'default n' being present or not: > > ... > One side effect of (and the main motivation for) this change is making > the following two definitions behave exactly the same: > > config FOO > bool > > config FOO > bool > default n > > With this change, neither of these will generate a > '# CONFIG_FOO is not set' line (assuming FOO isn't selected/implied). > That might make it clearer to people that a bare 'default n' is > redundant. > ... > > Signed-off-by: Bartlomiej Zolnierkiewicz ... > Index: b/drivers/misc/ocxl/Kconfig > =================================================================== > --- a/drivers/misc/ocxl/Kconfig > +++ b/drivers/misc/ocxl/Kconfig > @@ -4,7 +4,6 @@ > > config OCXL_BASE > bool > - default n > select PPC_COPRO_BASE > > config OCXL Acked-by: Michael Ellerman (ocxl) cheers