linux-next: build warning after merge of the clockevents tree

linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi Daniel,

After merging the clockevents tree, the file
drivers/clocksource/clksrc-of.c.rej appears ... it was commited as part
of commit 55af97749f2b ("Merge branch 'clockevents/3.14' of
git://git.linaro.org/people/daniel.lezcano/linux into timers/core") which
I suspect was added to the clockevents tree when you based (or merged)
the tip tree.  That commit no longer appears in the tip tree.

-- 
Cheers,
Stephen Rothwell                    sfr@canb.auug.org.au

[-- Attachment #2: Type: application/pgp-signature, Size: 836 bytes --]

linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi all,

After merging the clockevents tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

drivers/clocksource/timer-ti-dm.c: In function 'omap_dm_timer_probe':
drivers/clocksource/timer-ti-dm.c:798:13: warning: 'timer' may be used uninitialized in this function [-Wmaybe-uninitialized]
  798 |  timer->irq = platform_get_irq(pdev, 0);
      |  ~~~~~~~~~~~^~~~~~~~~~~~~~~~~~~~~~~~~~~

Introduced by commit

  8c82723414d5 ("clocksource/drivers/timer-ti-dm: Switch to platform_get_irq")

This is not a false positive ...

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi all,

After merging the clockevents tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x324 (section: .text) -> register_current_timer_delay (section: .init.text)
WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x4c4 (section: .text) -> register_current_timer_delay (section: .init.text)

Introduced by commit

  5d86e479193b ("clocksource/drivers/exynos_mct: Add module support")

and possibly

  7e477e9c4eb4 ("clocksource/drivers/exynos_mct: Fix section mismatch from the module conversion")

For this build,

CONFIG_CLKSRC_EXYNOS_MCT=y

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi Daniel,

On Wed, 16 Jul 2025 11:26:15 +0200 Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
>

> Right. Shall I fold the fix with the commit ?

That would be up to you.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi all,

On Wed, 16 Jul 2025 16:08:09 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>
> After merging the clockevents tree, today's linux-next build (arm
> multi_v7_defconfig) produced this warning:
> 
> WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x324 (section: .text) -> register_current_timer_delay (section: .init.text)
> WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x4c4 (section: .text) -> register_current_timer_delay (section: .init.text)
> 
> Introduced by commit
> 
>   5d86e479193b ("clocksource/drivers/exynos_mct: Add module support")
> 
> and possibly
> 
>   7e477e9c4eb4 ("clocksource/drivers/exynos_mct: Fix section mismatch from the module conversion")
> 
> For this build,
> 
> CONFIG_CLKSRC_EXYNOS_MCT=y

I am still seeing these warnings.  The above commit is now also commit

  338007c44c7f ("clocksource/drivers/exynos_mct: Add module support")

in the tip tree.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

Re: linux-next: build warning after merge of the clockevents tree

[Daniel Lezcano]

On 29/07/2025 03:40, Stephen Rothwell wrote:
> Hi all,
> 
> On Wed, 16 Jul 2025 16:08:09 +1000 Stephen Rothwell <sfr@canb.auug.org.au> wrote:
>>
>> After merging the clockevents tree, today's linux-next build (arm
>> multi_v7_defconfig) produced this warning:
>>
>> WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x324 (section: .text) -> register_current_timer_delay (section: .init.text)
>> WARNING: modpost: vmlinux: section mismatch in reference: mct_init_dt+0x4c4 (section: .text) -> register_current_timer_delay (section: .init.text)
>>
>> Introduced by commit
>>
>>    5d86e479193b ("clocksource/drivers/exynos_mct: Add module support")
>>
>> and possibly
>>
>>    7e477e9c4eb4 ("clocksource/drivers/exynos_mct: Fix section mismatch from the module conversion")
>>
>> For this build,
>>
>> CONFIG_CLKSRC_EXYNOS_MCT=y
> 
> I am still seeing these warnings.  The above commit is now also commit
> 
>    338007c44c7f ("clocksource/drivers/exynos_mct: Add module support")
> 
> in the tip tree.

This should be fixed now.

Also the clockevent branch has been reset.


-- 
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro:  <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

Re: linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi Daniel,

On Wed, 30 Jul 2025 14:22:27 +0200 Daniel Lezcano <daniel.lezcano@linaro.org> wrote:
>
> This should be fixed now.
> 
> Also the clockevent branch has been reset.

Thanks for both.

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]

linux-next: build warning after merge of the clockevents tree

[Stephen Rothwell]

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

Hi all,

After merging the clockevents tree, today's linux-next build (arm
multi_v7_defconfig) produced this warning:

In file included from include/linux/bitops.h:6,
                 from include/linux/log2.h:12,
                 from include/asm-generic/div64.h:55,
                 from arch/arm/include/asm/div64.h:114,
                 from include/linux/math.h:6,
                 from include/linux/math64.h:6,
                 from include/linux/time.h:6,
                 from include/uapi/linux/timex.h:56,
                 from include/linux/timex.h:56,
                 from include/linux/clocksource.h:13,
                 from include/linux/clockchips.h:14,
                 from drivers/clocksource/arm_arch_timer_mmio.c:13:
drivers/clocksource/arm_arch_timer_mmio.c: In function 'arch_timer_mmio_setup':
include/linux/bits.h:47:9: warning: conversion from 'long long unsigned int' to 'long unsigned int' changes value from '72057594037927935' to '4294967295' [-Woverflow]
   47 |         ((t)(GENMASK_INPUT_CHECK(h, l) +                        \
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   48 |              (type_max(t) << (l) &                              \
      |              ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   49 |               type_max(t) >> (BITS_PER_TYPE(t) - 1 - (h)))))
      |               ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
include/linux/bits.h:52:33: note: in expansion of macro 'GENMASK_TYPE'
   52 | #define GENMASK_ULL(h, l)       GENMASK_TYPE(unsigned long long, h, l)
      |                                 ^~~~~~~~~~~~
include/linux/clocksource.h:153:32: note: in expansion of macro 'GENMASK_ULL'
  153 | #define CLOCKSOURCE_MASK(bits) GENMASK_ULL((bits) - 1, 0)
      |                                ^~~~~~~~~~~
drivers/clocksource/arm_arch_timer_mmio.c:264:66: note: in expansion of macro 'CLOCKSOURCE_MASK'
  264 |         clockevents_config_and_register(&at->evt, at->rate, 0xf, CLOCKSOURCE_MASK(56));
      |                                                                  ^~~~~~~~~~~~~~~~

Introduced by commit

  fbdd82d7f782 ("clocksource/drivers/arm_arch_timer: Add standalone MMIO driver")

-- 
Cheers,
Stephen Rothwell

[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 488 bytes --]