Message ID | 1442936337-3104-2-git-send-email-vishnupatekar0510@gmail.com |
---|---|
State | New |
Headers | show |
On Tue, Sep 22, 2015 at 11:38 PM, Vishnu Patekar <vishnupatekar0510@gmail.com> wrote: > Allwinner A83T is octa-core cortex-a7 based SoC. > It's clock control unit and prcm, pinmux are different from previous sun8i > series. > Its processor cores are arragned in two clusters 4 cores each, > similar to A80. > > Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> > --- > Documentation/devicetree/bindings/arm/sunxi.txt | 1 + > arch/arm/mach-sunxi/sunxi.c | 1 + > drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ > 3 files changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt > index 67da205..cf5ed27 100644 > --- a/Documentation/devicetree/bindings/arm/sunxi.txt > +++ b/Documentation/devicetree/bindings/arm/sunxi.txt > @@ -11,4 +11,5 @@ using one of the following compatible strings: > allwinner,sun8i-a23 > allwinner,sun8i-a33 > allwinner,sun8i-h3 > + allwinner,sun8i-a83t Alphabetic order please. > allwinner,sun9i-a80 > diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c > index 65bab28..b04aefa 100644 > --- a/arch/arm/mach-sunxi/sunxi.c > +++ b/arch/arm/mach-sunxi/sunxi.c > @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { > "allwinner,sun8i-a23", > "allwinner,sun8i-a33", > "allwinner,sun8i-h3", > + "allwinner,sun8i-a83t", Same here. > NULL, > }; > > diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c > index 413070d..f216d5d 100644 > --- a/drivers/clk/sunxi/clk-sunxi.c > +++ b/drivers/clk/sunxi/clk-sunxi.c > @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); > CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); > CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); > > +static void __init sun8ia83t_init_clocks(struct device_node *node) > +{ > + sunxi_init_clocks(NULL, 0); > +} > +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); sun8i? Missing underscore too. I think you should add this once you actually have clock support. Otherwise this call basically does nothing. > + > static void __init sun9i_init_clocks(struct device_node *node) > { > sunxi_init_clocks(NULL, 0); On the side, I will send patches to remove sun9i_init_clocks. Regards ChenYu -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hi, On Tue, Sep 22, 2015 at 11:38:54PM +0800, Vishnu Patekar wrote: > Allwinner A83T is octa-core cortex-a7 based SoC. > It's clock control unit and prcm, pinmux are different from previous sun8i > series. > Its processor cores are arragned in two clusters 4 cores each, > similar to A80. > > Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> > --- > Documentation/devicetree/bindings/arm/sunxi.txt | 1 + > arch/arm/mach-sunxi/sunxi.c | 1 + > drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ > 3 files changed, 8 insertions(+) > > diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt > index 67da205..cf5ed27 100644 > --- a/Documentation/devicetree/bindings/arm/sunxi.txt > +++ b/Documentation/devicetree/bindings/arm/sunxi.txt > @@ -11,4 +11,5 @@ using one of the following compatible strings: > allwinner,sun8i-a23 > allwinner,sun8i-a33 > allwinner,sun8i-h3 > + allwinner,sun8i-a83t > allwinner,sun9i-a80 > diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c > index 65bab28..b04aefa 100644 > --- a/arch/arm/mach-sunxi/sunxi.c > +++ b/arch/arm/mach-sunxi/sunxi.c > @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { > "allwinner,sun8i-a23", > "allwinner,sun8i-a33", > "allwinner,sun8i-h3", > + "allwinner,sun8i-a83t", > NULL, > }; > > diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c > index 413070d..f216d5d 100644 > --- a/drivers/clk/sunxi/clk-sunxi.c > +++ b/drivers/clk/sunxi/clk-sunxi.c > @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); > CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); > CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); > > +static void __init sun8ia83t_init_clocks(struct device_node *node) Having an underscore between sun8i and a83t would be great :) > +{ > + sunxi_init_clocks(NULL, 0); > +} > +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); ^ Is this a sun9i or sun8i SoC ? Is the list of protected clocks always going to be empty, or will you add some new clocks in that list in the future? Thanks! Maxime
On Tue, Sep 22, 2015 at 11:47:40PM +0800, Chen-Yu Tsai wrote: > On Tue, Sep 22, 2015 at 11:38 PM, Vishnu Patekar > <vishnupatekar0510@gmail.com> wrote: > > Allwinner A83T is octa-core cortex-a7 based SoC. > > It's clock control unit and prcm, pinmux are different from previous sun8i > > series. > > Its processor cores are arragned in two clusters 4 cores each, > > similar to A80. > > > > Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> > > --- > > Documentation/devicetree/bindings/arm/sunxi.txt | 1 + > > arch/arm/mach-sunxi/sunxi.c | 1 + > > drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ > > 3 files changed, 8 insertions(+) > > > > diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt > > index 67da205..cf5ed27 100644 > > --- a/Documentation/devicetree/bindings/arm/sunxi.txt > > +++ b/Documentation/devicetree/bindings/arm/sunxi.txt > > @@ -11,4 +11,5 @@ using one of the following compatible strings: > > allwinner,sun8i-a23 > > allwinner,sun8i-a33 > > allwinner,sun8i-h3 > > + allwinner,sun8i-a83t > > Alphabetic order please. > > > allwinner,sun9i-a80 > > diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c > > index 65bab28..b04aefa 100644 > > --- a/arch/arm/mach-sunxi/sunxi.c > > +++ b/arch/arm/mach-sunxi/sunxi.c > > @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { > > "allwinner,sun8i-a23", > > "allwinner,sun8i-a33", > > "allwinner,sun8i-h3", > > + "allwinner,sun8i-a83t", > > Same here. > > > NULL, > > }; > > > > diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c > > index 413070d..f216d5d 100644 > > --- a/drivers/clk/sunxi/clk-sunxi.c > > +++ b/drivers/clk/sunxi/clk-sunxi.c > > @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); > > CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); > > CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); > > > > +static void __init sun8ia83t_init_clocks(struct device_node *node) > > +{ > > + sunxi_init_clocks(NULL, 0); > > +} > > +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); > > sun8i? Missing underscore too. > > I think you should add this once you actually have clock support. > Otherwise this call basically does nothing. That's not true, it actually initializes most of the clocks in the system. Maxime
On Tue, Sep 22, 2015 at 11:53 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > On Tue, Sep 22, 2015 at 11:47:40PM +0800, Chen-Yu Tsai wrote: >> On Tue, Sep 22, 2015 at 11:38 PM, Vishnu Patekar >> <vishnupatekar0510@gmail.com> wrote: >> > Allwinner A83T is octa-core cortex-a7 based SoC. >> > It's clock control unit and prcm, pinmux are different from previous sun8i >> > series. >> > Its processor cores are arragned in two clusters 4 cores each, >> > similar to A80. >> > >> > Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> >> > --- >> > Documentation/devicetree/bindings/arm/sunxi.txt | 1 + >> > arch/arm/mach-sunxi/sunxi.c | 1 + >> > drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ >> > 3 files changed, 8 insertions(+) >> > >> > diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt >> > index 67da205..cf5ed27 100644 >> > --- a/Documentation/devicetree/bindings/arm/sunxi.txt >> > +++ b/Documentation/devicetree/bindings/arm/sunxi.txt >> > @@ -11,4 +11,5 @@ using one of the following compatible strings: >> > allwinner,sun8i-a23 >> > allwinner,sun8i-a33 >> > allwinner,sun8i-h3 >> > + allwinner,sun8i-a83t >> >> Alphabetic order please. >> >> > allwinner,sun9i-a80 >> > diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c >> > index 65bab28..b04aefa 100644 >> > --- a/arch/arm/mach-sunxi/sunxi.c >> > +++ b/arch/arm/mach-sunxi/sunxi.c >> > @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { >> > "allwinner,sun8i-a23", >> > "allwinner,sun8i-a33", >> > "allwinner,sun8i-h3", >> > + "allwinner,sun8i-a83t", >> >> Same here. >> >> > NULL, >> > }; >> > >> > diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c >> > index 413070d..f216d5d 100644 >> > --- a/drivers/clk/sunxi/clk-sunxi.c >> > +++ b/drivers/clk/sunxi/clk-sunxi.c >> > @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); >> > CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); >> > CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); >> > >> > +static void __init sun8ia83t_init_clocks(struct device_node *node) >> > +{ >> > + sunxi_init_clocks(NULL, 0); >> > +} >> > +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); >> >> sun8i? Missing underscore too. >> >> I think you should add this once you actually have clock support. >> Otherwise this call basically does nothing. > > That's not true, it actually initializes most of the clocks in the > system. Right. Sorry, I think my thoughts strayed to sun9i. ChenYu -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello, On Tue, Sep 22, 2015 at 11:47 PM, Chen-Yu Tsai <wens@csie.org> wrote: > On Tue, Sep 22, 2015 at 11:38 PM, Vishnu Patekar > <vishnupatekar0510@gmail.com> wrote: >> Allwinner A83T is octa-core cortex-a7 based SoC. >> It's clock control unit and prcm, pinmux are different from previous sun8i >> series. >> Its processor cores are arragned in two clusters 4 cores each, >> similar to A80. >> >> Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> >> --- >> Documentation/devicetree/bindings/arm/sunxi.txt | 1 + >> arch/arm/mach-sunxi/sunxi.c | 1 + >> drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ >> 3 files changed, 8 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt >> index 67da205..cf5ed27 100644 >> --- a/Documentation/devicetree/bindings/arm/sunxi.txt >> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt >> @@ -11,4 +11,5 @@ using one of the following compatible strings: >> allwinner,sun8i-a23 >> allwinner,sun8i-a33 >> allwinner,sun8i-h3 >> + allwinner,sun8i-a83t > > Alphabetic order please. Okie, I'll correct. > >> allwinner,sun9i-a80 >> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c >> index 65bab28..b04aefa 100644 >> --- a/arch/arm/mach-sunxi/sunxi.c >> +++ b/arch/arm/mach-sunxi/sunxi.c >> @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { >> "allwinner,sun8i-a23", >> "allwinner,sun8i-a33", >> "allwinner,sun8i-h3", >> + "allwinner,sun8i-a83t", > > Same here. > >> NULL, >> }; >> >> diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c >> index 413070d..f216d5d 100644 >> --- a/drivers/clk/sunxi/clk-sunxi.c >> +++ b/drivers/clk/sunxi/clk-sunxi.c >> @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); >> CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); >> CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); >> >> +static void __init sun8ia83t_init_clocks(struct device_node *node) >> +{ >> + sunxi_init_clocks(NULL, 0); >> +} >> +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); > > sun8i? Missing underscore too. Sorry, yes, sun8i. will add _. > > I think you should add this once you actually have clock support. > Otherwise this call basically does nothing. > >> + >> static void __init sun9i_init_clocks(struct device_node *node) >> { >> sunxi_init_clocks(NULL, 0); > > On the side, I will send patches to remove sun9i_init_clocks. > > > Regards > ChenYu Thanks for review. -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
Hello, On Tue, Sep 22, 2015 at 11:51 PM, Maxime Ripard <maxime.ripard@free-electrons.com> wrote: > Hi, > > On Tue, Sep 22, 2015 at 11:38:54PM +0800, Vishnu Patekar wrote: >> Allwinner A83T is octa-core cortex-a7 based SoC. >> It's clock control unit and prcm, pinmux are different from previous sun8i >> series. >> Its processor cores are arragned in two clusters 4 cores each, >> similar to A80. >> >> Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> >> --- >> Documentation/devicetree/bindings/arm/sunxi.txt | 1 + >> arch/arm/mach-sunxi/sunxi.c | 1 + >> drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ >> 3 files changed, 8 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt >> index 67da205..cf5ed27 100644 >> --- a/Documentation/devicetree/bindings/arm/sunxi.txt >> +++ b/Documentation/devicetree/bindings/arm/sunxi.txt >> @@ -11,4 +11,5 @@ using one of the following compatible strings: >> allwinner,sun8i-a23 >> allwinner,sun8i-a33 >> allwinner,sun8i-h3 >> + allwinner,sun8i-a83t >> allwinner,sun9i-a80 >> diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c >> index 65bab28..b04aefa 100644 >> --- a/arch/arm/mach-sunxi/sunxi.c >> +++ b/arch/arm/mach-sunxi/sunxi.c >> @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { >> "allwinner,sun8i-a23", >> "allwinner,sun8i-a33", >> "allwinner,sun8i-h3", >> + "allwinner,sun8i-a83t", >> NULL, >> }; >> >> diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c >> index 413070d..f216d5d 100644 >> --- a/drivers/clk/sunxi/clk-sunxi.c >> +++ b/drivers/clk/sunxi/clk-sunxi.c >> @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); >> CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); >> CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); >> >> +static void __init sun8ia83t_init_clocks(struct device_node *node) > > Having an underscore between sun8i and a83t would be great :) > >> +{ >> + sunxi_init_clocks(NULL, 0); >> +} >> +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); > > ^ Is this a sun9i or sun8i SoC ? It's sun8i, sorry. > > Is the list of protected clocks always going to be empty, or will you > add some new clocks in that list in the future? Yes, I'll add new clocks as time permits. > > Thanks! > Maxime > > -- > Maxime Ripard, Free Electrons > Embedded Linux, Kernel and Android engineering > http://free-electrons.com Thanks for review. -- To unsubscribe from this list: send the line "unsubscribe linux-gpio" 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/Documentation/devicetree/bindings/arm/sunxi.txt b/Documentation/devicetree/bindings/arm/sunxi.txt index 67da205..cf5ed27 100644 --- a/Documentation/devicetree/bindings/arm/sunxi.txt +++ b/Documentation/devicetree/bindings/arm/sunxi.txt @@ -11,4 +11,5 @@ using one of the following compatible strings: allwinner,sun8i-a23 allwinner,sun8i-a33 allwinner,sun8i-h3 + allwinner,sun8i-a83t allwinner,sun9i-a80 diff --git a/arch/arm/mach-sunxi/sunxi.c b/arch/arm/mach-sunxi/sunxi.c index 65bab28..b04aefa 100644 --- a/arch/arm/mach-sunxi/sunxi.c +++ b/arch/arm/mach-sunxi/sunxi.c @@ -69,6 +69,7 @@ static const char * const sun8i_board_dt_compat[] = { "allwinner,sun8i-a23", "allwinner,sun8i-a33", "allwinner,sun8i-h3", + "allwinner,sun8i-a83t", NULL, }; diff --git a/drivers/clk/sunxi/clk-sunxi.c b/drivers/clk/sunxi/clk-sunxi.c index 413070d..f216d5d 100644 --- a/drivers/clk/sunxi/clk-sunxi.c +++ b/drivers/clk/sunxi/clk-sunxi.c @@ -1212,6 +1212,12 @@ CLK_OF_DECLARE(sun6i_a31s_clk_init, "allwinner,sun6i-a31s", sun6i_init_clocks); CLK_OF_DECLARE(sun8i_a23_clk_init, "allwinner,sun8i-a23", sun6i_init_clocks); CLK_OF_DECLARE(sun8i_a33_clk_init, "allwinner,sun8i-a33", sun6i_init_clocks); +static void __init sun8ia83t_init_clocks(struct device_node *node) +{ + sunxi_init_clocks(NULL, 0); +} +CLK_OF_DECLARE(sun9i_a83t_clk_init, "allwinner,sun8i-a83t", sun8ia83t_init_clocks); + static void __init sun9i_init_clocks(struct device_node *node) { sunxi_init_clocks(NULL, 0);
Allwinner A83T is octa-core cortex-a7 based SoC. It's clock control unit and prcm, pinmux are different from previous sun8i series. Its processor cores are arragned in two clusters 4 cores each, similar to A80. Signed-off-by: Vishnu Patekar <vishnupatekar0510@gmail.com> --- Documentation/devicetree/bindings/arm/sunxi.txt | 1 + arch/arm/mach-sunxi/sunxi.c | 1 + drivers/clk/sunxi/clk-sunxi.c | 6 ++++++ 3 files changed, 8 insertions(+)