All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH]: BeFs parser.h support in linux kernel v3.5
@ 2012-08-07  0:07 Christopher Sacchi
  2012-08-07  0:48 ` Randy Dunlap
  0 siblings, 1 reply; 2+ messages in thread
From: Christopher Sacchi @ 2012-08-07  0:07 UTC (permalink / raw)
  To: Linux Kernel Mailing List

This patch adds a new header, in the to-do list. There are no style issues.
Here's the patch:
--
Signed-off-by: Christopher P. Sacchi <chris.sacchi@gmail.com>
--- befs.h      2012-07-21 20:58:29.000000000 +0000
+++ befs.h      2012-08-06 19:52:05.000000000 +0000
@@ -9,6 +9,7 @@
 #define _LINUX_BEFS_H

 #include "befs_fs_types.h"
+#include "linux-3.5/include/parser.h"

 /* used in debug.c */
 #define BEFS_VERSION "0.9.3"

--

-- 
Christopher

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [PATCH]: BeFs parser.h support in linux kernel v3.5
  2012-08-07  0:07 [PATCH]: BeFs parser.h support in linux kernel v3.5 Christopher Sacchi
@ 2012-08-07  0:48 ` Randy Dunlap
  0 siblings, 0 replies; 2+ messages in thread
From: Randy Dunlap @ 2012-08-07  0:48 UTC (permalink / raw)
  To: Christopher Sacchi; +Cc: Linux Kernel Mailing List

On 08/06/2012 05:07 PM, Christopher Sacchi wrote:

> This patch adds a new header, in the to-do list. There are no style issues.
> Here's the patch:
> --
> Signed-off-by: Christopher P. Sacchi <chris.sacchi@gmail.com>
> --- befs.h      2012-07-21 20:58:29.000000000 +0000
> +++ befs.h      2012-08-06 19:52:05.000000000 +0000
> @@ -9,6 +9,7 @@
>  #define _LINUX_BEFS_H
> 
>  #include "befs_fs_types.h"
> +#include "linux-3.5/include/parser.h"
> 
>  /* used in debug.c */
>  #define BEFS_VERSION "0.9.3"
> 
> --


filesystem patches should also go to linux-fsdevel@vger.kernel.org .


from TODO file:
* See if Alexander Viro's option parser made it into the kernel tree. 
	Use that if we can. (include/linux/parser.h)

Yes, the option parser made it into the kernel tree several years ago,
but just adding the header doesn't make befs use the option parser,
so the patch is not enough.  And the header patch should be:

#include <linux/parser.h>



-- 
~Randy

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2012-08-07  0:49 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-08-07  0:07 [PATCH]: BeFs parser.h support in linux kernel v3.5 Christopher Sacchi
2012-08-07  0:48 ` Randy Dunlap

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.