Re: [PATCH 1/2] ELAN U132 Host Controller Driver: convert sw_lock to mutex

El Fri, Sep 07, 2007 at 12:08:34AM +0200 Oleg Verych ha dit:

> * Thu, 6 Sep 2007 23:18:15 +0200
> 
> []
> > @@ -184,7 +184,7 @@ struct u132_ring {
> >  struct u132 {
> 
> Subject: [PATCH 1/2] ELAN U132 Host Controller Driver: convert sw_lock to mutex
> []
> >          struct list_head u132_list;
> > -        struct semaphore sw_lock;
> > +        struct mutex sw_lock;
> >          struct semaphore scheduler_lock;
> 
> Subject: [PATCH 2/2] ELAN U132 Host Controller Driver: convert  scheduler_lock t
> []
> >         struct list_head u132_list;
> >         struct semaphore sw_lock;
> >-        struct semaphore scheduler_lock;
> >+        struct mutex scheduler_lock;
> 
> Are you sure, you have right patch sequence?

my idea was to sent two independent patches, would it be better to
have one patch based on the other?

-- 
Matthias Kaehlcke
Linux Application Developer
Barcelona

              Insanity: doing the same thing over and over
                again and expecting different results
                            (Albert Einstein)
                                                                 .''`.
    using free software / Debian GNU/Linux | http://debian.org  : :'  :
                                                                `. `'`
gpg --keyserver pgp.mit.edu --recv-keys 47D8E5D4                  `-