From:   Chancel Liu <chancel.liu@nxp.com>
To:     Mark Brown <broonie@kernel.org>
CC:     "lgirdwood@gmail.com" <lgirdwood@gmail.com>,
        "robh+dt@kernel.org" <robh+dt@kernel.org>,
        "krzysztof.kozlowski+dt@linaro.org" 
        <krzysztof.kozlowski+dt@linaro.org>,
        "perex@perex.cz" <perex@perex.cz>,
        "tiwai@suse.com" <tiwai@suse.com>,
        "ckeepax@opensource.cirrus.com" <ckeepax@opensource.cirrus.com>,
        "patches@opensource.cirrus.com" <patches@opensource.cirrus.com>,
        "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>
Subject: RE: Re: [PATCH 4/4] ASoC: wm8524: Delay some time to follow power up
 sequency
Thread-Topic: Re: [PATCH 4/4] ASoC: wm8524: Delay some time to follow power up
 sequency
Thread-Index: AQHZSD5wvadUyO/5A0CFUk73BVIm1A==
Date:   Fri, 24 Feb 2023 10:54:59 +0000
Message-ID: <VI1PR04MB42220617033D1DE404AF445DE3A89@VI1PR04MB4222.eurprd04.prod.outlook.com>
References: <20230222113945.3390672-1-chancel.liu@nxp.com>
 <20230222113945.3390672-4-chancel.liu@nxp.com>
 <Y/YPNMGMZcVZs3xy@sirena.org.uk>
