Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Wed, 3 Aug 2022 12:44:04 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Nathan Chancellor <nathan@kernel.org>
Cc:     Justin Stitt <justinstitt@google.com>,
        Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>,
        Liam Girdwood <lgirdwood@gmail.com>,
        Peter Ujfalusi <peter.ujfalusi@linux.intel.com>,
        Bard Liao <yung-chuan.liao@linux.intel.com>,
        Ranjani Sridharan <ranjani.sridharan@linux.intel.com>,
        Daniel Baluta <daniel.baluta@nxp.com>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>,
        Nick Desaulniers <ndesaulniers@google.com>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Tom Rix <trix@redhat.com>,
        sound-open-firmware@alsa-project.org, alsa-devel@alsa-project.org,
        linux-kernel@vger.kernel.org, llvm@lists.linux.dev, ardb@kernel.org
Subject: Re: [PATCH] soc: sof: fix clang -Wformat warnings
Message-ID: <YupfhNXObIt6yNqI@sirena.org.uk>
References: <20220721211218.4039288-1-justinstitt@google.com>
 <YumINAZ4WaM4rG7Q@dev-arch.thelio-3990X>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="dMPAc8CxvXVPL9lp"
Content-Disposition: inline
In-Reply-To: <YumINAZ4WaM4rG7Q@dev-arch.thelio-3990X>
Precedence: bulk


--dMPAc8CxvXVPL9lp
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline

On Tue, Aug 02, 2022 at 01:25:24PM -0700, Nathan Chancellor wrote:

> Is it too late for this patch to make 6.0? We are trying to enable
> -Wformat for clang in 6.0 and this instance of that warning was
> introduced this development cycle by commit 323aa1f093e6 ("ASoC: SOF:
> Add a new IPC op for parsing topology manifest"). If I am tracking all
> my patches correctly, this is the only instance of -Wformat that does
> not have a patch applied to a maintainer's tree so it would be really
> unfortunate if we could not sure it on for -rc1.

> We could probably route this via the Kbuild tree with an Ack along with
> the patch that enables -Wformat if it cannot go via Mark's or Takashi's
> ree.

We have a couple of months to get fixes into the next release so it's
not an emergency at this point.  If you want people to see things
promptly you really need to do things like send them with subject lines
that look like something that might be relevant for them to review
(generally, at least visually resemble how other commits in the area
look).  I don't know if I even opened this mail first time around
because based on the subject it looks like something for drivers/soc not
a subsystem I maintain.  Given that none of the SOF people responded
it's likely something similar applied there.

In any case, if you think something's been lost the content free pings
either here or on IRC aren't that helpful - they're not directly
actionable.  It is generally more helpful to resend, that way people
have the patch to hand and don't need to go looking for it.

--dMPAc8CxvXVPL9lp
Content-Type: application/pgp-signature; name="signature.asc"

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmLqX4MACgkQJNaLcl1U
h9CFOwf+JSow+bMD3gDkBQomWzJw8cGlWyTUcsXLdlE/CzXwQs5srMa5Or+NqyRW
/iU7VN7jq7imtSN6PHsdwgu9V5v6B3bK7aIO8aTICXnQhLrgjuRSeA0OMcIjCf7n
mpjg920Om3+BWhYMU9JK4sNilrA6cEeEK8FaZKPQu+bEpAzrtKGKRugbBc/C8Jbm
GULKrkZ01e5UUovr2h6va6ownqScAVgj58T25DyWcJMT3hiLouiHd/vdIFzSsLBS
uoxFUkJXQH3Gk+GnwtvmXmagvSlzvcIfQn4XzKWQP0wS12AV1VuZ8kF9YfOcBqF/
Xjm48O6XfeasYtW3gZV9hz8wX3rFNA==
=BSuP
-----END PGP SIGNATURE-----

--dMPAc8CxvXVPL9lp--