linux-kernel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl'
@ 2016-08-14  0:05 kbuild test robot
  2016-08-15  9:10 ` Andy Shevchenko
  0 siblings, 1 reply; 3+ messages in thread
From: kbuild test robot @ 2016-08-14  0:05 UTC (permalink / raw)
  To: Andy Shevchenko; +Cc: kbuild-all, linux-kernel, Linus Walleij, Mika Westerberg

[-- Attachment #1: Type: text/plain, Size: 3242 bytes --]

Hi Andy,

FYI, the error/warning still remains.

tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
head:   329f4152911c276b074bec75a0443f88821afdb7
commit: 4e80c8f505741cbdef3e10862ea36057e8d85e7c pinctrl: intel: Add Intel Merrifield pin controller support
date:   7 weeks ago
config: x86_64-randconfig-s2-08140746 (attached as .config)
compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
reproduce:
        git checkout 4e80c8f505741cbdef3e10862ea36057e8d85e7c
        # save the attached .config to linux build tree
        make ARCH=x86_64 

All errors (new ones prefixed by >>):

   drivers/pinctrl/intel/pinctrl-merrifield.c: In function 'mrfld_pin_dbg_show':
>> drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl'
   drivers/pinctrl/intel/pinctrl-merrifield.c: In function 'mrfld_update_bufcfg':
>> drivers/pinctrl/intel/pinctrl-merrifield.c:575: error: implicit declaration of function 'writel'

vim +/readl +518 drivers/pinctrl/intel/pinctrl-merrifield.c

   512		if (!mrfld_buf_available(mp, pin)) {
   513			seq_puts(s, "not available");
   514			return;
   515		}
   516	
   517		bufcfg = mrfld_get_bufcfg(mp, pin);
 > 518		value = readl(bufcfg);
   519	
   520		mode = (value & BUFCFG_PINMODE_MASK) >> BUFCFG_PINMODE_SHIFT;
   521		if (!mode)
   522			seq_puts(s, "GPIO ");
   523		else
   524			seq_printf(s, "mode %d ", mode);
   525	
   526		seq_printf(s, "0x%08x", value);
   527	}
   528	
   529	static const struct pinctrl_ops mrfld_pinctrl_ops = {
   530		.get_groups_count = mrfld_get_groups_count,
   531		.get_group_name = mrfld_get_group_name,
   532		.get_group_pins = mrfld_get_group_pins,
   533		.pin_dbg_show = mrfld_pin_dbg_show,
   534	};
   535	
   536	static int mrfld_get_functions_count(struct pinctrl_dev *pctldev)
   537	{
   538		struct mrfld_pinctrl *mp = pinctrl_dev_get_drvdata(pctldev);
   539	
   540		return mp->nfunctions;
   541	}
   542	
   543	static const char *mrfld_get_function_name(struct pinctrl_dev *pctldev,
   544						   unsigned int function)
   545	{
   546		struct mrfld_pinctrl *mp = pinctrl_dev_get_drvdata(pctldev);
   547	
   548		return mp->functions[function].name;
   549	}
   550	
   551	static int mrfld_get_function_groups(struct pinctrl_dev *pctldev,
   552					     unsigned int function,
   553					     const char * const **groups,
   554					     unsigned int * const ngroups)
   555	{
   556		struct mrfld_pinctrl *mp = pinctrl_dev_get_drvdata(pctldev);
   557	
   558		*groups = mp->functions[function].groups;
   559		*ngroups = mp->functions[function].ngroups;
   560		return 0;
   561	}
   562	
   563	static void mrfld_update_bufcfg(struct mrfld_pinctrl *mp, unsigned int pin,
   564					u32 bits, u32 mask)
   565	{
   566		void __iomem *bufcfg;
   567		u32 value;
   568	
   569		bufcfg = mrfld_get_bufcfg(mp, pin);
   570		value = readl(bufcfg);
   571	
   572		value &= ~mask;
   573		value |= bits & mask;
   574	
 > 575		writel(value, bufcfg);
   576	}
   577	
   578	static int mrfld_pinmux_set_mux(struct pinctrl_dev *pctldev,

---
0-DAY kernel test infrastructure                Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all                   Intel Corporation

[-- Attachment #2: .config.gz --]
[-- Type: application/octet-stream, Size: 26431 bytes --]

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl'
  2016-08-14  0:05 drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl' kbuild test robot
@ 2016-08-15  9:10 ` Andy Shevchenko
  2016-08-16  8:16   ` Linus Walleij
  0 siblings, 1 reply; 3+ messages in thread
From: Andy Shevchenko @ 2016-08-15  9:10 UTC (permalink / raw)
  To: kbuild test robot
  Cc: kbuild-all, linux-kernel, Linus Walleij, Mika Westerberg

On Sun, 2016-08-14 at 08:05 +0800, kbuild test robot wrote:
> Hi Andy,
> 
> FYI, the error/warning still remains.

Linus, can you apply the fix for next v4.8-rcX?

> 
> tree:   https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux
> .git master
> head:   329f4152911c276b074bec75a0443f88821afdb7
> commit: 4e80c8f505741cbdef3e10862ea36057e8d85e7c pinctrl: intel: Add
> Intel Merrifield pin controller support
> date:   7 weeks ago
> config: x86_64-randconfig-s2-08140746 (attached as .config)
> compiler: gcc-4.4 (Debian 4.4.7-8) 4.4.7
> reproduce:
>         git checkout 4e80c8f505741cbdef3e10862ea36057e8d85e7c
>         # save the attached .config to linux build tree
>         make ARCH=x86_64 
> 
> All errors (new ones prefixed by >>):
> 
>    drivers/pinctrl/intel/pinctrl-merrifield.c: In function
> 'mrfld_pin_dbg_show':
> > 
> > > 
> > > drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit
> > > declaration of function 'readl'
>    drivers/pinctrl/intel/pinctrl-merrifield.c: In function
> 'mrfld_update_bufcfg':
> > 
> > > 
> > > drivers/pinctrl/intel/pinctrl-merrifield.c:575: error: implicit
> > > declaration of function 'writel'
> 
> vim +/readl +518 drivers/pinctrl/intel/pinctrl-merrifield.c
> 
>    512		if (!mrfld_buf_available(mp, pin)) {
>    513			seq_puts(s, "not available");
>    514			return;
>    515		}
>    516	
>    517		bufcfg = mrfld_get_bufcfg(mp, pin);
>  > 518		value = readl(bufcfg);
>    519	
>    520		mode = (value & BUFCFG_PINMODE_MASK) >>
> BUFCFG_PINMODE_SHIFT;
>    521		if (!mode)
>    522			seq_puts(s, "GPIO ");
>    523		else
>    524			seq_printf(s, "mode %d ", mode);
>    525	
>    526		seq_printf(s, "0x%08x", value);
>    527	}
>    528	
>    529	static const struct pinctrl_ops mrfld_pinctrl_ops = {
>    530		.get_groups_count = mrfld_get_groups_count,
>    531		.get_group_name = mrfld_get_group_name,
>    532		.get_group_pins = mrfld_get_group_pins,
>    533		.pin_dbg_show = mrfld_pin_dbg_show,
>    534	};
>    535	
>    536	static int mrfld_get_functions_count(struct pinctrl_dev
> *pctldev)
>    537	{
>    538		struct mrfld_pinctrl *mp =
> pinctrl_dev_get_drvdata(pctldev);
>    539	
>    540		return mp->nfunctions;
>    541	}
>    542	
>    543	static const char *mrfld_get_function_name(struct
> pinctrl_dev *pctldev,
>    544						   unsigned int
> function)
>    545	{
>    546		struct mrfld_pinctrl *mp =
> pinctrl_dev_get_drvdata(pctldev);
>    547	
>    548		return mp->functions[function].name;
>    549	}
>    550	
>    551	static int mrfld_get_function_groups(struct pinctrl_dev
> *pctldev,
>    552					     unsigned int
> function,
>    553					     const char * const
> **groups,
>    554					     unsigned int *
> const ngroups)
>    555	{
>    556		struct mrfld_pinctrl *mp =
> pinctrl_dev_get_drvdata(pctldev);
>    557	
>    558		*groups = mp->functions[function].groups;
>    559		*ngroups = mp->functions[function].ngroups;
>    560		return 0;
>    561	}
>    562	
>    563	static void mrfld_update_bufcfg(struct mrfld_pinctrl
> *mp, unsigned int pin,
>    564					u32 bits, u32 mask)
>    565	{
>    566		void __iomem *bufcfg;
>    567		u32 value;
>    568	
>    569		bufcfg = mrfld_get_bufcfg(mp, pin);
>    570		value = readl(bufcfg);
>    571	
>    572		value &= ~mask;
>    573		value |= bits & mask;
>    574	
>  > 575		writel(value, bufcfg);
>    576	}
>    577	
>    578	static int mrfld_pinmux_set_mux(struct pinctrl_dev
> *pctldev,
> 
> ---
> 0-DAY kernel test infrastructure                Open Source Technology
> Center
> https://lists.01.org/pipermail/kbuild-all                   Intel
> Corporation

-- 
Andy Shevchenko <andriy.shevchenko@linux.intel.com>
Intel Finland Oy

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl'
  2016-08-15  9:10 ` Andy Shevchenko
@ 2016-08-16  8:16   ` Linus Walleij
  0 siblings, 0 replies; 3+ messages in thread
From: Linus Walleij @ 2016-08-16  8:16 UTC (permalink / raw)
  To: Andy Shevchenko
  Cc: kbuild test robot, kbuild-all, linux-kernel, Mika Westerberg

On Mon, Aug 15, 2016 at 11:10 AM, Andy Shevchenko
<andriy.shevchenko@linux.intel.com> wrote:
> On Sun, 2016-08-14 at 08:05 +0800, kbuild test robot wrote:
>> Hi Andy,
>>
>> FYI, the error/warning still remains.
>
> Linus, can you apply the fix for next v4.8-rcX?

It's been applied for some time.

Sent a pull request to Torvalds now, too much to do, that's all :)

Yours,
Linus Walleij

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2016-08-16  8:16 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-08-14  0:05 drivers/pinctrl/intel/pinctrl-merrifield.c:518: error: implicit declaration of function 'readl' kbuild test robot
2016-08-15  9:10 ` Andy Shevchenko
2016-08-16  8:16   ` Linus Walleij

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).