[permalink] [raw]

Subject: [PATCH 02/13] docs: Remove metag docs

Now that arch/metag/ has been removed, remove Meta architecture specific
documentation from the Documentation/ directory.

Signed-off-by: James Hogan <[email protected]>
Cc: Jonathan Corbet <[email protected]>
Cc: [email protected]
Cc: [email protected]
---
Documentation/00-INDEX | 2 -
Documentation/admin-guide/kernel-parameters.txt | 4 -
Documentation/devicetree/bindings/metag/meta.txt | 30 ---
Documentation/metag/00-INDEX | 4 -
Documentation/metag/kernel-ABI.txt | 256 -----------------------
5 files changed, 296 deletions(-)
delete mode 100644 Documentation/devicetree/bindings/metag/meta.txt
delete mode 100644 Documentation/metag/00-INDEX
delete mode 100644 Documentation/metag/kernel-ABI.txt

diff --git a/Documentation/00-INDEX b/Documentation/00-INDEX
index 7f3a0728ccf2..eae1e7193f50 100644
--- a/Documentation/00-INDEX
+++ b/Documentation/00-INDEX
@@ -276,8 +276,6 @@ memory-hotplug.txt
- Hotpluggable memory support, how to use and current status.
men-chameleon-bus.txt
- info on MEN chameleon bus.
-metag/
- - directory with info about Linux on Meta architecture.
mic/
- Intel Many Integrated Core (MIC) architecture device driver.
mips/
diff --git a/Documentation/admin-guide/kernel-parameters.txt b/Documentation/admin-guide/kernel-parameters.txt
index 1d1d53f85ddd..30a8d0635898 100644
--- a/Documentation/admin-guide/kernel-parameters.txt
+++ b/Documentation/admin-guide/kernel-parameters.txt
@@ -1347,10 +1347,6 @@
If specified, z/VM IUCV HVC accepts connections
from listed z/VM user IDs only.

- hwthread_map= [METAG] Comma-separated list of Linux cpu id to
- hardware thread id mappings.
- Format: <cpu>:<hwthread>
-
keep_bootcon [KNL]
Do not unregister boot console at start. This is only
useful for debugging when something happens in the window
diff --git a/Documentation/devicetree/bindings/metag/meta.txt b/Documentation/devicetree/bindings/metag/meta.txt
deleted file mode 100644
index f4457f57ab08..000000000000
diff --git a/Documentation/metag/00-INDEX b/Documentation/metag/00-INDEX
deleted file mode 100644
index db11c513bd5c..000000000000
diff --git a/Documentation/metag/kernel-ABI.txt b/Documentation/metag/kernel-ABI.txt
deleted file mode 100644
index 628216603198..000000000000
--
2.13.6

2018-02-22 01:28:11

by Guenter Roeck

[permalink] [raw]

Subject: Re: [PATCH 11/13] watchdog: imgpdc: Drop METAG dependency

On 02/21/2018 03:38 PM, James Hogan wrote:
> Now that arch/metag/ has been removed, remove the METAG dependency from
> the IMG IR device driver. The hardware is also present on MIPS SoCs so
> the driver still has value.
>
> Signed-off-by: James Hogan <[email protected]>
> Cc: Wim Van Sebroeck <[email protected]>
> Cc: Guenter Roeck <[email protected]>
> Cc: [email protected]
> Cc: [email protected]

Acked-by: Guenter Roeck <[email protected]>

> ---
> drivers/watchdog/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/watchdog/Kconfig b/drivers/watchdog/Kconfig
> index aff773bcebdb..152268010c45 100644
> --- a/drivers/watchdog/Kconfig
> +++ b/drivers/watchdog/Kconfig
> @@ -1602,7 +1602,7 @@ config BCM7038_WDT
> config IMGPDC_WDT
> tristate "Imagination Technologies PDC Watchdog Timer"
> depends on HAS_IOMEM
> - depends on METAG || MIPS || COMPILE_TEST
> + depends on MIPS || COMPILE_TEST
> select WATCHDOG_CORE
> help
> Driver for Imagination Technologies PowerDown Controller
>

