2022-04-06 13:57:24

by Stephen Rothwell

[permalink] [raw]

Subject: linux-next: build warning after merge of the drm-misc tree

---

Hi all,

After merging the drm-misc tree, today's linux-next build (x86_64
allmodconfig) produced this warning:

WARNING: modpost: drivers/gpu/drm/bridge/chipone-icn6211.prelink.o(.exit.data+0x0): Section mismatch in reference from the variable __cfi_jt_cleanup_module to the function .init.text:cleanup_module()
The variable __exitdata __cfi_jt_cleanup_module references
a function __init cleanup_module().
This is often seen when error handling in the exit function
uses functionality in the init path.
The fix is often to remove the __init annotation of
cleanup_module() so it may be used outside an init section.

Introduced by commit

8dde6f7452a1 ("drm: bridge: icn6211: Add I2C configuration support")

The chipone_exit function should not be marked __init.

--
Cheers,
Stephen Rothwell

---

Attachments:

(No filename) (499.00 B)
OpenPGP digital signature

2022-04-06 16:36:13

by Marek Vasut

[permalink] [raw]

Subject: Re: linux-next: build warning after merge of the drm-misc tree

---

On 4/6/22 02:58, Stephen Rothwell wrote:
> Hi all,

Hi,

> After merging the drm-misc tree, today's linux-next build (x86_64
> allmodconfig) produced this warning:
>
> WARNING: modpost: drivers/gpu/drm/bridge/chipone-icn6211.prelink.o(.exit.data+0x0): Section mismatch in reference from the variable __cfi_jt_cleanup_module to the function .init.text:cleanup_module()
> The variable __exitdata __cfi_jt_cleanup_module references
> a function __init cleanup_module().
> This is often seen when error handling in the exit function
> uses functionality in the init path.
> The fix is often to remove the __init annotation of
> cleanup_module() so it may be used outside an init section.
>
> Introduced by commit
>
> 8dde6f7452a1 ("drm: bridge: icn6211: Add I2C configuration support")
>
> The chipone_exit function should not be marked __init.

Thanks, fix is submitted.

Sigh.