Re: [PATCH 1/3] clk: qcom: branch: Extend invert logic for branch2 mem clocks

To: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx>
Subject: Re: [PATCH 1/3] clk: qcom: branch: Extend invert logic for branch2 mem clocks
From: Taniya Das <quic_tdas@xxxxxxxxxxx>
Date: Fri, 24 Jan 2025 16:04:15 +0530
Cc: Bjorn Andersson <andersson@xxxxxxxxxx>, Michael Turquette <mturquette@xxxxxxxxxxxx>, Stephen Boyd <sboyd@xxxxxxxxxx>, Rob Herring <robh@xxxxxxxxxx>, Krzysztof Kozlowski <krzk+dt@xxxxxxxxxx>, Conor Dooley <conor+dt@xxxxxxxxxx>, Ajit Pandey <quic_ajipan@xxxxxxxxxxx>, Imran Shaik <quic_imrashai@xxxxxxxxxxx>, Jagadeesh Kona <quic_jkona@xxxxxxxxxxx>, <linux-arm-msm@xxxxxxxxxxxxxxx>, <linux-clk@xxxxxxxxxxxxxxx>, <linux-kernel@xxxxxxxxxxxxxxx>, <devicetree@xxxxxxxxxxxxxxx>
In-reply-to: <oylnao5wjtqyczivesfg46eexfb5oj6o2azprtwf4bl7yhko2c@saqzbycbixqk>
References: <20241206-sm8750_videocc-v1-0-5da6e7eea2bd@quicinc.com> <20241206-sm8750_videocc-v1-1-5da6e7eea2bd@quicinc.com> <oylnao5wjtqyczivesfg46eexfb5oj6o2azprtwf4bl7yhko2c@saqzbycbixqk>
User-agent: Mozilla Thunderbird



On 12/8/2024 11:00 AM, Dmitry Baryshkov wrote:

Extend the support for mem ops implementation to handle the
sequence of enable/disable of the memories for the invert
logic.

This mostly duplicates the patch contents. Instead commit message should
explain what is the "invert logic" and why it is relevant.


Yeah Dmitry, I will update the commit text and update the patch.

--
Thanks & Regards,
Taniya Das.

References:
- [PATCH 0/3] Add the support for SM8750 Video clock controller
  - From: Taniya Das
- [PATCH 1/3] clk: qcom: branch: Extend invert logic for branch2 mem clocks
  - From: Taniya Das
- Re: [PATCH 1/3] clk: qcom: branch: Extend invert logic for branch2 mem clocks
  - From: Dmitry Baryshkov