1. 27 10月, 2010 2 次提交
  2. 26 10月, 2010 1 次提交
  3. 25 10月, 2010 12 次提交
  4. 21 10月, 2010 2 次提交
  5. 18 10月, 2010 8 次提交
  6. 02 9月, 2010 1 次提交
    • J
      drivers/net/can/mscan/mpc5xxx_can.c: Add of_node_put to avoid memory leak · aed5029e
      Julia Lawall 提交于
      Add a call to of_node_put in the error handling code following a call to
      of_find_matching_node.
      
      This patch also moves the existing call to of_node_put after the call to
      iounmap in the error handling code, to make it possible to jump to
      of_node_put without doing iounmap.  These appear to be disjoint operations,
      so the ordering doesn't matter.
      
      This patch furthermore changes the -ENODEV result in the error handling
      code for of_find_matching_node to a return of 0, as found in the error
      handling code for of_iomap, because the return type of the function is
      unsigned.
      
      The semantic match that finds this problem is as follows:
      (http://coccinelle.lip6.fr/)
      
      // <smpl>
      @r exists@
      local idexpression x;
      expression E,E1,E2;
      statement S;
      @@
      
      *x =
      (of_find_node_by_path
      |of_find_node_by_name
      |of_find_node_by_phandle
      |of_get_parent
      |of_get_next_parent
      |of_get_next_child
      |of_find_compatible_node
      |of_match_node
      |of_find_node_by_type
      |of_find_node_with_property
      |of_find_matching_node
      |of_parse_phandle
      )(...);
      ...
      if (x == NULL) S
      <... when != x = E
      *if (...) {
        ... when != of_node_put(x)
            when != if (...) { ... of_node_put(x); ... }
      (
        return <+...x...+>;
      |
      *  return ...;
      )
      }
      ...>
      (
      E2 = x;
      |
      of_node_put(x);
      )
      // </smpl>
      Signed-off-by: NJulia Lawall <julia@diku.dk>
      Reviewed-by: NWolfram Sang <w.sang@pengutronix.de>
      Acked-by: NWolfgang Grandegger <wg@grandegger.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      aed5029e
  7. 06 8月, 2010 1 次提交
  8. 04 8月, 2010 1 次提交
  9. 23 7月, 2010 1 次提交
  10. 03 6月, 2010 1 次提交
  11. 01 6月, 2010 1 次提交
  12. 28 5月, 2010 1 次提交
  13. 22 5月, 2010 1 次提交
    • G
      of: Remove duplicate fields from of_platform_driver · 4018294b
      Grant Likely 提交于
      .name, .match_table and .owner are duplicated in both of_platform_driver
      and device_driver.  This patch is a removes the extra copies from struct
      of_platform_driver and converts all users to the device_driver members.
      
      This patch is a pretty mechanical change.  The usage model doesn't change
      and if any drivers have been missed, or if anything has been fixed up
      incorrectly, then it will fail with a compile time error, and the fixup
      will be trivial.  This patch looks big and scary because it touches so
      many files, but it should be pretty safe.
      Signed-off-by: NGrant Likely <grant.likely@secretlab.ca>
      Acked-by: NSean MacLennan <smaclennan@pikatech.com>
      4018294b
  14. 21 5月, 2010 2 次提交
  15. 19 5月, 2010 2 次提交
    • G
      of: Always use 'struct device.of_node' to get device node pointer. · 61c7a080
      Grant Likely 提交于
      The following structure elements duplicate the information in
      'struct device.of_node' and so are being eliminated.  This patch
      makes all readers of these elements use device.of_node instead.
      
      (struct of_device *)->node
      (struct dev_archdata *)->prom_node (sparc)
      (struct dev_archdata *)->of_node (powerpc & microblaze)
      Signed-off-by: NGrant Likely <grant.likely@secretlab.ca>
      61c7a080
    • O
      can: Fix SJA1000 command register writes on SMP systems · 57c8a456
      Oliver Hartkopp 提交于
      The SJA1000 command register is concurrently written in the rx-path to free
      the receive buffer _and_ in the tx-path to start the transmission.
      
      The SJA1000 data sheet, 6.4.4 COMMAND REGISTER (CMR) states:
      "Between two commands at least one internal clock cycle is needed in
      order to proceed. The internal clock is half of the external oscillator
      frequency."
      
      On SMP systems the current implementation leads to a write stall in the
      tx-path, which can be solved by adding some general locking and some time
      to settle the write_reg() operation for the command register.
      
      Thanks to Klaus Hitschler for the original fix and detailed problem
      description.
      
      This patch applies on net-2.6 and (with some offsets) on net-next-2.6 .
      Signed-off-by: NOliver Hartkopp <socketcan@hartkopp.net>
      Acked-by: NWolfgang Grandegger <wg@grandegger.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      57c8a456
  16. 18 5月, 2010 1 次提交
  17. 14 5月, 2010 1 次提交
    • J
      drivers/net: Remove unnecessary returns from void function()s · a4b77097
      Joe Perches 提交于
      This patch removes from drivers/net/ all the unnecessary
      return; statements that precede the last closing brace of
      void functions.
      
      It does not remove the returns that are immediately
      preceded by a label as gcc doesn't like that.
      
      It also does not remove null void functions with return.
      
      Done via:
      $ grep -rP --include=*.[ch] -l "return;\n}" net/ | \
        xargs perl -i -e 'local $/ ; while (<>) { s/\n[ \t\n]+return;\n}/\n}/g; print; }'
      
      with some cleanups by hand.
      
      Compile tested x86 allmodconfig only.
      Signed-off-by: NJoe Perches <joe@perches.com>
      Signed-off-by: NDavid S. Miller <davem@davemloft.net>
      a4b77097
  18. 10 5月, 2010 1 次提交