2018-02-22 01:53:17

by Steven Rostedt

[permalink] [raw]

Subject: Re: [PATCH 04/13] Drop a bunch of metag references

On Wed, 21 Feb 2018 23:38:16 +0000
James Hogan <[email protected]> wrote:

> Now that arch/metag/ has been removed, drop a bunch of metag references
> in various codes across the whole tree:
> - VM_GROWSUP and __VM_ARCH_PECIFIC_1.
> - MT_METAG_* ELF note types.
> - METAG Kconfig dependencies (FRAME_POINTER) and ranges
> (MAX_STACK_SIZE_MB).
> - metag cases in tools (checkstack.pl, recordmcount.c, perf).
>
> Signed-off-by: James Hogan <[email protected]>
> Cc: Steven Rostedt <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: Peter Zijlstra <[email protected]>
> Cc: Arnaldo Carvalho de Melo <[email protected]>
> Cc: Alexander Shishkin <[email protected]>
> Cc: Jiri Olsa <[email protected]>
> Cc: Namhyung Kim <[email protected]>
> Cc: [email protected]
> Cc: [email protected]
> ---
> include/linux/mm.h | 2 --

> include/trace/events/mmflags.h | 2 +-

> include/uapi/linux/elf.h | 3 ---
> lib/Kconfig.debug | 2 +-
> mm/Kconfig | 7 +++----
> scripts/checkstack.pl | 4 ----

> scripts/recordmcount.c | 20 --------------------

Acked-by: Steven Rostedt (VMware) <[email protected]>

-- Steve

> tools/perf/perf-sys.h | 4 ----
> 8 files changed, 5 insertions(+), 39 deletions(-)

> diff --git a/include/trace/events/mmflags.h b/include/trace/events/mmflags.h
> index dbe1bb058c09..a81cffb76d89 100644
> --- a/include/trace/events/mmflags.h
> +++ b/include/trace/events/mmflags.h
> @@ -115,7 +115,7 @@ IF_HAVE_PG_IDLE(PG_idle, "idle" )
> #define __VM_ARCH_SPECIFIC_1 {VM_PAT, "pat" }
> #elif defined(CONFIG_PPC)
> #define __VM_ARCH_SPECIFIC_1 {VM_SAO, "sao" }
> -#elif defined(CONFIG_PARISC) || defined(CONFIG_METAG) || defined(CONFIG_IA64)
> +#elif defined(CONFIG_PARISC) || defined(CONFIG_IA64)
> #define __VM_ARCH_SPECIFIC_1 {VM_GROWSUP, "growsup" }
> #elif !defined(CONFIG_MMU)
> #define __VM_ARCH_SPECIFIC_1 {VM_MAPPED_COPY,"mappedcopy" }

> diff --git a/scripts/recordmcount.c b/scripts/recordmcount.c
> index 16e086dcc567..8c9691c3329e 100644
> --- a/scripts/recordmcount.c
> +++ b/scripts/recordmcount.c
> @@ -33,20 +33,6 @@
> #include <string.h>
> #include <unistd.h>
>
> -/*
> - * glibc synced up and added the metag number but didn't add the relocations.
> - * Work around this in a crude manner for now.
> - */
> -#ifndef EM_METAG
> -#define EM_METAG 174
> -#endif
> -#ifndef R_METAG_ADDR32
> -#define R_METAG_ADDR32 2
> -#endif
> -#ifndef R_METAG_NONE
> -#define R_METAG_NONE 3
> -#endif
> -
> #ifndef EM_AARCH64
> #define EM_AARCH64 183
> #define R_AARCH64_NONE 0
> @@ -538,12 +524,6 @@ do_file(char const *const fname)
> gpfx = '_';
> break;
> case EM_IA_64: reltype = R_IA64_IMM64; gpfx = '_'; break;
> - case EM_METAG: reltype = R_METAG_ADDR32;
> - altmcount = "_mcount_wrapper";
> - rel_type_nop = R_METAG_NONE;
> - /* We happen to have the same requirement as MIPS */
> - is_fake_mcount32 = MIPS32_is_fake_mcount;
> - break;
> case EM_MIPS: /* reltype: e_class */ gpfx = '_'; break;
> case EM_PPC: reltype = R_PPC_ADDR32; gpfx = '_'; break;
> case EM_PPC64: reltype = R_PPC64_ADDR64; gpfx = '_'; break;

