linux-kernel - Re: [PATCH v2 1/2] clk: qcom: videocc: Use HW_CTRL

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] [day] [month] [year] [list]

Message-ID: <812bd104-8661-456a-ae0f-56942bb5d4f4@quicinc.com>
Date: Tue, 14 Jan 2025 13:43:46 +0800
From: Renjiang Han <quic_renjiang@...cinc.com>
To: Bjorn Andersson <andersson@...nel.org>
CC: Michael Turquette <mturquette@...libre.com>,
        Stephen Boyd
	<sboyd@...nel.org>,
        Stanimir Varbanov <stanimir.k.varbanov@...il.com>,
        "Vikash Garodia" <quic_vgarodia@...cinc.com>,
        Bryan O'Donoghue
	<bryan.odonoghue@...aro.org>,
        Mauro Carvalho Chehab <mchehab@...nel.org>,
        <linux-arm-msm@...r.kernel.org>, <linux-clk@...r.kernel.org>,
        <linux-kernel@...r.kernel.org>, <linux-media@...r.kernel.org>,
        Taniya Das
	<quic_tdas@...cinc.com>
Subject: Re: [PATCH v2 1/2] clk: qcom: videocc: Use HW_CTRL_TRIGGER flag for
 video GDSC's


On 1/8/2025 12:41 PM, Bjorn Andersson wrote:
> On Thu, Jan 02, 2025 at 12:06:14PM +0800, Renjiang Han wrote:
>> On 12/26/2024 11:54 AM, Bjorn Andersson wrote:
>>>> The video driver will be using the newly introduced
>>>> dev_pm_genpd_set_hwmode() API to switch the video GDSC to HW and SW
>>>> control modes at runtime.
>>> "Will be using", does that imply then that if this patch is merged
>>> before (or without the venus patch) something unexpected will happen?
>>>
>>> Please clarify how you expect this to be merged, or clarify in the
>>> commit message that ordering is not of any concern.
>>>
>>> Regards,
>>> Bjorn
>>   Thanks for your comment. This patch series is to make the video driver
>>   to use dev_pm_genpd_set_hwmode() to switch GDSC mode. This patch and
>>   the venus driver patch need to be merged at the same time. Otherwise,
>>   the video will not work properly on these platforms.
>>
> The two patches are handled by different maintainers, of different
> subsystems and as such would not be expected to be merged together ever.
>
> If you have such requirements, it need to be made very clear to the
> maintainers that they will have to synchronize the effort.
>
>
> You're expected to always keep the tree "bisectable", i.e. the tree
> should function after each commit in the git history. Please clarify
> the best possible order here, and if the changes truly need to be merged
> in some specific order let's see if we can get Maruo's Acked-by and I
> could merge the pair through the clock tree.
>
> Regards,
> Bjorn
  Thank you for your reminder. As we synced offline, this patch series
  needs to be merged into the same release. If only the clock patch or
  the venus driver patch is merged, it will cause the venus driver to
  not work properly, but will not affect other functions of the system.

-- 
Best Regards,
Renjiang