Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-29 Thread Michael Ellerman
Nicholas Piggin writes: > On Mon, 29 May 2017 20:29:49 +1000 > Michael Ellerman wrote: >> Nicholas Piggin writes: ... >> >> In practice it works because the base CPU spec that we initialise in >> dt_cpu_ftrs.c is mostly OK, and skiboot also adds the cpu-version >> property which causes us to ca

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-29 Thread Michael Ellerman
Balbir Singh writes: > On Thu, 2017-05-11 at 21:24 +1000, Nicholas Piggin wrote: >> diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c >> b/arch/powerpc/kernel/dt_cpu_ftrs.c >> index fcc7588a96d6..050925b5b451 100644 >> --- a/arch/powerpc/kernel/dt_cpu_ftrs.c >> +++ b/arch/powerpc/kernel/dt_cpu_ftrs.

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-29 Thread Nicholas Piggin
On Mon, 29 May 2017 20:29:49 +1000 Michael Ellerman wrote: > Nicholas Piggin writes: > > > Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU > > feature discovery, and fall back to the "cputable" based version. > > I don't think this works properly. > > > diff --git a/arc

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-29 Thread Balbir Singh
On Thu, 2017-05-11 at 21:24 +1000, Nicholas Piggin wrote: > Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU > feature discovery, and fall back to the "cputable" based version. > > Also allow control of advertising unknown features to userspace and > with this parameter, and re

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-29 Thread Michael Ellerman
Nicholas Piggin writes: > Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU > feature discovery, and fall back to the "cputable" based version. I don't think this works properly. > diff --git a/arch/powerpc/kernel/dt_cpu_ftrs.c > b/arch/powerpc/kernel/dt_cpu_ftrs.c > index f

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-15 Thread Michael Ellerman
Paul Clarke writes: > On 05/11/2017 10:46 PM, Michael Ellerman wrote: >> Paul Clarke writes: >>> On 05/11/2017 06:24 AM, Nicholas Piggin wrote: Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU feature discovery, and fall back to the "cputable" based version. >>> >>>

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-12 Thread Paul Clarke
On 05/11/2017 10:46 PM, Michael Ellerman wrote: > Paul Clarke writes: >> On 05/11/2017 06:24 AM, Nicholas Piggin wrote: >>> Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU >>> feature discovery, and fall back to the "cputable" based version. >> >> This boat has already sailed,

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-11 Thread Michael Ellerman
Paul Clarke writes: > On 05/11/2017 06:24 AM, Nicholas Piggin wrote: >> Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU >> feature discovery, and fall back to the "cputable" based version. > > This boat has already sailed, I think, but "ftrs"? What you think vowels grow on t

Re: [PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-11 Thread Paul Clarke
On 05/11/2017 06:24 AM, Nicholas Piggin wrote: > Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU > feature discovery, and fall back to the "cputable" based version. This boat has already sailed, I think, but "ftrs"? Was it too difficult to type "features"? This seems like s

[PATCH] powerpc/64s: dt_cpu_ftrs boot time setup option

2017-05-11 Thread Nicholas Piggin
Provide a dt_cpu_ftrs= cmdline option to disable the dt_cpu_ftrs CPU feature discovery, and fall back to the "cputable" based version. Also allow control of advertising unknown features to userspace and with this parameter, and remove the clunky CONFIG option. Signed-off-by: Nicholas Piggin ---