2018-02-22 03:53:38

by Guenter Roeck

On Thu, Feb 22, 2018 at 10:26:54AM +0100, Peter Zijlstra wrote:
> On Wed, Feb 21, 2018 at 11:38:12PM +0000, James Hogan wrote:
> > So lets call it a day and drop the Meta architecture port from the
> > kernel. RIP Meta.
>
> So long, and thanks for all the fish!
>
> Nice cleanup though, most welcome :-)

I thought you might like it ;-)

> Acked-by: Peter Zijlstra (Intel) <[email protected]>

Thanks
James

Attachments:

(No filename) (427.00 B)
signature.asc (849.00 B)
Digital signature Download all attachments

2018-02-22 10:12:55

by Daniel Lezcano

[permalink] [raw]

Subject: Re: [PATCH 06/13] clocksource: Remove metag generic timer driver

On 22/02/2018 00:38, James Hogan wrote:
> Now that arch/metag/ has been removed, remove the metag generic
> per-thread timer driver. It is of no value without the architecture
> code.
>
> Signed-off-by: James Hogan <[email protected]>
> Cc: Daniel Lezcano <[email protected]>
> Cc: Thomas Gleixner <[email protected]>
> Cc: [email protected]

Acked-by: Daniel Lezcano <[email protected]>

--
<http://www.linaro.org/> Linaro.org │ Open source software for ARM SoCs

Follow Linaro: <http://www.facebook.com/pages/Linaro> Facebook |
<http://twitter.com/#!/linaroorg> Twitter |
<http://www.linaro.org/linaro-blog/> Blog

2018-02-22 10:43:47

by Graham Whaley

[permalink] [raw]

Subject: Re: [PATCH 00/13] Remove metag architecture

On 22 February 2018 at 09:48, James Hogan <[email protected]> wrote:
> On Thu, Feb 22, 2018 at 10:26:54AM +0100, Peter Zijlstra wrote:
>> On Wed, Feb 21, 2018 at 11:38:12PM +0000, James Hogan wrote:
>> > So lets call it a day and drop the Meta architecture port from the
>> > kernel. RIP Meta.
>>
>> So long, and thanks for all the fish!
>>
>> Nice cleanup though, most welcome :-)
>
> I thought you might like it ;-)
>
>> Acked-by: Peter Zijlstra (Intel) <[email protected]>
>
> Thanks
> James

RIP indeed. As I sit here listening to my META powered radio.... for
the series in general:

Acked-by: Graham Whaley <[email protected]>

2018-02-22 16:20:47

by Graham Whaley

[permalink] [raw]

Subject: Re: [PATCH 01/13] metag: Remove arch/metag/

>> >
>> > So lets call it a day and drop the Meta architecture port from the
>> > kernel. RIP Meta.

Oh, and James, I meant to ask/note - I guess that just leaves parisc
using STACK_GROWSUP... shame we could not ditch that as well then eh?

Graham

2018-02-22 16:40:34

On Fri, Feb 23, 2018 at 11:26:58AM +0100, Arnd Bergmann wrote:
> On Thu, Feb 22, 2018 at 12:38 AM, James Hogan <[email protected]> wrote:
> > So lets call it a day and drop the Meta architecture port from the
> > kernel. RIP Meta.
>
> Since I brought up the architecture removal independently, I could
> pick this up into a git tree that also has the removal of some of the
> other architectures.
>
> I see your tree is part of linux-next, so you could also just put it
> in there and send a pull request at the merge window if you prefer.
>
> The only real reason I see for a shared git tree would be to avoid
> conflicts when we touch the same Kconfig files or #ifdefs in driver,
> but Meta only appears in
>
> config FRAME_POINTER
> bool "Compile the kernel with frame pointers"
> depends on DEBUG_KERNEL && \
> (CRIS || M68K || FRV || UML || \
> SUPERH || BLACKFIN || MN10300 || METAG) || \
> ARCH_WANT_FRAME_POINTERS
>
> and
>
> include/trace/events/mmflags.h:#elif defined(CONFIG_PARISC) ||
> defined(CONFIG_METAG) || defined(CONFIG_IA64)
>
> so there is little risk.

