Received-SPF: pass (google.com: domain of linux-kernel+bounces-3511-linux.lists.archive=gmail.com@vger.kernel.org designates 2604:1380:4601:e00::3 as permitted sender) client-ip=2604:1380:4601:e00::3;
Date: Mon, 18 Dec 2023 13:51:14 +0200
From: Andy Shevchenko <andriy.shevchenko@intel.com>
To: Takashi Iwai <tiwai@suse.de>
Cc: Stuart Henderson <stuarth@opensource.cirrus.com>,
	patches@opensource.cirrus.com,
	Aleksandrs Vinarskis <alex.vinarskis@gmail.com>, perex@perex.cz,
	tiwai@suse.com, sbinding@opensource.cirrus.com,
	james.schulman@cirrus.com, david.rhodes@cirrus.com,
	alsa-devel@alsa-project.org, linux-kernel@vger.kernel.org,
	Jasper Smet <josbeir@gmail.com>
Subject: Re: [PATCH 1/1] ALSA: hda: cs35l41: Dell Fiorano add missing _DSD
 properties
Message-ID: <ZYAyMu9IuNDI69FP@smile.fi.intel.com>
References: <20231212195243.10666-1-alex.vinarskis@gmail.com>
 <87fs05qc65.wl-tiwai@suse.de>
 <d98a3157-6e58-4dd9-a35a-759bcdcbe0c9@opensource.cirrus.com>
 <87ttokpyws.wl-tiwai@suse.de>
Precedence: bulk
MIME-Version: 1.0
Content-Type: text/plain; charset=iso-8859-1
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <87ttokpyws.wl-tiwai@suse.de>
Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo

On Thu, Dec 14, 2023 at 04:49:23PM +0100, Takashi Iwai wrote:
> On Thu, 14 Dec 2023 16:39:58 +0100,
> Stuart Henderson wrote:
> > On 14/12/2023 11:02, Takashi Iwai wrote:
> > > On Tue, 12 Dec 2023 20:52:43 +0100,
> > > Aleksandrs Vinarskis wrote:

...

> > > Can Cirrus team review this?

> > The patch looks sensible, with some minor comments below, however
> > we're just at the tail end of testing a patch chain that genericises a
> > lot of this code and adds support for a rather large batch of other
> > laptops with incomplete DSD.? We're hoping to push this upstream on
> > Monday.
> > 
> > Can I be awkward and ask that we hold off on this patch chain until
> > then?? Then we can add this laptop using the new approach.
> > 
> > If/when the chain is accepted, we will add support for a few Dell
> > laptops as well, including this one.
> 
> It's fine to wait for a while for me.
> Hopefully we can make it in 6.7, and we can catch up in the next
> week.
> 
> (BTW, I'll be off from the next Tuesday, and the reply will be
> delayed, but I can eventually check and merge patches remotely :)

Since v6.7 is going to be an LTS, I think we are eagerly want to have
one or the other to be included. In case Cirrus is slow with their
patch, I would like to see this one being included. Let's make an
xmas/ny gift to all users of this codec!

-- 
With Best Regards,
Andy Shevchenko