* [PATCH 0/3] remove cast from void* @ 2010-09-28 2:04 matt mooney 2010-09-28 2:04 ` matt mooney 2010-09-28 2:04 ` matt mooney 0 siblings, 2 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: kernel-janitors Hi guys, This is a resend to see if anyone wants to pick these up. They are now being sent to the trivial mailing list. Thanks, mfm matt mooney (3): arm: remove cast from void* powerpc: remove cast from void* trace: remove cast from void* arch/arm/mach-omap2/mailbox.c | 8 ++++---- arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- kernel/trace/ftrace.c | 4 ++-- kernel/trace/trace.c | 2 +- 4 files changed, 8 insertions(+), 8 deletions(-) -- 1.7.2.1 ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/3 RESEND] arm: remove cast from void* 2010-09-28 2:04 [PATCH 0/3] remove cast from void* matt mooney 2010-09-28 2:04 ` matt mooney @ 2010-09-28 2:04 ` matt mooney 1 sibling, 0 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: linux-arm-kernel Unnecessary cast from void* in assignment. Signed-off-by: matt mooney <mfm@muteddisk.com> Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> --- arch/arm/mach-omap2/mailbox.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c index 42dbfa4..40ddeca 100644 --- a/arch/arm/mach-omap2/mailbox.c +++ b/arch/arm/mach-omap2/mailbox.c @@ -181,7 +181,7 @@ static int omap2_mbox_fifo_full(struct omap_mbox *mbox) static void omap2_mbox_enable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq = IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqenable); @@ -192,7 +192,7 @@ static void omap2_mbox_enable_irq(struct omap_mbox *mbox, static void omap2_mbox_disable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq = IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqdisable); l &= ~bit; @@ -202,7 +202,7 @@ static void omap2_mbox_disable_irq(struct omap_mbox *mbox, static void omap2_mbox_ack_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq = IRQ_TX) ? p->notfull_bit : p->newmsg_bit; mbox_write_reg(bit, p->irqstatus); @@ -214,7 +214,7 @@ static void omap2_mbox_ack_irq(struct omap_mbox *mbox, static int omap2_mbox_is_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq = IRQ_TX) ? p->notfull_bit : p->newmsg_bit; u32 enable = mbox_read_reg(p->irqenable); u32 status = mbox_read_reg(p->irqstatus); -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 1/3 RESEND] arm: remove cast from void* @ 2010-09-28 2:04 ` matt mooney 0 siblings, 0 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: linux-arm-kernel Unnecessary cast from void* in assignment. Signed-off-by: matt mooney <mfm@muteddisk.com> Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> --- arch/arm/mach-omap2/mailbox.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c index 42dbfa4..40ddeca 100644 --- a/arch/arm/mach-omap2/mailbox.c +++ b/arch/arm/mach-omap2/mailbox.c @@ -181,7 +181,7 @@ static int omap2_mbox_fifo_full(struct omap_mbox *mbox) static void omap2_mbox_enable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqenable); @@ -192,7 +192,7 @@ static void omap2_mbox_enable_irq(struct omap_mbox *mbox, static void omap2_mbox_disable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqdisable); l &= ~bit; @@ -202,7 +202,7 @@ static void omap2_mbox_disable_irq(struct omap_mbox *mbox, static void omap2_mbox_ack_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; mbox_write_reg(bit, p->irqstatus); @@ -214,7 +214,7 @@ static void omap2_mbox_ack_irq(struct omap_mbox *mbox, static int omap2_mbox_is_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; u32 enable = mbox_read_reg(p->irqenable); u32 status = mbox_read_reg(p->irqstatus); -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 1/3 RESEND] arm: remove cast from void* @ 2010-09-28 2:04 ` matt mooney 0 siblings, 0 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: Jiri Kosina Cc: Tony Lindgren, kernel-janitors, linux-omap, Russell King, linux-arm-kernel Unnecessary cast from void* in assignment. Signed-off-by: matt mooney <mfm@muteddisk.com> Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> --- arch/arm/mach-omap2/mailbox.c | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/arch/arm/mach-omap2/mailbox.c b/arch/arm/mach-omap2/mailbox.c index 42dbfa4..40ddeca 100644 --- a/arch/arm/mach-omap2/mailbox.c +++ b/arch/arm/mach-omap2/mailbox.c @@ -181,7 +181,7 @@ static int omap2_mbox_fifo_full(struct omap_mbox *mbox) static void omap2_mbox_enable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqenable); @@ -192,7 +192,7 @@ static void omap2_mbox_enable_irq(struct omap_mbox *mbox, static void omap2_mbox_disable_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 l, bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; l = mbox_read_reg(p->irqdisable); l &= ~bit; @@ -202,7 +202,7 @@ static void omap2_mbox_disable_irq(struct omap_mbox *mbox, static void omap2_mbox_ack_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; mbox_write_reg(bit, p->irqstatus); @@ -214,7 +214,7 @@ static void omap2_mbox_ack_irq(struct omap_mbox *mbox, static int omap2_mbox_is_irq(struct omap_mbox *mbox, omap_mbox_type_t irq) { - struct omap_mbox2_priv *p = (struct omap_mbox2_priv *)mbox->priv; + struct omap_mbox2_priv *p = mbox->priv; u32 bit = (irq == IRQ_TX) ? p->notfull_bit : p->newmsg_bit; u32 enable = mbox_read_reg(p->irqenable); u32 status = mbox_read_reg(p->irqstatus); -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 1/3 RESEND] arm: remove cast from void* 2010-09-28 2:04 ` matt mooney (?) @ 2010-09-28 16:55 ` Tony Lindgren -1 siblings, 0 replies; 13+ messages in thread From: Tony Lindgren @ 2010-09-28 16:55 UTC (permalink / raw) To: linux-arm-kernel * matt mooney <mfm@muteddisk.com> [100927 18:57]: > Unnecessary cast from void* in assignment. > > Signed-off-by: matt mooney <mfm@muteddisk.com> > Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> Thanks, queuing this for the upcoming merge window. Tony ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 1/3 RESEND] arm: remove cast from void* @ 2010-09-28 16:55 ` Tony Lindgren 0 siblings, 0 replies; 13+ messages in thread From: Tony Lindgren @ 2010-09-28 16:55 UTC (permalink / raw) To: linux-arm-kernel * matt mooney <mfm@muteddisk.com> [100927 18:57]: > Unnecessary cast from void* in assignment. > > Signed-off-by: matt mooney <mfm@muteddisk.com> > Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> Thanks, queuing this for the upcoming merge window. Tony ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 1/3 RESEND] arm: remove cast from void* @ 2010-09-28 16:55 ` Tony Lindgren 0 siblings, 0 replies; 13+ messages in thread From: Tony Lindgren @ 2010-09-28 16:55 UTC (permalink / raw) To: matt mooney Cc: Jiri Kosina, Russell King, linux-omap, linux-arm-kernel, kernel-janitors * matt mooney <mfm@muteddisk.com> [100927 18:57]: > Unnecessary cast from void* in assignment. > > Signed-off-by: matt mooney <mfm@muteddisk.com> > Acked-by: Hiroshi DOYU <Hiroshi.DOYU@nokia.com> Thanks, queuing this for the upcoming merge window. Tony ^ permalink raw reply [flat|nested] 13+ messages in thread
* [PATCH 2/3 RESEND] powerpc: remove cast from void* 2010-09-28 2:04 [PATCH 0/3] remove cast from void* matt mooney @ 2010-09-28 2:04 ` matt mooney 2010-09-28 2:04 ` matt mooney 1 sibling, 0 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: Jiri Kosina; +Cc: kernel-janitors, Paul Mackerras, linuxppc-dev Unnecessary cast from void* in assignment. Signed-off-by: matt mooney <mfm@muteddisk.com> --- arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c index e19ff02..f106662 100644 --- a/arch/powerpc/platforms/pseries/hvCall_inst.c +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) static int hc_show(struct seq_file *m, void *p) { unsigned long h_num = (unsigned long)p; - struct hcall_stats *hs = (struct hcall_stats *)m->private; + struct hcall_stats *hs = m->private; if (hs[h_num].num_calls) { if (cpu_has_feature(CPU_FTR_PURR)) -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* [PATCH 2/3 RESEND] powerpc: remove cast from void* @ 2010-09-28 2:04 ` matt mooney 0 siblings, 0 replies; 13+ messages in thread From: matt mooney @ 2010-09-28 2:04 UTC (permalink / raw) To: Jiri Kosina; +Cc: kernel-janitors, Paul Mackerras, linuxppc-dev Unnecessary cast from void* in assignment. Signed-off-by: matt mooney <mfm@muteddisk.com> --- arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c index e19ff02..f106662 100644 --- a/arch/powerpc/platforms/pseries/hvCall_inst.c +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) static int hc_show(struct seq_file *m, void *p) { unsigned long h_num = (unsigned long)p; - struct hcall_stats *hs = (struct hcall_stats *)m->private; + struct hcall_stats *hs = m->private; if (hs[h_num].num_calls) { if (cpu_has_feature(CPU_FTR_PURR)) -- 1.7.2.1 ^ permalink raw reply related [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3 RESEND] powerpc: remove cast from void* 2010-09-28 2:04 ` matt mooney @ 2010-11-03 14:15 ` Jiri Kosina -1 siblings, 0 replies; 13+ messages in thread From: Jiri Kosina @ 2010-11-03 14:15 UTC (permalink / raw) To: matt mooney; +Cc: kernel-janitors, Paul Mackerras, linuxppc-dev On Mon, 27 Sep 2010, matt mooney wrote: > Unnecessary cast from void* in assignment. > > Signed-off-by: matt mooney <mfm@muteddisk.com> > --- > arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c > index e19ff02..f106662 100644 > --- a/arch/powerpc/platforms/pseries/hvCall_inst.c > +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c > @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) > static int hc_show(struct seq_file *m, void *p) > { > unsigned long h_num = (unsigned long)p; > - struct hcall_stats *hs = (struct hcall_stats *)m->private; > + struct hcall_stats *hs = m->private; > > if (hs[h_num].num_calls) { > if (cpu_has_feature(CPU_FTR_PURR)) This doesn't seem to be present in 37-rc1, so I am taking it to trivial queue. If this has been already merged to another tree, let me know. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3 RESEND] powerpc: remove cast from void* @ 2010-11-03 14:15 ` Jiri Kosina 0 siblings, 0 replies; 13+ messages in thread From: Jiri Kosina @ 2010-11-03 14:15 UTC (permalink / raw) To: matt mooney; +Cc: kernel-janitors, Paul Mackerras, linuxppc-dev On Mon, 27 Sep 2010, matt mooney wrote: > Unnecessary cast from void* in assignment. > > Signed-off-by: matt mooney <mfm@muteddisk.com> > --- > arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- > 1 files changed, 1 insertions(+), 1 deletions(-) > > diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c > index e19ff02..f106662 100644 > --- a/arch/powerpc/platforms/pseries/hvCall_inst.c > +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c > @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) > static int hc_show(struct seq_file *m, void *p) > { > unsigned long h_num = (unsigned long)p; > - struct hcall_stats *hs = (struct hcall_stats *)m->private; > + struct hcall_stats *hs = m->private; > > if (hs[h_num].num_calls) { > if (cpu_has_feature(CPU_FTR_PURR)) This doesn't seem to be present in 37-rc1, so I am taking it to trivial queue. If this has been already merged to another tree, let me know. Thanks, -- Jiri Kosina SUSE Labs, Novell Inc. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3 RESEND] powerpc: remove cast from void* 2010-11-03 14:15 ` Jiri Kosina @ 2010-11-03 14:18 ` Benjamin Herrenschmidt -1 siblings, 0 replies; 13+ messages in thread From: Benjamin Herrenschmidt @ 2010-11-03 14:18 UTC (permalink / raw) To: Jiri Kosina; +Cc: matt mooney, linuxppc-dev, kernel-janitors, Paul Mackerras On Wed, 2010-11-03 at 10:15 -0400, Jiri Kosina wrote: > On Mon, 27 Sep 2010, matt mooney wrote: > > > Unnecessary cast from void* in assignment. > > > > Signed-off-by: matt mooney <mfm@muteddisk.com> > > --- > > arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c > > index e19ff02..f106662 100644 > > --- a/arch/powerpc/platforms/pseries/hvCall_inst.c > > +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c > > @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) > > static int hc_show(struct seq_file *m, void *p) > > { > > unsigned long h_num = (unsigned long)p; > > - struct hcall_stats *hs = (struct hcall_stats *)m->private; > > + struct hcall_stats *hs = m->private; > > > > if (hs[h_num].num_calls) { > > if (cpu_has_feature(CPU_FTR_PURR)) > > This doesn't seem to be present in 37-rc1, so I am taking it to trivial > queue. If this has been already merged to another tree, let me know. Looks like I missed it, but Ack. Cheers, Ben. ^ permalink raw reply [flat|nested] 13+ messages in thread
* Re: [PATCH 2/3 RESEND] powerpc: remove cast from void* @ 2010-11-03 14:18 ` Benjamin Herrenschmidt 0 siblings, 0 replies; 13+ messages in thread From: Benjamin Herrenschmidt @ 2010-11-03 14:18 UTC (permalink / raw) To: Jiri Kosina; +Cc: matt mooney, linuxppc-dev, kernel-janitors, Paul Mackerras On Wed, 2010-11-03 at 10:15 -0400, Jiri Kosina wrote: > On Mon, 27 Sep 2010, matt mooney wrote: > > > Unnecessary cast from void* in assignment. > > > > Signed-off-by: matt mooney <mfm@muteddisk.com> > > --- > > arch/powerpc/platforms/pseries/hvCall_inst.c | 2 +- > > 1 files changed, 1 insertions(+), 1 deletions(-) > > > > diff --git a/arch/powerpc/platforms/pseries/hvCall_inst.c b/arch/powerpc/platforms/pseries/hvCall_inst.c > > index e19ff02..f106662 100644 > > --- a/arch/powerpc/platforms/pseries/hvCall_inst.c > > +++ b/arch/powerpc/platforms/pseries/hvCall_inst.c > > @@ -55,7 +55,7 @@ static void hc_stop(struct seq_file *m, void *p) > > static int hc_show(struct seq_file *m, void *p) > > { > > unsigned long h_num = (unsigned long)p; > > - struct hcall_stats *hs = (struct hcall_stats *)m->private; > > + struct hcall_stats *hs = m->private; > > > > if (hs[h_num].num_calls) { > > if (cpu_has_feature(CPU_FTR_PURR)) > > This doesn't seem to be present in 37-rc1, so I am taking it to trivial > queue. If this has been already merged to another tree, let me know. Looks like I missed it, but Ack. Cheers, Ben. ^ permalink raw reply [flat|nested] 13+ messages in thread
end of thread, other threads:[~2010-11-03 14:18 UTC | newest] Thread overview: 13+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2010-09-28 2:04 [PATCH 0/3] remove cast from void* matt mooney 2010-09-28 2:04 ` [PATCH 1/3 RESEND] arm: " matt mooney 2010-09-28 2:04 ` matt mooney 2010-09-28 2:04 ` matt mooney 2010-09-28 16:55 ` Tony Lindgren 2010-09-28 16:55 ` Tony Lindgren 2010-09-28 16:55 ` Tony Lindgren 2010-09-28 2:04 ` [PATCH 2/3 RESEND] powerpc: " matt mooney 2010-09-28 2:04 ` matt mooney 2010-11-03 14:15 ` Jiri Kosina 2010-11-03 14:15 ` Jiri Kosina 2010-11-03 14:18 ` Benjamin Herrenschmidt 2010-11-03 14:18 ` Benjamin Herrenschmidt
This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.