- bouncing Prasad
Quoting Dmitry Baryshkov (2022-05-18 11:26:16)
> On 18/05/2022 20:59, Stephen Boyd wrote:
> > Quoting Dmitry Baryshkov (2022-05-13 10:53:37)
> >> diff --git a/drivers/clk/qcom/gcc-sm8450.c b/drivers/clk/qcom/gcc-sm8450.c
> >> index 593a195467ff..a140a89b73b4 100644
> >> --- a/drivers/clk/qcom/gcc-sm8450.c
> >> +++ b/drivers/clk/qcom/gcc-sm8450.c
> >> @@ -239,17 +218,21 @@ static const struct clk_parent_data gcc_parent_data_11[] = {
> >> { .fw_name = "bi_tcxo" },
> >> };
> >>
> >> -static struct clk_regmap_mux gcc_pcie_0_pipe_clk_src = {
> >> +static struct clk_regmap_phy_mux gcc_pcie_0_pipe_clk_src = {
> >> .reg = 0x7b060,
> >> .shift = 0,
> >> .width = 2,
> >> - .parent_map = gcc_parent_map_4,
> >> + .phy_src_val = 0, /* pipe_clk */
> >
> > Make a define? PCIE0_PIPE_CLK_SRC_VAL and drop the comment?
>
> This value can change between the muxes. Thus I'd prefer not to do this.
> Compare it with the parent_maps, where we do not use defines for the
> 'val' part.
>
We don't have defines for the parent maps because they have defines for
the other side.
