From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marex@denx.de>
Date: Fri, 7 Sep 2012 11:42:24 +0200
Subject: [U-Boot] [PATCH v4 1/2] Loop block device for sandbox
In-Reply-To: <3758685.YXC1Fgde2a@bloomfield>
References: <1346843795-4744-1-git-send-email-morpheus.ibis@gmail.com>
	<201209071126.48768.marex@denx.de> <3758685.YXC1Fgde2a@bloomfield>
Message-ID: <201209071142.25053.marex@denx.de>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

Dear Pavel Herrmann,

> On Friday 07 of September 2012 11:26:48 Marek Vasut wrote:
> > Dear Pavel Herrmann,
> > 
> > > On Friday 07 of September 2012 01:29:55 Marek Vasut wrote:
> > > > Dear Pavel Herrmann,
> > > > 
> > > > > This driver uses files as block devices, can be used for testing
> > > > > disk operations on sandbox.
> > > > > A new command "sata_loop" is introduced to load files in runtime.
> > > > 
> > > > WARNING: externs should be avoided in .c files
> > > > #141: FILE: drivers/block/sata_loopback.c:39:
> > > > +extern block_dev_desc_t sata_dev_desc[];
> > > > 
> > > > total: 0 errors, 1 warnings, 231 lines checked
> > > > 
> > > > NOTE: Ignored message types: COMPLEX_MACRO CONSIDER_KSTRTO MINMAX
> > > > MULTISTATEMENT_MACRO_USE_DO_WHILE
> > > 
> > > Yes, i know about that, and chose to ignore it.
> > 
> > Sorry, this is not how it works.
> > 
> > > i will not create a header file for a single line that checkpatch
> > > doesnt like (with just a warning), especially when other drivers have
> > > such a line in them as well.
> > 
> > So, implement proper accessor for this list? Or export the structure
> > through some sata-related header? How does other do it?
> 
> other drivers have the same line that i do, therefore ignore the checkpatch
> warning, same as i do

Oh god damned! This is definitelly not the way to go!

NAK! NAK! NAK!! NAK!!! NAK!!!!! NAK!!!!!!!! ... you get the idea ;-)

> see sata_dwc.c, ata_piix.c, fsl_sata.c, sata_sil3114.c

Sad state of codebase, thanks for pointing out the obvious :-( You know, now 
that you pointed it out, you also know what to do :-)

Besides, you'd have to do it anyway, so let's do it now.

> Pavel Herrmann

Best regards,
Marek Vasut