From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mail-ed1-f48.google.com (mail-ed1-f48.google.com [209.85.208.48]) (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 0C4861F4717 for ; Thu, 18 Sep 2025 09:47:43 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=209.85.208.48 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758188865; cv=none; b=RwEfmffbGvbHcUyybB81SIcTQwtIud3YgHfmzxFY+xjeENKIx1XtShfDm++3d+wx8nrNWBCz9dPaXBeb04kimzwnrh/Y7e2BBX2PMOdWlbDJeurQRuxSNb1jWork98xZwhY/abMNS5s8ftLfj9rIuR5hbZn/AuWpE2titA6DtNU= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1758188865; c=relaxed/simple; bh=ebIejkG9lO3H+4v19Kd3NsOO4t6uOqMYtPLh1XK6HPg=; h=Message-ID:Date:MIME-Version:Subject:To:Cc:References:From: In-Reply-To:Content-Type; b=h08HeKbB5X4ZJ8kg9JSn/zvKgeJY0qewQ8gktpoB5NazaKYr3tk/Q70brsysYn2lSE3dOfFk39zm4RPDiI1Wf9caQI8Ttz2na58rMyswZvpj4GJRJpRfkds+diHv1J+UMvtAXqKvbIV7OR21gkyqMqZB1eZ9SxMwm1c5EI/4GNE= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev; spf=pass smtp.mailfrom=tuxon.dev; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b=Ybv3BI+6; arc=none smtp.client-ip=209.85.208.48 Authentication-Results: smtp.subspace.kernel.org; dmarc=none (p=none dis=none) header.from=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=tuxon.dev Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=tuxon.dev header.i=@tuxon.dev header.b="Ybv3BI+6" Received: by mail-ed1-f48.google.com with SMTP id 4fb4d7f45d1cf-62f1987d547so933933a12.2 for ; Thu, 18 Sep 2025 02:47:43 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=tuxon.dev; s=google; t=1758188862; x=1758793662; darn=vger.kernel.org; h=content-transfer-encoding:in-reply-to:content-language:from :references:cc:to:subject:user-agent:mime-version:date:message-id :from:to:cc:subject:date:message-id:reply-to; bh=xY2fN2zgk9K5ePhnAH7PE+WNn7DSzrHO8ylDHhqkcZ4=; b=Ybv3BI+6pNabav/XvdmX6XIsppQ0M/5IxuGMMtXGZgPUwY0Lnd/MprI2Pk+tOS7kbu w2fp5utdwqax5a1S9sBlkkn7oAbBk1ju3xCZ0EHQv/lYLwEjgbeUIgxZIxTOhXj9nefr qPiv2mCwuGuJI9ezJUK6Xo9K1uZ9Ae08ij4XxyXDeSwiFt/BW9DxV3KC2zA3DKvDrGI6 N6nUXPpEtXLqIsyy3Q7F6qLx0OuqWg8OE6X6flkiKAoV4oFhP5E9C0Tvw9SvRzp8cluG yxv9j7zqAIQmCBkecM0MwzPVezr/UPQoij/GCYGfDzWnDgeb+mBWcoO4AzzC1oMpIkT4 cJMQ== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1758188862; x=1758793662; h=content-transfer-encoding:in-reply-to:content-language:from :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=xY2fN2zgk9K5ePhnAH7PE+WNn7DSzrHO8ylDHhqkcZ4=; b=f9BsqNZPwOzT8GmKGcbYLmC+zfo1OA8pydPZU+97mjY2ZjDjSFG75F2Txez1DUuuq0 bfpTZn6qPrrkQXtXnD0TPk6v85LBZks64yS3a9R60I8A4+V8CjS0Oeo4ZqmMcvJHYCJd +wsZWXkViBJp+Mzb2JGbJqaoJZPC3/GnolQAwK+zBkmblfRRdIf4cn5C/8Mlgo1m++be hsrFafcgzzD6pUVuPD31xScZx82jAYTWWfdB4jUncMt0r2r2bbHuxPQqI3SysXIWKPRS Fr1n0GGzUOS06SJKIumwVqIkHqA6hN9gyQ+lfqjbLn5AbupWQtMeXCnC80zVlMAY29CW 8wBQ== X-Forwarded-Encrypted: i=1; AJvYcCV7IvNOwqJNwPtj1O22TJzbwvrwzaW2WEgnpX28IWO5AV5/1GzPsh17o/WpYmPpeIKoVqdPYineMbJB@vger.kernel.org X-Gm-Message-State: AOJu0Yw4DuPQQ2WPyOah4p5XJVckZvOz8OY198Yod6+ZRFxexUMQTWyh +C6s7rA2TNmOTNu9b1Wl9seghHBL/nAvG3VXyuxtTnA+wMwBVMKqdCDGl0yeisH3Gcs= X-Gm-Gg: ASbGncvrlIYyDvelGjre3/DeAZr1oq8yOQ21+sZY1pLBqeYP0v9h2rP4kyFGPlgvj5a ygb0MknOGYNjvbAPLJxahE249lR4u0vws+cF3yWVMgCVl3FkQU50bvUjfisifBYWBeUIXQbktZK pz96OINhO6mZexrvvcqU+gH0iqf3r6xKv9OflUXZFPtpAu6G2DGcLu7Z5zq3emkXwy29zl1nf8Z ktbiDffnXPzECVnQ/b7tdQkCVZFfZvC3vB0TV8FN2JpKavkRrUGV3oRwCQQFWFrDkESn2iZ9/26 hbGLMsRJkJWjRjmUzcYLvXKmaY7y/gAIocLYX2FFt3wEVRirBVu4GBa2fFcLzBW1lha2PzX2pJy BUga8ORPjWP/8ndo2/JCdDh9eoUvsFb/I+xsG3xIj9EGpvLcciqz3jipXKS6Q X-Google-Smtp-Source: AGHT+IGm1gMecKNS7ulsOyutPp7cC/3sMrnG/x/HjeoFQdZsaPx3dh9Tz2I2uksJ4hOSdxJlyIcHBg== X-Received: by 2002:a17:906:730d:b0:b07:88ae:4b80 with SMTP id a640c23a62f3a-b1bbb7615cdmr515429166b.65.1758188862248; Thu, 18 Sep 2025 02:47:42 -0700 (PDT) Received: from [192.168.50.4] ([82.78.167.153]) by smtp.gmail.com with ESMTPSA id a640c23a62f3a-b1fd15cb2fesm161684766b.89.2025.09.18.02.47.40 (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Thu, 18 Sep 2025 02:47:41 -0700 (PDT) Message-ID: Date: Thu, 18 Sep 2025 12:47:40 +0300 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 v4 4/6] arm64: dts: renesas: rzg3s-smarc-som: Update dma-ranges for PCIe To: Geert Uytterhoeven Cc: bhelgaas@google.com, lpieralisi@kernel.org, kwilczynski@kernel.org, mani@kernel.org, robh@kernel.org, krzk+dt@kernel.org, conor+dt@kernel.org, magnus.damm@gmail.com, p.zabel@pengutronix.de, linux-pci@vger.kernel.org, linux-renesas-soc@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, Claudiu Beznea , Wolfram Sang References: <20250912122444.3870284-1-claudiu.beznea.uj@bp.renesas.com> <20250912122444.3870284-5-claudiu.beznea.uj@bp.renesas.com> From: Claudiu Beznea Content-Language: en-US In-Reply-To: Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Hi, Geert, On 9/18/25 12:09, Geert Uytterhoeven wrote: > Hi Claudiu, > > On Fri, 12 Sept 2025 at 14:24, Claudiu wrote: >> From: Claudiu Beznea >> >> The first 128MB of memory is reserved on this board for secure area. >> Secure area is a RAM region used by firmware. The rzg3s-smarc-som.dtsi >> memory node (memory@48000000) excludes the secure area. >> Update the PCIe dma-ranges property to reflect this. >> >> Tested-by: Wolfram Sang >> Signed-off-by: Claudiu Beznea > > Thanks for your patch! > >> --- a/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi >> +++ b/arch/arm64/boot/dts/renesas/rzg3s-smarc-som.dtsi >> @@ -214,6 +214,16 @@ &sdhi2 { >> }; >> #endif >> >> +&pcie { >> + /* First 128MB is reserved for secure area. */ > > Do you really have to take that into account here? I believe that > 128 MiB region will never be used anyway, as it is excluded from the > memory map (see memory@48000000). > >> + dma-ranges = <0x42000000 0 0x48000000 0 0x48000000 0x0 0x38000000>; > > Hence shouldn't you add > > dma-ranges = <0x42000000 0 0x48000000 0 0x48000000 0x0 0x38000000>; > > to the pcie node in arch/arm64/boot/dts/renesas/r9a08g045s33.dtsi > instead, like is done for all other Renesas SoCs that have PCIe? I chose to add it here as the rzg3s-smarc-som.dtsi is the one that defines the available memory for board, as the available memory is something board dependent. If you consider it is better to have it in the SoC file, please let me know. > >> +}; >> + >> +&pcie_port0 { >> + clocks = <&versa3 5>; >> + clock-names = "ref"; >> +}; > > This is not related. Ah, right! Could you please let me know if you prefer to have another patch or to update the patch description? Thank you, Claudiu > >> + >> &pinctrl { >> #if SW_CONFIG3 == SW_ON >> eth0-phy-irq-hog { > > Gr{oetje,eeting}s, > > Geert > > > -- > Geert Uytterhoeven -- There's lots of Linux beyond ia32 -- geert@linux-m68k.org > > In personal conversations with technical people, I call myself a hacker. But > when I'm talking to journalists I just say "programmer" or something like that. > -- Linus Torvalds >