drm/tegra: dpaux: Registers are 32-bit
authorThierry Reding <treding@nvidia.com>
Tue, 2 Jun 2015 11:13:01 +0000 (13:13 +0200)
committerThierry Reding <treding@nvidia.com>
Fri, 12 Jun 2015 14:26:04 +0000 (16:26 +0200)
Use a sized unsigned 32-bit data type (u32) to store register contents.
The DPAUX registers are 32 bits wide irrespective of the architecture's
data width.

Signed-off-by: Thierry Reding <treding@nvidia.com>
drivers/gpu/drm/tegra/dpaux.c

index a43a836e6f882b8062b8df74ca06371bbf8e4f6e..07b26972f487967cb1b4203488f0387c9e0736fc 100644 (file)
@@ -56,15 +56,14 @@ static inline struct tegra_dpaux *work_to_dpaux(struct work_struct *work)
        return container_of(work, struct tegra_dpaux, work);
 }
 
-static inline unsigned long tegra_dpaux_readl(struct tegra_dpaux *dpaux,
-                                             unsigned long offset)
+static inline u32 tegra_dpaux_readl(struct tegra_dpaux *dpaux,
+                                   unsigned long offset)
 {
        return readl(dpaux->regs + (offset << 2));
 }
 
 static inline void tegra_dpaux_writel(struct tegra_dpaux *dpaux,
-                                     unsigned long value,
-                                     unsigned long offset)
+                                     u32 value, unsigned long offset)
 {
        writel(value, dpaux->regs + (offset << 2));
 }
@@ -76,7 +75,7 @@ static void tegra_dpaux_write_fifo(struct tegra_dpaux *dpaux, const u8 *buffer,
 
        for (i = 0; i < DIV_ROUND_UP(size, 4); i++) {
                size_t num = min_t(size_t, size - i * 4, 4);
-               unsigned long value = 0;
+               u32 value = 0;
 
                for (j = 0; j < num; j++)
                        value |= buffer[i * 4 + j] << (j * 8);
@@ -92,7 +91,7 @@ static void tegra_dpaux_read_fifo(struct tegra_dpaux *dpaux, u8 *buffer,
 
        for (i = 0; i < DIV_ROUND_UP(size, 4); i++) {
                size_t num = min_t(size_t, size - i * 4, 4);
-               unsigned long value;
+               u32 value;
 
                value = tegra_dpaux_readl(dpaux, DPAUX_DP_AUXDATA_READ(i));
 
@@ -248,7 +247,7 @@ static irqreturn_t tegra_dpaux_irq(int irq, void *data)
 {
        struct tegra_dpaux *dpaux = data;
        irqreturn_t ret = IRQ_HANDLED;
-       unsigned long value;
+       u32 value;
 
        /* clear interrupts */
        value = tegra_dpaux_readl(dpaux, DPAUX_INTR_AUX);
@@ -271,7 +270,7 @@ static int tegra_dpaux_probe(struct platform_device *pdev)
 {
        struct tegra_dpaux *dpaux;
        struct resource *regs;
-       unsigned long value;
+       u32 value;
        int err;
 
        dpaux = devm_kzalloc(&pdev->dev, sizeof(*dpaux), GFP_KERNEL);
@@ -463,7 +462,7 @@ int tegra_dpaux_detach(struct tegra_dpaux *dpaux)
 
 enum drm_connector_status tegra_dpaux_detect(struct tegra_dpaux *dpaux)
 {
-       unsigned long value;
+       u32 value;
 
        value = tegra_dpaux_readl(dpaux, DPAUX_DP_AUXSTAT);
 
@@ -475,7 +474,7 @@ enum drm_connector_status tegra_dpaux_detect(struct tegra_dpaux *dpaux)
 
 int tegra_dpaux_enable(struct tegra_dpaux *dpaux)
 {
-       unsigned long value;
+       u32 value;
 
        value = DPAUX_HYBRID_PADCTL_AUX_CMH(2) |
                DPAUX_HYBRID_PADCTL_AUX_DRVZ(4) |
@@ -493,7 +492,7 @@ int tegra_dpaux_enable(struct tegra_dpaux *dpaux)
 
 int tegra_dpaux_disable(struct tegra_dpaux *dpaux)
 {
-       unsigned long value;
+       u32 value;
 
        value = tegra_dpaux_readl(dpaux, DPAUX_HYBRID_SPARE);
        value |= DPAUX_HYBRID_SPARE_PAD_POWER_DOWN;