Messages in this thread | | | Date | Sat, 8 Jan 2005 09:41:17 +0100 | From | Jens Axboe <> | Subject | Re: [PATCH 2.6] cciss typo fix |
| |
On Fri, Jan 07 2005, Miller, Mike (OS Dev) wrote: > > -----Original Message----- > > From: James Bottomley [mailto:James.Bottomley@SteelEye.com] > > > > > > On Fri, 2005-01-07 at 17:01 -0600, mike.miller@hp.com wrote: > > > - *total_size = be32_to_cpu(*((__be32 *) > > &buf->total_size[0]))+1; > > > - *block_size = be32_to_cpu(*((__be32 *) > > &buf->block_size[0])); > > > + *total_size = be32_to_cpu(*((__u32 *) > > &buf->total_size[0]))+1; > > > + *block_size = be32_to_cpu(*((__u32 *) > > &buf->block_size[0])); > > > > I don't think that's a typo. It was introduced by this patch: > > > > ChangeSet 1.1988.24.79 2004/10/06 07:55:02 viro@www.linux.org.uk > > [PATCH] cciss endianness and iomem annotations > > > > The idea being that BE and LE numbers should be annotated differently, > > so the __be32 annotations look correct to me. I think sparse > > will warn > > if you make this change. > > Hmmm, SuSE complained that __be32 was not defined in the kernel. Any > other thoughts, anyone?
Hmm odd, no one should have complained, it should just have been added to the compat header.
-- Jens Axboe
- 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/
| |