Re: Re[2]: [PATCH v2 1/3] mfd: syscon: Removed support for unloading

2013-02-12 Thread Arnd Bergmann
On Tuesday 12 February 2013, Alexander Shiyan wrote:
> > On Monday 11 February 2013, Alexander Shiyan wrote:
> > > The driver can be used in various subsystems and therefore should not
> > > be unloaded when it is defined in the kernel configuration, so remove
> > > support for unloading it.
> > > 
> > > Signed-off-by: Alexander Shiyan 
> > 
> > Can you describe a scenario where that would happen? Normally the
> > module should stay pinned as long as any other module refers
> > to its exported symbols.
> 
> Probably I wrote a bad description.
> The driver registered by "postcore_initcall". Therefore, if we unregister the
> driver, we have no way to register it back. Fixme please.

Ah, the driver is actually "bool" in Kconfig rather than "tristate", so
it is not possible to build it as a module. Makes sense then, and if 
we ever want to turn this into a module, we can still revert your patch.

Arnd
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re: Re[2]: [PATCH v2 1/3] mfd: syscon: Removed support for unloading

2013-02-12 Thread Arnd Bergmann
On Tuesday 12 February 2013, Alexander Shiyan wrote:
  On Monday 11 February 2013, Alexander Shiyan wrote:
   The driver can be used in various subsystems and therefore should not
   be unloaded when it is defined in the kernel configuration, so remove
   support for unloading it.
   
   Signed-off-by: Alexander Shiyan shc_w...@mail.ru
  
  Can you describe a scenario where that would happen? Normally the
  module should stay pinned as long as any other module refers
  to its exported symbols.
 
 Probably I wrote a bad description.
 The driver registered by postcore_initcall. Therefore, if we unregister the
 driver, we have no way to register it back. Fixme please.

Ah, the driver is actually bool in Kconfig rather than tristate, so
it is not possible to build it as a module. Makes sense then, and if 
we ever want to turn this into a module, we can still revert your patch.

Arnd
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/


Re[2]: [PATCH v2 1/3] mfd: syscon: Removed support for unloading

2013-02-11 Thread Alexander Shiyan
Hello.

> On Monday 11 February 2013, Alexander Shiyan wrote:
> > The driver can be used in various subsystems and therefore should not
> > be unloaded when it is defined in the kernel configuration, so remove
> > support for unloading it.
> > 
> > Signed-off-by: Alexander Shiyan 
> 
> Can you describe a scenario where that would happen? Normally the
> module should stay pinned as long as any other module refers
> to its exported symbols.

Probably I wrote a bad description.
The driver registered by "postcore_initcall". Therefore, if we unregister the
driver, we have no way to register it back. Fixme please.

---
N�r��yb�X��ǧv�^�)޺{.n�+{zX����ܨ}���Ơz�:+v���zZ+��+zf���h���~i���z��w���?�&�)ߢf��^jǫy�m��@A�a���
0��h���i

Re[2]: [PATCH v2 1/3] mfd: syscon: Removed support for unloading

2013-02-11 Thread Alexander Shiyan
Hello.

 On Monday 11 February 2013, Alexander Shiyan wrote:
  The driver can be used in various subsystems and therefore should not
  be unloaded when it is defined in the kernel configuration, so remove
  support for unloading it.
  
  Signed-off-by: Alexander Shiyan shc_w...@mail.ru
 
 Can you describe a scenario where that would happen? Normally the
 module should stay pinned as long as any other module refers
 to its exported symbols.

Probably I wrote a bad description.
The driver registered by postcore_initcall. Therefore, if we unregister the
driver, we have no way to register it back. Fixme please.

---
N�r��yb�X��ǧv�^�)޺{.n�+{zX����ܨ}���Ơz�j:+v���zZ+��+zf���h���~i���z��w���?��)ߢf��^jǫy�m��@A�a���
0��h���i