To be consistent with the use of attributes in the rest of the kernel
replace all use of __attribute_pure__ with __pure and delete the
definition of __attribute_pure__.Signed-off-by: Ralf Baechle <ralf@linux-mips.org>
---
Patch against v2.6.23-rc8-mm2.And btw why does Analog list half their employees in the MAINTAINERS entry?
Seems a little over the top ...Ralf
diff --git a/arch/arm/nwfpe/fpopcode.h b/arch/arm/nwfpe/fpopcode.h
index ec78e35..0090b19 100644
--- a/arch/arm/nwfpe/fpopcode.h
+++ b/arch/arm/nwfpe/fpopcode.h
@@ -369,20 +369,20 @@ TABLE 5
#define getRoundingMode(opcode) ((opcode & MASK_ROUNDING_MODE) >> 5)#ifdef CONFIG_FPE_NWFPE_XP
-static inline __attribute_pure__ floatx80 getExtendedConstant(const unsigned int nIndex)
+static inline floatx80 __pure getExtendedConstant(const unsigned int nIndex)
{
extern const floatx80 floatx80Constant[];
return floatx80Constant[nIndex];
}
#endif-static inline __attribute_pure__ float64 getDoubleConstant(const unsigned int nIndex)
+static inline float64 __pure getDoubleConstant(const unsigned int nIndex)
{
extern const float64 float64Constant[];
return float64Constant[nIndex];
}-static inline __attribute_pure__ float32 getSingleConstant(const unsigned int nIndex)
+static inline float32 __pure getSingleConstant(const unsigned int nIndex)
{
extern const float32 float32Constant[];
return float32Constant[nIndex];
diff --git a/drivers/media/video/v4l1-compat.c b/drivers/media/video/v4l1-compat.c
index 9eac65f..dcf22a3 100644
--- a/drivers/media/video/v4l1-compat.c
+++ b/drivers/media/video/v4l1-compat.c
@@ -144,7 +144,7 @@ const static unsigned int palette2pixelformat[] = {
[VIDEO_PALETTE_YUV422P] = V4L2_PIX_FMT_YUV422P,
};-static unsigned int __attribute_pure__
+static unsigned int __pure
palette_to_pixelformat(unsigned int palette)
{
if (palette < ARRAY_SIZE(palette2pixelformat))
diff --git a/include/asm-blackfin/processor.h b/include/asm-blackfin/pr...
Concern: __attribute_pure__ is very similar to __attribute_const__,
which is almost completely, but not totally unlike the keyword "const"...-hpa
-
Trimmed the CC list a bit
Yes, there's also the fact that __pure is a reserved GCC keyword.
Essentially according to GCC docs all of the GCC-specific keywords
are equivalently defined as "keyword", "__keyword", and
"__keyword__", with only the latter two defined in strict-ANSI mode.
The following is valid according to GCC docs:static int __attribute__((__pure)) my_strlen(const char *str);
With the proposed definition of __pure, that becomes a noticeably
invalid __attribute__((__attribute__((__pure__))))Cheers,
Kyle Moffett-
Yeah, the original submission got a little carried away...
We can cut that down to just Bryan and the mailing list I think.
-Robin
-
doesnt matter one way or the other to me, so:
this is supposed to be replaced with just the mailing list
-mike
-
Acked-by: Mauro Carvalho Chehab <mchehab@infradead.org>
Cheers,
Mauro-
| Bart Van Assche | Integration of SCST in the mainstream Linux kernel |
| Greg KH | [GIT PATCH] driver core patches against 2.6.24 |
| Roland McGrath | Re: Linus 2.6.23-rc1 |
| Rafael J. Wysocki | [Bug #10984] MMC print trace information when resume from suspend |
git: | |
| Gerrit Renker | [PATCH 03/37] dccp: List management for new feature negotiation |
| Corey Minyard | [PATCH 3/3] Convert the UDP hash lock to RCU |
| David Miller | [GIT]: Networking |
| David Miller | xfrm_state locking regression... |