I'm happy to put v2 in linux-next now (only patch 4 has changed, I just
sent an updated version), and send you a pull request early next week so
you can take it from there. The patches can't be directly applied with
git-am anyway thanks to the -D option to make them more concise.

Sound okay?

Thanks
James

Attachments:

(No filename) (1.46 kB)
signature.asc (849.00 B)
Digital signature Download all attachments

2018-02-23 12:27:11

by Arnd Bergmann

[permalink] [raw]

Subject: Re: [PATCH 00/13] Remove metag architecture

On Fri, Feb 23, 2018 at 12:02 PM, James Hogan <[email protected]> wrote:
> On Fri, Feb 23, 2018 at 11:26:58AM +0100, Arnd Bergmann wrote:
>> On Thu, Feb 22, 2018 at 12:38 AM, James Hogan <[email protected]> wrote:
>> > So lets call it a day and drop the Meta architecture port from the
>> > kernel. RIP Meta.
>>
>> Since I brought up the architecture removal independently, I could
>> pick this up into a git tree that also has the removal of some of the
>> other architectures.
>>
>> I see your tree is part of linux-next, so you could also just put it
>> in there and send a pull request at the merge window if you prefer.
>>
>> The only real reason I see for a shared git tree would be to avoid
>> conflicts when we touch the same Kconfig files or #ifdefs in driver,
>> but Meta only appears in
>>
>> config FRAME_POINTER
>> bool "Compile the kernel with frame pointers"
>> depends on DEBUG_KERNEL && \
>> (CRIS || M68K || FRV || UML || \
>> SUPERH || BLACKFIN || MN10300 || METAG) || \
>> ARCH_WANT_FRAME_POINTERS
>>
>> and
>>
>> include/trace/events/mmflags.h:#elif defined(CONFIG_PARISC) ||
>> defined(CONFIG_METAG) || defined(CONFIG_IA64)
>>
>> so there is little risk.
>
> I'm happy to put v2 in linux-next now (only patch 4 has changed, I just
> sent an updated version), and send you a pull request early next week so
> you can take it from there. The patches can't be directly applied with
> git-am anyway thanks to the -D option to make them more concise.
>
> Sound okay?

Yes, sounds good, thanks!

Arnd

2018-02-23 14:29:57

by Guenter Roeck

[permalink] [raw]

Subject: Re: [PATCH v2 04/13] Drop a bunch of metag references

On 02/23/2018 02:53 AM, James Hogan wrote:
> Now that arch/metag/ has been removed, drop a bunch of metag references
> in various codes across the whole tree:
> - VM_GROWSUP and __VM_ARCH_PECIFIC_1.
> - MT_METAG_* ELF note types.
> - METAG Kconfig dependencies (FRAME_POINTER) and ranges
> (MAX_STACK_SIZE_MB).
> - metag cases in tools (checkstack.pl, recordmcount.c, perf).
>
> Signed-off-by: James Hogan <[email protected]>
> Acked-by: Steven Rostedt (VMware) <[email protected]>
> Acked-by: Peter Zijlstra (Intel) <[email protected]>
> Cc: Ingo Molnar <[email protected]>
> Cc: Arnaldo Carvalho de Melo <[email protected]>
> Cc: Alexander Shishkin <[email protected]>
> Cc: Jiri Olsa <[email protected]>
> Cc: Namhyung Kim <[email protected]>
> Cc: Guenter Roeck <[email protected]>
> Cc: [email protected]
> Cc: [email protected]

Reviewed-by: Guenter Roeck <[email protected]>

