On Thu, 04 Feb 2021, Alessandrelli, Daniele wrote:
> On Thu, 2021-02-04 at 11:09 +0000, Lee Jones wrote:
> > Fixes the following W=1 kernel build warning(s):
> >
> > drivers/crypto/keembay/ocs-hcu.c:107: warning: expecting prototype for struct ocs_hcu_dma_list. Prototype was for struct ocs_hcu_dma_entry instead
> > drivers/crypto/keembay/ocs-hcu.c:127: warning: expecting prototype for struct ocs_dma_list. Prototype was for struct ocs_hcu_dma_list instead
> > drivers/crypto/keembay/ocs-hcu.c:610: warning: expecting prototype for ocs_hcu_digest(). Prototype was for ocs_hcu_hash_update() instead
> > drivers/crypto/keembay/ocs-hcu.c:648: warning: expecting prototype for ocs_hcu_hash_final(). Prototype was for ocs_hcu_hash_finup() instead
> >
> > Cc: Daniele Alessandrelli <[email protected]>
> > Cc: Declan Murphy <[email protected]>
> > Cc: Herbert Xu <[email protected]>
> > Cc: "David S. Miller" <[email protected]>
> > Cc: [email protected]
> > Signed-off-by: Lee Jones <[email protected]>
> > ---
>
> Acked-by: Daniele Alessandrelli <[email protected]>
>
>
> Thanks for fixing these.
>
> For some reason, if the issues are there, I don't get those warnings
> when compiling with W=1; the command I run is:
>
> make CROSS_COMPILE=<arm-compiler> ARCH=arm64 -j5 W=4 M=drivers/crypto/keembay
Not sure what would happen with 'W=4'.
Probably nothing, as it only goes up to 3 [0].
> Which command are you running exactly? I'll use it for my next
> submissions.
rm -rf ../builds/build-arm64/drivers/crypto/
make -f Makefile -j24 --quiet ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- KBUILD_OUTPUT=../builds/build-arm64 allmodconfig
make -f Makefile -j24 --quiet ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- KBUILD_OUTPUT=../builds/build-arm64 W=1 drivers/crypto/
Hope that helps.
[0] scripts/Makefile.extrawarn
--
Lee Jones [李琼斯]
Senior Technical Lead - Developer Services
Linaro.org │ Open source software for Arm SoCs
Follow Linaro: Facebook | Twitter | Blog
On Thu, 2021-02-04 at 18:55 +0000, Lee Jones wrote:
> On Thu, 04 Feb 2021, Alessandrelli, Daniele wrote:
>
> > On Thu, 2021-02-04 at 11:09 +0000, Lee Jones wrote:
> > > Fixes the following W=1 kernel build warning(s):
> > >
> > > drivers/crypto/keembay/ocs-hcu.c:107: warning: expecting prototype for struct ocs_hcu_dma_list. Prototype was for struct ocs_hcu_dma_entry instead
> > > drivers/crypto/keembay/ocs-hcu.c:127: warning: expecting prototype for struct ocs_dma_list. Prototype was for struct ocs_hcu_dma_list instead
> > > drivers/crypto/keembay/ocs-hcu.c:610: warning: expecting prototype for ocs_hcu_digest(). Prototype was for ocs_hcu_hash_update() instead
> > > drivers/crypto/keembay/ocs-hcu.c:648: warning: expecting prototype for ocs_hcu_hash_final(). Prototype was for ocs_hcu_hash_finup() instead
> > >
> > > Cc: Daniele Alessandrelli <[email protected]>
> > > Cc: Declan Murphy <[email protected]>
> > > Cc: Herbert Xu <[email protected]>
> > > Cc: "David S. Miller" <[email protected]>
> > > Cc: [email protected]
> > > Signed-off-by: Lee Jones <[email protected]>
> > > ---
> >
> > Acked-by: Daniele Alessandrelli <[email protected]>
> >
> >
> > Thanks for fixing these.
> >
> > For some reason, if the issues are there, I don't get those warnings
> > when compiling with W=1; the command I run is:
> >
> > make CROSS_COMPILE=<arm-compiler> ARCH=arm64 -j5 W=4 M=drivers/crypto/keembay
>
> Not sure what would happen with 'W=4'.
Sorry that was a typo (I meant to write W=1) :/
>
> Probably nothing, as it only goes up to 3 [0].
>
> > Which command are you running exactly? I'll use it for my next
> > submissions.
>
> rm -rf ../builds/build-arm64/drivers/crypto/
> make -f Makefile -j24 --quiet ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- KBUILD_OUTPUT=../builds/build-arm64 allmodconfig
> make -f Makefile -j24 --quiet ARCH=arm64 CROSS_COMPILE=aarch64-linux-gnu- KBUILD_OUTPUT=../builds/build-arm64 W=1 drivers/crypto/
>
> Hope that helps.
Thanks for providing your commands. Unfortunately, even if I run them I
don't get the above warnings.
I wonder if the issue is in my version of scripts/kernel-doc (which
seems to be the script called by the Makefile to to check the kernel-
doc).
I'll keep investigating this. Thanks again!
>
> [0] scripts/Makefile.extrawarn
>