bryanodonoghue: in eb73facec2c2 I see you added `vfe->genpd_link = NULL` to `pm_domain_off`. this would be fine if `vfe_put` was the only function to call it since it keeps track of `power_count` and ensures every `pm_domain_off` is matched with a previous `pm_domain_on`, but `ispif_reset` skips that and calls `pm_domain_put` directly, then when `vfe_put` is called it calls `pm_domain_put` again causing null pointer dereference. https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=eb73facec2c261c92ad271e87ee37894e0bef9a8 https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/media/platform/qcom/camss/camss-vfe.c?id=c0297e7dd50795d559f3534887a6de1756b35d0f#n488 https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/drivers/media/platform/qcom/camss/camss-ispif.c?id=c0297e7dd50795d559f3534887a6de1756b35d0f#n346