From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.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 AD0A31DA55 for ; Tue, 4 Nov 2025 12:32:06 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762259528; cv=none; b=IseQWQ2MHzTXNM5Zz+veWo8XYCj6B37MHoR0+y/xbmal+1H86vhqiG1Hsm+D2XRLcCfxrEsbWLDlzxpaCr/4J/ok9Ou0BlDUUJLCpJzDJBQ4Q9TYtDR9l0hWOBmEWieBm13cgqB/ml0PMJKh+xdsyPqHoRDJQrqhkc7oFEP3y2E= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1762259528; c=relaxed/simple; bh=iqu/Ya2dwS9ONJUXHsu1yCgirOeUVkBUk9APrAHQLD0=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=TqpQgoVv+/6IMT/9HSSgzAjhwM6YtrTXsABAH5/eCPzfwZa+eCDkyk4Ffp/aSva684KBl7BxFByqftHnu0TSsbGvT7pBRVyP16LPQ16NPWxFvNltGdLadiYyD1ZDv8k1B+jbGpQTQ43jXs00Uu4W+t/bwXOjHUs8hkiNC8pJUUY= 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=Sv3SlHj6; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=ZblhvTPA; arc=none smtp.client-ip=205.220.168.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="Sv3SlHj6"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="ZblhvTPA" Received: from pps.filterd (m0279863.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 5A48gRJ61419267 for ; Tue, 4 Nov 2025 12:32:06 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= LyszaJhakD561ZLBW57AsNnvbXTrFvY/B4WU2SNmYTo=; b=Sv3SlHj6gEach04s jN+XZyRC5yWUQFhTX0Vq6YR1+YYwZJJDfYRmRoYzmv+7V0jG4xHViOy364FUuss1 RZ0tuni2FschXt+5InLD1fJ1VioAH6mK9Z8M8EED557KMggzBkh3Z8D6Wy/M4nIT 8P10E1ZsgkFLjA++8y3wOs/TOREw6liytR3vh4pxDZ+Zxks+h9zaGahhSsRu8XhJ 6nXAa29j0gVMFeupUvVv3Q44lnxFykK1Ji6qWFwkESab3LrLkNplf6RZcO0Sk9d3 sKb8YB9x1O/vAF/15yHBzqZsnhMVlUiVzYeSfmLgu29O7rMOJyKc8evOxm85jdP7 7pX2+Q== Received: from mail-qt1-f198.google.com (mail-qt1-f198.google.com [209.85.160.198]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4a70fhtwcr-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Tue, 04 Nov 2025 12:32:05 +0000 (GMT) Received: by mail-qt1-f198.google.com with SMTP id d75a77b69052e-4ecf8fad13dso9517951cf.1 for ; Tue, 04 Nov 2025 04:32:05 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1762259525; x=1762864325; darn=vger.kernel.org; h=content-transfer-encoding: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; bh=LyszaJhakD561ZLBW57AsNnvbXTrFvY/B4WU2SNmYTo=; b=ZblhvTPAdi7aioM8HS+JxJxPELEMmr2tRjw2dWi+JWQKZIkcnX+cxwfT9mxXszQo6c iwJENit0TThr0psFc0DenmIPNhswn5Y3N4X2B/EqQFEe7ecEFd+pGwoHQbhflze8n+CK AvlKnlqkjKM2w8ZYVr2b9LtOcL1GXLRQWyNpmvRAScxyrJKc0+D0YFcJEFbD4Fv3hwhC QC3D9jj656KM0wSB6o2qITUJXoXAQGsietG9s5v+vC+jnKKmRpK1e/dVC4wy0AvnLF6K SZ0ZcqMFJBgDLWZE3YzMOA5gArdfqeB6NS9/Lns1re/izbNM4hVkFWfyUjmTQrbBXkeG 59Zw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1762259525; x=1762864325; h=content-transfer-encoding:in-reply-to:from:content-language :references:cc:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=LyszaJhakD561ZLBW57AsNnvbXTrFvY/B4WU2SNmYTo=; b=CtijBj74pgi1dr1rDG4z585J/F0GipmyXnmTMpxMEzHCdxwTjvDW1DZRf50zwuJVTO dAqRR97XGJykijajL9FVnxEap66uSexSsojcKFzcwi5BMC9nutU4N50sbvD8LqJ1Sso7 99o8OtmAuCe2qMZ9oX2OtMfkNOV8pvVPhZaGwRcyq3dSvKYxcRJOg1asZqZ4OLEF1Gpc jd455gv/RoCcH+T7m8UfgIcAXxV0sOwAq/cHLq9pYy1h3eOQF8vpgaCp/fI8q3ZPwKlo E1sBQPSjaCEBwNDsuQ+9jfPMC/9YWuTcJs/HTcBY+gnZCH0/4bQJtkb1wDLj8KW/8Yoq s6Ag== X-Forwarded-Encrypted: i=1; AJvYcCXuaM5EXwi3y7bWG8CgSqDD1wuNbep/0FqKG95pPDrvz6WjAg+a6BsMlsbQisJirL7m4bg4Y4xoJJh0@vger.kernel.org X-Gm-Message-State: AOJu0Yy+LfcHQt6cFq31t62ftgGTSQl588Jg829Cy6YOZ2LfLhvootMI HuwnUbovBOqQ0XnIpEAo1ZA84uliLTiInUUe2YF0IB2sDzaaukpQExKRwv+sULjhXfA33THOV2W zPcGrbl9gDMC+D2lApzbLwbSA3s9VGehBpdbj2eJxl/QiBAqCYyyv0CUKXqPDDlqy X-Gm-Gg: ASbGncsOe/SAjiRbTVu83fPMWtCNy0BnrJ5qv3T6wRtvby8Iwg5Q//kfr9eRkaggi8s Q3o8jwf+zXqZjCzRYatyJeducCAPi+qZgMF+axtc+cgjCvlQMrAZVbB3k1cLSrDfcQn41T1nX9Q Yo4k5ecRC5KcDCXk19uhNnhP5aKH3bjUVqCqjntYUW7gp9susKjf8Cd87kl/XsEwOKEDraGPTcH QPO2ySkTn/fqTjvcv8LSVRMRIdoiPGW3DvfmE3YgBVBmR/eGi82250CbmtkVPtPXhfvwPLIvp08 nO/otsaaKeKL6diarWGnaNLe9N85oSMW8Z7R+NKWILIczaTWKVFZNMzvRgIl81yruhoqgCVvJjr jmKJVQO6zIFnJAFds/7Z9+oT9mzhnak8L1jNL0HQgNx/ngDFlhh7JTxnj X-Received: by 2002:a05:622a:409:b0:4ec:f075:50cb with SMTP id d75a77b69052e-4ed30d4641dmr129558321cf.2.1762259524675; Tue, 04 Nov 2025 04:32:04 -0800 (PST) X-Google-Smtp-Source: AGHT+IG51Vci1fQYtyUF8ZqRWbO8+HNZTvYQqZoIMz008jdi7wzWhQWr6jzzFAkQFLU+efsFn3wONg== X-Received: by 2002:a05:622a:409:b0:4ec:f075:50cb with SMTP id d75a77b69052e-4ed30d4641dmr129558031cf.2.1762259524230; Tue, 04 Nov 2025 04:32:04 -0800 (PST) Received: from [192.168.119.202] (078088045245.garwolin.vectranet.pl. [78.88.45.245]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b72406568a2sm200000566b.73.2025.11.04.04.32.01 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 04 Nov 2025 04:32:03 -0800 (PST) Message-ID: <790ca394-cee2-412b-97d8-c6416b843010@oss.qualcomm.com> Date: Tue, 4 Nov 2025 13:32:00 +0100 Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH v3 1/2] dt-bindings: soc: qcom: Add qcom,kaanapali-imem compatible To: Krzysztof Kozlowski , Jingyi Wang Cc: Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Konrad Dybcio , Robert Marko , Das Srinagesh , aiqun.yu@oss.qualcomm.com, tingwei.zhang@oss.qualcomm.com, trilok.soni@oss.qualcomm.com, yijie.yang@oss.qualcomm.com, linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org References: <20251102-knp-soc-binding-v3-0-11255ec4a535@oss.qualcomm.com> <20251102-knp-soc-binding-v3-1-11255ec4a535@oss.qualcomm.com> <20251104-glaring-rebel-pillbug-a467ca@kuoka> Content-Language: en-US From: Konrad Dybcio In-Reply-To: <20251104-glaring-rebel-pillbug-a467ca@kuoka> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit X-Proofpoint-Spam-Details-Enc: AW1haW4tMjUxMTA0MDEwMyBTYWx0ZWRfX/dOOBEFpm40E 3ogkl8Hl0urxTAO+hBLzDBwBfc9fb91pIWtfudpG2YXxmO0N0Xx02Ffbk12WksIzKLZ9YCi6X5D 1ENYdS5xuFJRKI2aZ7ok1N88KXG1dC3PzQqNeiulngn2nyhe4uuDwXK9bPARiJk37Ktg7WSlmZO AIodxWGFscWlH/PYfAQOBnbTc8QhGWnoENp9kbyG4ILz7ORwBNSTRc5n7Kumq3NL7mthQK/qdby b4TeudKGujDmjzSxJQqEip/LTkJoScl9DZOOvH3jgSnbtLTQkLwY5I0pD+kqDoKyIFoyek49mXN Dr9QTh2DHz0OEWTds89tdskUPGP2TwnzIEaeDakd4AxPYLYLxVf6B8mKg39TY1i3IPJ178bj2E7 sBWSpOxiKOA2FNAPDBrWHq/S282FWg== X-Proofpoint-ORIG-GUID: OhuYjsM3o0MmyY-7iWfpcuzzDFFSIDJu X-Authority-Analysis: v=2.4 cv=CbsFJbrl c=1 sm=1 tr=0 ts=6909f245 cx=c_pps a=mPf7EqFMSY9/WdsSgAYMbA==:117 a=FpWmc02/iXfjRdCD7H54yg==:17 a=IkcTkHD0fZMA:10 a=6UeiqGixMTsA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=df4OXsTxo-j3LUg4bp0A:9 a=QEXdDO2ut3YA:10 a=dawVfQjAaf238kedN5IG:22 X-Proofpoint-GUID: OhuYjsM3o0MmyY-7iWfpcuzzDFFSIDJu X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1121,Hydra:6.1.9,FMLib:17.12.100.49 definitions=2025-11-04_01,2025-11-03_03,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 priorityscore=1501 spamscore=0 bulkscore=0 impostorscore=0 suspectscore=0 lowpriorityscore=0 clxscore=1015 malwarescore=0 adultscore=0 phishscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2510240001 definitions=main-2511040103 On 11/4/25 9:16 AM, Krzysztof Kozlowski wrote: > On Sun, Nov 02, 2025 at 11:25:06PM -0800, Jingyi Wang wrote: >> Document qcom,kaanapali-imem compatible. Kaanapali IMEM is not a syscon or >> simple-mfd, also "reboot reason" is not required on Kaanapali like some > > I do not see correlation. Something is not a syscon, so you add a new > generic compatible? No. > >> other platforms. So define a common "qcom,imem" binding and fallback to it. > > You did not define fallback to it! > > ... > >> + - items: >> + - enum: >> + - qcom,kaanapali-imem >> + - const: qcom,imem > > I do not understand what this generic compatible is supposed to express, > not explained in commit msg. Considering this wasn't before, it is a > major and really undesired change. It also makes no sesne. There was no > generic compatible before but "if not syscon" now this must have generic > compatible, what? So IMEM (or SYSTEM_IMEM more specifically as opposed to BOOT_IMEM which you can take your guesses what it's used for) is to the best of our understanding just a piece of SRAM that's accessible by multiple processors/subsystems on the SoC. A smaller region within it ("shared IMEM") is a little bit of a dumping ground for various (incl. runtime) configuration and debug magic data and that's usually what Linux is concerned with. IMEM is currently described as a simple-mfd+syscon, which it is clearly not. The former, as we've established in the past, was used as a hack to have something call of_platform_populate(). I think that in turn is only necessary for the old arm32 DTs which have a syscon-reboot-mode node under IMEM (and I think that's where the syscon compatible comes from). Should we make the switch to mmio-sram and settle this discussion? It would probably require convincing the sram maintainer to add that of_platform_populate() call in its probe func and making syscon-reboot not depend on a syscon (not like it's very hard) Konrad