| Message ID | 20230606105656.124355-3-mason.huo@starfivetech.com (mailing list archive) |
|---|---|
| State | Handled Elsewhere |
| Headers | show |
| Series | Add JH7110 cpufreq support | expand |
| Context | Check | Description |
|---|---|---|
| conchuod/cover_letter | success | Series has a cover letter |
| conchuod/tree_selection | success | Guessed tree name to be for-next at HEAD 90502d51ab90 |
| conchuod/fixes_present | success | Fixes tag not required for -next series |
| conchuod/maintainers_pattern | success | MAINTAINERS pattern errors before the patch: 6 and now 6 |
| conchuod/verify_signedoff | success | Signed-off-by tag matches author and committer |
| conchuod/kdoc | success | Errors and warnings before: 0 this patch: 0 |
| conchuod/build_rv64_clang_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/module_param | success | Was 0 now: 0 |
| conchuod/build_rv64_gcc_allmodconfig | success | Errors and warnings before: 8 this patch: 8 |
| conchuod/build_rv32_defconfig | success | Build OK |
| conchuod/dtb_warn_rv64 | success | Errors and warnings before: 3 this patch: 3 |
| conchuod/header_inline | success | No static functions without inline keyword in header files |
| conchuod/checkpatch | success | total: 0 errors, 0 warnings, 0 checks, 8 lines checked |
| conchuod/build_rv64_nommu_k210_defconfig | success | Build OK |
| conchuod/verify_fixes | success | No Fixes tag |
| conchuod/build_rv64_nommu_virt_defconfig | success | Build OK |
On 06-06-23, 18:56, Mason Huo wrote: > Add the compatible strings for supporting the generic > cpufreq driver on the StarFive JH7110 SoC. > > Signed-off-by: Mason Huo <mason.huo@starfivetech.com> > --- > drivers/cpufreq/cpufreq-dt-platdev.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > index 338cf6cc6596..14aa8281c7f4 100644 > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > @@ -85,6 +85,8 @@ static const struct of_device_id allowlist[] __initconst = { > { .compatible = "st-ericsson,u9500", }, > { .compatible = "st-ericsson,u9540", }, > > + { .compatible = "starfive,jh7110", }, > + > { .compatible = "ti,omap2", }, > { .compatible = "ti,omap4", }, > { .compatible = "ti,omap5", }, I thought I already merged it ?
On Tue, Jun 06, 2023 at 04:38:04PM +0530, Viresh Kumar wrote: > On 06-06-23, 18:56, Mason Huo wrote: > > Add the compatible strings for supporting the generic > > cpufreq driver on the StarFive JH7110 SoC. > > > > Signed-off-by: Mason Huo <mason.huo@starfivetech.com> > > --- > > drivers/cpufreq/cpufreq-dt-platdev.c | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c > > index 338cf6cc6596..14aa8281c7f4 100644 > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > @@ -85,6 +85,8 @@ static const struct of_device_id allowlist[] __initconst = { > > { .compatible = "st-ericsson,u9500", }, > > { .compatible = "st-ericsson,u9540", }, > > > > + { .compatible = "starfive,jh7110", }, > > + > > { .compatible = "ti,omap2", }, > > { .compatible = "ti,omap4", }, > > { .compatible = "ti,omap5", }, > > I thought I already merged it ? You did, 4b4c0d37164c ("cpufreq: dt-platdev: Add JH7110 SOC to the allowlist").
Hi Viresh & Conor, Sorry for the confusion. Should I re-send the patch series and remove this patch? Or you can just ignore it? Thanks Mason > On Tue, Jun 06, 2023 at 04:38:04PM +0530, Viresh Kumar wrote: > > On 06-06-23, 18:56, Mason Huo wrote: > > > Add the compatible strings for supporting the generic cpufreq driver > > > on the StarFive JH7110 SoC. > > > > > > Signed-off-by: Mason Huo <mason.huo@starfivetech.com> > > > --- > > > drivers/cpufreq/cpufreq-dt-platdev.c | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c > > > b/drivers/cpufreq/cpufreq-dt-platdev.c > > > index 338cf6cc6596..14aa8281c7f4 100644 > > > --- a/drivers/cpufreq/cpufreq-dt-platdev.c > > > +++ b/drivers/cpufreq/cpufreq-dt-platdev.c > > > @@ -85,6 +85,8 @@ static const struct of_device_id allowlist[] __initconst = { > > > { .compatible = "st-ericsson,u9500", }, > > > { .compatible = "st-ericsson,u9540", }, > > > > > > + { .compatible = "starfive,jh7110", }, > > > + > > > { .compatible = "ti,omap2", }, > > > { .compatible = "ti,omap4", }, > > > { .compatible = "ti,omap5", }, > > > > I thought I already merged it ? > > You did, 4b4c0d37164c ("cpufreq: dt-platdev: Add JH7110 SOC to the allowlist").
On 07-06-23, 02:04, Mason Huo wrote:
> Or you can just ignore it?
I can do that :)
On Wed, Jun 07, 2023 at 02:04:37AM +0000, Mason Huo wrote: > Hi Viresh & Conor, > > Sorry for the confusion. > Should I re-send the patch series and remove this patch? > Or you can just ignore it? I applied the two dts patches yesterday, you shouldn't need to do anything.
Hi Conor & Viresh, > On Wed, Jun 07, 2023 at 02:04:37AM +0000, Mason Huo wrote: > > Hi Viresh & Conor, > > > > Sorry for the confusion. > > Should I re-send the patch series and remove this patch? > > Or you can just ignore it? > > I applied the two dts patches yesterday, you shouldn't need to do anything. Thank you for your help & review. B.R Mason
diff --git a/drivers/cpufreq/cpufreq-dt-platdev.c b/drivers/cpufreq/cpufreq-dt-platdev.c index 338cf6cc6596..14aa8281c7f4 100644 --- a/drivers/cpufreq/cpufreq-dt-platdev.c +++ b/drivers/cpufreq/cpufreq-dt-platdev.c @@ -85,6 +85,8 @@ static const struct of_device_id allowlist[] __initconst = { { .compatible = "st-ericsson,u9500", }, { .compatible = "st-ericsson,u9540", }, + { .compatible = "starfive,jh7110", }, + { .compatible = "ti,omap2", }, { .compatible = "ti,omap4", }, { .compatible = "ti,omap5", },
Add the compatible strings for supporting the generic cpufreq driver on the StarFive JH7110 SoC. Signed-off-by: Mason Huo <mason.huo@starfivetech.com> --- drivers/cpufreq/cpufreq-dt-platdev.c | 2 ++ 1 file changed, 2 insertions(+)