[PATCH -next] fbdev: via: Fix section mismatch warning in via_core_init()

shangxiaojing shangxiaojing at huawei.com
Tue Nov 15 06:52:52 UTC 2022



On 2022/11/15 13:14, Helge Deller wrote:
> On 11/15/22 03:53, Shang XiaoJing wrote:
>> Due to viafb_exit() with "__exit" tag, it should not be called by the
>> __init function via_core_init().
> 
> I wonder if you can't instead of this and your previous patch 
> (ab885d8c7e15)
> turn the i2c and gpio drivers to proper platform drivers, e.g.
> adding to bottom of via/via_i2c.c:
> module_platform_driver(&via_i2c_driver)
> instead of viafb_i2c_init() and viafb_i2c_exit().
> 
> Shouldn't they then automatically be loaded/unloaded?
> 

I'm sorry that I have no idea how to change an i2c driver to a platform 
driver.

As for module_platform_driver(), I have checked and it looks like just a 
helper macro to definite XXX_init() and XXX_exit() instead of 
automatically load/unload the driver.

Besides, the XXX_init() and XXX_exit() definited by the 
module_platform_driver() can only call 
platform_driver_register()/platform_driver_unregister(), perhaps not 
suitable for via_driver? (or just need to call 
platform_driver_register() after change to platform driver)

Thanks,
-- 
Shang XiaoJing


More information about the dri-devel mailing list