Re: [PATCH 45/56] microblaze_v2: headers simple files - empty orredirect to asm-generic

From: Stephen Rothwell
Date: Tue May 06 2008 - 10:47:31 EST


On Mon, 05 May 2008 12:34:09 +1000 John Williams <john.williams@xxxxxxxxxxxxx> wrote:
>
>
> > include/asm-microblaze/auxvec.h | 14 ++++++++++++++
> > include/asm-microblaze/cputime.h | 16 ++++++++++++++++
> > include/asm-microblaze/div64.h | 16 ++++++++++++++++
> > include/asm-microblaze/emergency-restart.h | 16 ++++++++++++++++
> > include/asm-microblaze/errno.h | 16 ++++++++++++++++
> > include/asm-microblaze/futex.h | 16 ++++++++++++++++
> > include/asm-microblaze/kdebug.h | 19 +++++++++++++++++++
> > include/asm-microblaze/local.h | 16 ++++++++++++++++
> > include/asm-microblaze/mutex.h | 16 ++++++++++++++++
> > include/asm-microblaze/namei.h | 24 ++++++++++++++++++++++++
> > include/asm-microblaze/percpu.h | 16 ++++++++++++++++
> > include/asm-microblaze/resource.h | 16 ++++++++++++++++
> > include/asm-microblaze/user.h | 18 ++++++++++++++++++
>
> These are all just includes of asm-generic/* (hooray!).

Absolutely, but is it really necessary to have 14-16 lines of comment
(including a copyright notice) for a file whose single real line is just
to include another file? i.e. reduce all these to 1 line files.

--
Cheers,
Stephen Rothwell sfr@xxxxxxxxxxxxxxxx
http://www.canb.auug.org.au/~sfr/

Attachment: pgp00000.pgp
Description: PGP signature