Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Mon, 28 Sep 2020 10:10:26 +0200
From:   Thierry Reding <thierry.reding@gmail.com>
To:     =?utf-8?B?TWljaGHFgiBNaXJvc8WCYXc=?= <mirq-linux@rere.qmqm.pl>
Cc:     Jaroslav Kysela <perex@perex.cz>, Takashi Iwai <tiwai@suse.com>,
        Jonathan Hunter <jonathanh@nvidia.com>,
        Stephen Warren <swarren@nvidia.com>,
        alsa-devel@alsa-project.org, linux-tegra@vger.kernel.org,
        linux-kernel@vger.kernel.org
Subject: Re: [PATCH] ASoC: tegra20-spdif: remove "default m"
Message-ID: <20200928081026.GH2837573@ulmo>
References: <ede103cf7f6914054a73cf8f1d9725ee13a7cf5d.1601149261.git.mirq-linux@rere.qmqm.pl>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha256;
        protocol="application/pgp-signature"; boundary="SUk9VBj82R8Xhb8H"
Content-Disposition: inline
In-Reply-To: <ede103cf7f6914054a73cf8f1d9725ee13a7cf5d.1601149261.git.mirq-linux@rere.qmqm.pl>
User-Agent: Mutt/1.14.7 (2020-08-29)
Precedence: bulk


--SUk9VBj82R8Xhb8H
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Sat, Sep 26, 2020 at 09:42:40PM +0200, Micha=C5=82 Miros=C5=82aw wrote:
> Make tegra20-spdif default to N as all other drivers do.
>=20
> Signed-off-by: Micha=C5=82 Miros=C5=82aw <mirq-linux@rere.qmqm.pl>
> Fixes: 774fec338bfc ("ASoC: Tegra: Implement SPDIF CPU DAI")

I don't think this is warranted. This doesn't fix a bug or anything.
It's merely a change in the default configuration. The presence of a
Fixes: tag is typically used as a hint for people to pick this up into
stable releases, but I don't think this qualifies.

> ---
>  sound/soc/tegra/Kconfig | 1 -
>  1 file changed, 1 deletion(-)
>=20
> diff --git a/sound/soc/tegra/Kconfig b/sound/soc/tegra/Kconfig
> index 3d91bd3e59cd..a62cc87551ac 100644
> --- a/sound/soc/tegra/Kconfig
> +++ b/sound/soc/tegra/Kconfig
> @@ -39,7 +39,6 @@ config SND_SOC_TEGRA20_I2S
>  config SND_SOC_TEGRA20_SPDIF
>  	tristate "Tegra20 SPDIF interface"
>  	depends on SND_SOC_TEGRA
> -	default m
>  	help
>  	  Say Y or M if you want to add support for the Tegra20 SPDIF interface.
>  	  You will also need to select the individual machine drivers to support

So now by default this driver will be disabled, which means that Linux
is going to regress for people that rely on this driver.

You need to at least follow this up with a patch that makes the
corresponding change in both tegra_defconfig and multi_v7_defconfig to
ensure that this driver is going to get built by default.

Given the above it's probably also a good idea to explain a bit more in
the commit message about what you're trying to achieve. Yes, "default n"
is usually the right thing to do and I'm honestly not sure why Stephen
chose to make this "default m" back in the day. Given that it depends on
SND_SOC_TEGRA, which itself is "default n", I think this makes some
sense, even if in retrospect it ended up being a bit inconsistent (you
could probably argue that all patches after this are the ones that were
inconsistent instead). This was merged over 9 years ago and a lot of
common practices have changed over that period of time.

Thierry

--SUk9VBj82R8Xhb8H
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQIzBAABCAAdFiEEiOrDCAFJzPfAjcif3SOs138+s6EFAl9xmm8ACgkQ3SOs138+
s6EzAw//csXks9215vvcBCKjXwZz3HXXMNsnJveUZK0fzDUjSr5oQeILZbN+gfxa
6pVNIU0ePDvtIh3814xSwzhSli5OlRq4Vo+y+zI2rer0L3BB6HpM80ekckhtKMJV
J8gKAxWtjmnzxizchU7LX+Lds/aeYS9xrqxEsninWvGZ7slMPbb7fxfGUyHvyG0n
ypjLyBsFyzDRfAo5ptW3LVW9W++aMoObYgcubxpdZ4EVThkf+/GUvUdWry69ZTYE
XmjwM2uq0B5edVuBYN9ezQflcoeDGIrD+ZR6yOtTeXViKxXqwZZuHir4dJ1i/upV
58cg3rYjAaAPyjxhMilJ35iOan0pGnVQJtjS64p9k9cCJBnADJs2fUcczzdXnP/O
tze1XSeQXbqr9XZKnODmNHwOajUoHVwLM6RN4bNwYdtMkpETN0afqQ9pX4X3WMD5
QJWXFSPc13LXeQwhOQNC+u5vfPaWYhne7/7WbLmd82IWiPABXeMwKINyzbv5ddUA
PWJo5TMOSpEopNzvDoWa8VYMN+QmPujxpiclBMK1aNkcWIa+hGGqWe+tyYun88e5
z7m9FSDqm0o2eizPl6zfNzUf35ihah/Q86k8eiULWQUR7kLNImS0n8fQGOR1s4dO
NLs6phyiJyLZOjYJ6GcCm0Vk/vCt2ifr+7DnT/okpWxlO9hDmAY=
=/ZoJ
-----END PGP SIGNATURE-----

--SUk9VBj82R8Xhb8H--