[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <901dd93d-bd79-a62f-1ada-0328a5f04737@starfivetech.com>
Date: Thu, 18 May 2023 09:22:59 +0800
From: Xingyu Wu <xingyu.wu@...rfivetech.com>
To: Andreas Schwab <schwab@...e.de>
CC: <linux-riscv@...ts.infradead.org>, <devicetree@...r.kernel.org>,
"Michael Turquette" <mturquette@...libre.com>,
Stephen Boyd <sboyd@...nel.org>,
Krzysztof Kozlowski <krzysztof.kozlowski+dt@...aro.org>,
Philipp Zabel <p.zabel@...gutronix.de>,
Conor Dooley <conor@...nel.org>,
"Emil Renner Berthing" <kernel@...il.dk>,
Rob Herring <robh+dt@...nel.org>,
Paul Walmsley <paul.walmsley@...ive.com>,
Palmer Dabbelt <palmer@...belt.com>,
Albert Ou <aou@...s.berkeley.edu>,
Hal Feng <hal.feng@...rfivetech.com>,
<linux-kernel@...r.kernel.org>, <linux-clk@...r.kernel.org>
Subject: Re: [PATCH v5 08/10] reset: starfive: jh7110: Add StarFive
STG/ISP/VOUT resets support
On 2023/5/17 21:19, Andreas Schwab wrote:
> On Apr 24 2023, Xingyu Wu wrote:
>
>> @@ -58,6 +76,18 @@ static const struct auxiliary_device_id jh7110_reset_ids[] = {
>> .name = "clk_starfive_jh7110_sys.rst-aon",
>> .driver_data = (kernel_ulong_t)&jh7110_aon_info,
>> },
>> + {
>> + .name = "clk_starfive_jh7110_sys.rst-stg",
>> + .driver_data = (kernel_ulong_t)&jh7110_stg_info,
>> + },
>> + {
>> + .name = "clk_starfive_jh7110_sys.rst-isp",
>> + .driver_data = (kernel_ulong_t)&jh7110_isp_info,
>> + },
>> + {
>> + .name = "clk_starfive_jh7110_sys.rst-vout",
>
> "clk_starfive_jh7110_sys.rst-vout" is exactly AUXILIARY_NAME_SIZE long,
> thus lacks a null termination.
>
Yeah, it should be modified to "clk_starfive_jh7110_sys.rst-vo" in this and
use "rst-vo" instead of "rst-vout" in the probe of vout clock driver.
Then it will work.
Best regards,
Xingyu Wu
Powered by blists - more mailing lists