From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail.loongson.cn (mail.loongson.cn [114.242.206.163]) by smtp.subspace.kernel.org (Postfix) with ESMTP id 7DCEE18C933; Mon, 21 Sep 2026 03:05:42 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=114.242.206.163 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789959945; cv=none; b=soP5D74Ek5LDKrso9MeZ8ena/tzQptroGr9SjJf95YTzk+alwCufdw131cpiVpC8uRZy+ow73+r5C/BVNTRAUGbM70i/BjTUqi0ORHi7616UhJXDR+VMuWuvr6v0lQxbyYZKJi0MRukF7TcSfPyuytkRXexAnXG5hUsOQWcv6b4= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789959945; c=relaxed/simple; bh=jdb+IxpMyFt+Y1qN2qE5gq6MvJtNj8u1ie9E/V+sCtM=; h=Subject:From:To:Cc:References:Message-ID:Date:MIME-Version: In-Reply-To:Content-Type; b=X54LPZJgMcyL2tOa0EzVpE5tgo5J0ZVPFrESkr4NoMNpH48LVsC6lsem8WOC+SMqAdjRuWPtJhmyElonRLARJTwE0RyOofZJhw+Wmr92M7dlkBbLIXIIcLsln6Lmk5QhUhKEGjkrqZC1nSuEVeXF1pm7xjIjaLLd8rP+aL/LBxo= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn; spf=pass smtp.mailfrom=loongson.cn; arc=none smtp.client-ip=114.242.206.163 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=loongson.cn Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=loongson.cn Received: from loongson.cn (unknown [10.20.42.101]) by gateway (Coremail) with SMTP id _____8AxntL+nrBqiA4OAA--.41125S3; Mon, 21 Sep 2026 11:05:34 +0800 (CST) Received: from [10.20.42.101] (unknown [10.20.42.101]) by front1 (Coremail) with SMTP id qMiowJAxV838nrBq5jAjAA--.41865S3; Mon, 21 Sep 2026 11:05:32 +0800 (CST) Subject: Re: [PATCH v10 0/2] i2c: ls2x: Add clock- related properties and parsing From: Hongliang Wang To: Binbin Zhou , Andi Shyti , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Wolfram Sang Cc: linux-i2c@vger.kernel.org, devicetree@vger.kernel.org, loongarch@lists.linux.dev References: <20260817090721.30582-1-wanghongliang@loongson.cn> <668c5145-da87-d318-2338-738b0035a671@loongson.cn> <1cdc2657-9924-336d-1f10-f01ca2183f1f@loongson.cn> Message-ID: Date: Mon, 21 Sep 2026 11:04:34 +0800 User-Agent: Mozilla/5.0 (X11; Linux loongarch64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 Precedence: bulk X-Mailing-List: linux-i2c@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 In-Reply-To: <1cdc2657-9924-336d-1f10-f01ca2183f1f@loongson.cn> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Content-Language: en-US X-CM-TRANSID:qMiowJAxV838nrBq5jAjAA--.41865S3 X-CM-SenderInfo: pzdqwxxrqjzxhdqjqz5rrqw2lrqou0/ X-Coremail-Antispam: 1Uk129KBj93XoW3GFWUXryUAF47ZF1fGryUArc_yoW7XF15pF Z8GFWUGr1UCa48uw43t34SvFyYqwn8J3yUWr1xJa45uanFvr1qqr12gFn09FykZFZ7Cayj 9rnIg3ZF9Fy5AFcCm3ZEXasCq-sJn29KB7ZKAUJUUUU5529EdanIXcx71UUUUU7KY7ZEXa sCq-sGcSsGvfJ3Ic02F40EFcxC0VAKzVAqx4xG6I80ebIjqfuFe4nvWSU5nxnvy29KBjDU 0xBIdaVrnRJUUUv0b4IE77IF4wAFF20E14v26r1j6r4UM7CY07I20VC2zVCF04k26cxKx2 IYs7xG6rWj6s0DM7CIcVAFz4kK6r1Y6r17M28lY4IEw2IIxxk0rwA2F7IY1VAKz4vEj48v e4kI8wA2z4x0Y4vE2Ix0cI8IcVAFwI0_JFI_Gr1l84ACjcxK6xIIjxv20xvEc7CjxVAFwI 0_Gr0_Cr1l84ACjcxK6I8E87Iv67AKxVW8JVWxJwA2z4x0Y4vEx4A2jsIEc7CjxVAFwI0_ Gr0_Gr1UM2AIxVAIcxkEcVAq07x20xvEncxIr21l57IF6xkI12xvs2x26I8E6xACxx1l5I 8CrVACY4xI64kE6c02F40Ex7xfMcIj6xIIjxv20xvE14v26r1Y6r17McIj6I8E87Iv67AK xVWUJVW8JwAm72CE4IkC6x0Yz7v_Jr0_Gr1lF7xvr2IY64vIr41lc7I2V7IY0VAS07AlzV AYIcxG8wCF04k20xvY0x0EwIxGrwCFx2IqxVCFs4IE7xkEbVWUJVW8JwC20s026c02F40E 14v26r1j6r18MI8I3I0E7480Y4vE14v26r106r1rMI8E67AF67kF1VAFwI0_Jw0_GFylIx kGc2Ij64vIr41lIxAIcVC0I7IYx2IY67AKxVWUJVWUCwCI42IY6xIIjxv20xvEc7CjxVAF wI0_Jr0_Gr1lIxAIcVCF04k26cxKx2IYs7xG6r1j6r1xMIIF0xvEx4A2jsIE14v26r1j6r 4UMIIF0xvEx4A2jsIEc7CjxVAFwI0_Jr0_GrUvcSsGvfC2KfnxnUUI43ZEXa7IU8zwZ7UU UUU== On 2026/8/31 上午9:57, Hongliang Wang wrote: > > On 2026/8/17 下午5:23, Hongliang Wang wrote: >> Hi all, >> >> Sorry for the duplicate 0/2 cover‑letter message in this thread. >> Only the 09:07 UTC full V10 series (0/2, 1/2, 2/2) is the intended one, >> please ignore the earlier 09:01 UTC standalone 0/2 cover‑letter. >> >> The patches 1/2 and 2/2 are correct. >> >> Thanks, >> Hongliang Wang >> >> >> On 2026/8/17 下午5:07, Hongliang Wang wrote: >>> Hi all: >>> >>> This patch set adds clock related properties and parsing in dts and >>> acpi. >>> >>> ====== >>> V10: >>> Patch (2/2): >>> - Replace '/' operator with div64_u64() to avoid __udivdi3 linker >>> error on >>>    32‑bit architectures with CONFIG_COMPILE_TEST. >>> >>> Link to V9: >>> https://lore.kernel.org/all/20260810074927.25284-1-wanghongliang@loongson.cn/ >>> >>> >>> V9: >>> Patch (2/2): >>> - Rewrite the commit description >>> - Use 64-bit arithmetic to avoid integer overflow during the clock >>> calculation >>> - Add checks to reject zero clock rates and ensure valid clock inputs >>> - Modify the comment for clocks and clock-div properties >>> - Add Tested-by tag from Xi Ruoyao, thanks >>> >>> Link to V8: >>> https://lore.kernel.org/all/20260721122604.12717-1-wanghongliang@loongson.cn/ >>> >>> >>> V8: >>> Patch (2/2): >>> Fix the potential issues from Sashiko AI review: >>>   - Add error checking for the pointer returned by >>> devm_clk_get_optional_enabled(); >>>   - Add zero-value checking for the match data to provide default div. >>> >>> Link to V7: >>> https://lore.kernel.org/all/20260707035104.3092-1-wanghongliang@loongson.cn/ >>> >>> >>> V7: >>> Patch (2/2): >>> Fix the potential issues from Sashiko AI review: >>>   - Move the clock property parsing process to probe function to >>> solve the issue >>>     of calling `devm_clk_get_optional_enabled()` dynamically causes >>> unbounded memory >>>     leaks and unbalanced clock enable counts. >>>   - Add non-zero checking to priv->div. >>> >>> Link to V6: >>> https://lore.kernel.org/all/20260608024533.32419-1-wanghongliang@loongson.cn/ >>> >>> >>> V6: >>> - Add Reviewed-by tag from Huacai, thanks. >>> Patch (1/2): >>>   - Remove CC stable. >>> >>> Link to V5: >>> https://lore.kernel.org/all/20260604015848.18643-1-wanghongliang@loongson.cn/ >>> >>> >>> V5: >>> Patch (1/2): >>>   - Adjust the position of #include >>> ; >>>   - Add CC stable; >>>   - Fix Signed-off-by. >>> Patch (2/2): >>>   - Replace 2K0500/2K1000/2K2000 with LS2K0500/2K1000/2K2000; >>>   - Replace 7A1000/7A2000 with LS7A1000/7A2000; >>>   - Replace if (clk && !IS_ERR(clk)) with if(!IS_ERR_OR_NULL(clk)); >>>   - Add document that clocks and clock-div are only ACPI properties >>> in ACPI; >>>   - Remove unsigned int cast in code (unsigned >>> long)device_get_match_data(dev); >>>   - Add CC stable; >>>   - Fix Signed-off-by. >>> >>> Link to V4: >>> https://lore.kernel.org/all/20260526031021.32662-1-wanghongliang@loongson.cn/ >>> >>> >>> V4: >>> - Add Acked-by tag from Conor Dooley, thanks. >>> Patch (2/2): >>>   - Adjust the position of #include ; >>>   - Remove struct ls2x_i2c_chip_data and use macro to describe div; >>>   - Use div instead of factor in ls2x_i2c_adjust_bus_speed; >>>   - Reverse the "if & else" code logic in ls2x_i2c_adjust_bus_speed; >>> >>> Link to V2: >>> The PATCH v3 is incomplete, v4 is the replacement of v3, so the >>> previous patch link is v2. >>> >>> [PATCH v2] dt-bindings: i2c: ls2x-i2c: Add clocks and >>> clock-frequency properties >>> https://lore.kernel.org/all/20260507081010.12810-1-wanghongliang@loongson.cn/ >>> >>> [PATCH v2] i2c: ls2x: Add clocks property parsing and adjust bus speed >>> https://lore.kernel.org/all/20260507081010.12810-2-wanghongliang@loongson.cn/ >>> >>> >>> V2: >>> [PATCH v2] dt-bindings: i2c: ls2x-i2c: Add clocks and >>> clock-frequency properties >>> - Remove the custom properties clock-input and clock-div, use clock >>> framework; >>> [PATCH v2] i2c: ls2x: Add clocks property parsing and adjust bus speed >>> - Use clock framework to obtain the i2c reference clock in dts. >>> >>> Link to V1: >>> https://lore.kernel.org/all/20260325011852.19079-1-wanghongliang@loongson.cn/ >>> >>> >>> Hongliang Wang (2): >>>    dt-bindings: i2c: ls2x-i2c: Add clocks and clock-frequency >>> properties >>>    i2c: ls2x: Add clocks property parsing and adjust bus speed >>> >>>   .../bindings/i2c/loongson,ls2x-i2c.yaml       |  3 ++ >>>   drivers/i2c/busses/i2c-ls2x.c                 | 44 >>> +++++++++++++++++-- >>>   2 files changed, 44 insertions(+), 3 deletions(-) >>> > Hi, > > Gentle ping on the v10 patch series. Please let me know if there are > any further changes needed. > > Best regards, > Hongliang Wang Hi, Gentle ping on the v10 patch series. Please let me know if there are any further changes needed. Best regards, Hongliang Wang