Re: [PATCH] IDE 104

From: Marcin Dalecki (dalecki@evision.ag)
Date: Fri Jul 26 2002 - 04:07:00 EST


Alan Cox wrote:
> On Fri, 2002-07-26 at 08:23, Marcin Dalecki wrote:
>
>>- Make the bit-sliced data types in hdreg.h use the bit-slice data types
>> instead of the generic ones. This makes clear that those are supposed
>> to be register masks.
>
>
> Because its an external API it needs to remain constant. Swapping

???

The only thing which is really 'external API' there is the declaration
of the HDIO_XXX ioctl and among them in reality only HDIO_GETGEO
      is really used outside the scope of the dreddy hdparm application.
And
99% of times its usage is bogous anyway. Or do you know any better
examples I'm not aware of?

The remainings will be moved away from there soon becouse it doesn't
make any sense to include this at every single place out there where
HDIO_GETGEO is the needed declaration. If some application needs ATA
command macros - it should come with they own header declaring them
instead of peeing in to the kernel becouse they don't depend on the
kernel in question. They are cut in stone at www.t13.org ;-).

> uchar/uint looks fine and the structure is off the drive anyway. However
> they should be __u16 __u8 __u32 so that C libraries and apps can still
> use the header (its __u16 because 'u16' cannot be exposed to user space
> directly or via libc)

I know - see above.

-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



This archive was generated by hypermail 2b29 : Tue Jul 30 2002 - 14:00:23 EST