diff mbox series

crypto: bcm: remove -I. header search path and unused macro define

Message ID 1548396469-13395-1-git-send-email-yamada.masahiro@socionext.com (mailing list archive)
State Accepted
Delegated to: Herbert Xu
Headers show
Series crypto: bcm: remove -I. header search path and unused macro define | expand

Commit Message

Masahiro Yamada Jan. 25, 2019, 6:07 a.m. UTC
The header search path -I. in kernel Makefiles is very suspicious;
it allows the compiler to search for headers in the top of $(srctree),
where obviously no header file exists.

'git grep BCMDRIVER' has no hit. So, this macro is not referenced.

I was able to build this driver without the extra compiler options.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
---

 drivers/crypto/bcm/Makefile | 2 --
 1 file changed, 2 deletions(-)

Comments

Herbert Xu Feb. 1, 2019, 6:51 a.m. UTC | #1
On Fri, Jan 25, 2019 at 03:07:48PM +0900, Masahiro Yamada wrote:
> The header search path -I. in kernel Makefiles is very suspicious;
> it allows the compiler to search for headers in the top of $(srctree),
> where obviously no header file exists.
> 
> 'git grep BCMDRIVER' has no hit. So, this macro is not referenced.
> 
> I was able to build this driver without the extra compiler options.
> 
> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>
> ---
> 
>  drivers/crypto/bcm/Makefile | 2 --
>  1 file changed, 2 deletions(-)

Patch applied.  Thanks.
diff mbox series

Patch

diff --git a/drivers/crypto/bcm/Makefile b/drivers/crypto/bcm/Makefile
index 13cb80e..7469e19 100644
--- a/drivers/crypto/bcm/Makefile
+++ b/drivers/crypto/bcm/Makefile
@@ -11,5 +11,3 @@ 
 obj-$(CONFIG_CRYPTO_DEV_BCM_SPU) := bcm_crypto_spu.o
 
 bcm_crypto_spu-objs :=  util.o spu.o spu2.o cipher.o
-
-ccflags-y += -I. -DBCMDRIVER