Re: Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-08 Thread Dominik Brodowski 
On Sat, Nov 07, 2009 at 05:13:14PM +0100, Wolfram Sang wrote:
> > +   /*
> > +* Let's try to get the PCMCIA module for 16-bit PCMCIA support.
> > +* If it fails, it doesn't matter -- we still havae 32-bit CardBus
> > +* support to offer, so this is not a failure mode.
> > +*/
> > +   request_module_nowait("pcmcia");
> > +
> 
> Typo: "havae"

Thanks, fixed.

Best,
Dominik

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-07 Thread Komuro 


>Hey,
>
>> >yes, but why? Also, wouldn't request_module_nowait() suffice?
>> >
>> 
>
>What about this?
>

OK. Thanks,


Best Regards
Komuro


___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-07 Thread Wolfram Sang 
> + /*
> +  * Let's try to get the PCMCIA module for 16-bit PCMCIA support.
> +  * If it fails, it doesn't matter -- we still havae 32-bit CardBus
> +  * support to offer, so this is not a failure mode.
> +  */
> + request_module_nowait("pcmcia");
> +

Typo: "havae"

-- 
Pengutronix e.K.   | Wolfram Sang|
Industrial Linux Solutions | http://www.pengutronix.de/  |


signature.asc
Description: Digital signature
___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-07 Thread Dominik Brodowski 
Hey,

On Tue, Nov 03, 2009 at 08:22:48AM +0900, Komuro wrote:
> >> This patch integrates some of the pcmciautils feature into the kernel.
> >
> >yes, but why? Also, wouldn't request_module_nowait() suffice?
> >
> 
> (1)
> Because the "pcmcia" is kernel module, so I think it should be
> loaded by kernel, if kernel can do it.
> 
> (2)
> Even if distribution developer forgets to install the pcmciautils,
> the "pcmcia" is loaded with this patch.
> (especially embedded systems)
> 
> (3)
> I think request_module_nowait is better.

What about this?

Best,
Dominik


[PATCH] pcmcia: autoload module pcmcia

Attempt to load the "pcmcia" module for 16-bit PCMCIA cards, so that
PCMCIA support becomes available without pcmciautils/udev userspace
interaction. Based on a suggestion and a patch
Signed-off-by: Komuro 
but converted it to request_module_nowait() and move it to a later
stage.

Signed-off-by: Dominik Brodowski 

diff --git a/drivers/pcmcia/cs.c b/drivers/pcmcia/cs.c
index b229f6d..f56e88c 100644
--- a/drivers/pcmcia/cs.c
+++ b/drivers/pcmcia/cs.c
@@ -243,6 +243,13 @@ int pcmcia_register_socket(struct pcmcia_socket *socket)
 
pcmcia_parse_events(socket, SS_DETECT);
 
+   /*
+* Let's try to get the PCMCIA module for 16-bit PCMCIA support.
+* If it fails, it doesn't matter -- we still havae 32-bit CardBus
+* support to offer, so this is not a failure mode.
+*/
+   request_module_nowait("pcmcia");
+
return 0;
 
  err:

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-02 Thread Komuro 
Hi,

>> This patch integrates some of the pcmciautils feature into the kernel.
>
>yes, but why? Also, wouldn't request_module_nowait() suffice?
>

(1)
Because the "pcmcia" is kernel module, so I think it should be
loaded by kernel, if kernel can do it.

(2)
Even if distribution developer forgets to install the pcmciautils,
the "pcmcia" is loaded with this patch.
(especially embedded systems)

(3)
I think request_module_nowait is better.

Best Regards
Komuro



___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-02 Thread Dominik Brodowski 
Hey,

On Mon, Nov 02, 2009 at 09:37:51PM +0900, Komuro wrote:
> This patch integrates some of the pcmciautils feature into the kernel.

yes, but why? Also, wouldn't request_module_nowait() suffice?

Best,
Dominik

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-02 Thread Komuro 
Hi,

This patch integrates some of the pcmciautils feature into the kernel.


>
>As I'm a bit unsure about it: what's the general feeling about
>this patch?
>
>On Sun, Nov 01, 2009 at 12:04:30PM +0900, Komuro wrote:
>> 
>> load "pcmcia" module for 16-bit PCMCIA cards
>> without help of userland script.
>> 
>> 
>> Signed-off-by: Komuro 
>> 
>> ---
>> 
>> --- linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c.orig   2009-11-01 11:50:57.
0 +0900
>> +++ linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c2009-11-01 
>> 11:47:53.0
 +0900
>> @@ -239,6 +239,9 @@ int pcmcia_register_socket(struct pcmcia
>>  goto err;
>>  }
>>  
>> +/* loading "pcmcia" module for 16-bit PCMCIA cards */
>> +request_module("pcmcia");
>> +
>>  tsk = kthread_run(pccardd, socket, "pccardd");
>>  if (IS_ERR(tsk)) {
>>  ret = PTR_ERR(tsk);
>> 
>> ___
>> Linux PCMCIA reimplementation list
>> http://lists.infradead.org/mailman/listinfo/linux-pcmcia


___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

Re: [PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-11-01 Thread Dominik Brodowski 
As I'm a bit unsure about it: what's the general feeling about
this patch?

On Sun, Nov 01, 2009 at 12:04:30PM +0900, Komuro wrote:
> 
> load "pcmcia" module for 16-bit PCMCIA cards
> without help of userland script.
> 
> 
> Signed-off-by: Komuro 
> 
> ---
> 
> --- linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c.orig2009-11-01 
> 11:50:57.0 +0900
> +++ linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c 2009-11-01 11:47:53.0 
> +0900
> @@ -239,6 +239,9 @@ int pcmcia_register_socket(struct pcmcia
>   goto err;
>   }
>  
> + /* loading "pcmcia" module for 16-bit PCMCIA cards */
> + request_module("pcmcia");
> +
>   tsk = kthread_run(pccardd, socket, "pccardd");
>   if (IS_ERR(tsk)) {
>   ret = PTR_ERR(tsk);
> 
> ___
> Linux PCMCIA reimplementation list
> http://lists.infradead.org/mailman/listinfo/linux-pcmcia

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia

[PATCH] pcmcia: cs.c : load "pcmcia" module automatically.

2009-10-31 Thread Komuro 

load "pcmcia" module for 16-bit PCMCIA cards
without help of userland script.


Signed-off-by: Komuro 

---

--- linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c.orig  2009-11-01 
11:50:57.0 +0900
+++ linux-2.6.32-rc5-git3/drivers/pcmcia/cs.c   2009-11-01 11:47:53.0 
+0900
@@ -239,6 +239,9 @@ int pcmcia_register_socket(struct pcmcia
goto err;
}
 
+   /* loading "pcmcia" module for 16-bit PCMCIA cards */
+   request_module("pcmcia");
+
tsk = kthread_run(pccardd, socket, "pccardd");
if (IS_ERR(tsk)) {
ret = PTR_ERR(tsk);

___
Linux PCMCIA reimplementation list
http://lists.infradead.org/mailman/listinfo/linux-pcmcia