From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753534AbcHPN5f (ORCPT ); Tue, 16 Aug 2016 09:57:35 -0400 Received: from mail-wm0-f45.google.com ([74.125.82.45]:35804 "EHLO mail-wm0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752223AbcHPN5d (ORCPT ); Tue, 16 Aug 2016 09:57:33 -0400 Subject: Re: [PATCH] clocksource: mips-gic-timer: make gic_clocksource_of_init return int To: Paul Gortmaker , linux-kernel@vger.kernel.org References: <20160801033546.26472-1-paul.gortmaker@windriver.com> Cc: linux-mips@linux-mips.org From: Daniel Lezcano Message-ID: <57B31BC7.8000401@linaro.org> Date: Tue, 16 Aug 2016 15:57:27 +0200 User-Agent: Mozilla/5.0 (X11; Linux i686; rv:38.0) Gecko/20100101 Thunderbird/38.5.1 MIME-Version: 1.0 In-Reply-To: <20160801033546.26472-1-paul.gortmaker@windriver.com> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 08/01/2016 05:35 AM, Paul Gortmaker wrote: > In commit d8152bf85d2c057fc39c3e20a4d623f524d9f09c: > ("clocksource/drivers/mips-gic-timer: Convert init function to return error") > > several return values were added to a void function resulting in: > > clocksource/mips-gic-timer.c: In function 'gic_clocksource_of_init': > clocksource/mips-gic-timer.c:175:3: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c:183:4: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c:190:3: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c:195:3: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c:200:3: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c:211:2: warning: 'return' with a value, in function returning void [enabled by default] > clocksource/mips-gic-timer.c: At top level: > clocksource/mips-gic-timer.c:213:1: warning: comparison of distinct pointer types lacks a cast [enabled by default] > clocksource/mips-gic-timer.c: In function 'gic_clocksource_of_init': > clocksource/mips-gic-timer.c:183:18: warning: ignoring return value of 'PTR_ERR', declared with attribute warn_unused_result [-Wunused-result] > > Given that the addition of the return values was intentional, it seems > that the conversion of the containing function from void to int was > simply overlooked. > > Cc: Daniel Lezcano > Cc: linux-mips@linux-mips.org > Signed-off-by: Paul Gortmaker > --- Hi Paul, Thomas is in vacation and I'm returning back. So I applied this patch as a fix. Thanks! -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog