Merge branch 'for-linus-dma-masks' of git://git.linaro.org/people/rmk/linux-arm
authorLinus Torvalds <torvalds@linux-foundation.org>
Wed, 13 Nov 2013 22:55:21 +0000 (07:55 +0900)
committerLinus Torvalds <torvalds@linux-foundation.org>
Wed, 13 Nov 2013 22:55:21 +0000 (07:55 +0900)
Pull DMA mask updates from Russell King:
 "This series cleans up the handling of DMA masks in a lot of drivers,
  fixing some bugs as we go.

  Some of the more serious errors include:
   - drivers which only set their coherent DMA mask if the attempt to
     set the streaming mask fails.
   - drivers which test for a NULL dma mask pointer, and then set the
     dma mask pointer to a location in their module .data section -
     which will cause problems if the module is reloaded.

  To counter these, I have introduced two helper functions:
   - dma_set_mask_and_coherent() takes care of setting both the
     streaming and coherent masks at the same time, with the correct
     error handling as specified by the API.
   - dma_coerce_mask_and_coherent() which resolves the problem of
     drivers forcefully setting DMA masks.  This is more a marker for
     future work to further clean these locations up - the code which
     creates the devices really should be initialising these, but to fix
     that in one go along with this change could potentially be very
     disruptive.

  The last thing this series does is prise away some of Linux's addition
  to "DMA addresses are physical addresses and RAM always starts at
  zero".  We have ARM LPAE systems where all system memory is above 4GB
  physical, hence having DMA masks interpreted by (eg) the block layers
  as describing physical addresses in the range 0..DMAMASK fails on
  these platforms.  Santosh Shilimkar addresses this in this series; the
  patches were copied to the appropriate people multiple times but were
  ignored.

  Fixing this also gets rid of some ARM weirdness in the setup of the
  max*pfn variables, and brings ARM into line with every other Linux
  architecture as far as those go"

* 'for-linus-dma-masks' of git://git.linaro.org/people/rmk/linux-arm: (52 commits)
  ARM: 7805/1: mm: change max*pfn to include the physical offset of memory
  ARM: 7797/1: mmc: Use dma_max_pfn(dev) helper for bounce_limit calculations
  ARM: 7796/1: scsi: Use dma_max_pfn(dev) helper for bounce_limit calculations
  ARM: 7795/1: mm: dma-mapping: Add dma_max_pfn(dev) helper function
  ARM: 7794/1: block: Rename parameter dma_mask to max_addr for blk_queue_bounce_limit()
  ARM: DMA-API: better handing of DMA masks for coherent allocations
  ARM: 7857/1: dma: imx-sdma: setup dma mask
  DMA-API: firmware/google/gsmi.c: avoid direct access to DMA masks
  DMA-API: dcdbas: update DMA mask handing
  DMA-API: dma: edma.c: no need to explicitly initialize DMA masks
  DMA-API: usb: musb: use platform_device_register_full() to avoid directly messing with dma masks
  DMA-API: crypto: remove last references to 'static struct device *dev'
  DMA-API: crypto: fix ixp4xx crypto platform device support
  DMA-API: others: use dma_set_coherent_mask()
  DMA-API: staging: use dma_set_coherent_mask()
  DMA-API: usb: use new dma_coerce_mask_and_coherent()
  DMA-API: usb: use dma_set_coherent_mask()
  DMA-API: parport: parport_pc.c: use dma_coerce_mask_and_coherent()
  DMA-API: net: octeon: use dma_coerce_mask_and_coherent()
  DMA-API: net: nxp/lpc_eth: use dma_coerce_mask_and_coherent()
  ...

