From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0b-0031df01.pphosted.com (mx0b-0031df01.pphosted.com [205.220.180.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 228284399DD for ; Mon, 14 Sep 2026 11:22:59 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.180.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789384982; cv=none; b=Fdzh3Ym0CtV9++2q7lxvI8QA9OaANb8r4uKbcAb9EiKxRyJTUEH2Ay/lDlrrNOV6KifxbyVGnJINaT2PIwyvt7FBKQr/HHjy/geTFx1WnCmbk70ExISljXC/UyDv3TMDd/4gz9//s4ohBUfhXu3AK1Uj2ibdZWw8U3cjrjQMhUA= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1789384982; c=relaxed/simple; bh=qB15Tu0wQgvflvACMlEZFwV6PZUUfZT9UV5fY5oJj88=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=LxFqUAJiu/NTD4eQKFcvwA31xjps/hM7Zr0jpIBJGKm720Byp4Sn72p4Tv098dZPNSw8aCQFG/d5JfjCTCagF9qwkqa4FQ8Js1QW1/Z1uKsSNr59VRWLDn68Z9wbw8SajUvWsVjRUv3Qq5vyrTFLGN1v0eN0kuaTEk46Hz5+SXg= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=V6eXEJqa; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=TY8Xbk1k; arc=none smtp.client-ip=205.220.180.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="V6eXEJqa"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="TY8Xbk1k" Received: from pps.filterd (m0279873.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 68EAmYWb314057 for ; Mon, 14 Sep 2026 11:22:58 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= htoucfNJ1NBOznj2ucEtPoE5fys137bATrwbrwYNEEg=; b=V6eXEJqaW52sruVW KmAWqHedQKN92ge5YENWkiPjd+G4FUiYoi+rXuXHOXCHct/s2ruBO47fL403iyAH 2SylfxfndayU9tuNs74mBf0Xj12d3kC5/2MhrMELYG8moOkZhHp5DR7YyqO0kNHs 0Lw7qkc35AhQS+AgFCbu3IoOaFGpMQF7HP0nvezpOH9EjkcQG7/eI1EGSaOHqhHE tAFRbnOZo/0zdY6LMW69aKrcmdFxfdNVAuR98oQs9x/NZr3V+VuTPtOnn+xGBeb6 5M5JFDIf2VvoFSBwfEjJixhhuEPdxOYVn7knVSt6iSa8pTvUSJQzCq5yrtQwVTO8 3nO8Vw== Received: from mail-qv1-f70.google.com (mail-qv1-f70.google.com [209.85.219.70]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4gmy9c6cej-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Mon, 14 Sep 2026 11:22:58 +0000 (GMT) Received: by mail-qv1-f70.google.com with SMTP id 6a1803df08f44-9106bb33cdfso7384286d6.2 for ; Mon, 14 Sep 2026 04:22:58 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1789384978; x=1789989778; darn=vger.kernel.org; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:from:to:cc:subject:date:message-id:reply-to :content-type; bh=htoucfNJ1NBOznj2ucEtPoE5fys137bATrwbrwYNEEg=; b=TY8Xbk1k2YxpMzK/hLKk7URRPEjoLUwPSByfCcPtMKzYR4YH49rJsWXDYLRQlmKaKK X+ENG1DzvL5fLgYYq3GdL8Wwly+HGJhAveGLsEkJtE6v7AzLrHNjfeqhju7Dv0aTF8Hb OwK3nk5s5kMTo39eVvmD4RnW1iaGWL1CmGRAGrM6FNX1ffZKmP4xGihwoZkqGKAeJk1W yJMIlLGPL8K8wUQT/W5ez39I15/6K3kTWBaY5USnE5BmE5Qlom4OUMCEr+7SsFM7KPNh C6FeOLoA/Zw7gB4fFkp9Swjc8IO9zozRd2JwQRLvZOfL92GfFXENURAf2uYUTjZ81jC8 egEQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1789384978; x=1789989778; h=content-transfer-encoding:content-type:in-reply-to:from :content-language:references:cc:to:subject:user-agent:mime-version :date:message-id:x-gm-gg:x-gm-message-state:from:to:cc:subject:date :message-id:reply-to:content-type; bh=htoucfNJ1NBOznj2ucEtPoE5fys137bATrwbrwYNEEg=; b=hJdoA1mjtNQrrwvYoKZcMdDYYMyQDBn4GgE7b0G2/fH+8lZrYKcALZ919DbvIX/cJk iI0iUd5ouevLe6wKGWgd+UpgK+GNlJghXzwxM51lJ/HBLgScTw/mb136h4yzeXqUkViF ipVma1mYrEZ9dIcxZm6DyetlHUOmj3P7YHxNPq5I7GaXirpmh6u0o4aZpVqNkqT8nkPT cauFqXqhlqJhwnv98gJa677yKtb4n3rZRr3jqvBQKIZVp3lLYOKrEBEYFuWA68H0wIIL eRZP99ufFxFMkknpi61+Hxv+tFS5csDpw8Nvw8XL1EEbj7yDvWmMxb4BMfo25wqWmfAG 1PIw== X-Forwarded-Encrypted: i=1; AKwUvBxDYm8YUnAyr1gwbpgbMWUEcePnBIUhxEPIH0wztQpKd5L2HgVfVnNWPT2r0YeymIN9GdymBjflbkn8Xw==@vger.kernel.org X-Gm-Message-State: AFuF++niUDZOZ/G5VkCO0J9/n3zoZv0aW4yXkxhDHu0E7ojxVgUpC9GX lUMfNQi6o6CHCzTYefV4h8DVaxXS8OfxAxF41P6JRIjlfaNUzcKx+sjCUkvbnT8BONDKc98dbvz uMy3WJv39oYWDucp3J7onRm1s4MDCUymXes7c8t/MtYvDq9TzqsQH/1IgCzpTuUFVng== X-Gm-Gg: AYBFou2FAoiUMEcj4ONMlFUtEbHMR8iwHkgnpoHX6dxntLWBWjun1JIXntqQ0fCHCP1 24u+BpSxyT6yKH9940YTq4s9azxXQL0pe65MVr2sn4rexQgHtdbrwHs5WriQ/MsvDT8dV/lEvAK kYWY8Rkjpakcu5RcqGsolif6nF3i8sKrLyfcyKQKviDwFqIqv4rThtQ+MNtY1L7lx095uR5JKLM clp+Z/cxWWk2/1d8pHBrvBNgdQxFPfmGRbdi5S6nZPR7y9XAh0TxvMH/r3ciccWTBF/nQDMMxFJ KjMex/IZdCzPUeBOA79bnI0fbW7GKg6Zfr3HCV5d/w1Pm8gxVnoVpxfrs+EnSMYZKSmsths+kUh bC8IpTLEphSxj2/U67nz1Ww== X-Received: by 2002:a05:6214:4883:b0:912:1f0f:b204 with SMTP id 6a1803df08f44-9122e4ccd58mr27340576d6.1.1789384978177; Mon, 14 Sep 2026 04:22:58 -0700 (PDT) X-Received: by 2002:a05:6214:4883:b0:912:1f0f:b204 with SMTP id 6a1803df08f44-9122e4ccd58mr27339976d6.1.1789384977653; Mon, 14 Sep 2026 04:22:57 -0700 (PDT) Received: from [192.168.202.12] ([178.235.128.140]) by smtp.gmail.com with ESMTPSA id 4fb4d7f45d1cf-6a9b594f0fasm4216719a12.25.2026.09.14.04.22.52 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Mon, 14 Sep 2026 04:22:56 -0700 (PDT) Message-ID: Date: Mon, 14 Sep 2026 13:22:52 +0200 Precedence: bulk X-Mailing-List: linux-input@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH RFC 20/20] arm64: dts: qcom: generalmobile-shamrock: new device To: muzafferkadir@mainlining.org, Bjorn Andersson , Abel Vesa , Stephen Boyd , Brian Masney , Jerome Brunet , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Srinivas Kandagatla , Ulf Hansson , Konrad Dybcio , "Joerg Roedel (AMD)" , Will Deacon , Robin Murphy , Lee Jones , Rob Clark , Dmitry Baryshkov , Abhinav Kumar , Jessica Zhang , Sean Paul , Marijn Suijten , Maarten Lankhorst , Maxime Ripard , Thomas Zimmermann , David Airlie , Simona Vetter , Robert Marko , Guru Das Srinagesh , Amit Kucheria , Thara Gopinath , "Rafael J. Wysocki" , Daniel Lezcano , Zhang Rui , Lukasz Luba , Kees Cook , Tony Luck , "Guilherme G. Piccoli" , Dmitry Torokhov Cc: David Heidelberg , Dang Huynh , phone-devel@vger.kernel.org, linux-arm-msm@vger.kernel.org, linux-clk@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-mmc@vger.kernel.org, iommu@lists.linux.dev, mfd@lists.linux.dev, dri-devel@lists.freedesktop.org, freedreno@lists.freedesktop.org, Krzysztof Kozlowski , linux-pm@vger.kernel.org, linux-input@vger.kernel.org References: <20260912-msm8952-initial-support-v1-0-0e742578b524@mainlining.org> <20260912-msm8952-initial-support-v1-20-0e742578b524@mainlining.org> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20260912-msm8952-initial-support-v1-20-0e742578b524@mainlining.org> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-ORIG-GUID: PoF7S1zGX06VqaW9Lt9a0T-b1UokYXkB X-Proofpoint-GUID: PoF7S1zGX06VqaW9Lt9a0T-b1UokYXkB X-Authority-Analysis: v=2.4 cv=c5Y+0h9l c=1 sm=1 tr=0 ts=6aa7d912 cx=c_pps a=oc9J++0uMp73DTRD5QyR2A==:117 a=PRfkaYvzSr8QmIIGAkY2Sg==:17 a=IkcTkHD0fZMA:10 a=VdqzKS8jKosA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=rJkE3RaqiGZ5pbrm-msn:22 a=OuZLqq7tAAAA:8 a=orgwjQpZXFS75ENpSOkA:9 a=QEXdDO2ut3YA:10 a=iYH6xdkBrDN1Jqds4HTS:22 a=AKGiAy9iJ-JzxKVHQNES:22 X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwOTE0MDE2MSBTYWx0ZWRfX4JnKCtJQOD/C J/gGNUFGzG3022nvHgqPQCBUPc9+QHmrq8dzJl9DAzM1CeokJ/6bJzGyzCu487HCfn8Bv43CU5O If8TvGkp6841I3Bml18dJomL1qXVSTVfcecB8NvpP+o/8z+osCeqx2zKFd0qOEx2aos515f6xIJ /hborLRdrIGqio6NIlrBkFtfCcHKc6nez3qxs2lbcjyFHgjrLc3jUg/EMTfOog7Yc7aZfRsnBMK muPbZOa1QEFZTBaxMm/uAkQPxni2GWSKma1mz+FKSnOZFvGiD1wXjVh3WIevwXNCiQrdEzLqmyz +JCDNU5JxzGrwqOOaPLETs3u8QiGekCXoT8iZr3a3QpKfUTrGu83sqEo/t9U0Osaghi18M6UGJf ersaao2mptgKPlDNvZahW9hZDnmuMznvucvkThVsfCKlkixvNHM1nUT5pJljTgVsyN1lWny5wi5 OsvXmKXa/wq6LjzH2Tw== X-Proofpoint-Spam-Info: AW1haW4tMjYwOTE0MDE2MSBTYWx0ZWRfXzId43ED0M/Yu XASjBo8sXH5qDtQajvLFnbum4xPq4xuH/z4wP6HKw7gfLlHKSgN0G/vj8mayCVTe5kq2JQJbWia qWVD7ahjWNzC6/ImV9QhjVrCPpV0ywE= X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1176,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-09-14_03,2026-09-13_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 adultscore=0 clxscore=1015 priorityscore=1501 lowpriorityscore=0 bulkscore=0 impostorscore=0 suspectscore=0 malwarescore=0 phishscore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2609040000 definitions=main-2609140161 On 9/12/26 10:39 AM, Muzaffer Kadir via B4 Relay wrote: > From: Muzaffer Kadir > > Add General Mobile Shamrock (GM 5 Plus). > Shamrock is designed similar to L9300. > Downstream reference is [1]. Please feel free to wrap your messages at 72 characters, otherwise they are a bit hard to read! :D [...] > +/ { > + model = "General Mobile GM 5 Plus (shamrock)"; It's not any "policy", but other DTs tend to only put the "pretty name" here, without the codename [...] > + framebuffer { > + compatible = "simple-framebuffer"; > + memory-region = <&cont_splash_mem>; > + > + width = <1080>; > + height = <1920>; > + stride = <(1080 * 3)>; > + format = "r8g8b8"; > + > + power-domains = <&gcc MDSS_GDSC>; > + > + clocks = <&gcc GCC_MDSS_AHB_CLK>, > + <&gcc GCC_MDSS_AXI_CLK>, > + <&gcc GCC_MDSS_VSYNC_CLK>, > + <&gcc GCC_MDSS_MDP_CLK>, > + <&gcc GCC_MDSS_BYTE0_CLK>, > + <&gcc GCC_MDSS_PCLK0_CLK>, > + <&gcc GCC_MDSS_ESC0_CLK>; Please align the '<'s, make sure your tab width is set to 8 [...] > + multi-led { > + compatible = "leds-group-multicolor"; > + color = ; > + function = LED_FUNCTION_INDICATOR; > + leds = <&red_status>, <&green_status>, <&blue_status>; > + }; > + > +}; Stray \n above [...] > +&ramoops_mem { > + status = "okay"; > +}; > + > +&adsp { > + status = "okay"; > +}; > + > +&lpass_mem { > + status = "okay"; > +}; Please sort the label references alphabetically > + > +&pm8950_spmi_regulators { > + // PM8950 S5 + S6 = VDD_APC supply Let's use /* C-style comments */ [...] > +&tlmm { > + gpio-reserved-ranges = <20 4>; // Downstream doesn't use 21,22,23. 24 is mdss_te Let's keep a \n between the last property and the following subnode [...] > + tpa6130a2_default: tpa6130a2-default-state { > + pins = "gpio61"; > + function = "gpio"; > + drive-strength = <8>; > + bias-pull-down; > + output-low; Drop this property, the driver will take care of it Konrad