lkml.org 
[lkml]   [2010]   [Oct]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [RFCv3 20/21] tcm: Add Kbuild and Kconfig for drivers/target and Documentation/target/
From
Date
On Mon, 2010-10-18 at 15:48 -0700, Randy Dunlap wrote:
> On Mon, 18 Oct 2010 15:39:19 -0700 Nicholas A. Bellinger wrote:
>
> > From: Nicholas Bellinger <nab@linux-iscsi.org>
> >
> > Add TCM Kbuild and Kconfig files and update drivers/ to pick up drivers/target.
> > Add Docmentation/target/ for tcm_mod_builder.py script for generating new
> > TCM v4 fabric modules functional ConfigFS skeletons.
> >
> > Signed-off-by: Nicholas A. Bellinger <nab@linux-iscsi.org>
> > ---
> > Documentation/target/tcm_mod_builder.py | 1094 ++++++++++++++++++++++++++++++
> > Documentation/target/tcm_mod_builder.txt | 145 ++++
> > drivers/Kconfig | 2 +
> > drivers/Makefile | 1 +
> > drivers/target/Kbuild | 27 +
> > drivers/target/Kconfig | 34 +
> > 6 files changed, 1303 insertions(+), 0 deletions(-)
> > create mode 100755 Documentation/target/tcm_mod_builder.py
> > create mode 100644 Documentation/target/tcm_mod_builder.txt
> > create mode 100644 drivers/target/Kbuild
> > create mode 100644 drivers/target/Kconfig
>
>
> > diff --git a/drivers/target/Kconfig b/drivers/target/Kconfig
> > new file mode 100644
> > index 0000000..5c37653
> > --- /dev/null
> > +++ b/drivers/target/Kconfig
> > @@ -0,0 +1,34 @@
> > +config TARGET_CORE
> > + tristate "Generic Target Core Mod (TCM) and ConfigFS Infrastructure"
> > + select CONFIGFS_FS
>
> Indentation: use tabs, not spaces.
>

Whoops, fixed. Thanks for spotting this..

> > + default m
>
> Don't enable random drivers/modules.
> Let the user or defconfig do that.

Understood, changing to 'default n'.

Thanks!

--nab



\
 
 \ /
  Last update: 2010-10-19 01:11    [W:0.105 / U:0.104 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site