lkml.org 
[lkml]   [2009]   [Jul]   [20]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 4/5] bfa: Brocade BFA FC SCSI driver (make)
On Sun, Jul 19, 2009 at 11:26:54PM -0700, Jing Huang wrote:
> From: Jing Huang <huangj@brocade.com>
>
> This patch contains Makefile and Kconfig file for scsi and bfa.
>
> Signed-off-by: Jing Huang <huangj@brocade.com>
> diff -urpN orig/drivers/scsi/bfa/Makefile patch/drivers/scsi/bfa/Makefile
> --- orig/drivers/scsi/bfa/Makefile 1969-12-31 16:00:00.000000000 -0800
> +++ patch/drivers/scsi/bfa/Makefile 2009-07-19 17:48:15.000000000 -0700
> @@ -0,0 +1,30 @@
> +#
> +# Copyright (c) 2005-2008 Brocade Communications Systems, Inc.
> +# All rights reserved
> +# www.brocade.com
> +#
> +# Linux driver for Brocade Fibre Channel Host Bus Adapter.
> +#
> +# This program is free software; you can redistribute it and/or modify it
> +# under the terms of the GNU General Public License (GPL) Version 2 as
> +# published by the Free Software Foundation
> +#
> +# This program is distributed in the hope that it will be useful, but
> +# WITHOUT ANY WARRANTY; without even the implied warranty of
> +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> +# General Public License for more details.
> +#

A big boilerplate for a simple kbuild file...

> +obj-$(CONFIG_SCSI_BFA_FC) := bfa.o
OK - I assume the name of the module is remotely aligned with
what other scsi drivers uses.
That is - there is no scsi prefix or anything else that says this is
a scsi driver. If this is gennerally not used then you do not need it either.

> +
> +bfa-objs := bfad.o bfad_intr.o bfad_os.o bfad_im.o bfad_attr.o bfad_fwimg.o \
> + bfa_core.o bfa_ioc.o bfa_iocfc.o bfa_aen.o bfa_fcxp.o bfa_lps.o \
> + bfa_hw_cb.o bfa_hw_ct.o bfa_intr.o bfa_timer.o bfa_rport.o \
> + bfa_fcport.o bfa_port.o bfa_uf.o bfa_sgpg.o bfa_module.o bfa_ioim.o \
> + bfa_itnim.o bfa_fcpim.o bfa_tskim.o bfa_log.o bfa_log_module.o \
> + bfa_csdebug.o bfa_sm.o plog.o fcbuild.o fabric.o fcpim.o vfapi.o \
> + fcptm.o bfa_fcs.o bfa_fcs_port.o bfa_fcs_uf.o bfa_fcs_lport.o \
> + fab.o fdmi.o ms.o ns.o scn.o loop.o lport_api.o n2n.o rport.o \
> + rport_api.o rport_ftrs.o vport.o

1) Please us:
bfa-y := bfad.o ....

This allows you to later use:
bfa-$(SCSI_BFA_FEATURE) += bfa_feature.o


2) I personally prefer to avoid the line continuation.
So something like this would be *my* style:

bfa-y := bfad.o bfad_intr.o bfad_os.o bfad_im.o

# IO stuff
bfa-y += bfa_ioc.o bfa_iocfc.o

# HW near
bfa-y += bfa_hw_cb.o bfa_hw_ct.o bfa_intr.o bfa_timer.o
bfa-y += bfa_rport.o bfa_fcport.o bfa_port.o

[Understand the concept - I selected filenames almost randomly]

> +
> +EXTRA_CFLAGS += -I$(obj) -I$(obj)/include -I$(obj)/include/cna

Please use:
ccflags-y := -I$(obj)/include -I$(obj)/include/cna

I do not see why you need "-I$(obj)" - so I left it out on purpose.
It should only be needed if you generate files.


Sam


\
 
 \ /
  Last update: 2009-07-20 09:25    [W:0.024 / U:0.652 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site