Re: [PATCH v2 3/7] intel-speed-select: Add check for CascadeLake-N models

2019-10-07 Thread Srinivas Pandruvada
On Mon, 2019-10-07 at 13:03 +0300, Andy Shevchenko wrote: > On Fri, Oct 04, 2019 at 10:15:21AM -0700, Srinivas Pandruvada wrote: > > On Thu, 2019-10-03 at 08:11 -0400, Prarit Bhargava wrote: > > > + /* only three CascadeLake-N models are supported */ > > > + if (is_clx_n_platform()) { > > > +

Re: [PATCH v2 3/7] intel-speed-select: Add check for CascadeLake-N models

2019-10-07 Thread Andy Shevchenko
On Fri, Oct 04, 2019 at 10:15:21AM -0700, Srinivas Pandruvada wrote: > On Thu, 2019-10-03 at 08:11 -0400, Prarit Bhargava wrote: > > + /* only three CascadeLake-N models are supported */ > > + if (is_clx_n_platform()) { > > + FILE *fp; > > + size_t n; > > + char *

Re: [PATCH v2 3/7] intel-speed-select: Add check for CascadeLake-N models

2019-10-04 Thread Prarit Bhargava
On 10/4/19 1:15 PM, Srinivas Pandruvada wrote: > On Thu, 2019-10-03 at 08:11 -0400, Prarit Bhargava wrote: >> Three CascadeLake-N models (6252N, 6230N, and 5218N) have SST-PBF >> support. >> >> Return an error if the CascadeLake processor is not one of these >> specific >> models. >> > This patc

Re: [PATCH v2 3/7] intel-speed-select: Add check for CascadeLake-N models

2019-10-04 Thread Srinivas Pandruvada
On Thu, 2019-10-03 at 08:11 -0400, Prarit Bhargava wrote: > Three CascadeLake-N models (6252N, 6230N, and 5218N) have SST-PBF > support. > > Return an error if the CascadeLake processor is not one of these > specific > models. > This patch sigfaults immediately on CLX. > v2: Add is_clx_n_platfor