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
State Accepted
Commit 87fec0102dada9084b86a782287551b237efb914
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(-)

-- 
2.7.4

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.
-- 
Email: Herbert Xu <herbert@gondor.apana.org.au>
Home Page: http://gondor.apana.org.au/~herbert/
PGP Key: http://gondor.apana.org.au/~herbert/pubkey.txt
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