[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <c16070db-c086-45b8-bc0d-9e3bc02924b6@kernel.org>
Date: Mon, 4 Aug 2025 16:21:30 +0200
From: Krzysztof Kozlowski <krzk@...nel.org>
To: AngeloGioacchino Del Regno <angelogioacchino.delregno@...labora.com>,
Laura Nao <laura.nao@...labora.com>, wenst@...omium.org
Cc: conor+dt@...nel.org, devicetree@...r.kernel.org,
guangjie.song@...iatek.com, kernel@...labora.com, krzk+dt@...nel.org,
linux-arm-kernel@...ts.infradead.org, linux-clk@...r.kernel.org,
linux-kernel@...r.kernel.org, linux-mediatek@...ts.infradead.org,
matthias.bgg@...il.com, mturquette@...libre.com, netdev@...r.kernel.org,
nfraprado@...labora.com, p.zabel@...gutronix.de, richardcochran@...il.com,
robh@...nel.org, sboyd@...nel.org
Subject: Re: [PATCH v3 09/27] dt-bindings: clock: mediatek: Describe MT8196
clock controllers
On 04/08/2025 16:15, AngeloGioacchino Del Regno wrote:
> Il 04/08/25 15:58, Krzysztof Kozlowski ha scritto:
>> On 04/08/2025 15:27, AngeloGioacchino Del Regno wrote:
>>>
>>> We discussed about aggregating votes, yes, in software - this instead is a
>>> *broken* hardware that does the aggregation internally and does not require
>>> nor want external drivers to do the aggregation.
>>>
>>>> Maybe it is just the name, so avoid all the confusing "votes" if this is
>>>> not voting system. If this is a voting system, then don't use custom
>>>> phandles.
>>>
>>> Being it fundamentally *broken*, this being a voting system is what the hardware
>>> initially wanted to be - but effectively, since it requires YOU to:
>>> - Make sure that power supplies are turned on, if not, turn them on by "touching"
>>> HW registers (so, without any assistance from the voter MCU), if any;
>>> - Turn on parent clocks manually, if any, before using the "voter mcu" to try
>>> to ungate that clock; and
>>> - Enable the "FENC" manually, after the mcu says that the clock was ungated.
>>
>>
>> I understand that "YOU" as Linux driver, when you want to do something
>> (e.g. toggle) a clock?
>
> "you" == Linux driver, yes.
>
>> If so this looks a lot like power domain, although with some differences.
>>
>
> A power domain ungates power to something.
Does more, it is not a simple supply.
>
> These are clocks, giving a (x) (M)Hz signal to something.
Your earlier message about "YOU" said:
" - Make sure that power supplies are turned on, if not, turn them on
by "touching"
HW registers (so, without any assistance from the voter MCU), if any;"
so not a simple clocks stuff.
Best regards,
Krzysztof
Powered by blists - more mailing lists