mbox series

[0/3] ASoC: sof: cleanup Kconfig files

Message ID 20201014025633.4879-1-rdunlap@infradead.org
Headers show
Series ASoC: sof: cleanup Kconfig files | expand

Message

Randy Dunlap Oct. 14, 2020, 2:56 a.m. UTC
Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Cc: Liam Girdwood <lgirdwood@gmail.com>
Cc: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
Cc: Kai Vehmanen <kai.vehmanen@linux.intel.com>
Cc: Daniel Baluta <daniel.baluta@nxp.com>
Cc: sound-open-firmware@alsa-project.org
Cc: alsa-devel@alsa-project.org

Some general editing of sound/soc/sof/ Kconfig files:

 [PATCH 1/3] ASoC: sof: imx: fix Kconfig punctuation
 [PATCH 2/3] ASoC: sof: intel: fix Kconfig punctuation and wording
 [PATCH 3/3] ASoC: sof: Kconfig: fix Kconfig punctuation and wording

 sound/soc/sof/Kconfig       |   42 +++++++++++------------
 sound/soc/sof/imx/Kconfig   |   10 ++---
 sound/soc/sof/intel/Kconfig |   62 +++++++++++++++++-----------------
 3 files changed, 57 insertions(+), 57 deletions(-)

Comments

Pierre-Louis Bossart Oct. 14, 2020, 2:37 p.m. UTC | #1
On 10/13/20 9:56 PM, Randy Dunlap wrote:
> Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
> Cc: Liam Girdwood <lgirdwood@gmail.com>
> Cc: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
> Cc: Kai Vehmanen <kai.vehmanen@linux.intel.com>
> Cc: Daniel Baluta <daniel.baluta@nxp.com>
> Cc: sound-open-firmware@alsa-project.org
> Cc: alsa-devel@alsa-project.org
> 
> Some general editing of sound/soc/sof/ Kconfig files:

Thanks Randy!

That looks all good to me, but I see you didn't CC: Mark Brown and 
Takashi Iwai so not sure if Mark can apply this directly. Might need to 
resend the series?

I also can take this series in the SOF tree and combine it with other 
changes I am making to Kconfigs to remove hard-coded exclusions, that 
would mean less churn on the same files.
>   [PATCH 1/3] ASoC: sof: imx: fix Kconfig punctuation
>   [PATCH 2/3] ASoC: sof: intel: fix Kconfig punctuation and wording
>   [PATCH 3/3] ASoC: sof: Kconfig: fix Kconfig punctuation and wording
> 
>   sound/soc/sof/Kconfig       |   42 +++++++++++------------
>   sound/soc/sof/imx/Kconfig   |   10 ++---
>   sound/soc/sof/intel/Kconfig |   62 +++++++++++++++++-----------------
>   3 files changed, 57 insertions(+), 57 deletions(-)
>
Randy Dunlap Oct. 14, 2020, 4:11 p.m. UTC | #2
On 10/14/20 7:37 AM, Pierre-Louis Bossart wrote:
> 
> 
> On 10/13/20 9:56 PM, Randy Dunlap wrote:
>> Cc: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
>> Cc: Liam Girdwood <lgirdwood@gmail.com>
>> Cc: Ranjani Sridharan <ranjani.sridharan@linux.intel.com>
>> Cc: Kai Vehmanen <kai.vehmanen@linux.intel.com>
>> Cc: Daniel Baluta <daniel.baluta@nxp.com>
>> Cc: sound-open-firmware@alsa-project.org
>> Cc: alsa-devel@alsa-project.org
>>
>> Some general editing of sound/soc/sof/ Kconfig files:
> 
> Thanks Randy!
> 
> That looks all good to me, but I see you didn't CC: Mark Brown and Takashi Iwai so not sure if Mark can apply this directly. Might need to resend the series?

Oops, my bad.

> I also can take this series in the SOF tree and combine it with other changes I am making to Kconfigs to remove hard-coded exclusions, that would mean less churn on the same files.

Please go ahead with this.

>>   [PATCH 1/3] ASoC: sof: imx: fix Kconfig punctuation
>>   [PATCH 2/3] ASoC: sof: intel: fix Kconfig punctuation and wording
>>   [PATCH 3/3] ASoC: sof: Kconfig: fix Kconfig punctuation and wording
>>
>>   sound/soc/sof/Kconfig       |   42 +++++++++++------------
>>   sound/soc/sof/imx/Kconfig   |   10 ++---
>>   sound/soc/sof/intel/Kconfig |   62 +++++++++++++++++-----------------
>>   3 files changed, 57 insertions(+), 57 deletions(-)
>>

thanks.