1. 19 Feb, 2011 3 commits
    • Alan Cox's avatar
      644e6e4a
    • Pavel Machek's avatar
      pcmcia vs. MECR on pxa25x/sa1111 · 5b703683
      Pavel Machek authored
      After 2.6.34 changes, __pxa2xx_drv_pcmcia_probe() was replaced by
      sa1111_pcmcia_add(). That unfortunately means that configure_sockets()
      is not called, leading to MECR not being set properly, leading to
      strange crashes.
      
      Tested on pxa255+sa1111, I do not have lubbock board nearby. Perhaps
      cleaner solution exists?
      Signed-off-by: default avatarPavel Machek <pma@sysgo.com>
      Signed-off-by: default avatarDominik Brodowski <linux@dominikbrodowski.net>
      5b703683
    • Julia Lawall's avatar
      drivers/char/pcmcia/ipwireless/main.c: Convert release_resource to... · d5bb2923
      Julia Lawall authored
      drivers/char/pcmcia/ipwireless/main.c: Convert release_resource to release_region/release_mem_region
      
      Request_region should be used with release_region, not release_resource.
      
      This patch contains a number of changes, related to calls to request_region,
      request_mem_region, and the associated error handling code.
      
      1. For the call to request_region, the variable io_resource storing the
      result is dropped.  The call to release_resource at the end of the function
      is changed to a call to release_region with the first two arguments of
      request_region as its arguments.  The same call to release_region is also
      added to release_ipwireless.
      
      2. The first call to request_mem_region is now tested and ret is set to
      -EBUSY if the the call has failed.  This call was associated with the
      initialization of ipw->attr_memory.  But the error handling code was
      testing ipw->common_memory.  The definition of release_ipwireless also
      suggests that this call should be associated with ipw->common_memory, not
      ipw->attr_memory.
      
      3. The second call to request_mem_region is now tested and ret is
      set to -EBUSY if the the call has failed.
      
      4. The various gotos to the error handling code is adjusted so that there
      is no need for ifs.
      
      5. Return the value stored in the ret variable rather than -1.
      
      The semantic match that finds this problem is as follows:
      (http://coccinelle.lip6.fr/)
      
      // <smpl>
      @@
      expression x,E;
      @@
      (
      *x = request_region(...)
      |
      *x = request_mem_region(...)
      )
      ... when != release_region(x)
          when != x = E
      * release_resource(x);
      // </smpl>
      Signed-off-by: default avatarJulia Lawall <julia@diku.dk>
      Signed-off-by: default avatarJiri Kosina <jkosina@suse.cz>
      Signed-off-by: default avatarDominik Brodowski <linux@dominikbrodowski.net>
      d5bb2923
  2. 16 Feb, 2011 2 commits
  3. 15 Feb, 2011 31 commits
  4. 14 Feb, 2011 4 commits