From mboxrd@z Thu Jan 1 00:00:00 1970 From: Sergei Shtylyov Date: Sat, 08 Jun 2019 08:27:55 +0000 Subject: Re: [PATCH] sh: dma: Add missing IS_ERR test Message-Id: List-Id: References: <20190607115404.4557-1-embedded24@evers-fischer.de> In-Reply-To: <20190607115404.4557-1-embedded24@evers-fischer.de> MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: Rolf Evers-Fischer , ysato@users.sourceforge.jp, dalias@libc.org, linux-sh@vger.kernel.org, linux-kernel@vger.kernel.org Hello! On 07.06.2019 14:54, Rolf Evers-Fischer wrote: > get_dma_channel may return ERR_PTR, so a check is added. > > Signed-off-by: Rolf Evers-Fischer > --- > arch/sh/drivers/dma/dma-api.c | 20 +++++++++++++++++++- > arch/sh/drivers/dma/dma-sysfs.c | 2 +- > 2 files changed, 20 insertions(+), 2 deletions(-) > > diff --git a/arch/sh/drivers/dma/dma-api.c b/arch/sh/drivers/dma/dma-api.c > index ab9170494dcc..5d6f1a46cc5e 100644 > --- a/arch/sh/drivers/dma/dma-api.c > +++ b/arch/sh/drivers/dma/dma-api.c > @@ -94,7 +94,7 @@ int get_dma_residue(unsigned int chan) > struct dma_info *info = get_dma_info(chan); > struct dma_channel *channel = get_dma_channel(chan); > > - if (info->ops->get_residue) > + if (!IS_ERR(channel) && (info->ops->get_residue)) Extra parens not needed here. [...] MBR, Sergei