| Message ID | 20161012182550.2102-1-piotr.luc@intel.com (mailing list archive) |
|---|---|
| State | Not Applicable, archived |
| Headers | show |
On Wednesday, October 12, 2016 08:25:50 PM Piotr Luc wrote: > Add Knights Mill (KNM) to the list of CPUIDs supported by intel_pstate. > > Signed-off-by: Piotr Luc <piotr.luc@intel.com> > Reviewed-by: Dave Hansen <dave.hansen@intel.com> > Cc: Srinivas Pandruvada <srinivas.pandruvada@linux.intel.com> > Cc: Len Brown <lenb@kernel.org> > Cc: "Rafael J. Wysocki" <rjw@rjwysocki.net> > Cc: Viresh Kumar <viresh.kumar@linaro.org> > Cc: linux-pm@vger.kernel.org > Cc: linux-kernel@vger.kernel.org > --- > Depends-on: x86/cpu/intel: Add Knights Mill to Intel family > https://lkml.kernel.org/r/20161012180520.30976-1-piotr.luc () intel ! com First, putting broken URLs into patch tags doesn't really help. Second, please send all of these patches as a series where the first patch makes the change the rest depends on. That would make them easier to handle. > > drivers/cpufreq/intel_pstate.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c > index be9eade..4240aff 100644 > --- a/drivers/cpufreq/intel_pstate.c > +++ b/drivers/cpufreq/intel_pstate.c > @@ -1367,6 +1367,7 @@ static const struct x86_cpu_id intel_pstate_cpu_ids[] = { > ICPU(INTEL_FAM6_SKYLAKE_DESKTOP, core_params), > ICPU(INTEL_FAM6_BROADWELL_XEON_D, core_params), > ICPU(INTEL_FAM6_XEON_PHI_KNL, knl_params), > + ICPU(INTEL_FAM6_XEON_PHI_KNM, knl_params), > ICPU(INTEL_FAM6_ATOM_GOLDMONT, bxt_params), > {} > }; > Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
On Wed, 2016-10-12 at 21:39 +0200, Rafael J. Wysocki wrote: > On Wednesday, October 12, 2016 08:25:50 PM Piotr Luc wrote: > > > > --- > > Depends-on: x86/cpu/intel: Add Knights Mill to Intel family > > https://lkml.kernel.org/r/20161012180520.30976-1-piotr.luc () intel > > ! com > > First, putting broken URLs into patch tags doesn't really help. OK, with a series the link is not needed. I will refer to a patch inside the series. > > Second, please send all of these patches as a series where the first > patch makes the change the rest depends on. That would make them > easier to > handle. > I see. I just wonder if I should send the entire series to all maintainers that maintain individual modules/files or somehow limit 'TO' recipients only to their modules. Is any good method to it correctly? Thanks, Piotr
On Thursday, October 13, 2016 02:48:28 PM Luc, Piotr wrote: > On Wed, 2016-10-12 at 21:39 +0200, Rafael J. Wysocki wrote: > > On Wednesday, October 12, 2016 08:25:50 PM Piotr Luc wrote: > > > > > > --- > > > Depends-on: x86/cpu/intel: Add Knights Mill to Intel family > > > https://lkml.kernel.org/r/20161012180520.30976-1-piotr.luc () intel > > > ! com > > > > First, putting broken URLs into patch tags doesn't really help. > > OK, with a series the link is not needed. I will refer to a patch > inside the series. > > > > > Second, please send all of these patches as a series where the first > > patch makes the change the rest depends on. That would make them > > easier to > > handle. > > > I see. I just wonder if I should send the entire series to all > maintainers that maintain individual modules/files or somehow limit > 'TO' recipients only to their modules. > Is any good method to it correctly? The To: list should be possibly short. I would just send it To: x86@kernel.org and put the other addresses into the CC list. Generally speaking, CCs of the entire series should go to all of the relevant maintainers (so they have a chance to respond to your patches in the context of the series) though. Thanks, Rafael -- To unsubscribe from this list: send the line "unsubscribe linux-pm" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/drivers/cpufreq/intel_pstate.c b/drivers/cpufreq/intel_pstate.c index be9eade..4240aff 100644 --- a/drivers/cpufreq/intel_pstate.c +++ b/drivers/cpufreq/intel_pstate.c @@ -1367,6 +1367,7 @@ static const struct x86_cpu_id intel_pstate_cpu_ids[] = { ICPU(INTEL_FAM6_SKYLAKE_DESKTOP, core_params), ICPU(INTEL_FAM6_BROADWELL_XEON_D, core_params), ICPU(INTEL_FAM6_XEON_PHI_KNL, knl_params), + ICPU(INTEL_FAM6_XEON_PHI_KNM, knl_params), ICPU(INTEL_FAM6_ATOM_GOLDMONT, bxt_params), {} };