Re: [PATCH v2] mailbox: pcc: support polling mode when there is no platform IRQ

[Sudeep Holla <sudeep.holla@arm.com>]

On Wed, Nov 19, 2025 at 10:01:18AM +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.
> 
> peek_data is mapped to pcc_mbox_error_check_and_clear, so that it
> returns true if no error occurred while the platform processed last
> message, and therefore clients can fetch response data provided by the
> platform.
> 
> Tested-by: Thibault Cantori <thibault.cantori@sipearl.com>
> Co-developed-by: Olivier Dautricourt <olivier.dautricourt@sipearl.com>
> Signed-off-by: Olivier Dautricourt <olivier.dautricourt@sipearl.com>
> Signed-off-by: Andrea Tomassetti <andrea.tomassetti@sipearl.com>
> Reported-by: kernel test robot <lkp@intel.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/
> ---
> v2: Fix issues reported by the kernel test robot
>   - sparse: incorrect type in argument 2 (different address spaces)
>   - acpi_pcc.c:(.text+0x69): undefined reference to `__udivdi3'
> 
>  drivers/acpi/acpi_pcc.c | 78 ++++++++++++++++++++++++++++-------------
>  drivers/mailbox/pcc.c   | 12 ++++++-

Please separate the changes to the PCC OpRegion handler and the PCC mailbox
driver into individual patches. I’ve also done some cleanup work [1], which
has been tested and resolves part of the polling vs. IRQ flag initialization
issue.

I’d appreciate it if you could rebase your changes on top of those updates to
make the review process smoother and help us move forward more quickly.

-- 
Regards,
Sudeep

[1] https://lore.kernel.org/all/20251016-pcc_mb_updates-v1-0-0fba69616f69@arm.com/