> ---
> Changes in v2:
> - Drop CPUHP_AP_PERF_METAG_STARTING too (Guenter).
> ---
> include/linux/cpuhotplug.h | 1 -
> include/linux/mm.h | 2 --
> include/trace/events/mmflags.h | 2 +-
> include/uapi/linux/elf.h | 3 ---
> lib/Kconfig.debug | 2 +-
> mm/Kconfig | 7 +++----
> scripts/checkstack.pl | 4 ----
> scripts/recordmcount.c | 20 --------------------
> tools/perf/perf-sys.h | 4 ----
> 9 files changed, 5 insertions(+), 40 deletions(-)
>
> diff --git a/include/linux/cpuhotplug.h b/include/linux/cpuhotplug.h
> index 5172ad0daa7c..c7a950681f3a 100644
> --- a/include/linux/cpuhotplug.h
> +++ b/include/linux/cpuhotplug.h
> @@ -108,7 +108,6 @@ enum cpuhp_state {
> CPUHP_AP_PERF_X86_CQM_STARTING,
> CPUHP_AP_PERF_X86_CSTATE_STARTING,
> CPUHP_AP_PERF_XTENSA_STARTING,
> - CPUHP_AP_PERF_METAG_STARTING,
> CPUHP_AP_MIPS_OP_LOONGSON3_STARTING,
> CPUHP_AP_ARM_SDEI_STARTING,
> CPUHP_AP_ARM_VFP_STARTING,
> diff --git a/include/linux/mm.h b/include/linux/mm.h
> index ad06d42adb1a..ccac10682ce5 100644
> --- a/include/linux/mm.h
> +++ b/include/linux/mm.h
> @@ -241,8 +241,6 @@ extern unsigned int kobjsize(const void *objp);
> # define VM_SAO VM_ARCH_1 /* Strong Access Ordering (powerpc) */
> #elif defined(CONFIG_PARISC)
> # define VM_GROWSUP VM_ARCH_1
> -#elif defined(CONFIG_METAG)
> -# define VM_GROWSUP VM_ARCH_1
> #elif defined(CONFIG_IA64)
> # define VM_GROWSUP VM_ARCH_1
> #elif !defined(CONFIG_MMU)
> diff --git a/include/trace/events/mmflags.h b/include/trace/events/mmflags.h
> index dbe1bb058c09..a81cffb76d89 100644
> --- a/include/trace/events/mmflags.h
> +++ b/include/trace/events/mmflags.h
> @@ -115,7 +115,7 @@ IF_HAVE_PG_IDLE(PG_idle, "idle" )
> #define __VM_ARCH_SPECIFIC_1 {VM_PAT, "pat" }
> #elif defined(CONFIG_PPC)
> #define __VM_ARCH_SPECIFIC_1 {VM_SAO, "sao" }
> -#elif defined(CONFIG_PARISC) || defined(CONFIG_METAG) || defined(CONFIG_IA64)
> +#elif defined(CONFIG_PARISC) || defined(CONFIG_IA64)
> #define __VM_ARCH_SPECIFIC_1 {VM_GROWSUP, "growsup" }
> #elif !defined(CONFIG_MMU)
> #define __VM_ARCH_SPECIFIC_1 {VM_MAPPED_COPY,"mappedcopy" }
> diff --git a/include/uapi/linux/elf.h b/include/uapi/linux/elf.h
> index 3bf73fb58045..e2535d6dcec7 100644
> --- a/include/uapi/linux/elf.h
> +++ b/include/uapi/linux/elf.h
> @@ -420,9 +420,6 @@ typedef struct elf64_shdr {
> #define NT_ARM_HW_WATCH 0x403 /* ARM hardware watchpoint registers */
> #define NT_ARM_SYSTEM_CALL 0x404 /* ARM system call number */
> #define NT_ARM_SVE 0x405 /* ARM Scalable Vector Extension registers */
> -#define NT_METAG_CBUF 0x500 /* Metag catch buffer registers */
> -#define NT_METAG_RPIPE 0x501 /* Metag read pipeline state */
> -#define NT_METAG_TLS 0x502 /* Metag TLS pointer */
> #define NT_ARC_V2 0x600 /* ARCv2 accumulator/extra registers */
>
> /* Note header in a PT_NOTE section */
> diff --git a/lib/Kconfig.debug b/lib/Kconfig.debug
> index 6088408ef26c..d1c523e408e9 100644
> --- a/lib/Kconfig.debug
> +++ b/lib/Kconfig.debug
> @@ -356,7 +356,7 @@ config FRAME_POINTER
> bool "Compile the kernel with frame pointers"
> depends on DEBUG_KERNEL && \
> (CRIS || M68K || FRV || UML || \
> - SUPERH || BLACKFIN || MN10300 || METAG) || \
> + SUPERH || BLACKFIN || MN10300) || \
> ARCH_WANT_FRAME_POINTERS
> default y if (DEBUG_INFO && UML) || ARCH_WANT_FRAME_POINTERS
> help
> diff --git a/mm/Kconfig b/mm/Kconfig
> index c782e8fb7235..abefa573bcd8 100644
> --- a/mm/Kconfig
> +++ b/mm/Kconfig
> @@ -627,15 +627,14 @@ config GENERIC_EARLY_IOREMAP
> config MAX_STACK_SIZE_MB
> int "Maximum user stack size for 32-bit processes (MB)"
> default 80
> - range 8 256 if METAG
> range 8 2048
> depends on STACK_GROWSUP && (!64BIT || COMPAT)
> help
> This is the maximum stack size in Megabytes in the VM layout of 32-bit
> user processes when the stack grows upwards (currently only on parisc
> - and metag arch). The stack will be located at the highest memory
> - address minus the given value, unless the RLIMIT_STACK hard limit is
> - changed to a smaller value in which case that is used.
> + arch). The stack will be located at the highest memory address minus
> + the given value, unless the RLIMIT_STACK hard limit is changed to a
> + smaller value in which case that is used.
>
> A sane initial value is 80 MB.
>
> diff --git a/scripts/checkstack.pl b/scripts/checkstack.pl
> index cb993801e4b2..eeb9ac8dbcfb 100755
> --- a/scripts/checkstack.pl
> +++ b/scripts/checkstack.pl
> @@ -64,10 +64,6 @@ my (@stack, $re, $dre, $x, $xs, $funcre);
> # 2b6c: 4e56 fb70 linkw %fp,#-1168
> # 1df770: defc ffe4 addaw #-28,%sp
> $re = qr/.*(?:linkw %fp,|addaw )#-([0-9]{1,4})(?:,%sp)?$/o;
> - } elsif ($arch eq 'metag') {
> - #400026fc: 40 00 00 82 ADD A0StP,A0StP,#0x8
> - $re = qr/.*ADD.*A0StP,A0StP,\#(0x$x{1,8})/o;
> - $funcre = qr/^$x* <[^\$](.*)>:$/;
> } elsif ($arch eq 'mips64') {
> #8800402c: 67bdfff0 daddiu sp,sp,-16
> $re = qr/.*daddiu.*sp,sp,-(([0-9]{2}|[3-9])[0-9]{2})/o;
> diff --git a/scripts/recordmcount.c b/scripts/recordmcount.c
> index 16e086dcc567..8c9691c3329e 100644
> --- a/scripts/recordmcount.c
> +++ b/scripts/recordmcount.c
> @@ -33,20 +33,6 @@
> #include <string.h>
> #include <unistd.h>
>
> -/*
> - * glibc synced up and added the metag number but didn't add the relocations.
> - * Work around this in a crude manner for now.
> - */
> -#ifndef EM_METAG
> -#define EM_METAG 174
> -#endif
> -#ifndef R_METAG_ADDR32
> -#define R_METAG_ADDR32 2
> -#endif
> -#ifndef R_METAG_NONE
> -#define R_METAG_NONE 3
> -#endif
> -
> #ifndef EM_AARCH64
> #define EM_AARCH64 183
> #define R_AARCH64_NONE 0
> @@ -538,12 +524,6 @@ do_file(char const *const fname)
> gpfx = '_';
> break;
> case EM_IA_64: reltype = R_IA64_IMM64; gpfx = '_'; break;
> - case EM_METAG: reltype = R_METAG_ADDR32;
> - altmcount = "_mcount_wrapper";
> - rel_type_nop = R_METAG_NONE;
> - /* We happen to have the same requirement as MIPS */
> - is_fake_mcount32 = MIPS32_is_fake_mcount;
> - break;
> case EM_MIPS: /* reltype: e_class */ gpfx = '_'; break;
> case EM_PPC: reltype = R_PPC_ADDR32; gpfx = '_'; break;
> case EM_PPC64: reltype = R_PPC64_ADDR64; gpfx = '_'; break;
> diff --git a/tools/perf/perf-sys.h b/tools/perf/perf-sys.h
> index 36673f98d66b..3eb7a39169f6 100644
> --- a/tools/perf/perf-sys.h
> +++ b/tools/perf/perf-sys.h
> @@ -46,10 +46,6 @@
> #define CPUINFO_PROC {"Processor"}
> #endif
>
> -#ifdef __metag__
> -#define CPUINFO_PROC {"CPU"}
> -#endif
> -
> #ifdef __xtensa__
> #define CPUINFO_PROC {"core ID"}
> #endif
>

2018-02-26 11:32:29

by Sean Young

[permalink] [raw]

Subject: Re: [PATCH 12/13] media: img-ir: Drop METAG dependency

On Wed, Feb 21, 2018 at 11:38:24PM +0000, James Hogan wrote:
> Now that arch/metag/ has been removed, remove the METAG dependency from
> the IMG IR device driver. The hardware is also present on MIPS SoCs so
> the driver still has value.
>
> Signed-off-by: James Hogan <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: Mauro Carvalho Chehab <[email protected]>
> Cc: [email protected]
> Cc: [email protected]

Acked-by: Sean Young <[email protected]>

> ---
> drivers/media/rc/img-ir/Kconfig | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/drivers/media/rc/img-ir/Kconfig b/drivers/media/rc/img-ir/Kconfig
> index a896d3c83a1c..d2c6617d468e 100644
> --- a/drivers/media/rc/img-ir/Kconfig
> +++ b/drivers/media/rc/img-ir/Kconfig
> @@ -1,7 +1,7 @@
> config IR_IMG
> tristate "ImgTec IR Decoder"
> depends on RC_CORE
> - depends on METAG || MIPS || COMPILE_TEST
> + depends on MIPS || COMPILE_TEST
> select IR_IMG_HW if !IR_IMG_RAW
> help
> Say Y or M here if you want to use the ImgTec infrared decoder
> --
> 2.13.6

2018-02-27 13:40:24

On Thu, Feb 22, 2018 at 12:38 AM, James Hogan <[email protected]> wrote:
> These patches remove the metag architecture and tightly dependent
> drivers from the kernel. With the 4.16 kernel the ancient gcc 4.2.4
> based metag toolchain we have been using is hitting compiler bugs, so
> now seems a good time to drop it altogether.
>
> Quoting from patch 1:
>
> The earliest Meta architecture port of Linux I have a record of was an
> import of a Meta port of Linux v2.4.1 in February 2004, which was worked
> on significantly over the next few years by Graham Whaley, Will Newton,
> Matt Fleming, myself and others.
>
> Eventually the port was merged into mainline in v3.9 in March 2013, not
> long after Imagination Technologies bought MIPS Technologies and shifted
> its CPU focus over to the MIPS architecture.
>
> As a result, though the port was maintained for a while, kept on life
> support for a while longer, and useful for testing a few specific
> drivers for which I don't have ready access to the equivalent MIPS
> hardware, it is now essentially dead with no users.
>
> It is also stuck using an out-of-tree toolchain based on GCC 4.2.4 which
> is no longer maintained, now struggles to build modern kernels due to
> toolchain bugs, and doesn't itself build with a modern GCC. The latest
> buildroot port is still using an old uClibc snapshot which is no longer
> served, and the latest uClibc doesn't build with GCC 4.2.4.
>
> So lets call it a day and drop the Meta architecture port from the
> kernel. RIP Meta.

I've pulled it into my asm-generic tree now, which is also part of linux-next,
and followed up with patches removing frv, m32r, score, unicore32
and blackfin. I have not removed the device drivers yet, but I'm working
on that.

Arnd