| Message ID | 20230913070702.12707-1-quic_bibekkum@quicinc.com (mailing list archive) |
|---|---|
| State | Not Applicable |
| Headers | show |
| Series | [v5] mtd: rawnand: qcom: Unmap the right resource upon probe failure | expand |
On Wed, 2023-09-13 at 07:07:02 UTC, Bibek Kumar Patro wrote: > We currently provide the physical address of the DMA region > rather than the output of dma_map_resource() which is obviously wrong. > > Fixes: 7330fc505af4 ("mtd: rawnand: qcom: stop using phys_to_dma()") > Cc: stable@vger.kernel.org > Reviewed-by: Manivannan Sadhasivam <mani@kernel.org> > Signed-off-by: Bibek Kumar Patro <quic_bibekkum@quicinc.com> Applied to https://git.kernel.org/pub/scm/linux/kernel/git/mtd/linux.git mtd/fixes, thanks. Miquel
diff --git a/drivers/mtd/nand/raw/qcom_nandc.c b/drivers/mtd/nand/raw/qcom_nandc.c index 64499c1b3603..b079605c84d3 100644 --- a/drivers/mtd/nand/raw/qcom_nandc.c +++ b/drivers/mtd/nand/raw/qcom_nandc.c @@ -3444,7 +3444,7 @@ static int qcom_nandc_probe(struct platform_device *pdev) err_aon_clk: clk_disable_unprepare(nandc->core_clk); err_core_clk: - dma_unmap_resource(dev, res->start, resource_size(res), + dma_unmap_resource(dev, nandc->base_dma, resource_size(res), DMA_BIDIRECTIONAL, 0); return ret; }
We currently provide the physical address of the DMA region rather than the output of dma_map_resource() which is obviously wrong. Fixes: 7330fc505af4 ("mtd: rawnand: qcom: stop using phys_to_dma()") Cc: stable@vger.kernel.org Reviewed-by: Manivannan Sadhasivam <mani@kernel.org> Signed-off-by: Bibek Kumar Patro <quic_bibekkum@quicinc.com> --- v5: Incorporated suggestions from Miquel/Mani - Added tag to automatically include this patch in stable tree. v4: Incorporated suggestion from Miquel - Modified title and commit description. https://lore.kernel.org/all/20230912115903.1007-1-quic_bibekkum@quicinc.com/ v3: Incorporated comments from Miquel - Modified the commit message and title as per suggestions. https://lore.kernel.org/all/20230912101814.7748-1-quic_bibekkum@quicinc.com/ v2: Incorporated comments from Pavan/Mani. https://lore.kernel.org/all/20230911133026.29868-1-quic_bibekkum@quicinc.com/ v1: https://lore.kernel.org/all/20230907092854.11408-1-quic_bibekkum@quicinc.com/ drivers/mtd/nand/raw/qcom_nandc.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) -- 2.17.1