linux-kernel - Re: drivers/regulator/mt6380-regulator.c:333:34: warning: 'mt6380_of

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date:   Mon, 25 Jul 2022 10:00:56 +0200
From:   Jean Delvare <jdelvare@...e.de>
To:     kernel test robot <lkp@...el.com>
Cc:     Sean Wang <sean.wang@...iatek.com>, kbuild-all@...ts.01.org,
        linux-kernel@...r.kernel.org,
        Matthias Brugger <matthias.bgg@...il.com>
Subject: Re: drivers/regulator/mt6380-regulator.c:333:34: warning:
 'mt6380_of_match' defined but not used

On Sun, 24 Jul 2022 02:37:15 +0800, kernel test robot wrote:
> All warnings (new ones prefixed by >>):
> 
> >> drivers/regulator/mt6380-regulator.c:333:34: warning: 'mt6380_of_match' defined but not used [-Wunused-const-variable=]  
>      333 | static const struct of_device_id mt6380_of_match[] = {
>          |                                  ^~~~~~~~~~~~~~~
> --
> >> drivers/soc/mediatek/mtk-pmic-wrap.c:1436:34: warning: 'of_pwrap_match_tbl' defined but not used [-Wunused-const-variable=]  
>     1436 | static const struct of_device_id of_pwrap_match_tbl[] = {
>          |                                  ^~~~~~~~~~~~~~~~~~
> >> drivers/soc/mediatek/mtk-pmic-wrap.c:1370:34: warning: 'of_slave_match_tbl' defined but not used [-Wunused-const-variable=]  
>     1370 | static const struct of_device_id of_slave_match_tbl[] = {
>          |                                  ^~~~~~~~~~~~~~~~~~

Not sure what do to with this one. These warnings can be silenced with
__maybe_unused. But does it actually make sense to build these drivers
with CONFIG_OF disabled?

As I understand it, that's the condition which makes the warnings
appear, and from a quick look at the code, it seems to me that the
drivers are useless then. It might make more sense to make both drivers
depend on OF. What do you think?

-- 
Jean Delvare
SUSE L3 Support