* A change to scsi.h
@ 2003-02-11 15:54 Peter Leif Rasmussen (PLR)
2003-02-11 16:05 ` Christoph Hellwig
0 siblings, 1 reply; 2+ messages in thread
From: Peter Leif Rasmussen (PLR) @ 2003-02-11 15:54 UTC (permalink / raw)
To: 'linux-kernel@vger.kernel.org'
When looking into:
/usr/src/linux/include/scsi/scsi.h
in kernel revision 2.5.60 I find this in line 200 - 205:
/*
* ScsiLun: 8 byte LUN.
*/
typedef struct scsi_lun {
u8 scsi_lun[8];
} ScsiLun;
This produces problems when compiling a package that doesn't know about
'u8'.
In any case shouldn't it be 'uint8_t' so we get:
/*
* ScsiLun: 8 byte LUN.
*/
typedef struct scsi_lun {
uint8_t scsi_lun[8];
} ScsiLun;
Just curious from a non-regular on lkml (and I am getting out quick because
of the volume :-)
Thank you very much,
Peter
^ permalink raw reply [flat|nested] 2+ messages in thread* Re: A change to scsi.h
2003-02-11 15:54 A change to scsi.h Peter Leif Rasmussen (PLR)
@ 2003-02-11 16:05 ` Christoph Hellwig
0 siblings, 0 replies; 2+ messages in thread
From: Christoph Hellwig @ 2003-02-11 16:05 UTC (permalink / raw)
To: Peter Leif Rasmussen (PLR); +Cc: 'linux-kernel@vger.kernel.org'
On Tue, Feb 11, 2003 at 04:54:16PM +0100, Peter Leif Rasmussen (PLR) wrote:
> When looking into:
>
> /usr/src/linux/include/scsi/scsi.h
>
> in kernel revision 2.5.60 I find this in line 200 - 205:
>
> /*
> * ScsiLun: 8 byte LUN.
> */
> typedef struct scsi_lun {
> u8 scsi_lun[8];
> } ScsiLun;
>
> This produces problems when compiling a package that doesn't know about
> 'u8'.
Userland code is not supposed to include kernel headers.
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2003-02-11 15:55 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2003-02-11 15:54 A change to scsi.h Peter Leif Rasmussen (PLR)
2003-02-11 16:05 ` Christoph Hellwig
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®