Re: [PATCH] media: vsp1: mask interrupts before enabling

From: Nikita Yushchenko
Date: Mon Dec 13 2021 - 15:13:58 EST


Hi.

Now I'm finally looking at this again.

I think I would rather see the code to reset them done in
vsp1_reset_wpf(), rather than in probe directly as that is what we are
doing, and is I believe already in the call path.

Could you please explain, how that is intended to be called on the probe path?

As far as can read from the code, vsp1_reset_wpf() is only called from vsp1_device_init(), which in turn is called only from PM resume hook and only if vsp1->info is already set. However, in the probe path, pm_runtime_enable() is called before vsp1->info is set.

Nikita