diff mbox series

[1/9] ASoC: qcom: lpass-sc7280: Drop of_match_ptr for ID table

Message ID 20230310214333.274903-1-krzysztof.kozlowski@linaro.org
State New
Headers show
Series [1/9] ASoC: qcom: lpass-sc7280: Drop of_match_ptr for ID table | expand

Commit Message

Krzysztof Kozlowski March 10, 2023, 9:43 p.m. UTC
The driver is specific to ARCH_QCOM which depends on OF thus the driver
is OF-only.  Its of_device_id table is built unconditionally, thus
of_match_ptr() for ID table does not make sense.

Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
---
 sound/soc/qcom/lpass-sc7280.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Krzysztof Kozlowski March 10, 2023, 10:35 p.m. UTC | #1
On 10/03/2023 22:43, Krzysztof Kozlowski wrote:
> The Qualcomm SoC power-on driver is specific to ARCH_QCOM which depends
> on OF thus the driver is OF-only.  It's of_device_id table is built
> unconditionally, thus of_match_ptr() for ID table does not make sense.
> 
> Signed-off-by: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
> ---
>  drivers/power/reset/qcom-pon.c | 2 +-

This one is a duplicate - I already sent it separate. Please ignore.

Best regards,
Krzysztof
Mark Brown March 13, 2023, 7:07 p.m. UTC | #2
On Fri, 10 Mar 2023 22:43:24 +0100, Krzysztof Kozlowski wrote:
> The driver is specific to ARCH_QCOM which depends on OF thus the driver
> is OF-only.  Its of_device_id table is built unconditionally, thus
> of_match_ptr() for ID table does not make sense.
> 
> 

Applied to

   https://git.kernel.org/pub/scm/linux/kernel/git/broonie/sound.git for-next

Thanks!

[5/9] ASoC: codecs: adau1977-spi: Mark OF related data as maybe unused
      commit: a9048d64fb9605eb68c9912724e4a1346db8ba1a
[6/9] ASoC: codecs: pcm179x-spi: Mark OF related data as maybe unused
      commit: 413beba1a336e6068119f31a4441c51a7aeb1b55
[7/9] ASoC: codecs: rt1019: Mark OF related data as maybe unused
      commit: 0e8599a35823f05e90513918ff1201f8d210e315
[8/9] ASoC: codecs: src4xxx-i2c: Mark OF related data as maybe unused
      commit: 075a430db402eed0907b80493f1432d47a5120a7
[9/9] ASoC: codecs: zl38060: Mark OF related data as maybe unused
      commit: da1f22825b4fec7e25431fa5238252452f3dce46

All being well this means that it will be integrated into the linux-next
tree (usually sometime in the next 24 hours) and sent to Linus during
the next merge window (or sooner if it is a bug fix), however if
problems are discovered then the patch may be dropped or reverted.

You may get further e-mails resulting from automated or manual testing
and review of the tree, please engage with people reporting problems and
send followup patches addressing any issues that are reported if needed.

If any updates are required or you are submitting further changes they
should be sent as incremental updates against current git, existing
patches will not be replaced.

Please add any relevant lists and maintainers to the CCs when replying
to this mail.

Thanks,
Mark
diff mbox series

Patch

diff --git a/sound/soc/qcom/lpass-sc7280.c b/sound/soc/qcom/lpass-sc7280.c
index d43f480cbae3..956b974e322e 100644
--- a/sound/soc/qcom/lpass-sc7280.c
+++ b/sound/soc/qcom/lpass-sc7280.c
@@ -443,7 +443,7 @@  MODULE_DEVICE_TABLE(of, sc7280_lpass_cpu_device_id);
 static struct platform_driver sc7280_lpass_cpu_platform_driver = {
 	.driver = {
 		.name = "sc7280-lpass-cpu",
-		.of_match_table = of_match_ptr(sc7280_lpass_cpu_device_id),
+		.of_match_table = sc7280_lpass_cpu_device_id,
 		.pm = &sc7280_lpass_pm_ops,
 	},
 	.probe = asoc_qcom_lpass_cpu_platform_probe,