Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.32 as permitted sender) client-ip=23.128.96.32;
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>,
        "conor+dt@kernel.org" <conor+dt@kernel.org>,
        "shengjiu.wang@gmail.com" <shengjiu.wang@gmail.com>,
        "Xiubo.Lee@gmail.com" <Xiubo.Lee@gmail.com>,
        "festevam@gmail.com" <festevam@gmail.com>,
        "nicoleotsuka@gmail.com" <nicoleotsuka@gmail.com>,
        "perex@perex.cz" <perex@perex.cz>,
        "tiwai@suse.com" <tiwai@suse.com>,
        "shawnguo@kernel.org" <shawnguo@kernel.org>,
        "s.hauer@pengutronix.de" <s.hauer@pengutronix.de>,
        "kernel@pengutronix.de" <kernel@pengutronix.de>,
        "alsa-devel@alsa-project.org" <alsa-devel@alsa-project.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>,
        "devicetree@vger.kernel.org" <devicetree@vger.kernel.org>,
        "linux-arm-kernel@lists.infradead.org" 
        <linux-arm-kernel@lists.infradead.org>
Subject: RE: Re: [PATCH v2 1/2] ASoC: dt-bindings: fsl_rpmsg: List DAPM
 endpoints ignoring system suspend
Thread-Topic: Re: [PATCH v2 1/2] ASoC: dt-bindings: fsl_rpmsg: List DAPM
 endpoints ignoring system suspend
Thread-Index: AQHZ8O/MzWXts0FyqkyLsnXLlx8p+w==
Date:   Wed, 27 Sep 2023 03:07:49 +0000
Message-ID: <DB9PR04MB94987AC750B4AB02DCBC44C8E3C2A@DB9PR04MB9498.eurprd04.prod.outlook.com>
References: <20230925110946.3156100-1-chancel.liu@nxp.com>
 <ZRF8KI11IVf6NzpL@finisterre.sirena.org.uk>
