* [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock
@ 2017-09-01 14:53 Thierry Reding
2017-09-01 14:53 ` [PATCH 2/4] clk: tegra: Add peripheral clock registration helper Thierry Reding
` (3 more replies)
0 siblings, 4 replies; 10+ messages in thread
From: Thierry Reding @ 2017-09-01 14:53 UTC (permalink / raw)
To: Michael Turquette, Stephen Boyd
Cc: Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra
From: Thierry Reding <treding@nvidia.com>
The sor1_src clock implemented on Tegra210 is modelled the wrong way
around, which causes some issues with HDMI and DP support. This clock
implementation is provided by BPMP on Tegra186, which models this in
a more correct way. Since this introduces incompatibilities between
the two SoC generations which we want to avoid, the Tegra210 will be
fixed in subsequent patches.
This change adds sor1_out as an alias for sor1_src.
Signed-off-by: Thierry Reding <treding@nvidia.com>
---
include/dt-bindings/clock/tegra210-car.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/include/dt-bindings/clock/tegra210-car.h b/include/dt-bindings/clock/tegra210-car.h
index 46689cd3750b..43c4a8407333 100644
--- a/include/dt-bindings/clock/tegra210-car.h
+++ b/include/dt-bindings/clock/tegra210-car.h
@@ -309,6 +309,7 @@
#define TEGRA210_CLK_BLINK 280
/* 281 */
#define TEGRA210_CLK_SOR1_SRC 282
+#define TEGRA210_CLK_SOR1_OUT 282
/* 283 */
#define TEGRA210_CLK_XUSB_HOST_SRC 284
#define TEGRA210_CLK_XUSB_FALCON_SRC 285
--
2.13.3
^ permalink raw reply related [flat|nested] 10+ messages in thread* [PATCH 2/4] clk: tegra: Add peripheral clock registration helper 2017-09-01 14:53 [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Thierry Reding @ 2017-09-01 14:53 ` Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-09-01 14:53 ` [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() Thierry Reding ` (2 subsequent siblings) 3 siblings, 1 reply; 10+ messages in thread From: Thierry Reding @ 2017-09-01 14:53 UTC (permalink / raw) To: Michael Turquette, Stephen Boyd Cc: Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra From: Thierry Reding <treding@nvidia.com> There is a common pattern that registers individual peripheral clocks from an initialization table. Add a common implementation to remove the duplication from various call sites. Signed-off-by: Thierry Reding <treding@nvidia.com> --- drivers/clk/tegra/clk-periph.c | 8 ++++++++ drivers/clk/tegra/clk.h | 3 +++ 2 files changed, 11 insertions(+) diff --git a/drivers/clk/tegra/clk-periph.c b/drivers/clk/tegra/clk-periph.c index cf80831de79d..9475c00b7cf9 100644 --- a/drivers/clk/tegra/clk-periph.c +++ b/drivers/clk/tegra/clk-periph.c @@ -203,3 +203,11 @@ struct clk *tegra_clk_register_periph_nodiv(const char *name, return _tegra_clk_register_periph(name, parent_names, num_parents, periph, clk_base, offset, CLK_SET_RATE_PARENT); } + +struct clk *tegra_clk_register_periph_data(void __iomem *clk_base, + struct tegra_periph_init_data *init) +{ + return _tegra_clk_register_periph(init->name, init->p.parent_names, + init->num_parents, &init->periph, + clk_base, init->offset, init->flags); +} diff --git a/drivers/clk/tegra/clk.h b/drivers/clk/tegra/clk.h index 872f1189ad7f..3b2763df51c2 100644 --- a/drivers/clk/tegra/clk.h +++ b/drivers/clk/tegra/clk.h @@ -662,6 +662,9 @@ struct tegra_periph_init_data { _clk_num, _gate_flags, _clk_id,\ NULL, 0, NULL) +struct clk *tegra_clk_register_periph_data(void __iomem *clk_base, + struct tegra_periph_init_data *init); + /** * struct clk_super_mux - super clock * -- 2.13.3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 2/4] clk: tegra: Add peripheral clock registration helper 2017-09-01 14:53 ` [PATCH 2/4] clk: tegra: Add peripheral clock registration helper Thierry Reding @ 2017-11-02 8:16 ` Stephen Boyd 0 siblings, 0 replies; 10+ messages in thread From: Stephen Boyd @ 2017-11-02 8:16 UTC (permalink / raw) To: Thierry Reding Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra On 09/01, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > There is a common pattern that registers individual peripheral clocks > from an initialization table. Add a common implementation to remove the > duplication from various call sites. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() 2017-09-01 14:53 [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Thierry Reding 2017-09-01 14:53 ` [PATCH 2/4] clk: tegra: Add peripheral clock registration helper Thierry Reding @ 2017-09-01 14:53 ` Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-09-01 14:53 ` [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation Thierry Reding 2017-11-02 8:16 ` [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Stephen Boyd 3 siblings, 1 reply; 10+ messages in thread From: Thierry Reding @ 2017-09-01 14:53 UTC (permalink / raw) To: Michael Turquette, Stephen Boyd Cc: Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra From: Thierry Reding <treding@nvidia.com> Instead of open-coding the same pattern repeatedly, reuse the newly introduced tegra_clk_register_periph_data() helper that will unpack the initialization structure. Signed-off-by: Thierry Reding <treding@nvidia.com> --- drivers/clk/tegra/clk-tegra-periph.c | 5 +---- drivers/clk/tegra/clk-tegra114.c | 4 +--- drivers/clk/tegra/clk-tegra20.c | 4 +--- drivers/clk/tegra/clk-tegra30.c | 4 +--- 4 files changed, 4 insertions(+), 13 deletions(-) diff --git a/drivers/clk/tegra/clk-tegra-periph.c b/drivers/clk/tegra/clk-tegra-periph.c index 848255cc0209..090a5d792341 100644 --- a/drivers/clk/tegra/clk-tegra-periph.c +++ b/drivers/clk/tegra/clk-tegra-periph.c @@ -927,10 +927,7 @@ static void __init periph_clk_init(void __iomem *clk_base, continue; data->periph.gate.regs = bank; - clk = tegra_clk_register_periph(data->name, - data->p.parent_names, data->num_parents, - &data->periph, clk_base, data->offset, - data->flags); + clk = tegra_clk_register_periph_data(clk_base, data); *dt_clk = clk; } } diff --git a/drivers/clk/tegra/clk-tegra114.c b/drivers/clk/tegra/clk-tegra114.c index fd1a99c05c2d..63087d17c3e2 100644 --- a/drivers/clk/tegra/clk-tegra114.c +++ b/drivers/clk/tegra/clk-tegra114.c @@ -1092,9 +1092,7 @@ static __init void tegra114_periph_clk_init(void __iomem *clk_base, for (i = 0; i < ARRAY_SIZE(tegra_periph_clk_list); i++) { data = &tegra_periph_clk_list[i]; - clk = tegra_clk_register_periph(data->name, - data->p.parent_names, data->num_parents, - &data->periph, clk_base, data->offset, data->flags); + clk = tegra_clk_register_periph_data(clk_base, data); clks[data->clk_id] = clk; } diff --git a/drivers/clk/tegra/clk-tegra20.c b/drivers/clk/tegra/clk-tegra20.c index 837e5cbd60e9..2b839cb24436 100644 --- a/drivers/clk/tegra/clk-tegra20.c +++ b/drivers/clk/tegra/clk-tegra20.c @@ -850,9 +850,7 @@ static void __init tegra20_periph_clk_init(void) for (i = 0; i < ARRAY_SIZE(tegra_periph_clk_list); i++) { data = &tegra_periph_clk_list[i]; - clk = tegra_clk_register_periph(data->name, data->p.parent_names, - data->num_parents, &data->periph, - clk_base, data->offset, data->flags); + clk = tegra_clk_register_periph_data(clk_base, data); clks[data->clk_id] = clk; } diff --git a/drivers/clk/tegra/clk-tegra30.c b/drivers/clk/tegra/clk-tegra30.c index a2d163f759b4..98e8a55e4e1c 100644 --- a/drivers/clk/tegra/clk-tegra30.c +++ b/drivers/clk/tegra/clk-tegra30.c @@ -1079,9 +1079,7 @@ static void __init tegra30_periph_clk_init(void) for (i = 0; i < ARRAY_SIZE(tegra_periph_clk_list); i++) { data = &tegra_periph_clk_list[i]; - clk = tegra_clk_register_periph(data->name, data->p.parent_names, - data->num_parents, &data->periph, - clk_base, data->offset, data->flags); + clk = tegra_clk_register_periph_data(clk_base, data); clks[data->clk_id] = clk; } -- 2.13.3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() 2017-09-01 14:53 ` [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() Thierry Reding @ 2017-11-02 8:16 ` Stephen Boyd 0 siblings, 0 replies; 10+ messages in thread From: Stephen Boyd @ 2017-11-02 8:16 UTC (permalink / raw) To: Thierry Reding Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra On 09/01, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > Instead of open-coding the same pattern repeatedly, reuse the newly > introduced tegra_clk_register_periph_data() helper that will unpack > the initialization structure. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 10+ messages in thread
* [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation 2017-09-01 14:53 [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Thierry Reding 2017-09-01 14:53 ` [PATCH 2/4] clk: tegra: Add peripheral clock registration helper Thierry Reding 2017-09-01 14:53 ` [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() Thierry Reding @ 2017-09-01 14:53 ` Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-11-02 8:16 ` [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Stephen Boyd 3 siblings, 1 reply; 10+ messages in thread From: Thierry Reding @ 2017-09-01 14:53 UTC (permalink / raw) To: Michael Turquette, Stephen Boyd Cc: Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra From: Thierry Reding <treding@nvidia.com> This clock was previously called sor1_src and was modelled as an input to the sor1 module clock. However, it's really an output clock that can be fed either from the safe, the sor1_pad_clkout or the sor1 module clocks. sor1 itself can take input from either of the display PLLs. The same implementation for the sor1_out clock is used on Tegra186, so this nicely lines up both SoC generations to deal with this clock in a uniform way. Signed-off-by: Thierry Reding <treding@nvidia.com> --- drivers/clk/tegra/clk-tegra-periph.c | 16 ------------ drivers/clk/tegra/clk-tegra210.c | 47 ++++++++++++++++++++++++++++++++++++ 2 files changed, 47 insertions(+), 16 deletions(-) diff --git a/drivers/clk/tegra/clk-tegra-periph.c b/drivers/clk/tegra/clk-tegra-periph.c index 090a5d792341..c7694205573f 100644 --- a/drivers/clk/tegra/clk-tegra-periph.c +++ b/drivers/clk/tegra/clk-tegra-periph.c @@ -129,7 +129,6 @@ #define CLK_SOURCE_NVDEC 0x698 #define CLK_SOURCE_NVJPG 0x69c #define CLK_SOURCE_APE 0x6c0 -#define CLK_SOURCE_SOR1 0x410 #define CLK_SOURCE_SDMMC_LEGACY 0x694 #define CLK_SOURCE_QSPI 0x6c4 #define CLK_SOURCE_VI_I2C 0x6c8 @@ -278,7 +277,6 @@ static DEFINE_SPINLOCK(PLLP_OUTA_lock); static DEFINE_SPINLOCK(PLLP_OUTB_lock); static DEFINE_SPINLOCK(PLLP_OUTC_lock); static DEFINE_SPINLOCK(sor0_lock); -static DEFINE_SPINLOCK(sor1_lock); #define MUX_I2S_SPDIF(_id) \ static const char *mux_pllaout0_##_id##_2x_pllp_clkm[] = { "pll_a_out0", \ @@ -604,18 +602,6 @@ static u32 mux_pllp_plld_plld2_clkm_idx[] = { [0] = 0, [1] = 2, [2] = 5, [3] = 6 }; -static const char *mux_sor_safe_sor1_brick_sor1_src[] = { - /* - * Bit 0 of the mux selects sor1_brick, irrespective of bit 1, so the - * sor1_brick parent appears twice in the list below. This is merely - * to support clk_get_parent() if firmware happened to set these bits - * to 0b11. While not an invalid setting, code should always set the - * bits to 0b01 to select sor1_brick. - */ - "sor_safe", "sor1_brick", "sor1_src", "sor1_brick" -}; -#define mux_sor_safe_sor1_brick_sor1_src_idx NULL - static const char *mux_pllp_pllre_clkm[] = { "pll_p", "pll_re_out1", "clk_m" }; @@ -804,8 +790,6 @@ static struct tegra_periph_init_data periph_clks[] = { MUX8("nvdec", mux_pllc2_c_c3_pllp_plla1_clkm, CLK_SOURCE_NVDEC, 194, 0, tegra_clk_nvdec), MUX8("nvjpg", mux_pllc2_c_c3_pllp_plla1_clkm, CLK_SOURCE_NVJPG, 195, 0, tegra_clk_nvjpg), MUX8("ape", mux_plla_pllc4_out0_pllc_pllc4_out1_pllp_pllc4_out2_clkm, CLK_SOURCE_APE, 198, TEGRA_PERIPH_ON_APB, tegra_clk_ape), - MUX8_NOGATE_LOCK("sor1_src", mux_pllp_plld_plld2_clkm, CLK_SOURCE_SOR1, tegra_clk_sor1_src, &sor1_lock), - NODIV("sor1", mux_sor_safe_sor1_brick_sor1_src, CLK_SOURCE_SOR1, 14, MASK(2), 183, 0, tegra_clk_sor1, &sor1_lock), MUX8("sdmmc_legacy", mux_pllp_out3_clkm_pllp_pllc4, CLK_SOURCE_SDMMC_LEGACY, 193, TEGRA_PERIPH_ON_APB | TEGRA_PERIPH_NO_RESET, tegra_clk_sdmmc_legacy), MUX8("qspi", mux_pllp_pllc_pllc_out1_pllc4_out2_pllc4_out1_clkm_pllc4_out0, CLK_SOURCE_QSPI, 211, TEGRA_PERIPH_ON_APB, tegra_clk_qspi), I2C("vii2c", mux_pllp_pllc_clkm, CLK_SOURCE_VI_I2C, 208, tegra_clk_vi_i2c), diff --git a/drivers/clk/tegra/clk-tegra210.c b/drivers/clk/tegra/clk-tegra210.c index 6d7a613f2656..be7b736371f6 100644 --- a/drivers/clk/tegra/clk-tegra210.c +++ b/drivers/clk/tegra/clk-tegra210.c @@ -40,6 +40,7 @@ #define CLK_SOURCE_CSITE 0x1d4 #define CLK_SOURCE_EMC 0x19c +#define CLK_SOURCE_SOR1 0x410 #define PLLC_BASE 0x80 #define PLLC_OUT 0x84 @@ -264,6 +265,7 @@ static DEFINE_SPINLOCK(pll_d_lock); static DEFINE_SPINLOCK(pll_e_lock); static DEFINE_SPINLOCK(pll_re_lock); static DEFINE_SPINLOCK(pll_u_lock); +static DEFINE_SPINLOCK(sor1_lock); static DEFINE_SPINLOCK(emc_lock); /* possible OSC frequencies in Hz */ @@ -2628,10 +2630,35 @@ static int tegra210_init_pllu(void) return 0; } +static const char * const sor1_out_parents[] = { + /* + * Bit 0 of the mux selects sor1_pad_clkout, irrespective of bit 1, so + * the sor1_pad_clkout parent appears twice in the list below. This is + * merely to support clk_get_parent() if firmware happened to set + * these bits to 0b11. While not an invalid setting, code should + * always set the bits to 0b01 to select sor1_pad_clkout. + */ + "sor_safe", "sor1_pad_clkout", "sor1", "sor1_pad_clkout", +}; + +static const char * const sor1_parents[] = { + "pll_p", "pll_d_out0", "pll_d2_out0", "clk_m", +}; + +static u32 sor1_parents_idx[] = { 0, 2, 5, 6 }; + +static struct tegra_periph_init_data tegra210_periph[] = { + TEGRA_INIT_DATA_TABLE("sor1", NULL, NULL, sor1_parents, + CLK_SOURCE_SOR1, 29, 0x7, 0, 0, 8, 1, + TEGRA_DIVIDER_ROUND_UP, 183, 0, tegra_clk_sor1, + sor1_parents_idx, 0, &sor1_lock), +}; + static __init void tegra210_periph_clk_init(void __iomem *clk_base, void __iomem *pmc_base) { struct clk *clk; + unsigned int i; /* xusb_ss_div2 */ clk = clk_register_fixed_factor(NULL, "xusb_ss_div2", "xusb_ss_src", 0, @@ -2650,6 +2677,12 @@ static __init void tegra210_periph_clk_init(void __iomem *clk_base, 1, 17, 207); clks[TEGRA210_CLK_DPAUX1] = clk; + clk = clk_register_mux_table(NULL, "sor1_out", sor1_out_parents, + ARRAY_SIZE(sor1_out_parents), 0, + clk_base + CLK_SOURCE_SOR1, 14, 0x3, + 0, NULL, &sor1_lock); + clks[TEGRA210_CLK_SOR1_OUT] = clk; + /* pll_d_dsi_out */ clk = clk_register_gate(NULL, "pll_d_dsi_out", "pll_d_out0", 0, clk_base + PLLD_MISC0, 21, 0, &pll_d_lock); @@ -2694,6 +2727,20 @@ static __init void tegra210_periph_clk_init(void __iomem *clk_base, 0, NULL); clks[TEGRA210_CLK_ACLK] = clk; + for (i = 0; i < ARRAY_SIZE(tegra210_periph); i++) { + struct tegra_periph_init_data *init = &tegra210_periph[i]; + struct clk **clkp; + + clkp = tegra_lookup_dt_id(init->clk_id, tegra210_clks); + if (!clkp) { + pr_warn("clock %u not found\n", init->clk_id); + continue; + } + + clk = tegra_clk_register_periph_data(clk_base, init); + *clkp = clk; + } + tegra_periph_clk_init(clk_base, pmc_base, tegra210_clks, &pll_p_params); } -- 2.13.3 ^ permalink raw reply related [flat|nested] 10+ messages in thread
* Re: [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation 2017-09-01 14:53 ` [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation Thierry Reding @ 2017-11-02 8:16 ` Stephen Boyd 0 siblings, 0 replies; 10+ messages in thread From: Stephen Boyd @ 2017-11-02 8:16 UTC (permalink / raw) To: Thierry Reding Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra On 09/01, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > This clock was previously called sor1_src and was modelled as an input > to the sor1 module clock. However, it's really an output clock that can > be fed either from the safe, the sor1_pad_clkout or the sor1 module > clocks. sor1 itself can take input from either of the display PLLs. > > The same implementation for the sor1_out clock is used on Tegra186, so > this nicely lines up both SoC generations to deal with this clock in a > uniform way. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock 2017-09-01 14:53 [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Thierry Reding ` (2 preceding siblings ...) 2017-09-01 14:53 ` [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation Thierry Reding @ 2017-11-02 8:16 ` Stephen Boyd 2017-11-02 8:46 ` Thierry Reding 3 siblings, 1 reply; 10+ messages in thread From: Stephen Boyd @ 2017-11-02 8:16 UTC (permalink / raw) To: Thierry Reding Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra On 09/01, Thierry Reding wrote: > From: Thierry Reding <treding@nvidia.com> > > The sor1_src clock implemented on Tegra210 is modelled the wrong way > around, which causes some issues with HDMI and DP support. This clock > implementation is provided by BPMP on Tegra186, which models this in > a more correct way. Since this introduces incompatibilities between > the two SoC generations which we want to avoid, the Tegra210 will be > fixed in subsequent patches. > > This change adds sor1_out as an alias for sor1_src. > > Signed-off-by: Thierry Reding <treding@nvidia.com> > --- Applied to clk-next -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock 2017-11-02 8:16 ` [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Stephen Boyd @ 2017-11-02 8:46 ` Thierry Reding 2017-11-03 15:16 ` Stephen Boyd 0 siblings, 1 reply; 10+ messages in thread From: Thierry Reding @ 2017-11-02 8:46 UTC (permalink / raw) To: Stephen Boyd Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra [-- Attachment #1: Type: text/plain, Size: 1114 bytes --] On Thu, Nov 02, 2017 at 01:16:48AM -0700, Stephen Boyd wrote: > On 09/01, Thierry Reding wrote: > > From: Thierry Reding <treding@nvidia.com> > > > > The sor1_src clock implemented on Tegra210 is modelled the wrong way > > around, which causes some issues with HDMI and DP support. This clock > > implementation is provided by BPMP on Tegra186, which models this in > > a more correct way. Since this introduces incompatibilities between > > the two SoC generations which we want to avoid, the Tegra210 will be > > fixed in subsequent patches. > > > > This change adds sor1_out as an alias for sor1_src. > > > > Signed-off-by: Thierry Reding <treding@nvidia.com> > > --- > > Applied to clk-next Hi Stephen, sorry for not having been clear about this. This patch and the 2-4 are part of the pull request I sent out earlier (and for which I have an updated one to fix the regression I mentioned last week). I'd prefer to send this to you via pull request because there is a dependency on this from the ARM SoC tree (for a DT change). Any chance you could back this out? Thierry [-- Attachment #2: signature.asc --] [-- Type: application/pgp-signature, Size: 833 bytes --] ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock 2017-11-02 8:46 ` Thierry Reding @ 2017-11-03 15:16 ` Stephen Boyd 0 siblings, 0 replies; 10+ messages in thread From: Stephen Boyd @ 2017-11-03 15:16 UTC (permalink / raw) To: Thierry Reding Cc: Michael Turquette, Peter De Schrijver, Jonathan Hunter, linux-clk, linux-tegra On 11/02, Thierry Reding wrote: > On Thu, Nov 02, 2017 at 01:16:48AM -0700, Stephen Boyd wrote: > > On 09/01, Thierry Reding wrote: > > > From: Thierry Reding <treding@nvidia.com> > > > > > > The sor1_src clock implemented on Tegra210 is modelled the wrong way > > > around, which causes some issues with HDMI and DP support. This clock > > > implementation is provided by BPMP on Tegra186, which models this in > > > a more correct way. Since this introduces incompatibilities between > > > the two SoC generations which we want to avoid, the Tegra210 will be > > > fixed in subsequent patches. > > > > > > This change adds sor1_out as an alias for sor1_src. > > > > > > Signed-off-by: Thierry Reding <treding@nvidia.com> > > > --- > > > > Applied to clk-next > > Hi Stephen, > > sorry for not having been clear about this. This patch and the 2-4 are > part of the pull request I sent out earlier (and for which I have an > updated one to fix the regression I mentioned last week). I'd prefer to > send this to you via pull request because there is a dependency on this > from the ARM SoC tree (for a DT change). > Sure I can back out the changes. Next time please indicate your merge path/strategy in some sort of cover letter so I can ignore changes in the review queue. Also, if you pick up a patch from the list please send a note that you applied it to your tree so I know to wait for a PR later. Is the PR coming soon? I forgot about it because we were waiting for the second one. -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2017-11-03 15:16 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2017-09-01 14:53 [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Thierry Reding 2017-09-01 14:53 ` [PATCH 2/4] clk: tegra: Add peripheral clock registration helper Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-09-01 14:53 ` [PATCH 3/4] clk: tegra: Use tegra_clk_register_periph_data() Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-09-01 14:53 ` [PATCH 4/4] clk: tegra: Fix sor1_out clock implementation Thierry Reding 2017-11-02 8:16 ` Stephen Boyd 2017-11-02 8:16 ` [PATCH 1/4] dt-bindings: clock: tegra: Add sor1_out clock Stephen Boyd 2017-11-02 8:46 ` Thierry Reding 2017-11-03 15:16 ` Stephen Boyd
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).