lkml.org 
[lkml]   [2008]   [Sep]   [14]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/4] 24-bit types: typedef and functions for accessing 3-byte arrays as integers
Chris Leech wrote:
> On Thu, Sep 11, 2008 at 1:03 AM, Boaz Harrosh <bharrosh@panasas.com> wrote:
>>> diff --git a/fs/jfs/endian24.h b/fs/jfs/endian24.h
>>> index fa92f7f..45b6397 100644
>>> --- a/fs/jfs/endian24.h
>>> +++ b/fs/jfs/endian24.h
>>> @@ -33,6 +33,9 @@
>>> ((__x & (__u32)0x00ff0000UL) >> 16) )); \
>>> })
>>>
>>> +#undef __cpu_to_le24
>>> +#undef __le24_to_cpu
>>> +
>> Does #undef also work on inline functions? I didn't know that
>
> No, it doesn't and those two lines don't do anything. The two below
> do, so this file gets touched anyway, and the JFS patch deletes the
> entire file. I'll fix this is there's some other reason to redo the
> patchset again.
>
Sorry my bad, yes they get removed one patch later. No need to resubmit
just for that I think

>>> #if (defined(__KERNEL__) && defined(__LITTLE_ENDIAN)) || (defined(__BYTE_ORDER) && (__BYTE_ORDER == __LITTLE_ENDIAN))
>>> #define __cpu_to_le24(x) ((__u32)(x))
>>> #define __le24_to_cpu(x) ((__u32)(x))
>>> @@ -42,6 +45,8 @@
>>> #endif
>>>
>>> #ifdef __KERNEL__
>>> + #undef cpu_to_le24
>>> + #undef le24_to_cpu
>>> #define cpu_to_le24 __cpu_to_le24
>>> #define le24_to_cpu __le24_to_cpu
>>> #endif

Thanks again for doing this

Reviewed-by: Boaz Harrosh <bharrosh@panasas.com>

on all patches


\
 
 \ /
  Last update: 2008-09-14 12:49    [W:0.083 / U:0.048 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site