In-Reply-To: <ZRF8KI11IVf6NzpL@finisterre.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?1C0bhttnTRgcJK4Yw+96NZTFxOi8T2zN8aRB0c3VKWtC3yeyxwgSwDkopmB3?=
 =?us-ascii?Q?mUWBu7RiuhDo4v7z805gI5QuJD8CoP82ZKFYn3hFORJFMUHd3gt6wQLpwK71?=
 =?us-ascii?Q?JL/5iljaSibxgpPFZJAV+LZJym/n8iQhoK4LADvv+UCGmAD7V3oO1cM75/Cc?=
 =?us-ascii?Q?CIOKnF8sAP5D5aXuVODU/kZ+pwe2nTXYp2Xsu00oK3giOJAPYHQrbxJxsBk2?=
 =?us-ascii?Q?c/zEI0147Hv1JwYS/BFl9NQ16+nTUEHibrgcFqaHYxL4YkZ4j373QdULpqiJ?=
 =?us-ascii?Q?/LucIne/bL8TQ0T1xfdgBLXLzt7nW773KQX36rLQPQvj3JTq8kwjSpo/tlDq?=
 =?us-ascii?Q?T4+YPw+7vT3765nQq896Cj/pfoj6N0rrhJrpLZjVvYEy2+KsBBJbYl/deDdQ?=
 =?us-ascii?Q?bqwkY49oip/T+ZljM6FwXD0P7plnAhKotkqGz343WAoaIAPB/aceXsGf0WaL?=
 =?us-ascii?Q?W5AC721o/nXryT/d0FCB/n3RDBzJnLw1hGN6eMbgWDKbOlx8jYJFtgZz1P7f?=
 =?us-ascii?Q?EGyV37CoLdlQKEJF/YYmrzgGhvgGja6eCYJHp81voZRpezXzYy0hMJA6xylx?=
 =?us-ascii?Q?QBOXYajKwuO4LpTw3ZaGw/JwmWD3f3h9AF8USbsh+/l/8aUYAOxVkfYfendd?=
 =?us-ascii?Q?fojQx8x2GTe0VN9F2PVM01NQGqC2Q9YN2WSeZR3nBxrKZuqLHegIoe95G1K2?=
 =?us-ascii?Q?JTcNCOOQzam558IXnSNqB8FBIWo5YMFShz0L1rfZwSI97lL1XiMbc8yP2Xga?=
 =?us-ascii?Q?sd9dnmqdIdx3Fs0noZ7/Z6VAc4x65Wh7C+XX/WL5+YPs/bdwQNLmScdoK+Yh?=
 =?us-ascii?Q?36f3YrhnEgfLWk2jNDmuPYJ/FbgjLncsR84R8xZRMIv7pfEw6+iFDcaBihey?=
 =?us-ascii?Q?MBJ20vJXZOURjC25IOEuBZDXnr/u7rizHKVz5b/J30Em+z/LpAj56/LIITXn?=
 =?us-ascii?Q?zkaV7hWzDeRJNK2YIVHVDWo7xej/XFy9pbWMxT1cHjVBWpTK/4ZkSSPAL1mH?=
 =?us-ascii?Q?uEZ6RMxDbL88ViC2213rImR0NewiSCSfsy79UacT+vQv7ZJzl55EUVURlSZe?=
 =?us-ascii?Q?UNeRYvLebWDgaEGbKJmfAyyUd2dNpJbJJxEos+zIKBgQj+ExqRuWVmZ+pcN3?=
 =?us-ascii?Q?3Mc5kv5v6zrz3iwdBHYCLvn8BzZYf1wpg34uboA6Y3qdgXDVnLwdzrJpasok?=
 =?us-ascii?Q?lderqxgijebn6fHQch44+r9wlM+6sHyZ96n+k2UapgceHv3KnVNbt2fDzgmL?=
 =?us-ascii?Q?gKlHRcfpkS/K6BBBfANIR/3wG2YDb7wZpfL/sFAzOLPLbWTSkOGn/oV+m3Mg?=
 =?us-ascii?Q?3YbfDrU7a+aBmbWoMZRlWDpHpPElKYksn+qs4TGMjwn5KXa0rcoUqlyHO5r/?=
 =?us-ascii?Q?wmNVfT1ZBF8LbnhGPWL1k0pcd/InJgGVWsK85nNZjgU6/NWEUNfGlmk5Vufq?=
 =?us-ascii?Q?Zmu6/h+lIwXlzxm4D2K6d+ska30kEYUBBZm0sXooV/N4hhqx6e22JclFDCR6?=
 =?us-ascii?Q?1Xp/tnjcpqWCY5jm28i0xO6klEywAQFMbarYHqm4v4hZbDxChgZTlxbvC68y?=
 =?us-ascii?Q?U+5lVRQFtn0baSepdLFuKghtOsFgY92Hirn7btfC?=
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: DB9PR04MB9498.eurprd04.prod.outlook.com
X-MS-Exchange-CrossTenant-Network-Message-Id: acab4fa1-9e3e-407e-b29c-08dbbf06eea4
X-MS-Exchange-CrossTenant-originalarrivaltime: 27 Sep 2023 03:07:49.6976
 (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: S2M5oNtfCtM5n5M5CjOk4JC+XMBc4Cj2YI2dNznbwMs6gVpl6rEpxHkYb8G4nmLePnLrFEle0DpYwtj0cn1X4w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB9PR04MB10033
X-Spam-Status: No, score=-0.9 required=5.0 tests=DKIM_SIGNED,DKIM_VALID,
	DKIM_VALID_AU,HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,
	SPF_HELO_NONE,SPF_PASS autolearn=unavailable autolearn_force=no
	version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on agentk.vger.email
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
X-Greylist: Sender passed SPF test, not delayed by milter-greylist-4.6.4 (agentk.vger.email [0.0.0.0]); Tue, 26 Sep 2023 20:44:30 -0700 (PDT)

> > +  fsl,lpa-widgets:
> > +    $ref: /schemas/types.yaml#/definitions/non-unique-string-array
> > +    description: |
> > +      A list of DAPM endpoints which mark paths between these
> endpoints should
> > +      not be disabled when system enters in suspend state. LPA means l=
ow
power
> > +      audio case. On asymmetric multiprocessor, there are Cortex-A cor=
e
> and
> > +      Cortex-M core, Linux is running on Cortex-A core, RTOS or other =
OS is
> > +      running on Cortex-M core. The audio hardware devices can be
> controlled by
> > +      Cortex-M. LPA can be explained as a mechanism that Cortex-A
> allocates a
> > +      large buffer and fill audio data, then Cortex-A can enter into s=
uspend
> > +      for the purpose of power saving. Cortex-M continues to play the
> sound
> > +      during suspend phase of Cortex-A. When the data in buffer is
> consumed,
> > +      Cortex-M will trigger the Cortex-A to wakeup to fill data. LPA
> requires
> > +      some audio paths still enabled when Cortex-A enters into suspend=
.
>=20
> This is a fairly standard DSP playback case as far as I can see so it
> should work with DAPM without needing this obviously use case specific
> stuff peering into the Linux implementation.  Generally this is done by
> tagging endpoint widgets and DAIs as ignore_suspend, DAPM will then
> figure out the rest of the widgets in the path.

Yes, indeed I meant to let driver get DAPM endpoints from the "fsl,lpa-widg=
ets"
property and then set these endpoints as ignore_suspend if the sound card i=
s
running in this use case. Do you think the description for the use case can=
 be
simplified since it's a common use case?

Regards,=20
Chancel Liu