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, 12 Jul 2023 12:40:07 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Mastan Katragadda <mastan.katragadda@amd.com>
Cc:     "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
        "Hiregoudar, Basavaraj" <Basavaraj.Hiregoudar@amd.com>,
        "Dommati, Sunil-kumar" <Sunil-kumar.Dommati@amd.com>,
        "Mukunda, Vijendar" <Vijendar.Mukunda@amd.com>,
        "kondaveeti, Arungopal" <Arungopal.kondaveeti@amd.com>,
        "Reddy, V sujith kumar" <Vsujithkumar.Reddy@amd.com>,
        "Prasad, Prasad" <venkataprasad.potturu@amd.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>,
        Kai Vehmanen <kai.vehmanen@linux.intel.com>,
        Jaroslav Kysela <perex@perex.cz>,
        Takashi Iwai <tiwai@suse.com>,
        Ajye Huang <ajye_huang@compal.corp-partner.google.com>,
        Ajit Kumar Pandey <AjitKumar.Pandey@amd.com>,
        Paul Olaru <paul.olaru@nxp.com>,
        "moderated list:SOUND - SOUND OPEN FIRMWARE (SOF) DRIVERS" 
        <sound-open-firmware@alsa-project.org>,
        open list <linux-kernel@vger.kernel.org>
Subject: Re: [PATCH 2/2] ASoC: SOF: amd: refactor PSP smn_read
Message-ID: <fa7e890b-f268-4b32-bb17-46b1e7a5cab5@sirena.org.uk>
References: <20230630070544.2167421-1-Mastan.Katragadda@amd.com>
 <20230630070544.2167421-2-Mastan.Katragadda@amd.com>
 <f3b7d7fd-2ce4-a579-0570-d55b04950140@amd.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="eU1hy69DHhto7nHm"
Content-Disposition: inline
In-Reply-To: <f3b7d7fd-2ce4-a579-0570-d55b04950140@amd.com>
Precedence: bulk


--eU1hy69DHhto7nHm
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jul 12, 2023 at 04:38:46PM +0530, Mastan Katragadda wrote:
>=20
> On 30/06/23 12:35, Mastan Katragadda wrote:
> > Use the read_poll_timeout marco for PSP smn_read calls.
> This patch has not been merged.

Please don't send content free pings and please allow a reasonable time
for review.  People get busy, go on holiday, attend conferences and so=20
on so unless there is some reason for urgency (like critical bug fixes)
please allow at least a couple of weeks for review.  If there have been
review comments then people may be waiting for those to be addressed.

Sending content free pings adds to the mail volume (if they are seen at
all) which is often the problem and since they can't be reviewed
directly if something has gone wrong you'll have to resend the patches
anyway, so sending again is generally a better approach though there are
some other maintainers who like them - if in doubt look at how patches
for the subsystem are normally handled.

> > Signed-off-by: Mastan Katragadda <Mastan.Katragadda@amd.com>

Please delete unneeded context from mails when replying.  Doing this
makes it much easier to find your reply in the message, helping ensure
it won't be missed by people scrolling through the irrelevant quoted
material.

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

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

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAmSukRYACgkQJNaLcl1U
h9Cngwf/dct2rAH3pitDj4DNfze5HGZm+l4oqJQLMAjQulG2irV1Dd5uJrRh5MN/
leAU5ohLd68XSdmn7LMt5Vmokj4DG7ugVO17pQmKw7RukJSqyYuMn1wTWDq5/MaF
ySY16FHhSJiQqMUsVPu3TIlJMeeqk5gJo+Zad0HSkH6O3bQzoSEQdHxXDn8op73M
cwOJ944vq85V2ZhGSnZ+txZbGBY1TpQLx1ppyabUR9/DT1GDcviTx72yOHqeMM/g
dn/YmFke3OlgQ5gSy5wjPk5nF3X3+65/w95MmOmKiZD5vZsTUz9UusUKRBSqloeC
eLiafzfNZluMy0rEAQqfCeESh4cSlQ==
=yt9v
-----END PGP SIGNATURE-----

--eU1hy69DHhto7nHm--