| Message ID | 1450178204-26172-1-git-send-email-sudeep.holla@arm.com (mailing list archive) |
|---|---|
| State | Accepted, archived |
| Delegated to: | Rafael Wysocki |
| Headers | show |
Hi Rafael, On Tue, Dec 15, 2015 at 11:16 AM, Sudeep Holla <sudeep.holla@arm.com> wrote: > From: Dan Carpenter <dan.carpenter@oracle.com> > > The "domain" variable needs to be signed for the error handling to work. > > Fixes: 8def31034d03 ('cpufreq: arm_big_little: add SCPI interface driver') > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> > Acked-by: Sudeep Holla <sudeep.holla@arm.com> > --- > drivers/cpufreq/scpi-cpufreq.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > Hi Rafael, > > This patch seem to have slipped through the cracks somehow. I don't see > in patchwork too. So I am resending so that you can pick for your next > rc fix if you plan to send one else for -next > This got missed again, can you please ensure to pull this for your next -rc. If you have none planned, please pull this for-next. Regards, Sudeep -- 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 Wednesday, December 23, 2015 11:20:04 AM Sudeep Holla wrote: > Hi Rafael, > > On Tue, Dec 15, 2015 at 11:16 AM, Sudeep Holla <sudeep.holla@arm.com> wrote: > > From: Dan Carpenter <dan.carpenter@oracle.com> > > > > The "domain" variable needs to be signed for the error handling to work. > > > > Fixes: 8def31034d03 ('cpufreq: arm_big_little: add SCPI interface driver') > > Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> > > Acked-by: Viresh Kumar <viresh.kumar@linaro.org> > > Acked-by: Sudeep Holla <sudeep.holla@arm.com> > > --- > > drivers/cpufreq/scpi-cpufreq.c | 2 +- > > 1 file changed, 1 insertion(+), 1 deletion(-) > > > > Hi Rafael, > > > > This patch seem to have slipped through the cracks somehow. I don't see > > in patchwork too. So I am resending so that you can pick for your next > > rc fix if you plan to send one else for -next > > > > This got missed again, can you please ensure to pull this for your next -rc. > If you have none planned, please pull this for-next. Applied now, thanks! I'm going to push it for -rc7. 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 24/12/15 01:45, Rafael J. Wysocki wrote: > On Wednesday, December 23, 2015 11:20:04 AM Sudeep Holla wrote: >> Hi Rafael, >> >> On Tue, Dec 15, 2015 at 11:16 AM, Sudeep Holla <sudeep.holla@arm.com> wrote: >>> From: Dan Carpenter <dan.carpenter@oracle.com> >>> >>> The "domain" variable needs to be signed for the error handling to work. >>> >>> Fixes: 8def31034d03 ('cpufreq: arm_big_little: add SCPI interface driver') >>> Signed-off-by: Dan Carpenter <dan.carpenter@oracle.com> >>> Acked-by: Viresh Kumar <viresh.kumar@linaro.org> >>> Acked-by: Sudeep Holla <sudeep.holla@arm.com> >>> --- >>> drivers/cpufreq/scpi-cpufreq.c | 2 +- >>> 1 file changed, 1 insertion(+), 1 deletion(-) >>> >>> Hi Rafael, >>> >>> This patch seem to have slipped through the cracks somehow. I don't see >>> in patchwork too. So I am resending so that you can pick for your next >>> rc fix if you plan to send one else for -next >>> >> >> This got missed again, can you please ensure to pull this for your next -rc. >> If you have none planned, please pull this for-next. > > Applied now, thanks! > > I'm going to push it for -rc7. > Thanks, sorry for the nags.
diff --git a/drivers/cpufreq/scpi-cpufreq.c b/drivers/cpufreq/scpi-cpufreq.c index 2c3b16fd3a01..de5e89b2eaaa 100644 --- a/drivers/cpufreq/scpi-cpufreq.c +++ b/drivers/cpufreq/scpi-cpufreq.c @@ -31,7 +31,7 @@ static struct scpi_ops *scpi_ops; static struct scpi_dvfs_info *scpi_get_dvfs_info(struct device *cpu_dev) { - u8 domain = topology_physical_package_id(cpu_dev->id); + int domain = topology_physical_package_id(cpu_dev->id); if (domain < 0) return ERR_PTR(-EINVAL);