In-Reply-To: <Y/YPNMGMZcVZs3xy@sirena.org.uk>
Accept-Language: en-US
Content-Language: en-US
x-ms-exchange-antispam-messagedata-chunkcount: 1
x-ms-exchange-antispam-messagedata-0: =?us-ascii?Q?d82zA44PhvIW4lgT0OgH5SMCfRwZ0gVBekFLiP+FUe/iS85h+1ZDXSbnKhmr?=
 =?us-ascii?Q?nHJ70pimQrKn6YadmNSUHLtCRvla+hZhwW127xNPYandVo1kJ31AswHwtshe?=
 =?us-ascii?Q?qp5Sz7xY0cLD/GPB3CoR8xIt93zVlPrQw5S6NPQ2J0rFjS4F488xDoV7ISr/?=
 =?us-ascii?Q?QseA+6qHdSXYL2RMeXA5AAxHE+Bg6WUM0UH/eIflvk6vhLR1kXrdV8VOrn86?=
 =?us-ascii?Q?4VwELcRB5FH61WI3uuHQ9DiFh3AwwGBYMd94+I3SPOQoncQTYXweragVF4W8?=
 =?us-ascii?Q?fpA58TJHiaUHRVzZ31FZD8Bw4sdqIWK1MXRJ8vinFdqWmTsbviA6wOKnqJ6+?=
 =?us-ascii?Q?+SV1s/DBKqSCy1sSDJnBGDp/32idyhJODG8tknvBPCJ5w9olQOinkz+ZeB0g?=
 =?us-ascii?Q?tm0cS6asH71jGV4B86/hG39tS7kZdHXCmgo9bBeseMhcpTRii3PptVIB/1g4?=
 =?us-ascii?Q?oR7jrpR3YmgeC1ZO3OMuw7WYZvjNrGZw13mc2zk4bdJaOQUFem8LeBh2EfI3?=
 =?us-ascii?Q?zknBeG0qVm7NI1ZZYrj1EPraKTVV5fWB2WW2YPcFH6ZkkGNnbKR0u8EeaxIj?=
 =?us-ascii?Q?GoSwXwahM68/Fxz4C/xsBSWmmIzhqKY02+h7JxsxtS6fclFXlsTONbuNKwin?=
 =?us-ascii?Q?wgcdq5sNygOeCmigl2f9l7poRL3wES5Dau5MEtDfRvrAViPcqS+ZB6YGuYAb?=
 =?us-ascii?Q?9GWlx0qAzHPuKGoUvylUKvEz95pLm1nuXBEAhKLgLiIZlmSrfIts2c4FiwYp?=
 =?us-ascii?Q?nanSVj5g44NiKGreC5X2JGJ7rUJYYzsgHQEDexfPevaWTu1yUFIgwB2les/u?=
 =?us-ascii?Q?DM/FYFMoec5HwIQjaEOKF9VQrPYEuU9WYWpIIrvUu5kMuY0TKwG+hQkja639?=
 =?us-ascii?Q?OlsptpNf7dlSmEFV4kuBvROOvtqqer0cl7eGdyRTTgnZa/bO5iXcJiHkJgSp?=
 =?us-ascii?Q?DakdkcPb8E1VHDGFn455eRF947PEun4ySICxt7WfsXnqwrkons9JdZ3YfIqq?=
 =?us-ascii?Q?bgedijSyu5RZn7mCtL3Asmy31AdIzWYtHuu8MtrEQ+S/6szrsgynzux+tQd9?=
 =?us-ascii?Q?3Y4lZqbT8S5cgzta2tH1d1F6P4mJT1h9LylbNGOf7VScytyOT+diiY7sNVRw?=
 =?us-ascii?Q?ZhxKvymoU0OZCu3y+NW8xiGrmWPgtncoSenSp6Kl9SC/0dd7NvVHGi9Hwv7H?=
 =?us-ascii?Q?6fPFDvk1vkgD+rscWr3EdIyArzfgD2ehrKUWoqdkCR5xQvtMRHVxk80F130g?=
 =?us-ascii?Q?CmvX6ihLONhj+NPsnM7dqQ4p89PHQAsQ3ODhUcFajp0IHcx67iuSpNpbiL73?=
 =?us-ascii?Q?wwaOnjZuOHtG/AgAK9hGmdMSAUc9Tg6k2nls87/2yP0mSYzkerrsZ/UBlG8J?=
 =?us-ascii?Q?0TzVLsMU0cgL+O/wLO8Tx9ATd2C6TTBQmFUxBZm6yIkXvz2HTbVRGVsx7H+l?=
 =?us-ascii?Q?Kj9WwA/A06uzuwWa1Z1wcUSpopHccI9MWJPzq/gzqsvEGNVCbO/vKEWdIqO/?=
 =?us-ascii?Q?cmSWb927R4qnxwQQ6W/537u0G6NxWetTc2Z+sZ69ohZIq6ewKyEx8iTRmeHC?=
 =?us-ascii?Q?OAa3GXPJPXNZRdqG53YZ9oCndOHMOWZ/tCraNgpk?=
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
MIME-Version: 1.0
X-OriginatorOrg: nxp.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-AuthSource: VI1PR04MB4222.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 0be954e0-7ce6-4c64-1a18-08db165592a9
X-MS-Exchange-CrossTenant-originalarrivaltime: 24 Feb 2023 10:54:59.1487
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 686ea1d3-bc2b-4c6f-a92c-d99c5c301635
X-MS-Exchange-CrossTenant-mailboxtype: HOSTED
X-MS-Exchange-CrossTenant-userprincipalname: Seh2L9/mMfs1Ov4MzsgMyNxaNwcqnQRpE4JyvcZqRCZZF5FcXxJ7z48PaZqeV0SDEWmWnslD7uOp1LoAoOK/Qg==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: PA4PR04MB9687
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

> On Wed, Feb 22, 2023 at 07:39:45PM +0800, Chancel Liu wrote:
>=20
> > +	switch (cmd) {
> > +	case SNDRV_PCM_TRIGGER_START:
> > +	case SNDRV_PCM_TRIGGER_RESUME:
> > +	case SNDRV_PCM_TRIGGER_PAUSE_RELEASE:
> > +		mdelay(wm8524->power_up_delay);
> > +		break;
>=20
> Doing a 100ms busy wait in atomic context does not seem like a great
> idea, never mind a 1.5s one.  This shouldn't be done in trigger, it
> needs to be done later - digital_mute() might be a better time to hook
> in, though longer delays like this are really quite bad.

Yes, such long time delay in driver is very bad. But this device requires
waiting some time before able to output audio. We have to wait otherwise th=
e
beginning data may be lost.

The power up to audio out timing occurs after MCLK, BCLK and MUTE=3D1 are r=
eady.
I added the delay in trigger() because some CPU DAI drivers enable BCLK in
trigger(). You suggested moving the delay to digital_mute(). It seems
digital_mute() is called before cpu_dai->trigger. Please correct me if I'm
wrong.

Regards,=20
Chancel Liu