ARM: l2c: remove old .set_debug method
authorRussell King <rmk+kernel@arm.linux.org.uk>
Sun, 16 Mar 2014 19:38:25 +0000 (19:38 +0000)
committerRussell King <rmk+kernel@arm.linux.org.uk>
Thu, 29 May 2014 23:50:35 +0000 (00:50 +0100)
We no longer need or require the .set_debug method; we handle everything
it used to do via the .write_sec method instead.

Signed-off-by: Russell King <rmk+kernel@arm.linux.org.uk>
arch/arm/include/asm/outercache.h
arch/arm/mm/cache-l2x0.c

index 864afe2114d32612e7be1406b3dc67ada5db7d00..891a56b35bcf0c3ad4b8af956fdfa8509d699ff9 100644 (file)
@@ -35,7 +35,6 @@ struct outer_cache_fns {
        void (*resume)(void);
 
        /* This is an ARM L2C thing */
-       void (*set_debug)(unsigned long);
        void (*write_sec)(unsigned long, unsigned);
 };
 
index c4f3e8dc64ff07a24a292ccd40878f05631eeed3..ae6e71b3295cb791d7c4fb763851e376fd1694ca 100644 (file)
@@ -81,10 +81,7 @@ static void l2c_write_sec(unsigned long val, void __iomem *base, unsigned reg)
  */
 static inline void l2c_set_debug(void __iomem *base, unsigned long val)
 {
-       if (outer_cache.set_debug)
-               outer_cache.set_debug(val);
-       else
-               l2c_write_sec(val, base, L2X0_DEBUG_CTRL);
+       l2c_write_sec(val, base, L2X0_DEBUG_CTRL);
 }
 
 static void __l2c_op_way(void __iomem *reg)
@@ -155,8 +152,7 @@ static inline void cache_sync(void)
 #if defined(CONFIG_PL310_ERRATA_588369) || defined(CONFIG_PL310_ERRATA_727915)
 static inline void debug_writel(unsigned long val)
 {
-       if (outer_cache.set_debug || outer_cache.write_sec)
-               l2c_set_debug(l2x0_base, val);
+       l2c_set_debug(l2x0_base, val);
 }
 #else
 /* Optimised out for non-errata case */
@@ -514,11 +510,6 @@ static const struct l2c_init_data l2c220_data = {
  *     Affects: store buffer
  *     store buffer is not automatically drained.
  */
-static void l2c310_set_debug(unsigned long val)
-{
-       writel_relaxed(val, l2x0_base + L2X0_DEBUG_CTRL);
-}
-
 static void l2c310_inv_range_erratum(unsigned long start, unsigned long end)
 {
        void __iomem *base = l2x0_base;
@@ -695,10 +686,6 @@ static void __init l2c310_fixup(void __iomem *base, u32 cache_id,
        const char *errata[8];
        unsigned n = 0;
 
-       /* For compatibility */
-       if (revision <= L310_CACHE_ID_RTL_R3P0)
-               fns->set_debug = l2c310_set_debug;
-
        if (IS_ENABLED(CONFIG_PL310_ERRATA_588369) &&
            revision < L310_CACHE_ID_RTL_R2P0 &&
            /* For bcm compatibility */
@@ -759,7 +746,6 @@ static const struct l2c_init_data l2c310_init_fns __initconst = {
                .flush_all = l2c210_flush_all,
                .disable = l2c_disable,
                .sync = l2c210_sync,
-               .set_debug = l2c310_set_debug,
                .resume = l2c310_resume,
        },
 };
@@ -819,8 +805,6 @@ static void __init __l2c_init(const struct l2c_init_data *data,
        fns.write_sec = outer_cache.write_sec;
        if (data->fixup)
                data->fixup(l2x0_base, cache_id, &fns);
-       if (fns.write_sec)
-               fns.set_debug = NULL;
 
        /*
         * Check if l2x0 controller is already enabled.  If we are booting
@@ -1000,7 +984,6 @@ static const struct l2c_init_data of_l2c310_data __initconst = {
                .flush_all   = l2c210_flush_all,
                .disable     = l2c_disable,
                .sync        = l2c210_sync,
-               .set_debug   = l2c310_set_debug,
                .resume      = l2c310_resume,
        },
 };