Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::3:5 as permitted sender) client-ip=2620:137:e000::3:5;
Date:   Fri, 20 Oct 2023 12:42:59 +1100
From:   Stephen Rothwell <sfr@canb.auug.org.au>
To:     Mark Brown <broonie@kernel.org>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Takashi Iwai <tiwai@suse.de>
Cc:     Maarten Lankhorst <maarten.lankhorst@linux.intel.com>,
        Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Linux Kernel Mailing List <linux-kernel@vger.kernel.org>,
        Linux Next Mailing List <linux-next@vger.kernel.org>
Subject: linux-next: build failure after merge of the sound-asoc tree
Message-ID: <20231020124259.7d77b077@canb.auug.org.au>
MIME-Version: 1.0
Content-Type: multipart/signed; boundary="Sig_/vp5RwhVEh75twrm4tNXaI7H";
 protocol="application/pgp-signature"; micalg=pgp-sha256
Precedence: bulk

--Sig_/vp5RwhVEh75twrm4tNXaI7H
Content-Type: text/plain; charset=US-ASCII
Content-Transfer-Encoding: quoted-printable

Hi all,

After merging the sound-asoc tree, today's linux-next build (x86_64
allmodconfig) failed like this:

sound/soc/sof/intel/hda.c: In function 'hda_dsp_remove':
sound/soc/sof/intel/hda.c:1384:16: error: 'return' with a value, in functio=
n returning void [-Werror=3Dreturn-type]
 1384 |         return 0;
      |                ^
sound/soc/sof/intel/hda.c:1330:6: note: declared here
 1330 | void hda_dsp_remove(struct snd_sof_dev *sdev)
      |      ^~~~~~~~~~~~~~
sound/soc/sof/intel/hda.c: In function 'hda_dsp_remove_late':
sound/soc/sof/intel/hda.c:1392:1: error: control reaches end of non-void fu=
nction [-Werror=3Dreturn-type]
 1392 | }
      | ^
cc1: all warnings being treated as errors

Caused by a bad automatic merge between commit

  e4d09de3919b ("ASoC: SOF: make .remove callback return void")

and commit

  3d1a05581361 ("ASoC: SOF: Intel: Move binding to display driver outside o=
f deferred probe")

from the sound tree.

I have applied the follwoing merge fix patch.

From: Stephen Rothwell <sfr@canb.auug.org.au>
Date: Fri, 20 Oct 2023 12:37:26 +1100
Subject: [PATCH] fix for bad merge of "ASoC: SOF: Intel: Move binding to
 display driver outside of deferred probe"

with "ASoC: SOF: make .remove callback return void"

Signed-off-by: Stephen Rothwell <sfr@canb.auug.org.au>
---
 sound/soc/sof/intel/hda.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/sound/soc/sof/intel/hda.c b/sound/soc/sof/intel/hda.c
index 2f12265c472b..8342fcf52f52 100644
--- a/sound/soc/sof/intel/hda.c
+++ b/sound/soc/sof/intel/hda.c
@@ -1380,8 +1380,6 @@ void hda_dsp_remove(struct snd_sof_dev *sdev)
=20
 	if (!sdev->dspless_mode_selected)
 		iounmap(sdev->bar[HDA_DSP_BAR]);
-
-	return 0;
 }
=20
 int hda_dsp_remove_late(struct snd_sof_dev *sdev)
@@ -1389,6 +1387,8 @@ int hda_dsp_remove_late(struct snd_sof_dev *sdev)
 	iounmap(sof_to_bus(sdev)->remap_addr);
 	sof_hda_bus_exit(sdev);
 	hda_codec_i915_exit(sdev);
+
+	return 0;
 }
=20
 int hda_power_down_dsp(struct snd_sof_dev *sdev)
--=20
2.40.1

--=20
Cheers,
Stephen Rothwell

--Sig_/vp5RwhVEh75twrm4tNXaI7H
Content-Type: application/pgp-signature
Content-Description: OpenPGP digital signature

-----BEGIN PGP SIGNATURE-----

iQEzBAEBCAAdFiEENIC96giZ81tWdLgKAVBC80lX0GwFAmUx2yMACgkQAVBC80lX
0GyE8Qf+KPpRGYyXxqxsyyp51ZIxJDlXBy+18P4irT6UVAVo//OKygfM7Rje8M21
L2rgJqKRR7eqzZY9PJ06pOVjYoimLIhl4Jl8Trmbmq3ImKm3M4lZGXoIKnp9XSxH
2XogZuYPZkYOzcQG9vbdNXq0pQWi9kdVPLZYHXG6ge9sxSiLE4FmblGcpW9A/yQq
OpiTv1z75mCg9h52EV7F5uqY695ikiFnrozRm3LzhPHEr/U6/tbvXssnQkMqWV1f
GaZuu4sdLbHDIE5pqCw2Kh+qDBIvqbP82/vw2Ytqy/iw3oIEwIB1yVIRBRvqBbDa
zZIBxSvrmmM3YITpGGqLtcXzr2N2sA==
=wz9p
-----END PGP SIGNATURE-----

--Sig_/vp5RwhVEh75twrm4tNXaI7H--