From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andreas Ruprecht Subject: Re: [PATCH] ASoC: rockchip: i2s: Add dependency to I2S device driver Date: Fri, 21 Nov 2014 20:42:54 +0100 Message-ID: <546F95BE.8080808@einserver.de> References: <1416594913-30187-1-git-send-email-rupran@einserver.de> <20141121185856.GJ3815@sirena.org.uk> Mime-Version: 1.0 Content-Type: text/plain; charset=windows-1252 Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <20141121185856.GJ3815@sirena.org.uk> Sender: linux-kernel-owner@vger.kernel.org To: Mark Brown Cc: Heiko Stuebner , Liam Girdwood , Jaroslav Kysela , Takashi Iwai , linux-arm-kernel@lists.infradead.org, linux-rockchip@lists.infradead.org, alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org List-Id: alsa-devel@alsa-project.org -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 21.11.2014 19:58, Mark Brown wrote: > On Fri, Nov 21, 2014 at 07:35:13PM +0100, Andreas Ruprecht wrote: >> Currently, CONFIG_SND_SOC_ROCKCHIP_I2S could also be selected >> without having CONFIG_SND_SOC_ROCKCHIP enabled. >> >> As this makes no sense (and breaks the build, when the option >> CONFIG_SND_SOC_GENERIC_DMAENGINE_PCM - which would be selected >> by CONFIG_SND_SOC_ROCKCHIP - is not enabled), a Kconfig >> dependency is added to CONFIG_SND_SOC_ROCKCHIP_I2S. This will >> make the item visible only if CONFIG_SND_SOC_ROCKCHIP is >> enabled. > > This doesn't seem like an obvious fix - why is the fix not to put > the select on the I2S driver (which is the thing that actually > depends on the dmaengine stuff)? > I didn't look a lot further into it, I just saw the build error report and from the other commits touching the Kconfig file it looked like there could be more options in the future. I will submit a v2 of the patch in a minute with the select for the dmaengine support moved to the I2S option. Regards, Andreas -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJUb5W9AAoJEJBoQA9qo224gp0P/0tSHqlwTn1CnB3KQR+sEdnN KzJwEZr5NbdXZyu6eBq0n7IS7MdmusW4BZrFkO2YNwyTPCb0kQPo8gD8IRXYYqdv Hlkx0k4X8lZrkIFQJak1w3ZNVds6pl81ljf33GAZ82cYJA4c/wPDBv6HVggXjlwc 0sVn4i+JuHXnvhTuiQPs1kPC6S0NGo69yow1xK2W2t46WCFqiI4b6ZSPqC3RP3CA 8wMpdzYzA9VXKpwuAkMzFy+uTBfiJFEkAQhhb8kEiOMQoOsIO5SAMMHce7HodFpd m3nWfIl+C8ZPtIWuJPKcA8BY+L58bjE3OneCLTK50WRQn36AFeLQC7DPij5zHCDJ lBxEpCP78dv/N058h9hpIloRzbESCjtQxhvyW3nkNbFLYUqKy1MW9yJkKDCzC1v2 yAYmxlo6JAQHzvZGOcy3RVX8Rar5JS7raNygPizLkQCPqrUfqhWUg6h2h32wKgUN CXRO1pf0Sz9Uaxjx6CBowwxJlIZN9EiVQpCPmV+BeIeIuRgsPZF3rDPuHHw+WwCp F2Oih8U5ytNBfVwQgcTEM1doYmaYQpFTSBDMIerTTxZKTZsEa+x1EyX3f2aievIW EPJgOM15cODsAepRh74PsofmPYe6aFL+lTSIVwPv351exK2ELJwfCUBzsk50Hu8z MQOsYmY33VXh2VemTlek =xmsJ -----END PGP SIGNATURE----- From mboxrd@z Thu Jan 1 00:00:00 1970 From: rupran@einserver.de (Andreas Ruprecht) Date: Fri, 21 Nov 2014 20:42:54 +0100 Subject: [PATCH] ASoC: rockchip: i2s: Add dependency to I2S device driver In-Reply-To: <20141121185856.GJ3815@sirena.org.uk> References: <1416594913-30187-1-git-send-email-rupran@einserver.de> <20141121185856.GJ3815@sirena.org.uk> Message-ID: <546F95BE.8080808@einserver.de> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org -----BEGIN PGP SIGNED MESSAGE----- Hash: SHA1 On 21.11.2014 19:58, Mark Brown wrote: > On Fri, Nov 21, 2014 at 07:35:13PM +0100, Andreas Ruprecht wrote: >> Currently, CONFIG_SND_SOC_ROCKCHIP_I2S could also be selected >> without having CONFIG_SND_SOC_ROCKCHIP enabled. >> >> As this makes no sense (and breaks the build, when the option >> CONFIG_SND_SOC_GENERIC_DMAENGINE_PCM - which would be selected >> by CONFIG_SND_SOC_ROCKCHIP - is not enabled), a Kconfig >> dependency is added to CONFIG_SND_SOC_ROCKCHIP_I2S. This will >> make the item visible only if CONFIG_SND_SOC_ROCKCHIP is >> enabled. > > This doesn't seem like an obvious fix - why is the fix not to put > the select on the I2S driver (which is the thing that actually > depends on the dmaengine stuff)? > I didn't look a lot further into it, I just saw the build error report and from the other commits touching the Kconfig file it looked like there could be more options in the future. I will submit a v2 of the patch in a minute with the select for the dmaengine support moved to the I2S option. Regards, Andreas -----BEGIN PGP SIGNATURE----- Version: GnuPG v1 iQIcBAEBAgAGBQJUb5W9AAoJEJBoQA9qo224gp0P/0tSHqlwTn1CnB3KQR+sEdnN KzJwEZr5NbdXZyu6eBq0n7IS7MdmusW4BZrFkO2YNwyTPCb0kQPo8gD8IRXYYqdv Hlkx0k4X8lZrkIFQJak1w3ZNVds6pl81ljf33GAZ82cYJA4c/wPDBv6HVggXjlwc 0sVn4i+JuHXnvhTuiQPs1kPC6S0NGo69yow1xK2W2t46WCFqiI4b6ZSPqC3RP3CA 8wMpdzYzA9VXKpwuAkMzFy+uTBfiJFEkAQhhb8kEiOMQoOsIO5SAMMHce7HodFpd m3nWfIl+C8ZPtIWuJPKcA8BY+L58bjE3OneCLTK50WRQn36AFeLQC7DPij5zHCDJ lBxEpCP78dv/N058h9hpIloRzbESCjtQxhvyW3nkNbFLYUqKy1MW9yJkKDCzC1v2 yAYmxlo6JAQHzvZGOcy3RVX8Rar5JS7raNygPizLkQCPqrUfqhWUg6h2h32wKgUN CXRO1pf0Sz9Uaxjx6CBowwxJlIZN9EiVQpCPmV+BeIeIuRgsPZF3rDPuHHw+WwCp F2Oih8U5ytNBfVwQgcTEM1doYmaYQpFTSBDMIerTTxZKTZsEa+x1EyX3f2aievIW EPJgOM15cODsAepRh74PsofmPYe6aFL+lTSIVwPv351exK2ELJwfCUBzsk50Hu8z MQOsYmY33VXh2VemTlek =xmsJ -----END PGP SIGNATURE-----