| Message ID | 20220705202837.667-5-ansuelsmth@gmail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | clk: qcom: Drop use of pxo/cxo_board for rpm devices | expand |
On Tue, 5 Jul 2022 at 23:56, Christian Marangi <ansuelsmth@gmail.com> wrote: > > Drivers expect to call pxo/cxo_board clock as pxo/cxo. > Use clock-output-names to set the correct naming for these fixed clock > to save drivers that are migrating to parent_data and still use > parent_names. Please no. There is no reason to add these names. Let's follow the example of other platforms. > > Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> > --- > arch/arm/boot/dts/qcom-ipq8064.dtsi | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi > index 1425a4e4283f..720e27e29627 100644 > --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi > @@ -302,12 +302,14 @@ cxo_board: cxo_board { > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <25000000>; > + clock-output-names = "cxo"; > }; > > pxo_board: pxo_board { > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <25000000>; > + clock-output-names = "pxo"; > }; > > sleep_clk: sleep_clk { > -- > 2.36.1 >
On Wed, Jul 06, 2022 at 11:29:09AM +0300, Dmitry Baryshkov wrote: > On Tue, 5 Jul 2022 at 23:56, Christian Marangi <ansuelsmth@gmail.com> wrote: > > > > Drivers expect to call pxo/cxo_board clock as pxo/cxo. > > Use clock-output-names to set the correct naming for these fixed clock > > to save drivers that are migrating to parent_data and still use > > parent_names. > > Please no. There is no reason to add these names. Let's follow the > example of other platforms. > Okok... just don't like the '_board' suffix but will drop... > > > > Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> > > --- > > arch/arm/boot/dts/qcom-ipq8064.dtsi | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi > > index 1425a4e4283f..720e27e29627 100644 > > --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi > > +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi > > @@ -302,12 +302,14 @@ cxo_board: cxo_board { > > compatible = "fixed-clock"; > > #clock-cells = <0>; > > clock-frequency = <25000000>; > > + clock-output-names = "cxo"; > > }; > > > > pxo_board: pxo_board { > > compatible = "fixed-clock"; > > #clock-cells = <0>; > > clock-frequency = <25000000>; > > + clock-output-names = "pxo"; > > }; > > > > sleep_clk: sleep_clk { > > -- > > 2.36.1 > > > > > -- > With best wishes > Dmitry
On 5.07.2022 22:28, Christian Marangi wrote: > Drivers expect to call pxo/cxo_board clock as pxo/cxo. > Use clock-output-names to set the correct naming for these fixed clock > to save drivers that are migrating to parent_data and still use > parent_names. > For this you'd want to call them "cxo_board" and "pxo_board", as that's what they were named because of their node names. Konrad > Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> > --- > arch/arm/boot/dts/qcom-ipq8064.dtsi | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi > index 1425a4e4283f..720e27e29627 100644 > --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi > +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi > @@ -302,12 +302,14 @@ cxo_board: cxo_board { > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <25000000>; > + clock-output-names = "cxo"; > }; > > pxo_board: pxo_board { > compatible = "fixed-clock"; > #clock-cells = <0>; > clock-frequency = <25000000>; > + clock-output-names = "pxo"; > }; > > sleep_clk: sleep_clk {
diff --git a/arch/arm/boot/dts/qcom-ipq8064.dtsi b/arch/arm/boot/dts/qcom-ipq8064.dtsi index 1425a4e4283f..720e27e29627 100644 --- a/arch/arm/boot/dts/qcom-ipq8064.dtsi +++ b/arch/arm/boot/dts/qcom-ipq8064.dtsi @@ -302,12 +302,14 @@ cxo_board: cxo_board { compatible = "fixed-clock"; #clock-cells = <0>; clock-frequency = <25000000>; + clock-output-names = "cxo"; }; pxo_board: pxo_board { compatible = "fixed-clock"; #clock-cells = <0>; clock-frequency = <25000000>; + clock-output-names = "pxo"; }; sleep_clk: sleep_clk {
Drivers expect to call pxo/cxo_board clock as pxo/cxo. Use clock-output-names to set the correct naming for these fixed clock to save drivers that are migrating to parent_data and still use parent_names. Signed-off-by: Christian Marangi <ansuelsmth@gmail.com> --- arch/arm/boot/dts/qcom-ipq8064.dtsi | 2 ++ 1 file changed, 2 insertions(+)