2022-10-21 20:39:56

by Bjorn Helgaas

[permalink] [raw]
Subject: [PATCH] alpha: remove unused __SLOW_DOWN_IO and SLOW_DOWN_IO definitions

From: Bjorn Helgaas <[email protected]>

Remove unused __SLOW_DOWN_IO and SLOW_DOWN_IO definitions.

Signed-off-by: Bjorn Helgaas <[email protected]>
---

[Sorry for the noise; I sent this a few minutes ago, but it bounced from
all the individual recipients because of DMARC issues]

Previously posted v1 4/15/2022 at
https://lore.kernel.org/all/[email protected]/
and reposted v1 10/13/2022 at
https://lore.kernel.org/all/[email protected]/

Changes from v1 to v2:
- Drop the ia64 patch because Andrew applied it
https://lore.kernel.org/all/[email protected]/

- Split sh and alpha to separate series

arch/alpha/include/asm/io.h | 4 ----
1 file changed, 4 deletions(-)

diff --git a/arch/alpha/include/asm/io.h b/arch/alpha/include/asm/io.h
index 1c3605d874e9..7aeaf7c30a6f 100644
--- a/arch/alpha/include/asm/io.h
+++ b/arch/alpha/include/asm/io.h
@@ -14,10 +14,6 @@
the implementation we have here matches that interface. */
#include <asm-generic/iomap.h>

-/* We don't use IO slowdowns on the Alpha, but.. */
-#define __SLOW_DOWN_IO do { } while (0)
-#define SLOW_DOWN_IO do { } while (0)
-
/*
* Virtual -> physical identity mapping starts at this offset
*/
--
2.25.1


2023-02-26 02:05:59

by Matt Turner

[permalink] [raw]
Subject: Re: [PATCH] alpha: remove unused __SLOW_DOWN_IO and SLOW_DOWN_IO definitions

On Fri, Oct 21, 2022 at 4:18 PM Bjorn Helgaas <[email protected]> wrote:
>
> From: Bjorn Helgaas <[email protected]>
>
> Remove unused __SLOW_DOWN_IO and SLOW_DOWN_IO definitions.
>
> Signed-off-by: Bjorn Helgaas <[email protected]>
> ---
>
> [Sorry for the noise; I sent this a few minutes ago, but it bounced from
> all the individual recipients because of DMARC issues]
>
> Previously posted v1 4/15/2022 at
> https://lore.kernel.org/all/[email protected]/
> and reposted v1 10/13/2022 at
> https://lore.kernel.org/all/[email protected]/
>
> Changes from v1 to v2:
> - Drop the ia64 patch because Andrew applied it
> https://lore.kernel.org/all/[email protected]/
>
> - Split sh and alpha to separate series
>
> arch/alpha/include/asm/io.h | 4 ----
> 1 file changed, 4 deletions(-)
>
> diff --git a/arch/alpha/include/asm/io.h b/arch/alpha/include/asm/io.h
> index 1c3605d874e9..7aeaf7c30a6f 100644
> --- a/arch/alpha/include/asm/io.h
> +++ b/arch/alpha/include/asm/io.h
> @@ -14,10 +14,6 @@
> the implementation we have here matches that interface. */
> #include <asm-generic/iomap.h>
>
> -/* We don't use IO slowdowns on the Alpha, but.. */
> -#define __SLOW_DOWN_IO do { } while (0)
> -#define SLOW_DOWN_IO do { } while (0)
> -
> /*
> * Virtual -> physical identity mapping starts at this offset
> */
> --
> 2.25.1
>

Thanks for the patch! This was included in my pull request today and
is now upstream in Linus' tree.