• Linus Torvalds's avatar
    Merge branch 'devm_no_resource_check' of git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux · e5106682
    Linus Torvalds authored
    Pull devm usage cleanup from Wolfram Sang:
     "Lately, I have been experimenting how to improve the devm interface to
      make writing device drivers easier and less error prone while also
      getting rid of its subtle issues.  I think it has more potential but
      still needs work and definately conistency, especiall in its usage.
    
      The first thing I come up with is a low hanging fruit regarding
      devm_ioremap_resouce().  This function already checks if the passed
      resource is valid and gives an error message if not.  So, we can
      remove similar checks from the drivers and get rid of a bit of code
      and a number of inconsistent error strings.
    
      This series only removes the unneeded check iff devm_ioremap_resource
      follows platform_get_resource directly.  The previous version tried to
      shuffle code if needed, too, what lead to an embarrasing bug.  It
      turned out to me that shuffling code for all cases found will make the
      automated script too complex, so I am unsure if an automated cleanup
      is the proper tool for this case.  Removing the easy stuff seems
      worthwhile to me, though.
    
      Despite various architectures and platform dependencies, I managed to
      compile test 45 out of 57 modified files locally using heuristics and
      defconfigs."
    
    Pulled because: 296 deletions, 0 additions.
    
    * 'devm_no_resource_check' of git://git.kernel.org/pub/scm/linux/kernel/git/wsa/linux: (33 commits)
      sound/soc/kirkwood: don't check resource with devm_ioremap_resource
      sound/soc/fsl: don't check resource with devm_ioremap_resource
      arch/mips/lantiq/xway: don't check resource with devm_ioremap_resource
      arch/arm/plat-samsung: don't check resource with devm_ioremap_resource
      arch/arm/mach-tegra: don't check resource with devm_ioremap_resource
      drivers/watchdog: don't check resource with devm_ioremap_resource
      drivers/w1/masters: don't check resource with devm_ioremap_resource
      drivers/video/omap2/dss: don't check resource with devm_ioremap_resource
      drivers/video/omap2: don't check resource with devm_ioremap_resource
      drivers/usb/phy: don't check resource with devm_ioremap_resource
      drivers/usb/host: don't check resource with devm_ioremap_resource
      drivers/usb/gadget: don't check resource with devm_ioremap_resource
      drivers/usb/chipidea: don't check resource with devm_ioremap_resource
      drivers/thermal: don't check resource with devm_ioremap_resource
      drivers/staging/nvec: don't check resource with devm_ioremap_resource
      drivers/staging/dwc2: don't check resource with devm_ioremap_resource
      drivers/spi: don't check resource with devm_ioremap_resource
      drivers/rtc: don't check resource with devm_ioremap_resource
      drivers/pwm: don't check resource with devm_ioremap_resource
      drivers/pinctrl: don't check resource with devm_ioremap_resource
      ...
    e5106682
gptu.c 5.03 KB