From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752802AbcGYNMB (ORCPT ); Mon, 25 Jul 2016 09:12:01 -0400 Received: from mail-wm0-f47.google.com ([74.125.82.47]:37778 "EHLO mail-wm0-f47.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752579AbcGYNLz (ORCPT ); Mon, 25 Jul 2016 09:11:55 -0400 From: Daniel Lezcano Subject: Re: linux-next: build warning after merge of the tip tree To: Stephen Rothwell , Rich Felker References: <20160724153233.49f1574d@canb.auug.org.au> Cc: Thomas Gleixner , Ingo Molnar , "H. Peter Anvin" , Peter Zijlstra , linux-next@vger.kernel.org, linux-kernel@vger.kernel.org, Mark Brown , Marc Zyngier Message-ID: <57961014.7030902@linaro.org> Date: Mon, 25 Jul 2016 15:11:48 +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: 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 07/25/2016 09:16 AM, Thomas Gleixner wrote: > On Sun, 24 Jul 2016, Stephen Rothwell wrote: >> Hi all, >> >> After merging the tip tree, today's linux-next build (x86_64 allmodconfig) >> produced this warning: >> >> In file included from include/linux/clocksource.h:18:0, >> from include/linux/clockchips.h:13, >> from drivers/clocksource/jcore-pit.c:14: >> include/linux/of.h:1004:20: warning: comparison of distinct pointer types lacks a cast >> .data = (fn == (fn_type)NULL) ? fn : fn } >> ^ >> include/linux/of.h:1020:3: note: in expansion of macro '_OF_DECLARE' >> _OF_DECLARE(table, name, compat, fn, of_init_fn_1_ret) >> ^ >> include/linux/clocksource.h:247:2: note: in expansion of macro 'OF_DECLARE_1_RET' >> OF_DECLARE_1_RET(clksrc, name, compat, fn) >> ^ >> drivers/clocksource/jcore-pit.c:277:1: note: in expansion of macro 'CLOCKSOURCE_OF_DECLARE' >> CLOCKSOURCE_OF_DECLARE(jcore_pit, "jcore,pit", jcore_pit_init); >> ^ >> >> Introduced by commits >> >> b7c4db861683 ("clocksource/drivers/clksrc-probe: Introduce init functions with return code") >> 177cf6e52b0a ("clocksources: Switch back to the clksrc table") >> >> interacting with commit >> >> e0aa0655c60b ("clocksource: add J-Core timer/clocksource driver") >> >> from the sh tree. > > And why is that driver coming through the superh tree and not through the > clocksource maintainers? It's not only based on an old interface it's probably > unreviewed as well ... Rich, why are these changes in linux-next ? Except I am missing something, I don't see a new version sent for review on the mailing list. The interrupt controller driver is almost empty as stated by Marc Zyngier and there is no explanation / discussion about it. I don't know the goal of adding those patches in linux-next via your tree, may be you misunderstood how linux-next works and you should remove them. But if the purpose was to merge the patches, I remind you that being an arch maintainer does not give you the right to apply any patches, everywhere, at all cost, without review, because you want them in, you must follow the process, otherwise you take the risk to upset a lot of people and to be kicked out. -- Daniel -- Linaro.org │ Open source software for ARM SoCs Follow Linaro: Facebook | Twitter | Blog