From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mailout2.samsung.com ([203.254.224.25]:49589 "EHLO mailout2.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751593AbdCWL70 (ORCPT ); Thu, 23 Mar 2017 07:59:26 -0400 From: Bartlomiej Zolnierkiewicz 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 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> In-reply-to: <3019209.fgIhMIWD3x@amdc3058> MIME-version: 1.0 Content-transfer-encoding: 7Bit Content-type: text/plain; charset=us-ascii References: <1490255131-9597-1-git-send-email-m.szyprowski@samsung.com> <1490255131-9597-2-git-send-email-m.szyprowski@samsung.com> <3019209.fgIhMIWD3x@amdc3058> Sender: stable-owner@vger.kernel.org List-ID: 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, -- Bartlomiej Zolnierkiewicz Samsung R&D Institute Poland Samsung Electronics