From mboxrd@z Thu Jan 1 00:00:00 1970 Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.remote-ip=74.125.82.175 ARC-Seal: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; cv=none; t=1793551814; b=2BjRKpQJt1+Tm7H7CC6nRZU5/6/hSTJYJ6NKkR7BMedZCFt/EV+qQG7u7vJu7CgGn5m4 ANkYNl0xlgj0/jqFmg6CdVLGDERg83S3Y3mzvTiK6UljXJJTu1GmDPAxxiEzuH/juUvU0 uW3DyfJeSOLeRaMhD/9TW8JEJF898KGHdTHtszRTuM0MGfojFanNgwmrow4loU9hLEV1Q X4CBK47iCQqTmLfcF8VQrYfxUt8MmFnFaUxdFausxwbbtkqorrU1CCH7NABLthcGnJ0Aq 7xI8pJpM/rWr94mMW58ewMqmGTcNNC38P6OZswlU3nBycS2JQq3K04RPM31GAETPEjQ== ARC-Message-Signature: i=1; d=subspace.kernel.org; s=arc20261015; a=rsa-sha256; c=relaxed/relaxed; t=1793551814; h=DMARC-Filter:Received:DKIM-Signature:X-Google-DKIM-Signature: X-Forwarded-Encrypted:X-Gm-Message-State:X-Gm-Gg:X-Received:Received: From:To:Cc:Subject:Date:Message-ID:X-Mailer:In-Reply-To:References: MIME-Version:Content-Transfer-Encoding; bh=mZZe4bKEcD5TSYKYkVMyAQ6dVStKsxhjunUD6f+PU+0=; b=4s/bGsOw2fe/ID7UmZMt+UDIUctkxoCwfrbBKKX75whjcxmIcEd/ZP1x88CzLjkZlKdw ynF7H0oLOXXIfRjGx2g69LGu3RcUZCndiAsE/48lEPozWMMUB3mOsrGkhp5SCJq5LjkNC 8+rVdJclLhxcG0LK1N5cQMBiU3iJU13a3jk4Pzx/0Md8CgUZywlyUCZxikbiwTyCQ7oqQ SCdizg+0rz7NbN128rPc3Mst6Ib04Sm0Hciv3rG9vWLrn42Ml2EyOLhO30hdyBAYL9WMz vb0BsdxjaUY0rrNw4+Z7VvNlE3Tr8rNuvvZhh5jxe7AsMVOxCf93F/lBkjDh2gbOzfA== ARC-Authentication-Results: i=1; smtp.subspace.kernel.org; dmarc=pass header.from=gmail.com; spf=pass smtp.mailfrom=gmail.com; dkim=pass header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20260707 header.b=0jq2OaOv; arc=none smtp.remote-ip=74.125.82.175 Received: from mail-dy1-f175.google.com (mail-dy1-f175.google.com [74.125.82.175]) (using TLSv1.3 with cipher TLS_AES_128_GCM_SHA256 (128/128 bits) key-exchange X25519 server-signature ECDSA (prime256v1) server-digest SHA256) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 2A1B0606A326 for ; Sun, 1 Nov 2026 16:50:13 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.4.2 smtp.subspace.kernel.org 2A1B0606A326 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=none dis=none) header.from=gmail.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=gmail.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key, unprotected) header.d=gmail.com header.i=@gmail.com header.a=rsa-sha256 header.s=20260707 header.b=0jq2OaOv Received: by mail-dy1-f175.google.com with SMTP id 5a478bee46e88-36a17a332ecso3731439eec.1 for ; Sun, 01 Nov 2026 08:50:13 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20260707; t=1793551813; x=1794156613; darn=vger.kernel.org; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:from:to:cc:subject:date :message-id:reply-to:content-type; bh=mZZe4bKEcD5TSYKYkVMyAQ6dVStKsxhjunUD6f+PU+0=; b=0jq2OaOv/VB8alV10OIyFhzRgwYGwdbak1sNcvg1B4TViZnUcpBAp2z38SSNRYDRFm iWL4NL6qgRA55EC9Nok9ZP8gNMVq53QnCspXz1P64Zkgy+AWhvqcouMzwsduyTkZ5012 f4nQrmaIrmukypuridQE1Rqc2bQO2XtKr1wXs4gKemRM6oXqppdY9DKk6ZwU1tpgVLRs Yht0Pcfho/JnMuEIPOp4Ns3v2a6FQiGqn6Zq8Z9d2cYZdwoU6NVPn///Vpb2mGCsXGpG nHarEKEXX6wypiLqRLeopMvysMA2SEdZjNx/LDGITkRP6YXHoQfCv2vaKmKvnjsp/MVq 5idw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20260707; t=1793551813; x=1794156613; h=content-transfer-encoding:mime-version:references:in-reply-to :message-id:date:subject:cc:to:from:x-gm-gg:x-gm-message-state:from :to:cc:subject:date:message-id:reply-to:content-type; bh=mZZe4bKEcD5TSYKYkVMyAQ6dVStKsxhjunUD6f+PU+0=; b=PJANaurdKoyjVZPyd4q6YWsYhYMrKxgni0qReyVfePNtF/U68iOAxhlxhoCHHKA73G 3Dld4PoQTFCtQyyOJbwzzIQr2BAJLBdzsPwpGQomRZQZqt/9O/E2dGj2jpLa+vYrBjSI 8LGmHkMgEtVGeuwV+kGK8vvtJ/aDp9rKxnUV6moCr/D0xoJNiB/GLTCm1qK+T3qXEIJl kVr6nxhy5r6PDF7xDPPXZno45deaNfTkaGVf1ZlUOV2xdZ/6aE+hc3KdC/pxtqt8Fc13 3O314027D0NEcL+3M3JoMwlov/yxH9kXe5c50BKQH2ye93BBgmG2gw5ZRu1FwLk3h9j5 Y3mg== X-Forwarded-Encrypted: i=1; AKwUvBxgzZKrcKagQ9r0u451fVE9aJ6Jfzs3u+F6Y0kBYPlFW9NNQPr+pnZcSbPbS46Efg7AsekEvre229+dyE3F@vger.kernel.org X-Gm-Message-State: APuU4Z8u5003WL3k2RN5sLfrTkFku3YDZUB/y+dxotc5OsNRGNqz/XSI VwdoEe8RNUHYEs+1/YDdmvo9yxaS68JG9CRiG+zGflr4PIrXU5Mj5n4X X-Gm-Gg: AYBFou2s+8wuDVSd1VPMBQ2S3zDaUq6baZRKi3YAk7EfSZvFyJsZEHH32WNfoYm1cUi dO8Ams84kBHGTlUJiBIrFy1CHzKZhcMVPzY4NZJk00in3U8e4YuX/1enYHu2XtI+4hEQmkinJta rNWt7EXcthJX6rAsYAdZAZNhpSEnWihy+Jg4P9rhQtl9Rbl0C/kW9NhCfPvpZhcF/WxZvj+Z3Xi 0OEJueiqJWH73OZH6ysOJmg7zHFl0tmMwhzYqMnaL0YCvhK09AeWMmDQSnGX20FvXlA1kDxkIKm NxRDM05OoSa9gJKx+POKFYVqsEirO6GZj7jAsFZVbYSzgeKc6XUjgi0PDuNQKrc/8pp+cHljuIc Nbp3fkxsZEYINAQwYC0qA7jNdjfsxLxDPyBnvvU9/1PzCDd7Nv5AMLFS4MFxlXnGqwJUFOTmLue dn+jV0tSKFItiMFA+kkt9LdUWEfn+WLXNfwzLvlDuQXyvfUuftU7VWihOnKZPEEIyQKIHbaiPbl gJ2xJQVtKUT31hT4Rh8glpYRuOhTd1WJ4rcYZiRPC8NCLOzeWf8nx4XVP0pIFl8Skk4bThKsBrB UJgPdlZohMN4oGRgCxmc1wNydJVE2Nqg4cZpoHpQ9f1c0gYMsAg9Tdaqima1GGmfn4nP X-Received: by 2002:a05:693c:87cd:20b0:36a:32de:a239 with SMTP id 5a478bee46e88-36a420643f1mr11900427eec.0.1793551813029; Sun, 01 Nov 2026 08:50:13 -0800 (PST) Received: from localhost.localdomain ([121.101.131.247]) by smtp.gmail.com with ESMTPSA id 5a478bee46e88-36a42935b4asm39444940eec.16.2026.11.01.08.50.06 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 01 Nov 2026 08:50:12 -0800 (PST) From: Amir Hanip To: barnabas.czeman@mainlining.org Cc: abelvesa@kernel.org, andersson@kernel.org, conor+dt@kernel.org, devicetree@vger.kernel.org, killdeath104@gmail.com, konradybcio@kernel.org, krzk+dt@kernel.org, linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org, robh@kernel.org, a39skl@gmail.com Subject: Re: [PATCH v2 2/2] arm64: dts: qcom: Add Xiaomi Poco M3 Date: Sun, 1 Nov 2026 23:49:30 +0700 Message-ID: <20261101164930.17976-1-killdeath104@gmail.com> X-Mailer: git-send-email 2.47.3 In-Reply-To: <239d46726c0aad84feccc3def4eafab2@mainlining.org> References: <239d46726c0aad84feccc3def4eafab2@mainlining.org> X-Mailing-List: linux-arm-msm@vger.kernel.org List-Id: List-Post: List-Help: List-Subscribe: List-Unsubscribe: List-Owner: Precedence: list MIME-Version: 1.0 Content-Transfer-Encoding: 8bit On Fri, 30 Oct 2026 13:30:28 +0100, Barnabas Czeman wrote: > Poco M3 (citrus) is same HW like Redmi 9T (lime) and Remi 9T NFC > (lemon) basically is a lime with NFC. > So i think it would be better make a common base from lemon's > devicetree instead of making the same devicetree again for the > different variants where only the compatible and the model property > will be the difference. > Check sm6115-xiaomi-gingkgo-common for example, it was the same case > for Redmi Note 8 (ginkgo) and 8T (willow). Hi Barnabas, Adam, Thanks for the review, that makes sense. I am preparing a v3 that moves the shared parts of lemon into sm6115-xiaomi-lime-common.dtsi and has both lemon and citrus include it. I used the name lime-common following your naming. Happy to rename it. Plan for v3: 1. dt-bindings (unchanged) 2. lemon: move the shared nodes into the common dtsi, no functional change intended 3. citrus on top of the common dtsi I can only test citrus. I do not own a Redmi 9T, so lemon is untested on hardware. Instead, I compared the lemon DTS before and after the split, normalized with dtc (dtc -I dts -O dts -s). The only difference is the numbering of two auto-generated phandles. I will describe the check in the cover letter. Adam, would you be able to test lemon once v3 is posted, or look at the split beforehand? The IPA carve-outs (pil_ipa_fw_mem and pil_ipa_gsi_mem) are identical on both, so they now live in the common dtsi. I kept the following per variant because I cannot tell whether they are real hardware differences or just differences between the stock firmware images: - ADSP carve-out: 0x53800000/0x2200000 on citrus, 0x53400000/0x2600000 on lemon. Both regions end at 0x55a00000, so only the start and size differ. I can only confirm that the citrus value works on my device. - GPU carve-out: 0x55a15000 on citrus (right after pil_ipa_gsi_mem, like the SoC dtsi defaults), 0xfa400000 on lemon. The citrus value works: the GPU runs with a full DT that includes the display pipeline, which is not part of this series. I cannot verify lemon's value. Adam, is 0xfa400000 intentional? - PM6125 regulators: seven have different voltage ranges (s7, l5, l7, l11, l17, l18, l22). l24 has the same range, but only citrus sets regulator-allow-set-load. lemon also marks l5, l7 and l22 as always-on, citrus does not. - citrus deletes cdsp_sec_mem and the orientation-switch property of usb_qmpphy, lemon does not. Barnabas, Adam: do you know whether these differences are real, or whether they only come from the firmware of each model? Thanks, Amir