* [PATCH] compat: crc8: undef pr_fmt
@ 2012-03-30 11:13 Johannes Berg
2012-03-30 11:49 ` Arend van Spriel
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2012-03-30 11:13 UTC (permalink / raw)
To: Luis R. Rodriguez; +Cc: linux-wireless
From: Johannes Berg <johannes.berg@intel.com>
Signed-off-by: Johannes Berg <johannes.berg@intel.com>
---
compat/crc8.c | 3 +++
1 files changed, 3 insertions(+), 0 deletions(-)
diff --git a/compat/crc8.c b/compat/crc8.c
index fa8382d..753a5cb 100644
--- a/compat/crc8.c
+++ b/compat/crc8.c
@@ -14,6 +14,9 @@
* CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
*/
+#ifdef pr_fmt
+#undef pr_fmt
+#endif
#define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
#include <linux/module.h>
--
1.7.9.1
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: [PATCH] compat: crc8: undef pr_fmt
2012-03-30 11:13 [PATCH] compat: crc8: undef pr_fmt Johannes Berg
@ 2012-03-30 11:49 ` Arend van Spriel
2012-03-30 11:54 ` Johannes Berg
0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2012-03-30 11:49 UTC (permalink / raw)
To: Johannes Berg; +Cc: Luis R. Rodriguez, linux-wireless
On 03/30/2012 01:13 PM, Johannes Berg wrote:
>
> +#ifdef pr_fmt
> +#undef pr_fmt
> +#endif
> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>
> #include<linux/module.h>
Trivial remark. Not really necessary to do the #ifdef. #undef works fine
if pr_fmt has not been defined.
Gr. AvS
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compat: crc8: undef pr_fmt
2012-03-30 11:49 ` Arend van Spriel
@ 2012-03-30 11:54 ` Johannes Berg
2012-03-30 12:14 ` Arend van Spriel
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2012-03-30 11:54 UTC (permalink / raw)
To: Arend van Spriel; +Cc: Luis R. Rodriguez, linux-wireless
On Fri, 2012-03-30 at 13:49 +0200, Arend van Spriel wrote:
> On 03/30/2012 01:13 PM, Johannes Berg wrote:
> >
> > +#ifdef pr_fmt
> > +#undef pr_fmt
> > +#endif
> > #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> >
> > #include<linux/module.h>
>
> Trivial remark. Not really necessary to do the #ifdef. #undef works fine
> if pr_fmt has not been defined.
Really? I thought it would give a warning then, but couldn't test it
since it was defined for me.
johannes
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compat: crc8: undef pr_fmt
2012-03-30 11:54 ` Johannes Berg
@ 2012-03-30 12:14 ` Arend van Spriel
2012-03-30 12:28 ` Johannes Berg
0 siblings, 1 reply; 6+ messages in thread
From: Arend van Spriel @ 2012-03-30 12:14 UTC (permalink / raw)
To: Johannes Berg; +Cc: Luis R. Rodriguez, linux-wireless
On 03/30/2012 01:54 PM, Johannes Berg wrote:
> On Fri, 2012-03-30 at 13:49 +0200, Arend van Spriel wrote:
>> On 03/30/2012 01:13 PM, Johannes Berg wrote:
>>>
>>> +#ifdef pr_fmt
>>> +#undef pr_fmt
>>> +#endif
>>> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
>>>
>>> #include<linux/module.h>
>>
>> Trivial remark. Not really necessary to do the #ifdef. #undef works fine
>> if pr_fmt has not been defined.
>
> Really? I thought it would give a warning then, but couldn't test it
> since it was defined for me.
>
> johannes
>
>
I made a little test program to confirm my thought, before sending my
previous email. Retried compiling with '-W -Wall -ansi -pedantic' and no
complaints about the #undef.
Gr. AvS
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compat: crc8: undef pr_fmt
2012-03-30 12:14 ` Arend van Spriel
@ 2012-03-30 12:28 ` Johannes Berg
2012-03-30 21:23 ` Luis R. Rodriguez
0 siblings, 1 reply; 6+ messages in thread
From: Johannes Berg @ 2012-03-30 12:28 UTC (permalink / raw)
To: Arend van Spriel; +Cc: Luis R. Rodriguez, linux-wireless
On Fri, 2012-03-30 at 14:14 +0200, Arend van Spriel wrote:
> On 03/30/2012 01:54 PM, Johannes Berg wrote:
> > On Fri, 2012-03-30 at 13:49 +0200, Arend van Spriel wrote:
> >> On 03/30/2012 01:13 PM, Johannes Berg wrote:
> >>>
> >>> +#ifdef pr_fmt
> >>> +#undef pr_fmt
> >>> +#endif
> >>> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> >>>
> >>> #include<linux/module.h>
> >>
> >> Trivial remark. Not really necessary to do the #ifdef. #undef works fine
> >> if pr_fmt has not been defined.
> >
> > Really? I thought it would give a warning then, but couldn't test it
> > since it was defined for me.
> >
> > johannes
> >
> >
>
> I made a little test program to confirm my thought, before sending my
> previous email. Retried compiling with '-W -Wall -ansi -pedantic' and no
> complaints about the #undef.
Ok. I'm too lazy to send a new patch, but maybe Luis wants to change it
when he applies it :)
johannes
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] compat: crc8: undef pr_fmt
2012-03-30 12:28 ` Johannes Berg
@ 2012-03-30 21:23 ` Luis R. Rodriguez
0 siblings, 0 replies; 6+ messages in thread
From: Luis R. Rodriguez @ 2012-03-30 21:23 UTC (permalink / raw)
To: Johannes Berg; +Cc: Arend van Spriel, Luis R. Rodriguez, linux-wireless
On Fri, Mar 30, 2012 at 02:28:30PM +0200, Johannes Berg wrote:
> On Fri, 2012-03-30 at 14:14 +0200, Arend van Spriel wrote:
> > On 03/30/2012 01:54 PM, Johannes Berg wrote:
> > > On Fri, 2012-03-30 at 13:49 +0200, Arend van Spriel wrote:
> > >> On 03/30/2012 01:13 PM, Johannes Berg wrote:
> > >>>
> > >>> +#ifdef pr_fmt
> > >>> +#undef pr_fmt
> > >>> +#endif
> > >>> #define pr_fmt(fmt) KBUILD_MODNAME ": " fmt
> > >>>
> > >>> #include<linux/module.h>
> > >>
> > >> Trivial remark. Not really necessary to do the #ifdef. #undef works fine
> > >> if pr_fmt has not been defined.
> > >
> > > Really? I thought it would give a warning then, but couldn't test it
> > > since it was defined for me.
> > >
> > > johannes
> > >
> > >
> >
> > I made a little test program to confirm my thought, before sending my
> > previous email. Retried compiling with '-W -Wall -ansi -pedantic' and no
> > complaints about the #undef.
>
> Ok. I'm too lazy to send a new patch, but maybe Luis wants to change it
> when he applies it :)
I changed it, and applied, thanks!!!
Luis
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2012-03-30 21:23 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2012-03-30 11:13 [PATCH] compat: crc8: undef pr_fmt Johannes Berg
2012-03-30 11:49 ` Arend van Spriel
2012-03-30 11:54 ` Johannes Berg
2012-03-30 12:14 ` Arend van Spriel
2012-03-30 12:28 ` Johannes Berg
2012-03-30 21:23 ` Luis R. Rodriguez
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).