From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-io1-f44.google.com (mail-io1-f44.google.com [209.85.166.44]) (using TLSv1.2 with cipher ECDHE-RSA-AES128-GCM-SHA256 (128/128 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id A51AC81749 for ; Fri, 23 May 2025 17:59:27 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.166.44 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748023170; cv=none; b=J6mtjQ1AtZ75t+PHQ+03ewPzs9OMrh3ouAf7uIRqLgNzG/OyAmwjaTtNTosDfmVUQgAs+50NxBnpq3+w2wP4hiz6ltjDczRuoJy3zr9CJKfR2EGa2hkzvZcnDzCt1fTn+Cg1XtolUMfxytrpO5pQwc+pg8bRHiT68hhF6x3YeR0= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1748023170; c=relaxed/simple; bh=xLtmQ0TWKB+NkLXhdkRYKl8da1Hyt1mfZQezqD35yEs=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=rbYumN2N+KewT0j3JPymqVMP3XS3VYQjb2nH7ogl6xU5GkP/AoLH2T0m+WCIdW/w6xZiidqjl/GHh2nrhgf3a5aPb0aUELF+TRTvecKPAIedBc2kzp8CvpU2rfwgbZLNjeYHn3YifDD7bzvPTB3ghLSy5O/C94pvB+IxDFv0YL0= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ieee.org; spf=pass smtp.mailfrom=ieee.org; dkim=pass (1024-bit key) header.d=ieee.org header.i=@ieee.org header.b=C8N9LT75; arc=none smtp.client-ip=209.85.166.44 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=quarantine dis=none) header.from=ieee.org Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=ieee.org Authentication-Results: smtp.subspace.kernel.org; dkim=pass (1024-bit key) header.d=ieee.org header.i=@ieee.org header.b="C8N9LT75" Received: by mail-io1-f44.google.com with SMTP id ca18e2360f4ac-86a464849faso6877239f.2 for ; Fri, 23 May 2025 10:59:27 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ieee.org; s=google; t=1748023167; x=1748627967; 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=SGtEFmZ2rVp0Bhcu1O5oY42icjo5BCFEnZcJ+QVCi5k=; b=C8N9LT75+cLuq9p0NKlfwoqcK2L0qV4LlVGpw4e+WxnSPyUezwE8GlWCZgm9bqcXTM ckisH+OgjZMQcmh1GS3A3piIuoyHnt5QWCQNRQwnh/BPWTReT8wDzXK0rXaHqX5u8hhN 2N3x/NvQKLYWfQxd/km41ADGnFffl8teWi0l8= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1748023167; x=1748627967; 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=SGtEFmZ2rVp0Bhcu1O5oY42icjo5BCFEnZcJ+QVCi5k=; b=bnp9qBIL5HWQ794/4lfh0b+MxdwadmVCk6ERh5YdjQmwfFQ0uIMTaRCSiPsfMkivo7 CcfPmCqYtY9VFTshbNuF/eayMj86oDeUQNk8gnP3Mf+4a3Cddc+axz89qyAMcqWZLRlf GpWZfTWDoNEjHCxkE7288n+0NFVh+LRaFHsvXOJAmKl5OwBmOHKOdtGExSd/XqNf/11T /mFantTTSNPVDVOxwdxVuyiMs1BoOlyy6v80TerCJybvNBznhfpjydyDzdrdVz7FNJnF it4tVI991V58VvYJ+oZO+EGykzsnPceQRJAVtMh5W1PVALMT0LVTwfk5GQr9mjGKl30t FLQA== X-Forwarded-Encrypted: i=1; AJvYcCVC/p0O/pJxhZFS/IiHCu/EijQ2mAJFOB8iZQ/N56oZK6yOrrHFkcdZo6H7rauyxgSFlzihtQVl8nah978=@vger.kernel.org X-Gm-Message-State: AOJu0Yy43ED1xJQerSYM1tILKIfVGr9hngukThuH+0i2ozqwY3IqO8op yu3cARQWrybt0bMYOvxkTt8/lQszIOe1xCc+m0y3HdrkF66ggJFD++pbA58KfTKxGQ== X-Gm-Gg: ASbGncujPqEyKVsxcSo5ijd0aim3/PBgIK5EhL2BCSzD3j2if3dYoojlknfA0hQC/1y JelPkkE3F7EAqCMATuE7yL/+1hZL1NqZM4QEuERc050SBvzmoUw1m2244cmtYCPojNsaKtSEATo 6hdo2nTOQyESuIv/RJ6YKjdKfsi3ntiGZPW9oZ1nvcbIY1R1XkSf7CWT6JDBeCADJgxeZKI5eUo Xs70E694NricLaX0LtAeoy/zHSkrk0I1zb3Uor5FIUEz1htSe7AFVBK1ZTYUx92dbq/R1gpDLX7 y6jk2MuJY8bzm6vwHeVCs18rmvtNU73DoRfc4BswPoso54oA0PhZ3SE/xS61byHk4YNDxLuljjk LIBZ053aDHFHHS3sruHS3 X-Google-Smtp-Source: AGHT+IEpb1IcvOHrK/fzMraPZSR5ZnUsdHuhgqtnR1Igx7QRkPEVOT3iAbQKQTTUDJaVKlOxTZqGag== X-Received: by 2002:a05:6e02:3308:b0:3d8:2023:d057 with SMTP id e9e14a558f8ab-3dc9b6cc069mr1052295ab.11.1748023166709; Fri, 23 May 2025 10:59:26 -0700 (PDT) Received: from [172.22.22.28] (c-73-228-159-35.hsd1.mn.comcast.net. [73.228.159.35]) by smtp.googlemail.com with ESMTPSA id e9e14a558f8ab-3dc7f126188sm18218695ab.65.2025.05.23.10.59.25 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Fri, 23 May 2025 10:59:26 -0700 (PDT) Message-ID: <7707b574-6fcf-487d-909a-d24874f9d686@ieee.org> Date: Fri, 23 May 2025 12:59:24 -0500 Precedence: bulk X-Mailing-List: linux-kernel@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 User-Agent: Mozilla Thunderbird Subject: Re: [PATCH 1/3] dt-bindings: sram: qcom,imem: Allow modem-tables To: Konrad Dybcio , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Bjorn Andersson , Andrew Lunn , "David S. Miller" , Eric Dumazet , Jakub Kicinski , Paolo Abeni , Alex Elder Cc: Marijn Suijten , linux-arm-msm@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, netdev@vger.kernel.org, Konrad Dybcio References: <20250523-topic-ipa_imem-v1-0-b5d536291c7f@oss.qualcomm.com> <20250523-topic-ipa_imem-v1-1-b5d536291c7f@oss.qualcomm.com> Content-Language: en-US From: Alex Elder In-Reply-To: <20250523-topic-ipa_imem-v1-1-b5d536291c7f@oss.qualcomm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit On 5/22/25 6:08 PM, Konrad Dybcio wrote: > From: Konrad Dybcio > > The IP Accelerator hardware/firmware owns a sizeable region within the > IMEM, ominously named 'modem-tables', presumably having to do with some > internal IPA-modem specifics. > > It's not actually accessed by the OS, although we have to IOMMU-map it > with the IPA device, so that presumably the firmware can act upon it. > > Allow it as a subnode of IMEM. > > Signed-off-by: Konrad Dybcio So this will just show up as a subnode of an sram@... node, the way "qcom,pil-reloc-info" does. This is great. Is it called "modem-tables" in internal documentation? Or did you choose this ominous name? Reviewed-by: Alex Elder > --- > Documentation/devicetree/bindings/sram/qcom,imem.yaml | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/Documentation/devicetree/bindings/sram/qcom,imem.yaml b/Documentation/devicetree/bindings/sram/qcom,imem.yaml > index 2711f90d9664b70fcd1e2f7e2dfd3386ed5c1952..7c882819222dc04190db357ac6f9a3a35137cc9e 100644 > --- a/Documentation/devicetree/bindings/sram/qcom,imem.yaml > +++ b/Documentation/devicetree/bindings/sram/qcom,imem.yaml > @@ -51,6 +51,9 @@ properties: > $ref: /schemas/power/reset/syscon-reboot-mode.yaml# > > patternProperties: > + "^modem-tables@[0-9a-f]+$": > + description: Region reserved for the IP Accelerator > + > "^pil-reloc@[0-9a-f]+$": > $ref: /schemas/remoteproc/qcom,pil-info.yaml# > description: Peripheral image loader relocation region >