On 07/10/2024 10:04, sunyeal.hong wrote:
> Hello Krzysztof,
>
>> -----Original Message-----
>> From: Krzysztof Kozlowski <krzk@xxxxxxxxxx>
>> Sent: Monday, September 30, 2024 8:36 PM
>> To: Sunyeal Hong <sunyeal.hong@xxxxxxxxxxx>; Krzysztof Kozlowski
>> <krzk+dt@xxxxxxxxxx>; Rob Herring <robh@xxxxxxxxxx>; Conor Dooley
>> <conor+dt@xxxxxxxxxx>; Alim Akhtar <alim.akhtar@xxxxxxxxxxx>; Sylwester
>> Nawrocki <s.nawrocki@xxxxxxxxxxx>; Chanwoo Choi <cw00.choi@xxxxxxxxxxx>;
>> Michael Turquette <mturquette@xxxxxxxxxxxx>; Stephen Boyd
>> <sboyd@xxxxxxxxxx>
>> Cc: devicetree@xxxxxxxxxxxxxxx; linux-arm-kernel@xxxxxxxxxxxxxxxxxxx;
>> linux-samsung-soc@xxxxxxxxxxxxxxx; linux-kernel@xxxxxxxxxxxxxxx; linux-
>> clk@xxxxxxxxxxxxxxx
>> Subject: Re: [PATCH 2/3] clk: samsung: exynosautov920: add peric1, misc
>> and hsi0/1 clock support
>>
>> On 12/09/2024 12:38, Sunyeal Hong wrote:
>>> Like CMU_PERIC1, this provides clocks for USI09 ~ USI17, USI_I2C and
>> USI_I3C.
>>> Like CMU_MISC, this provides clocks for MISC, GIC and OTP.
>>> Like CMU_HSI0, this provides clocks for PCIE.
>>> Like CMU_HSI1, this provides clocks for USB and MMC.
>>>
>>> Signed-off-by: Sunyeal Hong <sunyeal.hong@xxxxxxxxxxx>
>>> ---
>>
>> ...
>>
>>> +
>>> static int __init exynosautov920_cmu_probe(struct platform_device
>>> *pdev) {
>>> const struct samsung_cmu_info *info; @@ -1154,6 +1431,19 @@ static
>>> const struct of_device_id exynosautov920_cmu_of_match[] = {
>>> {
>>> .compatible = "samsung,exynosautov920-cmu-peric0",
>>> .data = &peric0_cmu_info,
>>> + }, {
>>> + .compatible = "samsung,exynosautov920-cmu-peric1",
>>> + .data = &peric1_cmu_info,
>>> + }, {
>>> + .compatible = "samsung,exynosautov920-cmu-misc",
>>> + .data = &misc_cmu_info,
>>> + }, {
>>> + .compatible = "samsung,exynosautov920-cmu-hsi0",
>>> + .data = &hsi0_cmu_info,
>>> + }, {
>>> + .compatible = "samsung,exynosautov920-cmu-hsi1",
>>> + .data = &hsi1_cmu_info,
>>> + }, {
>>
>> This is unrelated change. Please rebase.
>>
> Could you please explain in more detail the comment mentioned above?
You were growing this list, didn't you? Then adding sentinel is unrelated.
Best regards,
Krzysztof
