All of lore.kernel.org
 help / color / mirror / Atom feed
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
Date: Mon, 18 Dec 2023 13:51:14 +0200	[thread overview]
Message-ID: <ZYAyMu9IuNDI69FP@smile.fi.intel.com> (raw)
In-Reply-To: <87ttokpyws.wl-tiwai@suse.de>

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



  parent reply	other threads:[~2023-12-18 11:52 UTC|newest]

Thread overview: 13+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-12-12 19:52 [PATCH 1/1] ALSA: hda: cs35l41: Dell Fiorano add missing _DSD properties Aleksandrs Vinarskis
2023-12-14 11:02 ` Takashi Iwai
2023-12-14 15:39   ` Stuart Henderson
2023-12-14 15:49     ` Takashi Iwai
2023-12-14 19:36       ` Aleksandrs Vinarskis
2023-12-18 11:51       ` Andy Shevchenko [this message]
2023-12-20  7:38       ` [PATCH v2 0/2] ALSA: hda: cs35l41: Support Dell XPS 9530 (2023) Aleksandrs Vinarskis
2023-12-20  7:38         ` [PATCH v2 1/2] ALSA: hda: cs35l41: Safety-guard against capped SPI speed Aleksandrs Vinarskis
2023-12-20 13:43           ` Stefan Binding
2023-12-21 11:24             ` Aleksandrs Vinarskis
2023-12-21 15:09               ` Stefan Binding
2023-12-20  7:38         ` [PATCH v2 2/2] ALSA: hda: cs35l41: Support Dell XPS 9530 (2023) Aleksandrs Vinarskis
2023-12-20 13:47           ` Stefan Binding

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=ZYAyMu9IuNDI69FP@smile.fi.intel.com \
    --to=andriy.shevchenko@intel.com \
    --cc=alex.vinarskis@gmail.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=david.rhodes@cirrus.com \
    --cc=james.schulman@cirrus.com \
    --cc=josbeir@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=patches@opensource.cirrus.com \
    --cc=perex@perex.cz \
    --cc=sbinding@opensource.cirrus.com \
    --cc=stuarth@opensource.cirrus.com \
    --cc=tiwai@suse.com \
    --cc=tiwai@suse.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.