linux-kernel - Re: [PATCH V4 0/4] Add camera clock controller support for SM8550

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <cc4e1339-f1d5-58e5-ba41-7bd5085f3c89@quicinc.com>
Date:   Tue, 13 Jun 2023 15:29:22 +0530
From:   Jagadeesh Kona <quic_jkona@...cinc.com>
To:     Krzysztof Kozlowski <krzysztof.kozlowski@...aro.org>,
        Bryan O'Donoghue <bryan.odonoghue@...aro.org>,
        Andy Gross <agross@...nel.org>,
        Konrad Dybcio <konrad.dybcio@...aro.org>,
        Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd <sboyd@...nel.org>,
        Rob Herring <robh+dt@...nel.org>,
        "Krzysztof Kozlowski" <krzysztof.kozlowski+dt@...aro.org>,
        Conor Dooley <conor+dt@...nel.org>
CC:     Bjorn Andersson <andersson@...nel.org>,
        Vladimir Zapolskiy <vladimir.zapolskiy@...aro.org>,
        <linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
        <devicetree@...r.kernel.org>, <linux-kernel@...r.kernel.org>,
        Taniya Das <quic_tdas@...cinc.com>,
        "Satya Priya Kakitapalli" <quic_skakitap@...cinc.com>,
        Imran Shaik <quic_imrashai@...cinc.com>,
        Ajit Pandey <quic_ajipan@...cinc.com>
Subject: Re: [PATCH V4 0/4] Add camera clock controller support for SM8550



On 6/13/2023 2:07 PM, Krzysztof Kozlowski wrote:
> On 12/06/2023 04:25, Bryan O'Donoghue wrote:
>> On 09/06/2023 12:50, Jagadeesh Kona wrote:
>>> Add bindings, driver and devicetree node for camera clock controller on
>>> SM8550.
>>
>> This is very confusing.
>>
>> Your cover letter doesn't detail any changes and your individual patches
>> all say "no changes since v3", "no changes since v2"
>>
>> If this is a RESEND then mark it as a RESEND.
> 
> That's indeed odd. Three versions without changes...
> 
> Best regards,
> Krzysztof
> 

This is not a RESEND, actually there were changes from each version to 
next version and change details were updated in respective patches. But 
the patches in which changes were present were dropped in v4 based on 
review comments. Will take care of updating cover letter as well with 
changes across versions if we push the next series.

Please find the summary of changes across versions till v4.

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
  - Link to v3: 
https://patchwork.kernel.org/project/linux-clk/list/?series=753150

Changes in v2:
  - Took care of review comments from v1
  - Added 2 extra patches updating L configuration value across chipsets 
to include CAL_L and RINGOSC_CAL_L fields and removed setting CAL_L 
field in clk_lucid_evo_pll_configure().
  - Link to v2: 
https://patchwork.kernel.org/project/linux-clk/list/?series=751058

v1:
  - Initial CAMCC changes for SM8550
  - Link to v1: 
https://patchwork.kernel.org/project/linux-clk/list/?series=749294

Thanks,
Jagadeesh