Re: sound/soc/ti/j721e-evm.c:528:34: warning: unused variable 'j721e_audio_of_match'

2020-09-29 Thread Mark Brown
On Tue, Sep 29, 2020 at 09:00:02AM +0300, Peter Ujfalusi wrote: > The ifdef would be preferred if the driver could work in non DT boot (to > save few bytes) but since it is not the case here: What I'd *really* like would be something in the of_match_ptr() that flagged that there's a reference to

Re: sound/soc/ti/j721e-evm.c:528:34: warning: unused variable 'j721e_audio_of_match'

2020-09-28 Thread Peter Ujfalusi
On 28/09/2020 21.04, Mark Brown wrote: > On Mon, Sep 28, 2020 at 10:52:54AM -0700, Nick Desaulniers wrote: >> On Mon, Sep 28, 2020 at 12:27 AM 'Peter Ujfalusi' via Clang Built > >>> Right, in the attached .config: >>> # CONFIG_OF is not set > >>> There must be a clean way to handle this withou

Re: sound/soc/ti/j721e-evm.c:528:34: warning: unused variable 'j721e_audio_of_match'

2020-09-28 Thread Mark Brown
On Mon, Sep 28, 2020 at 10:52:54AM -0700, Nick Desaulniers wrote: > On Mon, Sep 28, 2020 at 12:27 AM 'Peter Ujfalusi' via Clang Built > > Right, in the attached .config: > > # CONFIG_OF is not set > > There must be a clean way to handle this without extensive ifedfery... > Pretty sure I just saw

Re: sound/soc/ti/j721e-evm.c:528:34: warning: unused variable 'j721e_audio_of_match'

2020-09-28 Thread Nick Desaulniers
nux build tree > > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross > > ARCH=x86_64 > > > > If you fix the issue, kindly add following tag as appropriate > > Reported-by: kernel test robot > > > > All warnings (new ones

Re: sound/soc/ti/j721e-evm.c:528:34: warning: unused variable 'j721e_audio_of_match'

2020-09-28 Thread Peter Ujfalusi
7e42dfa975879fb275099f0b2 > # save the attached .config to linux build tree > COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross > ARCH=x86_64 > > If you fix the issue, kindly add following tag as appropriate > Reported-by: kernel test robot > > All w