From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Chen-Yu Tsai Cc: Rob Herring , Maxime Ripard , Mark Rutland , Mike Turquette , Stephen Boyd , David Airlie , Archit Taneja , Andrzej Hajda , devicetree , linux-arm-kernel , linux-kernel , linux-clk , dri-devel , linux-sunxi , Icenowy Zheng Subject: Re: [PATCH 08/27] clk: sunxi-ng: Add support for H6 DE3 clocks Date: Wed, 12 Sep 2018 16:55:07 +0200 Message-ID: <1702684.jtWEXlagAO@jernej-laptop> In-Reply-To: References: <20180902072643.4917-1-jernej.skrabec@siol.net> <1677668.BePkXd9qNa@jernej-laptop> MIME-Version: 1.0 Content-Type: text/plain; charset="UTF-8" List-ID: Dne sreda, 12. september 2018 ob 14:20:08 CEST je Chen-Yu Tsai napisal(a): > On Wed, Sep 5, 2018 at 1:46 AM Jernej =C5=A0krabec =20 wrote: > > Dne torek, 04. september 2018 ob 11:04:21 CEST je Chen-Yu Tsai napisal(= a): > > > On Sun, Sep 2, 2018 at 3:27 PM Jernej Skrabec > >=20 > > wrote: > > > > Support for mixer0, mixer1, writeback and rotation units is added. > > > >=20 > > > > Signed-off-by: Jernej Skrabec > > > > Signed-off-by: Icenowy Zheng > > > > --- > > > >=20 > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.c | 65 > > > > ++++++++++++++++++++++++++++ > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.h | 1 + > > > > 2 files changed, 66 insertions(+) > > > >=20 > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c index > > > > bae5ee67a797..4535c1c27d27 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > @@ -31,6 +31,8 @@ static SUNXI_CCU_GATE(bus_mixer1_clk, "bus-mixer1= ", > > > > "bus-de",> > > > >=20 > > > > 0x04, BIT(1), 0); > > > > =20 > > > > static SUNXI_CCU_GATE(bus_wb_clk, "bus-wb", "bus-de", > > > > =20 > > > > 0x04, BIT(2), 0); > > > >=20 > > > > +static SUNXI_CCU_GATE(bus_rot_clk, "bus-rot", "bus-de", > > > > + 0x04, BIT(3), 0); > > > >=20 > > > > static SUNXI_CCU_GATE(mixer0_clk, "mixer0", "mixer0-div= ", > > > > =20 > > > > 0x00, BIT(0), CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -38,6 +40,8 @@ static SUNXI_CCU_GATE(mixer1_clk, "mixer1", > > > > "mixer1-div",> > > > >=20 > > > > 0x00, BIT(1), CLK_SET_RATE_PARENT); > > > > =20 > > > > static SUNXI_CCU_GATE(wb_clk, "wb", "wb-div", > > > > =20 > > > > 0x00, BIT(2), CLK_SET_RATE_PARENT); > > > >=20 > > > > +static SUNXI_CCU_GATE(rot_clk, "rot", "rot-div", > > > > + 0x00, BIT(3), CLK_SET_RATE_PARENT); > > > >=20 > > > > static SUNXI_CCU_M(mixer0_div_clk, "mixer0-div", "de", 0x0c, 0, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -45,6 +49,8 @@ static SUNXI_CCU_M(mixer1_div_clk, "mixer1-div", > > > > "de", > > > > 0x0c, 4, 4,> > > > >=20 > > > > CLK_SET_RATE_PARENT); > > > > =20 > > > > static SUNXI_CCU_M(wb_div_clk, "wb-div", "de", 0x0c, 8, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > +static SUNXI_CCU_M(rot_div_clk, "rot-div", "de", 0x0c, 0x0c, 4, > > > > + CLK_SET_RATE_PARENT); > > > >=20 > > > > static SUNXI_CCU_M(mixer0_div_a83_clk, "mixer0-div", "pll-de", 0x0= c, > > > > 0, > > > > 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -53,6 +59,24 @@ static SUNXI_CCU_M(mixer1_div_a83_clk, > > > > "mixer1-div", > > > > "pll-de", 0x0c, 4, 4,> > > > >=20 > > > > static SUNXI_CCU_M(wb_div_a83_clk, "wb-div", "pll-de", 0x0c, 8, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > +static struct ccu_common *sun50i_h6_de3_clks[] =3D { > > > > + &mixer0_clk.common, > > > > + &mixer1_clk.common, > > > > + &wb_clk.common, > > > > + > > > > + &bus_mixer0_clk.common, > > > > + &bus_mixer1_clk.common, > > > > + &bus_wb_clk.common, > > > > + > > > > + &mixer0_div_clk.common, > > > > + &mixer1_div_clk.common, > > > > + &wb_div_clk.common, > > > > + > > > > + &bus_rot_clk.common, > > > > + &rot_clk.common, > > > > + &rot_div_clk.common, > > > > +}; > > > > + > > > >=20 > > > > static struct ccu_common *sun8i_a83t_de2_clks[] =3D { > > > > =20 > > > > &mixer0_clk.common, > > > > &mixer1_clk.common, > > > >=20 > > > > @@ -92,6 +116,26 @@ static struct ccu_common *sun8i_v3s_de2_clks[] = =3D { > > > >=20 > > > > &wb_div_clk.common, > > > > =20 > > > > }; > > > >=20 > > > > +static struct clk_hw_onecell_data sun50i_h6_de3_hw_clks =3D { > > > > + .hws =3D { > > > > + [CLK_MIXER0] =3D &mixer0_clk.common.hw, > > > > + [CLK_MIXER1] =3D &mixer1_clk.common.hw, > > > > + [CLK_WB] =3D &wb_clk.common.hw, > > > > + [CLK_ROT] =3D &rot_clk.common.hw, > > > > + > > > > + [CLK_BUS_MIXER0] =3D &bus_mixer0_clk.common.= hw, > > > > + [CLK_BUS_MIXER1] =3D &bus_mixer1_clk.common.= hw, > > > > + [CLK_BUS_WB] =3D &bus_wb_clk.common.hw, > > > > + [CLK_BUS_ROT] =3D &bus_rot_clk.common.hw, > > > > + > > > > + [CLK_MIXER0_DIV] =3D &mixer0_div_clk.common.= hw, > > > > + [CLK_MIXER1_DIV] =3D &mixer1_div_clk.common.= hw, > > > > + [CLK_WB_DIV] =3D &wb_div_clk.common.hw, > > > > + [CLK_ROT_DIV] =3D &rot_div_clk.common.hw, > > > > + }, > > > > + .num =3D 12, > > >=20 > > > It's best not to openly code these. It is error prone, like having > > > an index beyond .num, which then never gets registered. > > >=20 > > > Instead, please update CLK_NUMBERS and use that instead. > > > sunxi_ccu_probe() > > > can handle holes in .hws. > >=20 > > I'm not sure this will work. All newly introduced indices are at the en= d, > > so other arrays will still have same length (hole at the end). You will > > just claim that arrays are larger than they really are, which means bad > > things. > >=20 > > But I take any other suggestion. I really can't think of better solutio= n. >=20 > Then maybe have macros for both cases instead? > CLK_NUMBER_WITH_ROT / CLK_NUMBER_WITHOUT_ROT? That sounds reasonable. Do you want separate patch which renames original=20 macro CLK_NUMBER to CLK_NUMBER_WITHOUT_ROT? Best regards, Jernej >=20 > ChenYu >=20 > > Best regards, > > Jernej > >=20 > > > On the other hand, it can't handle holes in the ccu_reset_map. Hope we > > > never have to deal with such an instance. > > >=20 > > > ChenYu > > >=20 > > > > +}; > > > > + > > > >=20 > > > > static struct clk_hw_onecell_data sun8i_a83t_de2_hw_clks =3D { > > > > =20 > > > > .hws =3D { > > > > =20 > > > > [CLK_MIXER0] =3D &mixer0_clk.common.hw, > > > >=20 > > > > @@ -156,6 +200,13 @@ static struct ccu_reset_map > > > > sun50i_a64_de2_resets[] =3D > > > > {> > > > >=20 > > > > [RST_WB] =3D { 0x08, BIT(2) }, > > > > =20 > > > > }; > > > >=20 > > > > +static struct ccu_reset_map sun50i_h6_de3_resets[] =3D { > > > > + [RST_MIXER0] =3D { 0x08, BIT(0) }, > > > > + [RST_MIXER1] =3D { 0x08, BIT(1) }, > > > > + [RST_WB] =3D { 0x08, BIT(2) }, > > > > + [RST_ROT] =3D { 0x08, BIT(3) }, > > > > +}; > > > > + > > > >=20 > > > > static const struct sunxi_ccu_desc sun8i_a83t_de2_clk_desc =3D { > > > > =20 > > > > .ccu_clks =3D sun8i_a83t_de2_clks, > > > > .num_ccu_clks =3D ARRAY_SIZE(sun8i_a83t_de2_clks), > > > >=20 > > > > @@ -186,6 +237,16 @@ static const struct sunxi_ccu_desc > > > > sun50i_a64_de2_clk_desc =3D {> > > > >=20 > > > > .num_resets =3D ARRAY_SIZE(sun50i_a64_de2_resets), > > > > =20 > > > > }; > > > >=20 > > > > +static const struct sunxi_ccu_desc sun50i_h6_de3_clk_desc =3D { > > > > + .ccu_clks =3D sun50i_h6_de3_clks, > > > > + .num_ccu_clks =3D ARRAY_SIZE(sun50i_h6_de3_clks), > > > > + > > > > + .hw_clks =3D &sun50i_h6_de3_hw_clks, > > > > + > > > > + .resets =3D sun50i_h6_de3_resets, > > > > + .num_resets =3D ARRAY_SIZE(sun50i_h6_de3_resets), > > > > +}; > > > > + > > > >=20 > > > > static const struct sunxi_ccu_desc sun8i_v3s_de2_clk_desc =3D { > > > > =20 > > > > .ccu_clks =3D sun8i_v3s_de2_clks, > > > > .num_ccu_clks =3D ARRAY_SIZE(sun8i_v3s_de2_clks), > > > >=20 > > > > @@ -296,6 +357,10 @@ static const struct of_device_id > > > > sunxi_de2_clk_ids[] > > > > =3D {> > > > >=20 > > > > .compatible =3D "allwinner,sun50i-h5-de2-clk", > > > > .data =3D &sun50i_a64_de2_clk_desc, > > > > =20 > > > > }, > > > >=20 > > > > + { > > > > + .compatible =3D "allwinner,sun50i-h6-de3-clk", > > > > + .data =3D &sun50i_h6_de3_clk_desc, > > > > + }, > > > >=20 > > > > { } > > > > =20 > > > > }; > > > >=20 > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h index > > > > 530c006e0ae9..27bd88539f42 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > @@ -22,6 +22,7 @@ > > > >=20 > > > > #define CLK_MIXER0_DIV 3 > > > > #define CLK_MIXER1_DIV 4 > > > > #define CLK_WB_DIV 5 > > > >=20 > > > > +#define CLK_ROT_DIV 11 > > > >=20 > > > > #define CLK_NUMBER (CLK_WB + 1) > > > >=20 > > > > -- > > > > 2.18.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: jernej.skrabec@siol.net (Jernej =?utf-8?B?xaBrcmFiZWM=?=) Date: Wed, 12 Sep 2018 16:55:07 +0200 Subject: [PATCH 08/27] clk: sunxi-ng: Add support for H6 DE3 clocks In-Reply-To: References: <20180902072643.4917-1-jernej.skrabec@siol.net> <1677668.BePkXd9qNa@jernej-laptop> Message-ID: <1702684.jtWEXlagAO@jernej-laptop> To: linux-arm-kernel@lists.infradead.org List-Id: linux-arm-kernel.lists.infradead.org Dne sreda, 12. september 2018 ob 14:20:08 CEST je Chen-Yu Tsai napisal(a): > On Wed, Sep 5, 2018 at 1:46 AM Jernej ?krabec wrote: > > Dne torek, 04. september 2018 ob 11:04:21 CEST je Chen-Yu Tsai napisal(a): > > > On Sun, Sep 2, 2018 at 3:27 PM Jernej Skrabec > > > > wrote: > > > > Support for mixer0, mixer1, writeback and rotation units is added. > > > > > > > > Signed-off-by: Jernej Skrabec > > > > Signed-off-by: Icenowy Zheng > > > > --- > > > > > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.c | 65 > > > > ++++++++++++++++++++++++++++ > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.h | 1 + > > > > 2 files changed, 66 insertions(+) > > > > > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c index > > > > bae5ee67a797..4535c1c27d27 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > @@ -31,6 +31,8 @@ static SUNXI_CCU_GATE(bus_mixer1_clk, "bus-mixer1", > > > > "bus-de",> > > > > > > > > 0x04, BIT(1), 0); > > > > > > > > static SUNXI_CCU_GATE(bus_wb_clk, "bus-wb", "bus-de", > > > > > > > > 0x04, BIT(2), 0); > > > > > > > > +static SUNXI_CCU_GATE(bus_rot_clk, "bus-rot", "bus-de", > > > > + 0x04, BIT(3), 0); > > > > > > > > static SUNXI_CCU_GATE(mixer0_clk, "mixer0", "mixer0-div", > > > > > > > > 0x00, BIT(0), CLK_SET_RATE_PARENT); > > > > > > > > @@ -38,6 +40,8 @@ static SUNXI_CCU_GATE(mixer1_clk, "mixer1", > > > > "mixer1-div",> > > > > > > > > 0x00, BIT(1), CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_GATE(wb_clk, "wb", "wb-div", > > > > > > > > 0x00, BIT(2), CLK_SET_RATE_PARENT); > > > > > > > > +static SUNXI_CCU_GATE(rot_clk, "rot", "rot-div", > > > > + 0x00, BIT(3), CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(mixer0_div_clk, "mixer0-div", "de", 0x0c, 0, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > @@ -45,6 +49,8 @@ static SUNXI_CCU_M(mixer1_div_clk, "mixer1-div", > > > > "de", > > > > 0x0c, 4, 4,> > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(wb_div_clk, "wb-div", "de", 0x0c, 8, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > +static SUNXI_CCU_M(rot_div_clk, "rot-div", "de", 0x0c, 0x0c, 4, > > > > + CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(mixer0_div_a83_clk, "mixer0-div", "pll-de", 0x0c, > > > > 0, > > > > 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > @@ -53,6 +59,24 @@ static SUNXI_CCU_M(mixer1_div_a83_clk, > > > > "mixer1-div", > > > > "pll-de", 0x0c, 4, 4,> > > > > > > > > static SUNXI_CCU_M(wb_div_a83_clk, "wb-div", "pll-de", 0x0c, 8, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > +static struct ccu_common *sun50i_h6_de3_clks[] = { > > > > + &mixer0_clk.common, > > > > + &mixer1_clk.common, > > > > + &wb_clk.common, > > > > + > > > > + &bus_mixer0_clk.common, > > > > + &bus_mixer1_clk.common, > > > > + &bus_wb_clk.common, > > > > + > > > > + &mixer0_div_clk.common, > > > > + &mixer1_div_clk.common, > > > > + &wb_div_clk.common, > > > > + > > > > + &bus_rot_clk.common, > > > > + &rot_clk.common, > > > > + &rot_div_clk.common, > > > > +}; > > > > + > > > > > > > > static struct ccu_common *sun8i_a83t_de2_clks[] = { > > > > > > > > &mixer0_clk.common, > > > > &mixer1_clk.common, > > > > > > > > @@ -92,6 +116,26 @@ static struct ccu_common *sun8i_v3s_de2_clks[] = { > > > > > > > > &wb_div_clk.common, > > > > > > > > }; > > > > > > > > +static struct clk_hw_onecell_data sun50i_h6_de3_hw_clks = { > > > > + .hws = { > > > > + [CLK_MIXER0] = &mixer0_clk.common.hw, > > > > + [CLK_MIXER1] = &mixer1_clk.common.hw, > > > > + [CLK_WB] = &wb_clk.common.hw, > > > > + [CLK_ROT] = &rot_clk.common.hw, > > > > + > > > > + [CLK_BUS_MIXER0] = &bus_mixer0_clk.common.hw, > > > > + [CLK_BUS_MIXER1] = &bus_mixer1_clk.common.hw, > > > > + [CLK_BUS_WB] = &bus_wb_clk.common.hw, > > > > + [CLK_BUS_ROT] = &bus_rot_clk.common.hw, > > > > + > > > > + [CLK_MIXER0_DIV] = &mixer0_div_clk.common.hw, > > > > + [CLK_MIXER1_DIV] = &mixer1_div_clk.common.hw, > > > > + [CLK_WB_DIV] = &wb_div_clk.common.hw, > > > > + [CLK_ROT_DIV] = &rot_div_clk.common.hw, > > > > + }, > > > > + .num = 12, > > > > > > It's best not to openly code these. It is error prone, like having > > > an index beyond .num, which then never gets registered. > > > > > > Instead, please update CLK_NUMBERS and use that instead. > > > sunxi_ccu_probe() > > > can handle holes in .hws. > > > > I'm not sure this will work. All newly introduced indices are at the end, > > so other arrays will still have same length (hole at the end). You will > > just claim that arrays are larger than they really are, which means bad > > things. > > > > But I take any other suggestion. I really can't think of better solution. > > Then maybe have macros for both cases instead? > CLK_NUMBER_WITH_ROT / CLK_NUMBER_WITHOUT_ROT? That sounds reasonable. Do you want separate patch which renames original macro CLK_NUMBER to CLK_NUMBER_WITHOUT_ROT? Best regards, Jernej > > ChenYu > > > Best regards, > > Jernej > > > > > On the other hand, it can't handle holes in the ccu_reset_map. Hope we > > > never have to deal with such an instance. > > > > > > ChenYu > > > > > > > +}; > > > > + > > > > > > > > static struct clk_hw_onecell_data sun8i_a83t_de2_hw_clks = { > > > > > > > > .hws = { > > > > > > > > [CLK_MIXER0] = &mixer0_clk.common.hw, > > > > > > > > @@ -156,6 +200,13 @@ static struct ccu_reset_map > > > > sun50i_a64_de2_resets[] = > > > > {> > > > > > > > > [RST_WB] = { 0x08, BIT(2) }, > > > > > > > > }; > > > > > > > > +static struct ccu_reset_map sun50i_h6_de3_resets[] = { > > > > + [RST_MIXER0] = { 0x08, BIT(0) }, > > > > + [RST_MIXER1] = { 0x08, BIT(1) }, > > > > + [RST_WB] = { 0x08, BIT(2) }, > > > > + [RST_ROT] = { 0x08, BIT(3) }, > > > > +}; > > > > + > > > > > > > > static const struct sunxi_ccu_desc sun8i_a83t_de2_clk_desc = { > > > > > > > > .ccu_clks = sun8i_a83t_de2_clks, > > > > .num_ccu_clks = ARRAY_SIZE(sun8i_a83t_de2_clks), > > > > > > > > @@ -186,6 +237,16 @@ static const struct sunxi_ccu_desc > > > > sun50i_a64_de2_clk_desc = {> > > > > > > > > .num_resets = ARRAY_SIZE(sun50i_a64_de2_resets), > > > > > > > > }; > > > > > > > > +static const struct sunxi_ccu_desc sun50i_h6_de3_clk_desc = { > > > > + .ccu_clks = sun50i_h6_de3_clks, > > > > + .num_ccu_clks = ARRAY_SIZE(sun50i_h6_de3_clks), > > > > + > > > > + .hw_clks = &sun50i_h6_de3_hw_clks, > > > > + > > > > + .resets = sun50i_h6_de3_resets, > > > > + .num_resets = ARRAY_SIZE(sun50i_h6_de3_resets), > > > > +}; > > > > + > > > > > > > > static const struct sunxi_ccu_desc sun8i_v3s_de2_clk_desc = { > > > > > > > > .ccu_clks = sun8i_v3s_de2_clks, > > > > .num_ccu_clks = ARRAY_SIZE(sun8i_v3s_de2_clks), > > > > > > > > @@ -296,6 +357,10 @@ static const struct of_device_id > > > > sunxi_de2_clk_ids[] > > > > = {> > > > > > > > > .compatible = "allwinner,sun50i-h5-de2-clk", > > > > .data = &sun50i_a64_de2_clk_desc, > > > > > > > > }, > > > > > > > > + { > > > > + .compatible = "allwinner,sun50i-h6-de3-clk", > > > > + .data = &sun50i_h6_de3_clk_desc, > > > > + }, > > > > > > > > { } > > > > > > > > }; > > > > > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h index > > > > 530c006e0ae9..27bd88539f42 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > @@ -22,6 +22,7 @@ > > > > > > > > #define CLK_MIXER0_DIV 3 > > > > #define CLK_MIXER1_DIV 4 > > > > #define CLK_WB_DIV 5 > > > > > > > > +#define CLK_ROT_DIV 11 > > > > > > > > #define CLK_NUMBER (CLK_WB + 1) > > > > > > > > -- > > > > 2.18.0 From mboxrd@z Thu Jan 1 00:00:00 1970 From: Jernej =?utf-8?B?xaBrcmFiZWM=?= Subject: Re: [PATCH 08/27] clk: sunxi-ng: Add support for H6 DE3 clocks Date: Wed, 12 Sep 2018 16:55:07 +0200 Message-ID: <1702684.jtWEXlagAO@jernej-laptop> References: <20180902072643.4917-1-jernej.skrabec@siol.net> <1677668.BePkXd9qNa@jernej-laptop> Reply-To: jernej.skrabec-gGgVlfcn5nU@public.gmane.org Mime-Version: 1.0 Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable Return-path: Sender: linux-sunxi-/JYPxA39Uh5TLH3MbocFFw@public.gmane.org In-Reply-To: List-Post: , List-Help: , List-Archive: , List-Unsubscribe: , To: Chen-Yu Tsai Cc: Rob Herring , Maxime Ripard , Mark Rutland , Mike Turquette , Stephen Boyd , David Airlie , Archit Taneja , Andrzej Hajda , devicetree , linux-arm-kernel , linux-kernel , linux-clk , dri-devel , linux-sunxi , Icenowy Zheng List-Id: devicetree@vger.kernel.org Dne sreda, 12. september 2018 ob 14:20:08 CEST je Chen-Yu Tsai napisal(a): > On Wed, Sep 5, 2018 at 1:46 AM Jernej =C5=A0krabec =20 wrote: > > Dne torek, 04. september 2018 ob 11:04:21 CEST je Chen-Yu Tsai napisal(= a): > > > On Sun, Sep 2, 2018 at 3:27 PM Jernej Skrabec > >=20 > > wrote: > > > > Support for mixer0, mixer1, writeback and rotation units is added. > > > >=20 > > > > Signed-off-by: Jernej Skrabec > > > > Signed-off-by: Icenowy Zheng > > > > --- > > > >=20 > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.c | 65 > > > > ++++++++++++++++++++++++++++ > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.h | 1 + > > > > 2 files changed, 66 insertions(+) > > > >=20 > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c index > > > > bae5ee67a797..4535c1c27d27 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > @@ -31,6 +31,8 @@ static SUNXI_CCU_GATE(bus_mixer1_clk, "bus-mixer1= ", > > > > "bus-de",> > > > >=20 > > > > 0x04, BIT(1), 0); > > > > =20 > > > > static SUNXI_CCU_GATE(bus_wb_clk, "bus-wb", "bus-de", > > > > =20 > > > > 0x04, BIT(2), 0); > > > >=20 > > > > +static SUNXI_CCU_GATE(bus_rot_clk, "bus-rot", "bus-de", > > > > + 0x04, BIT(3), 0); > > > >=20 > > > > static SUNXI_CCU_GATE(mixer0_clk, "mixer0", "mixer0-div= ", > > > > =20 > > > > 0x00, BIT(0), CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -38,6 +40,8 @@ static SUNXI_CCU_GATE(mixer1_clk, "mixer1", > > > > "mixer1-div",> > > > >=20 > > > > 0x00, BIT(1), CLK_SET_RATE_PARENT); > > > > =20 > > > > static SUNXI_CCU_GATE(wb_clk, "wb", "wb-div", > > > > =20 > > > > 0x00, BIT(2), CLK_SET_RATE_PARENT); > > > >=20 > > > > +static SUNXI_CCU_GATE(rot_clk, "rot", "rot-div", > > > > + 0x00, BIT(3), CLK_SET_RATE_PARENT); > > > >=20 > > > > static SUNXI_CCU_M(mixer0_div_clk, "mixer0-div", "de", 0x0c, 0, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -45,6 +49,8 @@ static SUNXI_CCU_M(mixer1_div_clk, "mixer1-div", > > > > "de", > > > > 0x0c, 4, 4,> > > > >=20 > > > > CLK_SET_RATE_PARENT); > > > > =20 > > > > static SUNXI_CCU_M(wb_div_clk, "wb-div", "de", 0x0c, 8, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > +static SUNXI_CCU_M(rot_div_clk, "rot-div", "de", 0x0c, 0x0c, 4, > > > > + CLK_SET_RATE_PARENT); > > > >=20 > > > > static SUNXI_CCU_M(mixer0_div_a83_clk, "mixer0-div", "pll-de", 0x0= c, > > > > 0, > > > > 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > @@ -53,6 +59,24 @@ static SUNXI_CCU_M(mixer1_div_a83_clk, > > > > "mixer1-div", > > > > "pll-de", 0x0c, 4, 4,> > > > >=20 > > > > static SUNXI_CCU_M(wb_div_a83_clk, "wb-div", "pll-de", 0x0c, 8, 4, > > > > =20 > > > > CLK_SET_RATE_PARENT); > > > >=20 > > > > +static struct ccu_common *sun50i_h6_de3_clks[] =3D { > > > > + &mixer0_clk.common, > > > > + &mixer1_clk.common, > > > > + &wb_clk.common, > > > > + > > > > + &bus_mixer0_clk.common, > > > > + &bus_mixer1_clk.common, > > > > + &bus_wb_clk.common, > > > > + > > > > + &mixer0_div_clk.common, > > > > + &mixer1_div_clk.common, > > > > + &wb_div_clk.common, > > > > + > > > > + &bus_rot_clk.common, > > > > + &rot_clk.common, > > > > + &rot_div_clk.common, > > > > +}; > > > > + > > > >=20 > > > > static struct ccu_common *sun8i_a83t_de2_clks[] =3D { > > > > =20 > > > > &mixer0_clk.common, > > > > &mixer1_clk.common, > > > >=20 > > > > @@ -92,6 +116,26 @@ static struct ccu_common *sun8i_v3s_de2_clks[] = =3D { > > > >=20 > > > > &wb_div_clk.common, > > > > =20 > > > > }; > > > >=20 > > > > +static struct clk_hw_onecell_data sun50i_h6_de3_hw_clks =3D { > > > > + .hws =3D { > > > > + [CLK_MIXER0] =3D &mixer0_clk.common.hw, > > > > + [CLK_MIXER1] =3D &mixer1_clk.common.hw, > > > > + [CLK_WB] =3D &wb_clk.common.hw, > > > > + [CLK_ROT] =3D &rot_clk.common.hw, > > > > + > > > > + [CLK_BUS_MIXER0] =3D &bus_mixer0_clk.common.= hw, > > > > + [CLK_BUS_MIXER1] =3D &bus_mixer1_clk.common.= hw, > > > > + [CLK_BUS_WB] =3D &bus_wb_clk.common.hw, > > > > + [CLK_BUS_ROT] =3D &bus_rot_clk.common.hw, > > > > + > > > > + [CLK_MIXER0_DIV] =3D &mixer0_div_clk.common.= hw, > > > > + [CLK_MIXER1_DIV] =3D &mixer1_div_clk.common.= hw, > > > > + [CLK_WB_DIV] =3D &wb_div_clk.common.hw, > > > > + [CLK_ROT_DIV] =3D &rot_div_clk.common.hw, > > > > + }, > > > > + .num =3D 12, > > >=20 > > > It's best not to openly code these. It is error prone, like having > > > an index beyond .num, which then never gets registered. > > >=20 > > > Instead, please update CLK_NUMBERS and use that instead. > > > sunxi_ccu_probe() > > > can handle holes in .hws. > >=20 > > I'm not sure this will work. All newly introduced indices are at the en= d, > > so other arrays will still have same length (hole at the end). You will > > just claim that arrays are larger than they really are, which means bad > > things. > >=20 > > But I take any other suggestion. I really can't think of better solutio= n. >=20 > Then maybe have macros for both cases instead? > CLK_NUMBER_WITH_ROT / CLK_NUMBER_WITHOUT_ROT? That sounds reasonable. Do you want separate patch which renames original= =20 macro CLK_NUMBER to CLK_NUMBER_WITHOUT_ROT? Best regards, Jernej >=20 > ChenYu >=20 > > Best regards, > > Jernej > >=20 > > > On the other hand, it can't handle holes in the ccu_reset_map. Hope w= e > > > never have to deal with such an instance. > > >=20 > > > ChenYu > > >=20 > > > > +}; > > > > + > > > >=20 > > > > static struct clk_hw_onecell_data sun8i_a83t_de2_hw_clks =3D { > > > > =20 > > > > .hws =3D { > > > > =20 > > > > [CLK_MIXER0] =3D &mixer0_clk.common.hw, > > > >=20 > > > > @@ -156,6 +200,13 @@ static struct ccu_reset_map > > > > sun50i_a64_de2_resets[] =3D > > > > {> > > > >=20 > > > > [RST_WB] =3D { 0x08, BIT(2) }, > > > > =20 > > > > }; > > > >=20 > > > > +static struct ccu_reset_map sun50i_h6_de3_resets[] =3D { > > > > + [RST_MIXER0] =3D { 0x08, BIT(0) }, > > > > + [RST_MIXER1] =3D { 0x08, BIT(1) }, > > > > + [RST_WB] =3D { 0x08, BIT(2) }, > > > > + [RST_ROT] =3D { 0x08, BIT(3) }, > > > > +}; > > > > + > > > >=20 > > > > static const struct sunxi_ccu_desc sun8i_a83t_de2_clk_desc =3D { > > > > =20 > > > > .ccu_clks =3D sun8i_a83t_de2_clks, > > > > .num_ccu_clks =3D ARRAY_SIZE(sun8i_a83t_de2_clks), > > > >=20 > > > > @@ -186,6 +237,16 @@ static const struct sunxi_ccu_desc > > > > sun50i_a64_de2_clk_desc =3D {> > > > >=20 > > > > .num_resets =3D ARRAY_SIZE(sun50i_a64_de2_resets), > > > > =20 > > > > }; > > > >=20 > > > > +static const struct sunxi_ccu_desc sun50i_h6_de3_clk_desc =3D { > > > > + .ccu_clks =3D sun50i_h6_de3_clks, > > > > + .num_ccu_clks =3D ARRAY_SIZE(sun50i_h6_de3_clks), > > > > + > > > > + .hw_clks =3D &sun50i_h6_de3_hw_clks, > > > > + > > > > + .resets =3D sun50i_h6_de3_resets, > > > > + .num_resets =3D ARRAY_SIZE(sun50i_h6_de3_resets), > > > > +}; > > > > + > > > >=20 > > > > static const struct sunxi_ccu_desc sun8i_v3s_de2_clk_desc =3D { > > > > =20 > > > > .ccu_clks =3D sun8i_v3s_de2_clks, > > > > .num_ccu_clks =3D ARRAY_SIZE(sun8i_v3s_de2_clks), > > > >=20 > > > > @@ -296,6 +357,10 @@ static const struct of_device_id > > > > sunxi_de2_clk_ids[] > > > > =3D {> > > > >=20 > > > > .compatible =3D "allwinner,sun50i-h5-de2-clk", > > > > .data =3D &sun50i_a64_de2_clk_desc, > > > > =20 > > > > }, > > > >=20 > > > > + { > > > > + .compatible =3D "allwinner,sun50i-h6-de3-clk", > > > > + .data =3D &sun50i_h6_de3_clk_desc, > > > > + }, > > > >=20 > > > > { } > > > > =20 > > > > }; > > > >=20 > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h index > > > > 530c006e0ae9..27bd88539f42 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > @@ -22,6 +22,7 @@ > > > >=20 > > > > #define CLK_MIXER0_DIV 3 > > > > #define CLK_MIXER1_DIV 4 > > > > #define CLK_WB_DIV 5 > > > >=20 > > > > +#define CLK_ROT_DIV 11 > > > >=20 > > > > #define CLK_NUMBER (CLK_WB + 1) > > > >=20 > > > > -- > > > > 2.18.0 --=20 You received this message because you are subscribed to the Google Groups "= linux-sunxi" group. To unsubscribe from this group and stop receiving emails from it, send an e= mail to linux-sunxi+unsubscribe-/JYPxA39Uh5TLH3MbocFF+G/Ez6ZCGd0@public.gmane.org For more options, visit https://groups.google.com/d/optout. From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org X-Spam-Level: X-Spam-Status: No, score=-0.6 required=3.0 tests=FROM_EXCESS_BASE64, HEADER_FROM_DIFFERENT_DOMAINS,MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id 859C0C46469 for ; Wed, 12 Sep 2018 14:55:20 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 2A48620839 for ; Wed, 12 Sep 2018 14:55:20 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 2A48620839 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=siol.net Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1728264AbeILUAK convert rfc822-to-8bit (ORCPT ); Wed, 12 Sep 2018 16:00:10 -0400 Received: from mailoutvs21.siol.net ([185.57.226.212]:59920 "EHLO mail.siol.net" rhost-flags-OK-OK-OK-FAIL) by vger.kernel.org with ESMTP id S1727968AbeILUAK (ORCPT ); Wed, 12 Sep 2018 16:00:10 -0400 Received: from localhost (localhost [127.0.0.1]) by mail.siol.net (Zimbra) with ESMTP id 1A385523146; Wed, 12 Sep 2018 16:55:15 +0200 (CEST) X-Virus-Scanned: amavisd-new at psrvmta12.zcs-production.pri Received: from mail.siol.net ([127.0.0.1]) by localhost (psrvmta12.zcs-production.pri [127.0.0.1]) (amavisd-new, port 10032) with ESMTP id C2VgctkCNgiU; Wed, 12 Sep 2018 16:55:13 +0200 (CEST) Received: from mail.siol.net (localhost [127.0.0.1]) by mail.siol.net (Zimbra) with ESMTPS id C7502523132; Wed, 12 Sep 2018 16:55:13 +0200 (CEST) Received: from jernej-laptop.localnet (cpe1-8-82.cable.triera.net [213.161.8.82]) (Authenticated sender: 031275009) by mail.siol.net (Zimbra) with ESMTPA id 05A8C523146; Wed, 12 Sep 2018 16:55:08 +0200 (CEST) From: Jernej =?utf-8?B?xaBrcmFiZWM=?= To: Chen-Yu Tsai Cc: Rob Herring , Maxime Ripard , Mark Rutland , Mike Turquette , Stephen Boyd , David Airlie , Archit Taneja , Andrzej Hajda , devicetree , linux-arm-kernel , linux-kernel , linux-clk , dri-devel , linux-sunxi , Icenowy Zheng Subject: Re: [PATCH 08/27] clk: sunxi-ng: Add support for H6 DE3 clocks Date: Wed, 12 Sep 2018 16:55:07 +0200 Message-ID: <1702684.jtWEXlagAO@jernej-laptop> In-Reply-To: References: <20180902072643.4917-1-jernej.skrabec@siol.net> <1677668.BePkXd9qNa@jernej-laptop> MIME-Version: 1.0 Content-Transfer-Encoding: 8BIT Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Dne sreda, 12. september 2018 ob 14:20:08 CEST je Chen-Yu Tsai napisal(a): > On Wed, Sep 5, 2018 at 1:46 AM Jernej Škrabec wrote: > > Dne torek, 04. september 2018 ob 11:04:21 CEST je Chen-Yu Tsai napisal(a): > > > On Sun, Sep 2, 2018 at 3:27 PM Jernej Skrabec > > > > wrote: > > > > Support for mixer0, mixer1, writeback and rotation units is added. > > > > > > > > Signed-off-by: Jernej Skrabec > > > > Signed-off-by: Icenowy Zheng > > > > --- > > > > > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.c | 65 > > > > ++++++++++++++++++++++++++++ > > > > drivers/clk/sunxi-ng/ccu-sun8i-de2.h | 1 + > > > > 2 files changed, 66 insertions(+) > > > > > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c index > > > > bae5ee67a797..4535c1c27d27 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.c > > > > @@ -31,6 +31,8 @@ static SUNXI_CCU_GATE(bus_mixer1_clk, "bus-mixer1", > > > > "bus-de",> > > > > > > > > 0x04, BIT(1), 0); > > > > > > > > static SUNXI_CCU_GATE(bus_wb_clk, "bus-wb", "bus-de", > > > > > > > > 0x04, BIT(2), 0); > > > > > > > > +static SUNXI_CCU_GATE(bus_rot_clk, "bus-rot", "bus-de", > > > > + 0x04, BIT(3), 0); > > > > > > > > static SUNXI_CCU_GATE(mixer0_clk, "mixer0", "mixer0-div", > > > > > > > > 0x00, BIT(0), CLK_SET_RATE_PARENT); > > > > > > > > @@ -38,6 +40,8 @@ static SUNXI_CCU_GATE(mixer1_clk, "mixer1", > > > > "mixer1-div",> > > > > > > > > 0x00, BIT(1), CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_GATE(wb_clk, "wb", "wb-div", > > > > > > > > 0x00, BIT(2), CLK_SET_RATE_PARENT); > > > > > > > > +static SUNXI_CCU_GATE(rot_clk, "rot", "rot-div", > > > > + 0x00, BIT(3), CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(mixer0_div_clk, "mixer0-div", "de", 0x0c, 0, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > @@ -45,6 +49,8 @@ static SUNXI_CCU_M(mixer1_div_clk, "mixer1-div", > > > > "de", > > > > 0x0c, 4, 4,> > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(wb_div_clk, "wb-div", "de", 0x0c, 8, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > +static SUNXI_CCU_M(rot_div_clk, "rot-div", "de", 0x0c, 0x0c, 4, > > > > + CLK_SET_RATE_PARENT); > > > > > > > > static SUNXI_CCU_M(mixer0_div_a83_clk, "mixer0-div", "pll-de", 0x0c, > > > > 0, > > > > 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > @@ -53,6 +59,24 @@ static SUNXI_CCU_M(mixer1_div_a83_clk, > > > > "mixer1-div", > > > > "pll-de", 0x0c, 4, 4,> > > > > > > > > static SUNXI_CCU_M(wb_div_a83_clk, "wb-div", "pll-de", 0x0c, 8, 4, > > > > > > > > CLK_SET_RATE_PARENT); > > > > > > > > +static struct ccu_common *sun50i_h6_de3_clks[] = { > > > > + &mixer0_clk.common, > > > > + &mixer1_clk.common, > > > > + &wb_clk.common, > > > > + > > > > + &bus_mixer0_clk.common, > > > > + &bus_mixer1_clk.common, > > > > + &bus_wb_clk.common, > > > > + > > > > + &mixer0_div_clk.common, > > > > + &mixer1_div_clk.common, > > > > + &wb_div_clk.common, > > > > + > > > > + &bus_rot_clk.common, > > > > + &rot_clk.common, > > > > + &rot_div_clk.common, > > > > +}; > > > > + > > > > > > > > static struct ccu_common *sun8i_a83t_de2_clks[] = { > > > > > > > > &mixer0_clk.common, > > > > &mixer1_clk.common, > > > > > > > > @@ -92,6 +116,26 @@ static struct ccu_common *sun8i_v3s_de2_clks[] = { > > > > > > > > &wb_div_clk.common, > > > > > > > > }; > > > > > > > > +static struct clk_hw_onecell_data sun50i_h6_de3_hw_clks = { > > > > + .hws = { > > > > + [CLK_MIXER0] = &mixer0_clk.common.hw, > > > > + [CLK_MIXER1] = &mixer1_clk.common.hw, > > > > + [CLK_WB] = &wb_clk.common.hw, > > > > + [CLK_ROT] = &rot_clk.common.hw, > > > > + > > > > + [CLK_BUS_MIXER0] = &bus_mixer0_clk.common.hw, > > > > + [CLK_BUS_MIXER1] = &bus_mixer1_clk.common.hw, > > > > + [CLK_BUS_WB] = &bus_wb_clk.common.hw, > > > > + [CLK_BUS_ROT] = &bus_rot_clk.common.hw, > > > > + > > > > + [CLK_MIXER0_DIV] = &mixer0_div_clk.common.hw, > > > > + [CLK_MIXER1_DIV] = &mixer1_div_clk.common.hw, > > > > + [CLK_WB_DIV] = &wb_div_clk.common.hw, > > > > + [CLK_ROT_DIV] = &rot_div_clk.common.hw, > > > > + }, > > > > + .num = 12, > > > > > > It's best not to openly code these. It is error prone, like having > > > an index beyond .num, which then never gets registered. > > > > > > Instead, please update CLK_NUMBERS and use that instead. > > > sunxi_ccu_probe() > > > can handle holes in .hws. > > > > I'm not sure this will work. All newly introduced indices are at the end, > > so other arrays will still have same length (hole at the end). You will > > just claim that arrays are larger than they really are, which means bad > > things. > > > > But I take any other suggestion. I really can't think of better solution. > > Then maybe have macros for both cases instead? > CLK_NUMBER_WITH_ROT / CLK_NUMBER_WITHOUT_ROT? That sounds reasonable. Do you want separate patch which renames original macro CLK_NUMBER to CLK_NUMBER_WITHOUT_ROT? Best regards, Jernej > > ChenYu > > > Best regards, > > Jernej > > > > > On the other hand, it can't handle holes in the ccu_reset_map. Hope we > > > never have to deal with such an instance. > > > > > > ChenYu > > > > > > > +}; > > > > + > > > > > > > > static struct clk_hw_onecell_data sun8i_a83t_de2_hw_clks = { > > > > > > > > .hws = { > > > > > > > > [CLK_MIXER0] = &mixer0_clk.common.hw, > > > > > > > > @@ -156,6 +200,13 @@ static struct ccu_reset_map > > > > sun50i_a64_de2_resets[] = > > > > {> > > > > > > > > [RST_WB] = { 0x08, BIT(2) }, > > > > > > > > }; > > > > > > > > +static struct ccu_reset_map sun50i_h6_de3_resets[] = { > > > > + [RST_MIXER0] = { 0x08, BIT(0) }, > > > > + [RST_MIXER1] = { 0x08, BIT(1) }, > > > > + [RST_WB] = { 0x08, BIT(2) }, > > > > + [RST_ROT] = { 0x08, BIT(3) }, > > > > +}; > > > > + > > > > > > > > static const struct sunxi_ccu_desc sun8i_a83t_de2_clk_desc = { > > > > > > > > .ccu_clks = sun8i_a83t_de2_clks, > > > > .num_ccu_clks = ARRAY_SIZE(sun8i_a83t_de2_clks), > > > > > > > > @@ -186,6 +237,16 @@ static const struct sunxi_ccu_desc > > > > sun50i_a64_de2_clk_desc = {> > > > > > > > > .num_resets = ARRAY_SIZE(sun50i_a64_de2_resets), > > > > > > > > }; > > > > > > > > +static const struct sunxi_ccu_desc sun50i_h6_de3_clk_desc = { > > > > + .ccu_clks = sun50i_h6_de3_clks, > > > > + .num_ccu_clks = ARRAY_SIZE(sun50i_h6_de3_clks), > > > > + > > > > + .hw_clks = &sun50i_h6_de3_hw_clks, > > > > + > > > > + .resets = sun50i_h6_de3_resets, > > > > + .num_resets = ARRAY_SIZE(sun50i_h6_de3_resets), > > > > +}; > > > > + > > > > > > > > static const struct sunxi_ccu_desc sun8i_v3s_de2_clk_desc = { > > > > > > > > .ccu_clks = sun8i_v3s_de2_clks, > > > > .num_ccu_clks = ARRAY_SIZE(sun8i_v3s_de2_clks), > > > > > > > > @@ -296,6 +357,10 @@ static const struct of_device_id > > > > sunxi_de2_clk_ids[] > > > > = {> > > > > > > > > .compatible = "allwinner,sun50i-h5-de2-clk", > > > > .data = &sun50i_a64_de2_clk_desc, > > > > > > > > }, > > > > > > > > + { > > > > + .compatible = "allwinner,sun50i-h6-de3-clk", > > > > + .data = &sun50i_h6_de3_clk_desc, > > > > + }, > > > > > > > > { } > > > > > > > > }; > > > > > > > > diff --git a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h index > > > > 530c006e0ae9..27bd88539f42 > > > > 100644 > > > > --- a/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > +++ b/drivers/clk/sunxi-ng/ccu-sun8i-de2.h > > > > @@ -22,6 +22,7 @@ > > > > > > > > #define CLK_MIXER0_DIV 3 > > > > #define CLK_MIXER1_DIV 4 > > > > #define CLK_WB_DIV 5 > > > > > > > > +#define CLK_ROT_DIV 11 > > > > > > > > #define CLK_NUMBER (CLK_WB + 1) > > > > > > > > -- > > > > 2.18.0