public inbox for linux-kernel@vger.kernel.org
 help / color / mirror / Atom feed
From: "Shuming [范書銘]" <shumingf@realtek.com>
To: Charles Keepax <ckeepax@opensource.cirrus.com>,
	"broonie@kernel.org" <broonie@kernel.org>,
	"vkoul@kernel.org" <vkoul@kernel.org>,
	Oder Chiou <oder_chiou@realtek.com>
Cc: Jack Yu <jack.yu@realtek.com>,
	"lgirdwood@gmail.com" <lgirdwood@gmail.com>,
	"peter.ujfalusi@linux.intel.com" <peter.ujfalusi@linux.intel.com>,
	"yung-chuan.liao@linux.intel.com"
	<yung-chuan.liao@linux.intel.com>,
	"sanyog.r.kale@intel.com" <sanyog.r.kale@intel.com>,
	"pierre-louis.bossart@linux.dev" <pierre-louis.bossart@linux.dev>,
	"linux-sound@vger.kernel.org" <linux-sound@vger.kernel.org>,
	"linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
	"patches@opensource.cirrus.com" <patches@opensource.cirrus.com>,
	"Flove(HsinFu)" <flove@realtek.com>
Subject: RE: [PATCH 5/5] ASoC: rt722-sdca: Make use of new expanded MBQ regmap
Date: Wed, 18 Dec 2024 07:56:49 +0000	[thread overview]
Message-ID: <67b4ae369da2499da6f511671b61b13a@realtek.com> (raw)
In-Reply-To: <20241217135921.3572496-6-ckeepax@opensource.cirrus.com>

> Now the MBQ regmap implementation handles multiple sizes, this driver can
> combine its two register maps into one. So remove mbq_regmap and combine
> all the registers into regmap.
> 
> Also as rt722_sdca_adc_mux_get/put() only exist to access mbq_regmap,
> rather than doing any processing, these can now be dropped and the normal
> DAPM helpers used.
> 
> Signed-off-by: Charles Keepax <ckeepax@opensource.cirrus.com>
> ---
> 
> Note this patch is only build tested, as I do not have rt722 hardware, would be
> great is someone could test.
> 
> Thanks,
> Charles

Hi Charles,

Thanks for these patches to expand SdW MBQ regmap support.
We need to do more tests with rt722 before merging this change of rt722 codec driver.



  reply	other threads:[~2024-12-18  7:57 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-12-17 13:59 [PATCH 0/5] Expand SoundWire MBQ register map support Charles Keepax
2024-12-17 13:59 ` [PATCH 1/5] soundwire: SDCA: Add additional SDCA address macros Charles Keepax
2024-12-23  6:13   ` Vinod Koul
2024-12-17 13:59 ` [PATCH 2/5] ASoC: SDCA: Update list of entity_0 controls Charles Keepax
2024-12-17 13:59 ` [PATCH 3/5] regmap: sdw-mbq: Add support for further MBQ register sizes Charles Keepax
2024-12-17 13:59 ` [PATCH 4/5] regmap: sdw-mbq: Add support for SDCA deferred controls Charles Keepax
2024-12-17 13:59 ` [PATCH 5/5] ASoC: rt722-sdca: Make use of new expanded MBQ regmap Charles Keepax
2024-12-18  7:56   ` Shuming [范書銘] [this message]
2025-01-08 13:32 ` (subset) [PATCH 0/5] Expand SoundWire MBQ register map support Mark Brown

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=67b4ae369da2499da6f511671b61b13a@realtek.com \
    --to=shumingf@realtek.com \
    --cc=broonie@kernel.org \
    --cc=ckeepax@opensource.cirrus.com \
    --cc=flove@realtek.com \
    --cc=jack.yu@realtek.com \
    --cc=lgirdwood@gmail.com \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-sound@vger.kernel.org \
    --cc=oder_chiou@realtek.com \
    --cc=patches@opensource.cirrus.com \
    --cc=peter.ujfalusi@linux.intel.com \
    --cc=pierre-louis.bossart@linux.dev \
    --cc=sanyog.r.kale@intel.com \
    --cc=vkoul@kernel.org \
    --cc=yung-chuan.liao@linux.intel.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox