[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <b9b5b3c7-43ac-b856-6783-2c430fa098af@linux.intel.com>
Date: Thu, 14 Jan 2021 16:37:38 -0600
From: Pierre-Louis Bossart <pierre-louis.bossart@...ux.intel.com>
To: Arnd Bergmann <arnd@...nel.org>
Cc: ALSA Development Mailing List <alsa-devel@...a-project.org>,
Cezary Rojewski <cezary.rojewski@...el.com>,
Arnd Bergmann <arnd@...db.de>,
Kai Vehmanen <kai.vehmanen@...ux.intel.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
Takashi Iwai <tiwai@...e.com>,
Liam Girdwood <lgirdwood@...il.com>,
Ranjani Sridharan <ranjani.sridharan@...ux.intel.com>,
Mark Brown <broonie@...nel.org>,
Daniel Baluta <daniel.baluta@....com>,
Rander Wang <rander.wang@...ux.intel.com>,
sound-open-firmware@...a-project.org
Subject: Re: [Sound-open-firmware] [PATCH 1/2] [v2] ALSA: hda: fix
SND_INTEL_DSP_CONFIG dependency
On 1/14/21 3:19 PM, Arnd Bergmann wrote:
> On Thu, Jan 14, 2021 at 7:07 PM Pierre-Louis Bossart
> <pierre-louis.bossart@...ux.intel.com> wrote:
>> On 1/14/21 9:09 AM, Arnd Bergmann wrote:
>>> On Tue, Jan 12, 2021 at 9:32 PM Arnd Bergmann <arnd@...nel.org> wrote:
>>>> ---
>>>> v2: fix various build issues in the first version, now passes
>>>> all randconfig builds I tried
>>>
>>> Please disregard this version, while I have a tree that passes
>>> the randconfig builds now, this was not the patch from it that
>>> I wanted to send...
>>
>> No worries, I reworked the PCI case completely. Still running tests to
>> make sure there's no regression
>> https://github.com/thesofproject/linux/pull/2683
>
> Ok, I see. I had not realized that you already did the PCI bits as well,
> and applied my original patch to your tree, as I had meant to
> resend my "ASoC: SOF: ACPI: avoid reverse module dependency"
> patch after fixing some additional build failures in it. I have
> now applied the relevant commits from your branch to my
> randconfig tree, and will let you know if that finds anything more.
>
> From what I can tell so far, you already included the fixups that I had
> locally, and more.
Thanks Arnd.
Yes we added a couple of things. I missed the fact that we hard-coded
'sof-acpi' to determine if we use the legacy or SOF driver, and other
problems with driver names that I screwed-up.
The dependencies part should be finished now, I am just testing on my
side that nothing broke on the test devices (which is slower than I
wanted due to other ACPI scan things I had to revert locally). I will
add a couple of suggestions from Guennadi but this should be done tomorrow.
Powered by blists - more mailing lists