[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <CAMuHMdVz=_XyzG+ss3VFq=PSG5b4VE2MkYWTPJyVdzrw_1RisQ@mail.gmail.com>
Date: Fri, 6 May 2022 09:23:27 +0200
From: Geert Uytterhoeven <geert@...ux-m68k.org>
To: Arnd Bergmann <arnd@...db.de>
Cc: kernel test robot <lkp@...el.com>, kbuild-all@...ts.01.org,
Linux Kernel Mailing List <linux-kernel@...r.kernel.org>,
Greg Ungerer <gerg@...ux-m68k.org>,
Daniel Palmer <daniel@...f.com>
Subject: Re: time.c:undefined reference to `mach_get_rtc_pll'
Hi Arnd,
On Fri, May 6, 2022 at 9:21 AM Arnd Bergmann <arnd@...db.de> wrote:
> On Wed, May 4, 2022 at 3:33 PM Geert Uytterhoeven <geert@...ux-m68k.org> wrote:
> > On Wed, May 4, 2022 at 2:25 PM Arnd Bergmann <arnd@...db.de> wrote:
> > > On Wed, May 4, 2022 at 11:14 AM kernel test robot <lkp@...el.com> wrote:
> > > > tree: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git master
> > > > head: 107c948d1d3e61d10aee9d0f7c3d81bbee9842af
> > > > commit: 8b22820efb35f93d98638563b0a8f4094e8ee399 m68k: m68328: remove duplicate code
> > > >
> > > > If you fix the issue, kindly add following tag as appropriate
> > > > Reported-by: kernel test robot <lkp@...el.com>
> > > >
> > > > All errors (new ones prefixed by >>):
> > > >
> > > > m68k-linux-ld: arch/m68k/kernel/time.o: in function `rtc_ioctl':
> > > > >> time.c:(.text+0x82): undefined reference to `mach_get_rtc_pll'
> > > > m68k-linux-ld: time.c:(.text+0xbc): undefined reference to `mach_set_rtc_pll'
> > > > m68k-linux-ld: time.c:(.text+0xf4): undefined reference to `mach_set_rtc_pll'
> > > >
> > >
> > > I think this is just another build failure that was hidden before my patch,
> > > not caused by my patch.
> >
> > That's an m68knommu kernel build with CONFIG_RTC_DRV_GENERIC=y,
> > I guess?
>
> Correct, the .config file is linked in the original report, just double-checked
> that. More importantly, it's a nommu M68KCLASSIC (dragonball) config,
Sorry, the original report email was delayed by 9 hours, so I
got your response first.
> so the existing #ifdef CONFIG_M68KCLASSIC check in
> arch/m68k/kernel/time.c is insufficient.
Yeah, need to fix that, one day....
Gr{oetje,eeting}s,
Geert
--
Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@...ux-m68k.org
In personal conversations with technical people, I call myself a hacker. But
when I'm talking to journalists I just say "programmer" or something like that.
-- Linus Torvalds
Powered by blists - more mailing lists