From mboxrd@z Thu Jan 1 00:00:00 1970 From: Bartlomiej Zolnierkiewicz Subject: Re: [PATCH 2/2] serial: samsung: Add missing checks for dma_map_single failure Date: Thu, 23 Mar 2017 12:59:18 +0100 Message-ID: <1721523.vQEeMhhi8C@amdc3058> References: <1490255131-9597-1-git-send-email-m.szyprowski@samsung.com> <1490255131-9597-2-git-send-email-m.szyprowski@samsung.com> <3019209.fgIhMIWD3x@amdc3058> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Transfer-Encoding: 7Bit Return-path: In-reply-to: <3019209.fgIhMIWD3x@amdc3058> Sender: stable-owner@vger.kernel.org To: Marek Szyprowski Cc: linux-samsung-soc@vger.kernel.org, linux-serial@vger.kernel.org, Sylwester Nawrocki , Krzysztof Kozlowski , Greg Kroah-Hartman , Seung-Woo Kim , Joonyoung Shim , Inki Dae , stable@vger.kernel.org List-Id: linux-serial@vger.kernel.org On Thursday, March 23, 2017 12:04:02 PM Bartlomiej Zolnierkiewicz wrote: > On Thursday, March 23, 2017 08:45:22 AM Marek Szyprowski wrote: > > This patch adds missing checks for dma_map_single() failure and proper error > > reporting. While touching this part of the code, it also removes unnecessary > > spinlock calls around dma_map_single() for TX buffer. This finally solves all > > the issues reported by DMA API debug framework. > > > > Reported-by: Seung-Woo Kim > > Fixes: 2c37eedb74c8 ("serial: samsung: add dma reqest/release functions") > > The right commit id is 62c37eedb74c8. BTW With this fixed you may add: Reviewed-by: Bartlomiej Zolnierkiewicz to both patches. > > CC: stable@vger.kernel.org # v4.10+ > > Signed-off-by: Marek Szyprowski Best regards,