| Message ID | 20230414134812.16812-2-quic_devipriy@quicinc.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
| Series | Incremental patches on minimal boot support | expand |
On 14/04/2023 15:48, Devi Priya wrote: > Remove bias_pll_ubi_nc_clk from the binding as it has been removed from > the Device Tree. Also added Bjorn Andersson to the maintainers list. Was it really removed? Where? > > Signed-off-by: Devi Priya <quic_devipriy@quicinc.com> Best regards, Krzysztof
On 4/14/2023 7:47 PM, Krzysztof Kozlowski wrote: > On 14/04/2023 15:48, Devi Priya wrote: >> Remove bias_pll_ubi_nc_clk from the binding as it has been removed from >> the Device Tree. Also added Bjorn Andersson to the maintainers list. > > Was it really removed? Where? > It has been removed from the Device tree and binding in V11 https://lore.kernel.org/linux-arm-msm/20230404101622.5394-1-quic_devipriy@quicinc.com/ Thanks, Devi Priya >> >> Signed-off-by: Devi Priya <quic_devipriy@quicinc.com> > > > Best regards, > Krzysztof >
On 14/04/2023 16:22, Devi Priya wrote: > > > On 4/14/2023 7:47 PM, Krzysztof Kozlowski wrote: >> On 14/04/2023 15:48, Devi Priya wrote: >>> Remove bias_pll_ubi_nc_clk from the binding as it has been removed from >>> the Device Tree. Also added Bjorn Andersson to the maintainers list. >> >> Was it really removed? Where? >> > It has been removed from the Device tree and binding in V11 > https://lore.kernel.org/linux-arm-msm/20230404101622.5394-1-quic_devipriy@quicinc.com/ I still see it in current next. Are you sure you refer to something already merged? Best regards, Krzysztof
On 4/14/2023 8:39 PM, Krzysztof Kozlowski wrote: > On 14/04/2023 16:22, Devi Priya wrote: >> >> >> On 4/14/2023 7:47 PM, Krzysztof Kozlowski wrote: >>> On 14/04/2023 15:48, Devi Priya wrote: >>>> Remove bias_pll_ubi_nc_clk from the binding as it has been removed from >>>> the Device Tree. Also added Bjorn Andersson to the maintainers list. >>> >>> Was it really removed? Where? >>> >> It has been removed from the Device tree and binding in V11 >> https://lore.kernel.org/linux-arm-msm/20230404101622.5394-1-quic_devipriy@quicinc.com/ > > I still see it in current next. Are you sure you refer to something > already merged? This change was made in V11 and Currently V9 is merged in linux-next. So, the delta changes between V9 & V12 is posted in this series. The device tree change where the clock is removed is added part of this incremental patch series and the binding is updated to reflect the same. Apologies, if the confusion is because of "has been removed from device tree" in the commit message. Thanks, Devi Priya > > Best regards, > Krzysztof >
On 14/04/2023 18:04, Devi Priya wrote: > > > On 4/14/2023 8:39 PM, Krzysztof Kozlowski wrote: >> On 14/04/2023 16:22, Devi Priya wrote: >>> >>> >>> On 4/14/2023 7:47 PM, Krzysztof Kozlowski wrote: >>>> On 14/04/2023 15:48, Devi Priya wrote: >>>>> Remove bias_pll_ubi_nc_clk from the binding as it has been removed from >>>>> the Device Tree. Also added Bjorn Andersson to the maintainers list. >>>> >>>> Was it really removed? Where? >>>> >>> It has been removed from the Device tree and binding in V11 >>> https://lore.kernel.org/linux-arm-msm/20230404101622.5394-1-quic_devipriy@quicinc.com/ >> >> I still see it in current next. Are you sure you refer to something >> already merged? > > This change was made in V11 and Currently V9 is merged in linux-next. > So, the delta changes between V9 & V12 is posted in this series. > The device tree change where the clock is removed is added part of this > incremental patch series and the binding is updated to reflect the same. > Apologies, if the confusion is because of "has been removed from device > tree" in the commit message. Your commit indicated that removal from DTS happened. If it did not happen, you need to come with proper reason (and fix the tense not to confuse). Best regards, Krzysztof
On 4/14/2023 10:41 PM, Krzysztof Kozlowski wrote: > On 14/04/2023 18:04, Devi Priya wrote: >> >> >> On 4/14/2023 8:39 PM, Krzysztof Kozlowski wrote: >>> On 14/04/2023 16:22, Devi Priya wrote: >>>> >>>> >>>> On 4/14/2023 7:47 PM, Krzysztof Kozlowski wrote: >>>>> On 14/04/2023 15:48, Devi Priya wrote: >>>>>> Remove bias_pll_ubi_nc_clk from the binding as it has been removed from >>>>>> the Device Tree. Also added Bjorn Andersson to the maintainers list. >>>>> >>>>> Was it really removed? Where? >>>>> >>>> It has been removed from the Device tree and binding in V11 >>>> https://lore.kernel.org/linux-arm-msm/20230404101622.5394-1-quic_devipriy@quicinc.com/ >>> >>> I still see it in current next. Are you sure you refer to something >>> already merged? >> >> This change was made in V11 and Currently V9 is merged in linux-next. >> So, the delta changes between V9 & V12 is posted in this series. >> The device tree change where the clock is removed is added part of this >> incremental patch series and the binding is updated to reflect the same. >> Apologies, if the confusion is because of "has been removed from device >> tree" in the commit message. > > Your commit indicated that removal from DTS happened. If it did not > happen, you need to come with proper reason (and fix the tense not to > confuse). Got it, will update the commit message. Thanks, Devi Priya > > Best regards, > Krzysztof >
diff --git a/Documentation/devicetree/bindings/clock/qcom,ipq9574-gcc.yaml b/Documentation/devicetree/bindings/clock/qcom,ipq9574-gcc.yaml index afc68eb9d7cc..8581de266750 100644 --- a/Documentation/devicetree/bindings/clock/qcom,ipq9574-gcc.yaml +++ b/Documentation/devicetree/bindings/clock/qcom,ipq9574-gcc.yaml @@ -7,6 +7,7 @@ $schema: http://devicetree.org/meta-schemas/core.yaml# title: Qualcomm Global Clock & Reset Controller on IPQ9574 maintainers: + - Bjorn Andersson <andersson@kernel.org> - Anusha Rao <quic_anusha@quicinc.com> description: | @@ -25,7 +26,6 @@ properties: items: - description: Board XO source - description: Sleep clock source - - description: Bias PLL ubi clock source - description: PCIE30 PHY0 pipe clock source - description: PCIE30 PHY1 pipe clock source - description: PCIE30 PHY2 pipe clock source @@ -48,7 +48,6 @@ examples: reg = <0x01800000 0x80000>; clocks = <&xo_board_clk>, <&sleep_clk>, - <&bias_pll_ubi_nc_clk>, <&pcie30_phy0_pipe_clk>, <&pcie30_phy1_pipe_clk>, <&pcie30_phy2_pipe_clk>,
Remove bias_pll_ubi_nc_clk from the binding as it has been removed from the Device Tree. Also added Bjorn Andersson to the maintainers list. Signed-off-by: Devi Priya <quic_devipriy@quicinc.com> --- Changes since V9: - Dropped the clock description for Bias PLL ubi clock source to reflect the gcc node in the DT - Added Bjorn Andersson to the maintainers list Documentation/devicetree/bindings/clock/qcom,ipq9574-gcc.yaml | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-)