Re: [PATCH 2/2] soc: xilinx: Fix event manager CPU hotplug state cleanup

[Jay Buddhabhatti <jbuddhab@amd.com>]

Hi Prasanna,

On Mon, Nov 02, 2026 at 10:35:15PM -0800, Prasanna Kumar T S M wrote:
 > On driver removal, the event manager removes the CPUHP_AP_ONLINE_DYN
 > CPU hotplug state instead of the its actual state. This can leave the
 > event manager callbacks dangling. Use the actual CPU hotplug state
 > during driver removal.
 >
 > Hotplug registration failures are also ignored, allowing probe to
 > succeed without the callbacks needed to enable event interrupts on
 > CPUs brought online later.
 >
 > Fixes: c7fdb2404f66 ("drivers: soc: xilinx: add xilinx event 
management driver")
 > Signed-off-by: Prasanna Kumar T S M ptsm@linux.microsoft.com

This looks good to me.

A couple of small nits:

- "instead of the its actual state" -> "instead of its actual state"
- Minor alignment on the cpuhp_setup_state() continuation lines;
   checkpatch --strict complains.
	CHECK: Alignment should match open parenthesis
	#49: FILE: drivers/soc/xilinx/xlnx_event_manager.c:663:
	+       hp_state = cpuhp_setup_state(CPUHP_AP_ONLINE_DYN,
	+                       "soc/event:starting", xlnx_event_cpuhp_start,

	total: 0 errors, 0 warnings, 1 checks, 44 lines checked

Worth Cc: stable@vger.kernel.org as well, given the Fixes tag.

Reviewed-by: Jay Buddhabhatti <jay.buddhabhatti@amd.com>

Thanks,
Jay