From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1756057AbbJGWTI (ORCPT ); Wed, 7 Oct 2015 18:19:08 -0400 Received: from v094114.home.net.pl ([79.96.170.134]:47296 "HELO v094114.home.net.pl" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with SMTP id S1752385AbbJGWTF (ORCPT ); Wed, 7 Oct 2015 18:19:05 -0400 From: "Rafael J. Wysocki" To: Stephen Rothwell Cc: Viresh Kumar , Greg KH , linux-next list , Linux Kernel Mailing List Subject: Re: linux-next: build warning after merge of the driver-core tree Date: Thu, 08 Oct 2015 00:47:36 +0200 Message-ID: <1846544.jnahGernn9@vostro.rjw.lan> User-Agent: KMail/4.11.5 (Linux/4.1.0-rc5+; KDE/4.11.5; x86_64; ; ) In-Reply-To: <20151008090427.61690eef@canb.auug.org.au> References: <20151007155146.41c4528f@canb.auug.org.au> <8672077.UYXmy4VzzX@vostro.rjw.lan> <20151008090427.61690eef@canb.auug.org.au> MIME-Version: 1.0 Content-Transfer-Encoding: 7Bit Content-Type: text/plain; charset="utf-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Thursday, October 08, 2015 09:04:27 AM Stephen Rothwell wrote: > Hi Rafael, > > On Wed, 07 Oct 2015 23:48:11 +0200 "Rafael J. Wysocki" wrote: > > > > On Wednesday, October 07, 2015 08:40:59 AM Viresh Kumar wrote: > > > > > > On 7 October 2015 at 06:51, Stephen Rothwell wrote: > > > > > > > > After merging the driver-core tree, today's linux-next build (arm > > > > multi_v7_defconfig) produced this warning: > > > > > > > > drivers/base/power/opp/debugfs.c: In function 'opp_debug_create_one': > > > > drivers/base/power/opp/debugfs.c:48:7: warning: passing argument 4 of 'debugfs_create_bool' from incompatible pointer type > > > > if (!debugfs_create_bool("available", S_IRUGO, d, > > > > ^ > > > > In file included from drivers/base/power/opp/debugfs.c:13:0: > > > > include/linux/debugfs.h:94:16: note: expected 'bool *' but argument is of type 'u32 *' > > > > struct dentry *debugfs_create_bool(const char *name, umode_t mode, > > > > ^ > > > > drivers/base/power/opp/debugfs.c:52:7: warning: passing argument 4 of 'debugfs_create_bool' from incompatible pointer type > > > > if (!debugfs_create_bool("dynamic", S_IRUGO, d, (u32 *)&opp->dynamic)) > > > > ^ > > > > In file included from drivers/base/power/opp/debugfs.c:13:0: > > > > include/linux/debugfs.h:94:16: note: expected 'bool *' but argument is of type 'u32 *' > > > > struct dentry *debugfs_create_bool(const char *name, umode_t mode, > > > > ^ > > > > drivers/base/power/opp/debugfs.c:55:7: warning: passing argument 4 of 'debugfs_create_bool' from incompatible pointer type > > > > if (!debugfs_create_bool("turbo", S_IRUGO, d, (u32 *)&opp->turbo)) > > > > ^ > > > > In file included from drivers/base/power/opp/debugfs.c:13:0: > > > > include/linux/debugfs.h:94:16: note: expected 'bool *' but argument is of type 'u32 *' > > > > struct dentry *debugfs_create_bool(const char *name, umode_t mode, > > > > ^ > > > > > > > > Introduced by commit > > > > > > > > 621a5f7ad9cd ("debugfs: Pass bool pointer to debugfs_create_bool()") > > > > > > > > interacting with commit > > > > > > > > 5cb5fdbf3877 ("PM / OPP: Add debugfs support") > > > > > > > > from the pm tree. > > > > > > That was expected and the PM tree needs to get updated to get it fixed. > > > And I will be doing that soon. > > > > For the record, I'm tired of your breaking things on a regular basis like this. > > > > I'm dropping 5cb5fdbf3877 from the PM tree. > > To be fair, this is caused by in interaction between your tree and > Greg's ... i.e. the API for debugfs_create_bool() was updated in Greg's > tree while a new use of it was added to your tree. It can only be > fixed by one of you merging the other's tree (or part of it) or by > someone giving me a merge resolution patch (or just a hint) and me > carrying that (and someone remembering to tell Linus about it. Well, the conflicting commits here were from the same developer which is kind of annoying. The new use in my tree can wait until the API update in the Greg's tree is merged IMO. That's why I've now dropped the commit that added it. :-) Thanks, Rafael