2009-10-09 15:05:02

by Jiri Kosina

[permalink] [raw]
Subject: Re: [PATCH] char/vme_scc: adding __init macro to vme_scc.c

On Thu, 20 Aug 2009, Matheos Worku wrote:

> > Alan or Geert, please have a look at the small patch and either pull it
> > through your tree, or please ack' it so Jiri can pull it through the trivial
> > tree.
> >
> > linux version 2.6.31-rc6 - linus git tree, Do 20. Aug 22:26:06 CEST 2009
> >
> > Signed-off-by: Peter Huewe <[email protected]>
> > ---
> > diff --git a/drivers/char/vme_scc.c b/drivers/char/vme_scc.c
> > index 994e1a5..8b24729 100644
> > --- a/drivers/char/vme_scc.c
> > +++ b/drivers/char/vme_scc.c
> > @@ -136,7 +136,7 @@ static const struct tty_port_operations scc_port_ops = {
> > * vme_scc_init() and support functions
> > *---------------------------------------------------------------------------*/
> > -static int scc_init_drivers(void)
> > +static int __init scc_init_drivers(void)
> > {
> > int error;
> > @@ -172,7 +172,7 @@ static int scc_init_drivers(void)
> > /* ports[] array is indexed by line no (i.e. [0] for ttyS0, [1] for ttyS1).
> > */
> > -static void scc_init_portstructs(void)
> > +static void __init scc_init_portstructs(void)
> > {
> > struct scc_port *port;
> > int i;
> > @@ -195,7 +195,7 @@ static void scc_init_portstructs(void)
> > #ifdef CONFIG_MVME147_SCC
> > -static int mvme147_scc_init(void)
> > +static int __init mvme147_scc_init(void)
> > {
> > struct scc_port *port;
> > int error;
> > @@ -298,7 +298,7 @@ fail:
> > #ifdef CONFIG_MVME162_SCC
> > -static int mvme162_scc_init(void)
> > +static int __init mvme162_scc_init(void)
> > {
> > struct scc_port *port;
> > int error;
> > @@ -404,7 +404,7 @@ fail:
> > #ifdef CONFIG_BVME6000_SCC
> > -static int bvme6000_scc_init(void)
> > +static int __init bvme6000_scc_init(void)
> > {
> > struct scc_port *port;
> > int error;
> > @@ -503,7 +503,7 @@ fail_free_b_rx:
> > #endif
> > -static int vme_scc_init(void)
> > +static int __init vme_scc_init(void)
> > {
> > int res = -ENODEV;
> Then there are issnvram.txt iss.conf etc ... I guess we need to change
> these as well. How about error messages? I assume they would have ISS in
> them.

Confused ... how is this related to the patch?

--
Jiri Kosina
SUSE Labs, Novell Inc.


2009-12-22 07:10:40

by Peter Huewe

[permalink] [raw]
Subject: Re: [PATCH] char/vme_scc: adding __init macro to vme_scc.c

Am Freitag 09 Oktober 2009 17:04:22 schrieb Jiri Kosina:
> On Thu, 20 Aug 2009, Matheos Worku wrote:
> > > Alan or Geert, please have a look at the small patch and either pull it
> > > through your tree, or please ack' it so Jiri can pull it through the
> > > trivial tree.
> > >
> > > linux version 2.6.31-rc6 - linus git tree, Do 20. Aug 22:26:06 CEST
> > > 2009
> > >
> > > Signed-off-by: Peter Huewe <[email protected]>
> > > ---
> > > diff --git a/drivers/char/vme_scc.c b/drivers/char/vme_scc.c
> > > index 994e1a5..8b24729 100644
> > > --- a/drivers/char/vme_scc.c
> > > +++ b/drivers/char/vme_scc.c
> > > @@ -136,7 +136,7 @@ static const struct tty_port_operations
> > > scc_port_ops = { * vme_scc_init() and support functions
> > >
> > > *----------------------------------------------------------------------
> > >-----*/ -static int scc_init_drivers(void)
> > > +static int __init scc_init_drivers(void)
> > > {
> > > int error;
> > > @@ -172,7 +172,7 @@ static int scc_init_drivers(void)
> > > /* ports[] array is indexed by line no (i.e. [0] for ttyS0, [1] for
> > > ttyS1). */
> > > -static void scc_init_portstructs(void)
> > > +static void __init scc_init_portstructs(void)
> > > {
> > > struct scc_port *port;
> > > int i;
> > > @@ -195,7 +195,7 @@ static void scc_init_portstructs(void)
> > > #ifdef CONFIG_MVME147_SCC
> > > -static int mvme147_scc_init(void)
> > > +static int __init mvme147_scc_init(void)
> > > {
> > > struct scc_port *port;
> > > int error;
> > > @@ -298,7 +298,7 @@ fail:
> > > #ifdef CONFIG_MVME162_SCC
> > > -static int mvme162_scc_init(void)
> > > +static int __init mvme162_scc_init(void)
> > > {
> > > struct scc_port *port;
> > > int error;
> > > @@ -404,7 +404,7 @@ fail:
> > > #ifdef CONFIG_BVME6000_SCC
> > > -static int bvme6000_scc_init(void)
> > > +static int __init bvme6000_scc_init(void)
> > > {
> > > struct scc_port *port;
> > > int error;
> > > @@ -503,7 +503,7 @@ fail_free_b_rx:
> > > #endif
> > > -static int vme_scc_init(void)
> > > +static int __init vme_scc_init(void)
> > > {
> > > int res = -ENODEV;
> >
> > Then there are issnvram.txt iss.conf etc ... I guess we need to change
> > these as well. How about error messages? I assume they would have ISS in
> > them.
>
> Confused ... how is this related to the patch?
>

Hi,

any updates on this one? This patch floats around for quite a while now
(2009/08/20)

@Matheos: Can you perhaps clearify your remark/objection?


Thanks,
Peter