Re: [PATCH 2.6.13 5/14] sas-class: sas_discover.c Discover process (end devices)

[Luben Tuikov <luben_tuikov@adaptec.com>]

On 09/13/05 17:37, Stefan Richter wrote:
> Luben Tuikov wrote:
> 
>>I've never seen the symbols "lun".
> 
> 
> It is merely an encoding of a variable name or struct member name 
> according to the coding style spec.

I appreciate your insight.

I've never seen a "lun" in _any_ spec.  It is always abbreviated
"LUN".

"lun" exists only in the SCSI Core.  Even other OSs use "LUN".

So after a while, after someone reads enough specs, they see
only "LUN".  "lun" seems foreign.

> 
>>"task->ssp_task.LUN"
> 
> 
> But SSP is a TLA too, isn't it? ;-)

Not quite.  I can actually _see_ "LUN" in the frame.

As I said, after a while it becomes second nature to you,
due to the layout of the frame you're working with.

The pattern that the brain sees is "LUN": in the
transport frame and in the code.

	Luben

P.S. Trust me, using "lun" would be quite ugly and it would
show that whoever coded it has had little experience reading
SCSI specs.  What you want to use is "u8 LUN[8]".

PPS. I hope I don't have to put up this sort of convincing
emails back and forth for each and every little thing.  We'd
get nowhere, no code will be written and no hardware would
work.