mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
From: Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>
To: Jagadeesh Kona <quic_jkona@quicinc.com>,
	Andy Gross <agross@kernel.org>,
	Bjorn Andersson <andersson@kernel.org>,
	Konrad Dybcio <konrad.dybcio@linaro.org>,
	Michael Turquette <mturquette@baylibre.com>,
	Stephen Boyd <sboyd@kernel.org>, Rob Herring <robh+dt@kernel.org>,
	Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>,
	Conor Dooley <conor+dt@kernel.org>
Cc: Vladimir Zapolskiy <vladimir.zapolskiy@linaro.org>,
	linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org,
	devicetree@vger.kernel.org, linux-kernel@vger.kernel.org,
	Taniya Das <quic_tdas@quicinc.com>,
	Satya Priya Kakitapalli <quic_skakitap@quicinc.com>,
	Imran Shaik <quic_imrashai@quicinc.com>,
	Ajit Pandey <quic_ajipan@quicinc.com>
Subject: Re: [PATCH V6 0/5] Add camera clock controller support for SM8550
Date: Thu, 31 Aug 2023 10:24:12 +0200	[thread overview]
Message-ID: <8fa0874a-1a4e-3e4b-cb14-fae67f757401@linaro.org> (raw)
In-Reply-To: <484fad11-b44a-0383-c34b-5ae30cd24bb9@quicinc.com>

On 31/08/2023 10:20, Jagadeesh Kona wrote:
> 
> 
> On 7/7/2023 9:27 AM, Jagadeesh Kona wrote:
>> Add bindings, driver and devicetree node for camera clock controller on
>> SM8550.
>>
>> Changes in v6:
>>   - Updated parent map and frequency table of cam_cc_xo_clk_src to use
>>     active only source P_BI_TCXO_AO instead of P_BI_TCXO
>>
>> Changes in v5:
>>   - Added clk_lucid_ole_pll_configure() to configure lucid ole PLL's
>>   - Used module_platform_driver() instead of subsys_initcall()
>>   - Fixed overloading .l config with CAL_L and RINGOSC_CAL_L fields
>>
>> Changes in v4:
>>   - Dropped the extra patches added in v2, since the review comments on
>>     v3 recommended an alternate solution
>>
>> Changes in v3:
>>   - Squashed 2 extra patches added in v2 into single patch as per review
>>     comments
>>
>> Changes in v2:
>>   - Took care of review comments from v1
>>       + Removed new YAML file and reused SM8450 CAMCC YAML file for SM8550
>>       + Sorted the PLL names in proper order
>>       + Updated all PLL configurations to lower case hex
>>       + Reused evo ops instead of adding new ops for ole pll
>>       + Moved few clocks to separate patch to fix patch too long error
>>       + Padded non-zero address part to 8 hex digits in DT change
>>   - Added 2 extra patches updating .l config value across chipsets to
>>     include CAL_L and RINGOSC_CAL_L fields and removed setting CAL_L
>>     field explicitly in clk_lucid_evo_pll_configure().
>>
>> v1:
>>    - Initial CAMCC changes for SM8550
>>
>> Previous series:
>> v5 - https://patchwork.kernel.org/project/linux-clk/list/?series=759863
>> v4 - https://patchwork.kernel.org/project/linux-clk/list/?series=755683
>> v3 - https://patchwork.kernel.org/project/linux-clk/list/?series=753150
>> v2 - https://patchwork.kernel.org/project/linux-clk/list/?series=751058
>> v1 - https://patchwork.kernel.org/project/linux-clk/list/?series=749294
>>
>> Jagadeesh Kona (5):
>>    dt-bindings: clock: qcom: Add SM8550 camera clock controller
>>    clk: qcom: clk-alpha-pll: Add support for lucid ole pll configure
>>    clk: qcom: camcc-sm8550: Add camera clock controller driver for SM8550
>>    clk: qcom: camcc-sm8550: Add support for qdss, sleep and xo clocks
>>    arm64: dts: qcom: sm8550: Add camera clock controller
>>
>>   .../bindings/clock/qcom,sm8450-camcc.yaml     |    8 +-
>>   arch/arm64/boot/dts/qcom/sm8550.dtsi          |   15 +
>>   drivers/clk/qcom/Kconfig                      |    7 +
>>   drivers/clk/qcom/Makefile                     |    1 +
>>   drivers/clk/qcom/camcc-sm8550.c               | 3564 +++++++++++++++++
>>   drivers/clk/qcom/clk-alpha-pll.c              |   29 +
>>   drivers/clk/qcom/clk-alpha-pll.h              |    2 +
>>   include/dt-bindings/clock/qcom,sm8550-camcc.h |  187 +
>>   8 files changed, 3811 insertions(+), 2 deletions(-)
>>   create mode 100644 drivers/clk/qcom/camcc-sm8550.c
>>   create mode 100644 include/dt-bindings/clock/qcom,sm8550-camcc.h
>>
> 
> Hi Bjorn,
> 
> All patches in this series are in reviewed state, could you please help 
> to pick this series in the next release? Thanks!

We are in merge window, so please refrain from simple pings for
patchsets which are not fixes.

Best regards,
Krzysztof


  reply	other threads:[~2023-08-31  8:24 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-07-07  3:57 Jagadeesh Kona
2023-07-07  3:57 ` [PATCH V6 1/5] dt-bindings: clock: qcom: Add SM8550 camera clock controller Jagadeesh Kona
2023-07-07  3:57 ` [PATCH V6 2/5] clk: qcom: clk-alpha-pll: Add support for lucid ole pll configure Jagadeesh Kona
2023-07-07  3:57 ` [PATCH V6 3/5] clk: qcom: camcc-sm8550: Add camera clock controller driver for SM8550 Jagadeesh Kona
2023-07-07  3:57 ` [PATCH V6 4/5] clk: qcom: camcc-sm8550: Add support for qdss, sleep and xo clocks Jagadeesh Kona
2023-07-07 11:05   ` Dmitry Baryshkov
2023-07-07  3:57 ` [PATCH V6 5/5] arm64: dts: qcom: sm8550: Add camera clock controller Jagadeesh Kona
2023-08-31  8:20 ` [PATCH V6 0/5] Add camera clock controller support for SM8550 Jagadeesh Kona
2023-08-31  8:24   ` Krzysztof Kozlowski [this message]
2023-09-19 23:07 ` (subset) " Bjorn Andersson

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8fa0874a-1a4e-3e4b-cb14-fae67f757401@linaro.org \
    --to=krzysztof.kozlowski@linaro.org \
    --cc=agross@kernel.org \
    --cc=andersson@kernel.org \
    --cc=conor+dt@kernel.org \
    --cc=devicetree@vger.kernel.org \
    --cc=konrad.dybcio@linaro.org \
    --cc=krzysztof.kozlowski+dt@linaro.org \
    --cc=linux-arm-msm@vger.kernel.org \
    --cc=linux-clk@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mturquette@baylibre.com \
    --cc=quic_ajipan@quicinc.com \
    --cc=quic_imrashai@quicinc.com \
    --cc=quic_jkona@quicinc.com \
    --cc=quic_skakitap@quicinc.com \
    --cc=quic_tdas@quicinc.com \
    --cc=robh+dt@kernel.org \
    --cc=sboyd@kernel.org \
    --cc=vladimir.zapolskiy@linaro.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®