Re: [PATCH v5 4/5] remoteproc: qcom: pas: Add late attach support for subsystems

[Shawn Guo <shengchao.guo@oss.qualcomm.com>]

On Tue, Apr 14, 2026 at 12:56:11PM +0800, Jingyi Wang wrote:
> 
> 
> On 4/11/2026 10:59 AM, Bjorn Andersson wrote:
> > On Thu, Apr 09, 2026 at 01:52:27AM -0700, Jingyi Wang wrote:
> > > Subsystems can be brought out of reset by entities such as bootloaders.
> > > As the irq enablement could be later than subsystem bring up, the state
> > > of subsystem should be checked by reading SMP2P bits and performing ping
> > > test.
> > > 
> 
> Hi Bjorn,
> 
> > 
> > I still don't understand.
> > 
> > Are you saying that devm_request_threaded_irq() will succeed and then
> > calling irq_get_irqchip_state() will not work? Or are you saying that
> > SMP2P driver isn't reliable and we're loosing the ready or fatal bits?
> > 
> 
> This says the ready state is getting from irq_get_irqchip_state()
> instead of q6v5_ready_interrupt(like what rproc start do)

Hi Jingyi,

I guess Bjorn is questioning "performing ping test" part. Is that really
required?

I'm testing this patch with Nord ADSP which is brought up by bootloader.
I have to remove the ping test to get the ADSP attached, as it appears
that Nord ADSP doesn't support pong. Are we good to just check ready bit
for telling whether a subsystem/dsp is already running and remoteproc
should attach to instead of powering up it?

May I ask you to copy me on the new posting of the series? I would like
to make sure it works for Nord ADSP as well. Thanks!

Shawn