[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20140519074510.74e622fd@chukar.edge2.net>
Date: Mon, 19 May 2014 07:45:10 -0600
From: Jake Edge <jake@....net>
To: Anton Saraev <antonysaraev@...il.com>
Cc: gregkh@...uxfoundation.org, jason@...edaemon.net,
devel@...verdev.osuosl.org, linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 06/06] staging: crypto: skein: rename files
On Mon, 19 May 2014 12:09:59 +0400 Anton Saraev wrote:
> --- a/drivers/staging/skein/Makefile
> +++ b/drivers/staging/skein/Makefile
> @@ -4,10 +4,10 @@
> subdir-ccflags-y := -I$(src)/include/
as part of the rearranging of files, I think you want to get rid of
this include directory, move the files up to the main directory and
include them as "skein.h" (rather than <skein.h>).
Then this line can be removed from the Makefile.
> --- a/drivers/staging/skein/threefish1024Block.c
> +++ b/drivers/staging/skein/threefish_1024_block.c
> @@ -1,5 +1,5 @@
> #include <linux/string.h>
> -#include <threefishApi.h>
> +#include <threefish_api.h>
incidentally, none of the threefish_*_block.c files actually needs
linux/string.h ... there may be other files where that's true too, it
seems to have just gotten added everywhere ...
jake
--
Jake Edge - LWN - jake@....net - http://lwn.net
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists