[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20251204125938.3025022-1-andrea.tomassetti@sipearl.com>
Date: Thu, 4 Dec 2025 13:59:38 +0100
From: Andrea Tomassetti <andrea.tomassetti@...earl.com>
To: <sudeep.holla@....com>
CC: <andrea.tomassetti@...earl.com>, <jassisinghbrar@...il.com>,
<lenb@...nel.org>, <linux-acpi@...r.kernel.org>,
<linux-kernel@...r.kernel.org>, <lkp@...el.com>,
<oe-kbuild-all@...ts.linux.dev>, <olivier.dautricourt@...earl.com>,
<olivierdautricourt@...il.com>, <rafael@...nel.org>,
<thibault.cantori@...earl.com>
Subject: Re: Re: [PATCH v3 1/2] mailbox: pcc: support polling mode when there is no platform IRQ
On 25/12/03 10:28AM, Sudeep Holla wrote:
> On Tue, Dec 02, 2025 at 11:12:14AM +0100, Andrea Tomassetti wrote:
> > The goal is to allow clients to submit a message in both irq and polling
> > mode of the pcc mailbox. The ACPI specification does not require a
> > platform irq for pcc channels. Let's implement the case where it is not
> > available.
> >
>
> Just curious if you have a real use case for this polling mode on your
> platforms or ...
>
> > Tested-by: Thibault Cantori <thibault.cantori@...earl.com>
> > Co-developed-by: Olivier Dautricourt <olivier.dautricourt@...earl.com>
> > Signed-off-by: Olivier Dautricourt <olivier.dautricourt@...earl.com>
> > Signed-off-by: Andrea Tomassetti <andrea.tomassetti@...earl.com>
> > Reported-by: kernel test robot <lkp@...el.com>
> > Closes: https://lore.kernel.org/oe-kbuild-all/202511120243.soxAFpqQ-lkp@intel.com/
> > Closes: https://lore.kernel.org/oe-kbuild-all/202511120558.Cln7LF6M-lkp@intel.com/
> > Closes: https://lore.kernel.org/oe-kbuild-all/202511120729.R3XQNSnx-lkp@intel.com/
>
> You are just trying to fix these warnings. If it is latter, we don't have to
> add support for polling mode especially if it can't be tested on real
> platforms.
>
In our target product, we're still investigating if PCC-based SCMI communication will
rely on interrupts or polling. When we started looking into it we realized that polling
wasn't supported and that's why we decided to work on and send this patch. We thought it
could have been beneficial to other members of the community and it brings the driver a
bit closer to the ACPI specifications.
We're using ARM Fast Models for prototyping and that's how we validated and tested this patch.
Best regards,
Andrea
> --
> Regards,
> Sudeep
Powered by blists - more mailing lists