From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id D6FB3173 for ; Wed, 12 Jan 2022 20:45:51 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1642020352; x=1673556352; h=date:from:to:cc:subject:message-id:references: mime-version:in-reply-to; bh=3bd0+kvppXqt1g9WkYwL2BkeZvx2WQ/v4buHeuWib6I=; b=YHaxIoM+ZPFfbjShYZ4IGN4DQoJpacf1SkctzY2MAox0Tzy5AJGTEdoq NpMIB5FZXNdNbMwFmtkVoYjGUD6ivAtKuXI0F/AmO8ER1QzRSDxMzWmmb jlObdp2bdYu0rtk7ponhLtFrkg0Esh/IIXc47Wdya4U37xQ/sQFa5VVa5 oPIaWKpmcJApNVN6vm7wCg3/akAwjlxBN3ye51CZhVO3VfFO6736SN6bR uxVrMzFyW5DVyYKAzsxME2NJVGRftlmmYYYfeMJ+DOXxa2uzIr0KZ42ei 3F6waERrcvZtrAsuJ7B25pegbomQHZ00vrbKsbx9fzdcDozTWqzrzdIUK Q==; X-IronPort-AV: E=McAfee;i="6200,9189,10225"; a="241406501" X-IronPort-AV: E=Sophos;i="5.88,284,1635231600"; d="scan'208";a="241406501" Received: from orsmga005.jf.intel.com ([10.7.209.41]) by fmsmga102.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Jan 2022 12:45:50 -0800 X-IronPort-AV: E=Sophos;i="5.88,284,1635231600"; d="scan'208";a="691533858" Received: from smile.fi.intel.com ([10.237.72.61]) by orsmga005-auth.jf.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 12 Jan 2022 12:45:47 -0800 Received: from andy by smile.fi.intel.com with local (Exim 4.95) (envelope-from ) id 1n7kTx-009vDY-L5; Wed, 12 Jan 2022 22:44:33 +0200 Date: Wed, 12 Jan 2022 22:44:33 +0200 From: Andy Shevchenko To: Hans de Goede Cc: kernel test robot , Mika Westerberg , Andy Shevchenko , Bartosz Golaszewski , Linus Walleij , llvm@lists.linux.dev, kbuild-all@lists.01.org, linux-gpio@vger.kernel.org Subject: Re: [PATCH v3] pinctrl: baytrail: Clear direct_irq_en flag on broken configs Message-ID: References: <20220107234456.148389-1-hdegoede@redhat.com> <202201090203.kgCw6bSd-lkp@intel.com> Precedence: bulk X-Mailing-List: llvm@lists.linux.dev List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo On Wed, Jan 12, 2022 at 08:58:00PM +0100, Hans de Goede wrote: > On 1/8/22 19:54, kernel test robot wrote: > >>> drivers/pinctrl/intel/pinctrl-baytrail.c:1483:58: warning: format specifies type 'long' but the argument has type 'int' [-Wformat] > > dev_dbg(vg->dev, "Pin %i: uses direct IRQ %ld\n", pin, match - direct_irq); > > ~~~ ^~~~~~~~~~~~~~~~~~ > > %d > > include/linux/dev_printk.h:163:47: note: expanded from macro 'dev_dbg' > > dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARGS__); \ > > ~~~ ^~~~~~~~~~~ > > include/linux/dev_printk.h:129:34: note: expanded from macro 'dev_printk' > > _dev_printk(level, dev, fmt, ##__VA_ARGS__); \ > > ~~~ ^~~~~~~~~~~ > > 1 warning generated. > > Hmm, ok. so x86_64 needs a %ld for the pointer arithmic result on i386 needs a %d > without the 'l' what fun. I'll just store it in a temp int variable in the next > version. Why not to use uintptr_t and corresponding specifier (or ptrdiff_t)? -- With Best Regards, Andy Shevchenko From mboxrd@z Thu Jan 1 00:00:00 1970 Content-Type: multipart/mixed; boundary="===============1812259142485364606==" MIME-Version: 1.0 From: Andy Shevchenko To: kbuild-all@lists.01.org Subject: Re: [PATCH v3] pinctrl: baytrail: Clear direct_irq_en flag on broken configs Date: Wed, 12 Jan 2022 22:44:33 +0200 Message-ID: In-Reply-To: List-Id: --===============1812259142485364606== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: quoted-printable On Wed, Jan 12, 2022 at 08:58:00PM +0100, Hans de Goede wrote: > On 1/8/22 19:54, kernel test robot wrote: > >>> drivers/pinctrl/intel/pinctrl-baytrail.c:1483:58: warning: format spe= cifies type 'long' but the argument has type 'int' [-Wformat] > > dev_dbg(vg->dev, "Pin %i: uses direct IRQ %ld\n", pi= n, match - direct_irq); > > ~~~ = ^~~~~~~~~~~~~~~~~~ > > %d > > include/linux/dev_printk.h:163:47: note: expanded from macro 'dev_db= g' > > dev_printk(KERN_DEBUG, dev, dev_fmt(fmt), ##__VA_ARG= S__); \ > > ~~~ ^~~~~~~~= ~~~ > > include/linux/dev_printk.h:129:34: note: expanded from macro 'dev_pr= intk' > > _dev_printk(level, dev, fmt, ##__VA_ARGS__); = \ > > ~~~ ^~~~~~~~~~~ > > 1 warning generated. > = > Hmm, ok. so x86_64 needs a %ld for the pointer arithmic result on i386 ne= eds a %d > without the 'l' what fun. I'll just store it in a temp int variable in th= e next > version. Why not to use uintptr_t and corresponding specifier (or ptrdiff_t)? -- = With Best Regards, Andy Shevchenko --===============1812259142485364606==--