Messages in this thread | | | Date | Thu, 29 Sep 2022 22:14:34 +0530 | From | Vinod Koul <> | Subject | Re: [PATCH] dmaengine: qcom-adm: fix wrong calling convention for prep_slave_sg |
| |
On 16-09-22, 06:12, Christian Marangi wrote: > The calling convention for pre_slave_sg is to return NULL on error and > provide an error log to the system. Qcom-adm instead provide error > pointer when an error occur. This indirectly cause kernel panic for > example for the nandc driver that checks only if the pointer returned by > device_prep_slave_sg is not NULL. Returning an error pointer makes nandc > think the device_prep_slave_sg function correctly completed and makes > the kernel panics later in the code. > > While nandc is the one that makes the kernel crash, it was pointed out > that the real problem is qcom-adm not following calling convention for > that function. > > To fix this, drop returning error pointer and return NULL with an error > log.
Applied, thanks
-- ~Vinod
| |