38 files changed:
1  2 
arch/arm/mm/dma-mapping.c
arch/arm/mm/init.c
arch/powerpc/kernel/vio.c
drivers/dma/edma.c
drivers/net/ethernet/broadcom/b44.c
drivers/net/ethernet/broadcom/bnx2x/bnx2x_main.c
drivers/net/ethernet/brocade/bna/bnad.c
drivers/net/ethernet/emulex/benet/be_main.c
drivers/net/ethernet/intel/e1000/e1000_main.c
drivers/net/ethernet/intel/e1000e/netdev.c
drivers/net/ethernet/intel/igb/igb_main.c
drivers/net/ethernet/intel/igbvf/netdev.c
drivers/net/ethernet/intel/ixgbe/ixgbe_main.c
drivers/net/ethernet/intel/ixgbevf/ixgbevf_main.c
drivers/net/ethernet/octeon/octeon_mgmt.c
drivers/of/platform.c
drivers/staging/dwc2/platform.c
drivers/staging/et131x/et131x.c
drivers/staging/imx-drm/imx-drm-core.c
drivers/staging/imx-drm/ipuv3-crtc.c
drivers/usb/chipidea/ci_hdrc_imx.c
drivers/usb/host/ehci-atmel.c
drivers/usb/host/ehci-exynos.c
drivers/usb/host/ehci-tegra.c
drivers/usb/host/ohci-at91.c
drivers/usb/host/ohci-exynos.c
drivers/usb/host/ohci-nxp.c
drivers/usb/host/ohci-omap3.c
drivers/usb/host/ohci-pxa27x.c
drivers/usb/host/ohci-spear.c
drivers/usb/host/uhci-platform.c
sound/soc/atmel/atmel-pcm.c
sound/soc/davinci/davinci-pcm.c
sound/soc/fsl/fsl_dma.c
sound/soc/fsl/imx-pcm-fiq.c
sound/soc/fsl/mpc5200_dma.c
sound/soc/kirkwood/kirkwood-dma.c
sound/soc/s6000/s6000-pcm.c

Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
index bb2f20291509392192545f495a290cec93f60e1d,e6c3e663f6c085faa47c36f638897a2af7476978..e622cc1f96ffe58336ee8c24f12c54e1679e11b7
@@@ -12140,12 -12072,9 +12140,8 @@@ static int bnx2x_set_coherency_mask(str
  {
        struct device *dev = &bp->pdev->dev;
  
-       if (dma_set_mask(dev, DMA_BIT_MASK(64)) == 0) {
-               if (dma_set_coherent_mask(dev, DMA_BIT_MASK(64)) != 0) {
-                       dev_err(dev, "dma_set_coherent_mask failed, aborting\n");
-                       return -EIO;
-               }
-       } else if (dma_set_mask(dev, DMA_BIT_MASK(32)) != 0) {
 -      if (dma_set_mask_and_coherent(dev, DMA_BIT_MASK(64)) == 0) {
 -              bp->flags |= USING_DAC_FLAG;
 -      } else if (dma_set_mask_and_coherent(dev, DMA_BIT_MASK(32)) != 0) {
++      if (dma_set_mask_and_coherent(dev, DMA_BIT_MASK(64)) != 0 &&
++          dma_set_mask_and_coherent(dev, DMA_BIT_MASK(32)) != 0) {
                dev_err(dev, "System does not support DMA, aborting\n");
                return -EIO;
        }
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
index 016352e0f5a7b326bbdb84a22af2e440a4b2fddf,0000000000000000000000000000000000000000..e97c198e052fd88aa99a98679952769678728a75
mode 100644,000000..100644
--- /dev/null
@@@ -1,301 -1,0 +1,300 @@@
-       if (!pdev->dev.dma_mask)
-               pdev->dev.dma_mask = &pdev->dev.coherent_dma_mask;
-       if (!pdev->dev.coherent_dma_mask)
-               pdev->dev.coherent_dma_mask = DMA_BIT_MASK(32);
 +/*
 + * SAMSUNG EXYNOS USB HOST EHCI Controller
 + *
 + * Copyright (C) 2011 Samsung Electronics Co.Ltd
 + * Author: Jingoo Han <jg1.han@samsung.com>
 + * Author: Joonyoung Shim <jy0922.shim@samsung.com>
 + *
 + * This program is free software; you can redistribute  it and/or modify it
 + * under  the terms of  the GNU General  Public License as published by the
 + * Free Software Foundation;  either version 2 of the  License, or (at your
 + * option) any later version.
 + *
 + */
 +
 +#include <linux/clk.h>
 +#include <linux/dma-mapping.h>
 +#include <linux/io.h>
 +#include <linux/kernel.h>
 +#include <linux/module.h>
 +#include <linux/of.h>
 +#include <linux/of_gpio.h>
 +#include <linux/platform_device.h>
 +#include <linux/usb/phy.h>
 +#include <linux/usb/samsung_usb_phy.h>
 +#include <linux/usb.h>
 +#include <linux/usb/hcd.h>
 +#include <linux/usb/otg.h>
 +
 +#include "ehci.h"
 +
 +#define DRIVER_DESC "EHCI EXYNOS driver"
 +
 +#define EHCI_INSNREG00(base)                  (base + 0x90)
 +#define EHCI_INSNREG00_ENA_INCR16             (0x1 << 25)
 +#define EHCI_INSNREG00_ENA_INCR8              (0x1 << 24)
 +#define EHCI_INSNREG00_ENA_INCR4              (0x1 << 23)
 +#define EHCI_INSNREG00_ENA_INCRX_ALIGN                (0x1 << 22)
 +#define EHCI_INSNREG00_ENABLE_DMA_BURST       \
 +      (EHCI_INSNREG00_ENA_INCR16 | EHCI_INSNREG00_ENA_INCR8 | \
 +       EHCI_INSNREG00_ENA_INCR4 | EHCI_INSNREG00_ENA_INCRX_ALIGN)
 +
 +static const char hcd_name[] = "ehci-exynos";
 +static struct hc_driver __read_mostly exynos_ehci_hc_driver;
 +
 +struct exynos_ehci_hcd {
 +      struct clk *clk;
 +      struct usb_phy *phy;
 +      struct usb_otg *otg;
 +};
 +
 +#define to_exynos_ehci(hcd) (struct exynos_ehci_hcd *)(hcd_to_ehci(hcd)->priv)
 +
 +static void exynos_setup_vbus_gpio(struct platform_device *pdev)
 +{
 +      struct device *dev = &pdev->dev;
 +      int err;
 +      int gpio;
 +
 +      if (!dev->of_node)
 +              return;
 +
 +      gpio = of_get_named_gpio(dev->of_node, "samsung,vbus-gpio", 0);
 +      if (!gpio_is_valid(gpio))
 +              return;
 +
 +      err = devm_gpio_request_one(dev, gpio, GPIOF_OUT_INIT_HIGH,
 +                                  "ehci_vbus_gpio");
 +      if (err)
 +              dev_err(dev, "can't request ehci vbus gpio %d", gpio);
 +}
 +
 +static int exynos_ehci_probe(struct platform_device *pdev)
 +{
 +      struct exynos_ehci_hcd *exynos_ehci;
 +      struct usb_hcd *hcd;
 +      struct ehci_hcd *ehci;
 +      struct resource *res;
 +      struct usb_phy *phy;
 +      int irq;
 +      int err;
 +
 +      /*
 +       * Right now device-tree probed devices don't get dma_mask set.
 +       * Since shared usb code relies on it, set it here for now.
 +       * Once we move to full device tree support this will vanish off.
 +       */
++      err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
++      if (err)
++              return err;
 +
 +      exynos_setup_vbus_gpio(pdev);
 +
 +      hcd = usb_create_hcd(&exynos_ehci_hc_driver,
 +                           &pdev->dev, dev_name(&pdev->dev));
 +      if (!hcd) {
 +              dev_err(&pdev->dev, "Unable to create HCD\n");
 +              return -ENOMEM;
 +      }
 +      exynos_ehci = to_exynos_ehci(hcd);
 +
 +      if (of_device_is_compatible(pdev->dev.of_node,
 +                                      "samsung,exynos5440-ehci"))
 +              goto skip_phy;
 +
 +      phy = devm_usb_get_phy(&pdev->dev, USB_PHY_TYPE_USB2);
 +      if (IS_ERR(phy)) {
 +              usb_put_hcd(hcd);
 +              dev_warn(&pdev->dev, "no platform data or transceiver defined\n");
 +              return -EPROBE_DEFER;
 +      } else {
 +              exynos_ehci->phy = phy;
 +              exynos_ehci->otg = phy->otg;
 +      }
 +
 +skip_phy:
 +
 +      exynos_ehci->clk = devm_clk_get(&pdev->dev, "usbhost");
 +
 +      if (IS_ERR(exynos_ehci->clk)) {
 +              dev_err(&pdev->dev, "Failed to get usbhost clock\n");
 +              err = PTR_ERR(exynos_ehci->clk);
 +              goto fail_clk;
 +      }
 +
 +      err = clk_prepare_enable(exynos_ehci->clk);
 +      if (err)
 +              goto fail_clk;
 +
 +      res = platform_get_resource(pdev, IORESOURCE_MEM, 0);
 +      if (!res) {
 +              dev_err(&pdev->dev, "Failed to get I/O memory\n");
 +              err = -ENXIO;
 +              goto fail_io;
 +      }
 +
 +      hcd->rsrc_start = res->start;
 +      hcd->rsrc_len = resource_size(res);
 +      hcd->regs = devm_ioremap(&pdev->dev, res->start, hcd->rsrc_len);
 +      if (!hcd->regs) {
 +              dev_err(&pdev->dev, "Failed to remap I/O memory\n");
 +              err = -ENOMEM;
 +              goto fail_io;
 +      }
 +
 +      irq = platform_get_irq(pdev, 0);
 +      if (!irq) {
 +              dev_err(&pdev->dev, "Failed to get IRQ\n");
 +              err = -ENODEV;
 +              goto fail_io;
 +      }
 +
 +      if (exynos_ehci->otg)
 +              exynos_ehci->otg->set_host(exynos_ehci->otg, &hcd->self);
 +
 +      if (exynos_ehci->phy)
 +              usb_phy_init(exynos_ehci->phy);
 +
 +      ehci = hcd_to_ehci(hcd);
 +      ehci->caps = hcd->regs;
 +
 +      /* DMA burst Enable */
 +      writel(EHCI_INSNREG00_ENABLE_DMA_BURST, EHCI_INSNREG00(hcd->regs));
 +
 +      err = usb_add_hcd(hcd, irq, IRQF_SHARED);
 +      if (err) {
 +              dev_err(&pdev->dev, "Failed to add USB HCD\n");
 +              goto fail_add_hcd;
 +      }
 +
 +      platform_set_drvdata(pdev, hcd);
 +
 +      return 0;
 +
 +fail_add_hcd:
 +      if (exynos_ehci->phy)
 +              usb_phy_shutdown(exynos_ehci->phy);
 +fail_io:
 +      clk_disable_unprepare(exynos_ehci->clk);
 +fail_clk:
 +      usb_put_hcd(hcd);
 +      return err;
 +}
 +
 +static int exynos_ehci_remove(struct platform_device *pdev)
 +{
 +      struct usb_hcd *hcd = platform_get_drvdata(pdev);
 +      struct exynos_ehci_hcd *exynos_ehci = to_exynos_ehci(hcd);
 +
 +      usb_remove_hcd(hcd);
 +
 +      if (exynos_ehci->otg)
 +              exynos_ehci->otg->set_host(exynos_ehci->otg, &hcd->self);
 +
 +      if (exynos_ehci->phy)
 +              usb_phy_shutdown(exynos_ehci->phy);
 +
 +      clk_disable_unprepare(exynos_ehci->clk);
 +
 +      usb_put_hcd(hcd);
 +
 +      return 0;
 +}
 +
 +#ifdef CONFIG_PM
 +static int exynos_ehci_suspend(struct device *dev)
 +{
 +      struct usb_hcd *hcd = dev_get_drvdata(dev);
 +      struct exynos_ehci_hcd *exynos_ehci = to_exynos_ehci(hcd);
 +
 +      bool do_wakeup = device_may_wakeup(dev);
 +      int rc;
 +
 +      rc = ehci_suspend(hcd, do_wakeup);
 +
 +      if (exynos_ehci->otg)
 +              exynos_ehci->otg->set_host(exynos_ehci->otg, &hcd->self);
 +
 +      if (exynos_ehci->phy)
 +              usb_phy_shutdown(exynos_ehci->phy);
 +
 +      clk_disable_unprepare(exynos_ehci->clk);
 +
 +      return rc;
 +}
 +
 +static int exynos_ehci_resume(struct device *dev)
 +{
 +      struct usb_hcd *hcd = dev_get_drvdata(dev);
 +      struct exynos_ehci_hcd *exynos_ehci = to_exynos_ehci(hcd);
 +
 +      clk_prepare_enable(exynos_ehci->clk);
 +
 +      if (exynos_ehci->otg)
 +              exynos_ehci->otg->set_host(exynos_ehci->otg, &hcd->self);
 +
 +      if (exynos_ehci->phy)
 +              usb_phy_init(exynos_ehci->phy);
 +
 +      /* DMA burst Enable */
 +      writel(EHCI_INSNREG00_ENABLE_DMA_BURST, EHCI_INSNREG00(hcd->regs));
 +
 +      ehci_resume(hcd, false);
 +      return 0;
 +}
 +#else
 +#define exynos_ehci_suspend   NULL
 +#define exynos_ehci_resume    NULL
 +#endif
 +
 +static const struct dev_pm_ops exynos_ehci_pm_ops = {
 +      .suspend        = exynos_ehci_suspend,
 +      .resume         = exynos_ehci_resume,
 +};
 +
 +#ifdef CONFIG_OF
 +static const struct of_device_id exynos_ehci_match[] = {
 +      { .compatible = "samsung,exynos4210-ehci" },
 +      { .compatible = "samsung,exynos5440-ehci" },
 +      {},
 +};
 +MODULE_DEVICE_TABLE(of, exynos_ehci_match);
 +#endif
 +
 +static struct platform_driver exynos_ehci_driver = {
 +      .probe          = exynos_ehci_probe,
 +      .remove         = exynos_ehci_remove,
 +      .shutdown       = usb_hcd_platform_shutdown,
 +      .driver = {
 +              .name   = "exynos-ehci",
 +              .owner  = THIS_MODULE,
 +              .pm     = &exynos_ehci_pm_ops,
 +              .of_match_table = of_match_ptr(exynos_ehci_match),
 +      }
 +};
 +static const struct ehci_driver_overrides exynos_overrides __initdata = {
 +      .extra_priv_size = sizeof(struct exynos_ehci_hcd),
 +};
 +
 +static int __init ehci_exynos_init(void)
 +{
 +      if (usb_disabled())
 +              return -ENODEV;
 +
 +      pr_info("%s: " DRIVER_DESC "\n", hcd_name);
 +      ehci_init_driver(&exynos_ehci_hc_driver, &exynos_overrides);
 +      return platform_driver_register(&exynos_ehci_driver);
 +}
 +module_init(ehci_exynos_init);
 +
 +static void __exit ehci_exynos_cleanup(void)
 +{
 +      platform_driver_unregister(&exynos_ehci_driver);
 +}
 +module_exit(ehci_exynos_cleanup);
 +
 +MODULE_DESCRIPTION(DRIVER_DESC);
 +MODULE_ALIAS("platform:exynos-ehci");
 +MODULE_AUTHOR("Jingoo Han");
 +MODULE_AUTHOR("Joonyoung Shim");
 +MODULE_LICENSE("GPL v2");
Simple merge
Simple merge
index a87baedc0aa79555bcb60ed72a27036fc757566f,866f2464f9de64c84ac96114c96dccb52a472d32..91ec9b2cd37868f600eca4dd41cdf6180eed806e
@@@ -71,19 -114,14 +71,18 @@@ static int exynos_ohci_probe(struct pla
         * Since shared usb code relies on it, set it here for now.
         * Once we move to full device tree support this will vanish off.
         */
-       if (!pdev->dev.dma_mask)
-               pdev->dev.dma_mask = &pdev->dev.coherent_dma_mask;
-       if (!pdev->dev.coherent_dma_mask)
-               pdev->dev.coherent_dma_mask = DMA_BIT_MASK(32);
+       err = dma_coerce_mask_and_coherent(&pdev->dev, DMA_BIT_MASK(32));
+       if (err)
+               return err;
  
 -      exynos_ohci = devm_kzalloc(&pdev->dev, sizeof(struct exynos_ohci_hcd),
 -                                      GFP_KERNEL);
 -      if (!exynos_ohci)
 +      hcd = usb_create_hcd(&exynos_ohci_hc_driver,
 +                              &pdev->dev, dev_name(&pdev->dev));
 +      if (!hcd) {
 +              dev_err(&pdev->dev, "Unable to create HCD\n");
                return -ENOMEM;
 +      }
 +
 +      exynos_ohci = to_exynos_ohci(hcd);
  
        if (of_device_is_compatible(pdev->dev.of_node,
                                        "samsung,exynos5440-ohci"))
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge
Simple merge