lkml.org 
[lkml]   [2014]   [May]   [5]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH 1/1] FS/CIFS: remove obsolete __constant
On Sun, 4 May 2014 18:52:43 -0400
Jeff Layton <jlayton@poochiereds.net> wrote:

> On Sat, May 3, 2014 at 4:15 PM, Fabian Frederick <fabf@skynet.be> wrote:
> > Replacing all __constant_foo to foo()
> > except in smb2status.h (1700 lines to update).
> >
> > Cc: linux-cifs@vger.kernel.org
> > Cc: Steve French <sfrench@samba.org>
> > Cc: Andrew Morton <akpm@linux-foundation.org>
> > Signed-off-by: Fabian Frederick <fabf@skynet.be>
> > ---
> > fs/cifs/cifsacl.c | 2 +-
> > fs/cifs/cifssmb.c | 20 ++++++++++----------
> > fs/cifs/sess.c | 2 +-
> > fs/cifs/smb2misc.c | 38 +++++++++++++++++++-------------------
> > fs/cifs/smb2ops.c | 2 +-
> > fs/cifs/smb2pdu.c | 2 +-
> > fs/cifs/smb2pdu.h | 28 ++++++++++++++--------------
> > 7 files changed, 47 insertions(+), 47 deletions(-)
> >
> > diff --git a/fs/cifs/cifsacl.c b/fs/cifs/cifsacl.c
> > index 7ff866d..54ac0e8 100644
> > --- a/fs/cifs/cifsacl.c
> > +++ b/fs/cifs/cifsacl.c
> > @@ -38,7 +38,7 @@ static const struct cifs_sid sid_everyone = {
> > 1, 1, {0, 0, 0, 0, 0, 1}, {0} };
> > /* security id for Authenticated Users system group */
> > static const struct cifs_sid sid_authusers = {
> > - 1, 1, {0, 0, 0, 0, 0, 5}, {__constant_cpu_to_le32(11)} };
> > + 1, 1, {0, 0, 0, 0, 0, 5}, {cpu_to_le32(11)} };
>
>
> Does the build still work on BE arches with this? I know at one point
> the above wouldn't compile on those
> arches. See commit 536abdb0802f, for an explanation.

Untested on BE but it would mean checkpatch
"don't use __constant<foo> functions outside of include/uapi/" stuff is wrong ?


\
 
 \ /
  Last update: 2014-05-05 07:21    [W:0.900 / U:0.064 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site