Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Message-ID: <3a783b8d-4970-663f-816a-15aad5a66a48@nvidia.com>
Date:   Fri, 4 Nov 2022 15:46:03 +0000
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.2.2
Subject: Re: [PATCH v3 1/8] memory: tegra: Add API for retrieving carveout
 bounds
Content-Language: en-US
To:     Krzysztof Kozlowski <krzysztof.kozlowski@linaro.org>,
        Thierry Reding <thierry.reding@gmail.com>,
        Mikko Perttunen <cyndis@kapsi.fi>,
        Krzysztof Kozlowski <krzysztof.kozlowski+dt@linaro.org>
Cc:     David Airlie <airlied@linux.ie>, Daniel Vetter <daniel@ffwll.ch>,
        Rob Herring <robh+dt@kernel.org>,
        Mikko Perttunen <mperttunen@nvidia.com>,
        Ashish Mhetre <amhetre@nvidia.com>,
        Sameer Pujar <spujar@nvidia.com>,
        dri-devel@lists.freedesktop.org, linux-tegra@vger.kernel.org,
        devicetree@vger.kernel.org, linux-kernel@vger.kernel.org
References: <20220920081203.3237744-1-cyndis@kapsi.fi>
 <20220920081203.3237744-2-cyndis@kapsi.fi> <Y1aP7kDi8cJUTcGN@orome>
 <ee3073ba-d621-8b02-c986-3cf44cfbba96@nvidia.com>
 <b9b20abc-d37f-f74a-a416-2e210bf944c4@linaro.org>
From:   Jon Hunter <jonathanh@nvidia.com>
In-Reply-To: <b9b20abc-d37f-f74a-a416-2e210bf944c4@linaro.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Y0JNOEtuK3JHTlRNS08vUG1SSC9ZVW1BS2xHZTJWYmhLU0NPOTg2dmVwRkxY?=
 =?utf-8?B?VlMvRUFQdUErYW01Q1ZKUWRWMzNuS1lTb3FzeHc2YmsrakVmSXFKVzREeWdL?=
 =?utf-8?B?ZVdya21nRmRZdG1iNXhxM1V0YmRkSFlGUmlqY0lsNDR2NmZBWVhuODJjZVV4?=
 =?utf-8?B?MUQySHRkdklCSVdKMFlVR0JOVkc1Z29kVmo0ZjdPQzNXQVBLZ1UxTG9tbVl2?=
 =?utf-8?B?MHAxeERMdFBOOHJ4c29zb2djRXo1S2ZJRjRPRERxbWVTTEtmNXFWZFNCR001?=
 =?utf-8?B?aTBzSXVGdm01djgyMXBzUEVKenVXRDRhT0d6WC9qemlHeDM1c3BCUGlscGg1?=
 =?utf-8?B?a3YvWFZlaVdJdnlmaCtpTlY2aU5zMytSbEkycXVWc2JlOSt0Ynd6RXVvY1NP?=
 =?utf-8?B?dFVMZGpWaU1uenFtYzN6YTFNK2t1cms5T05KSFpiL0VSekpqelExa3RmTWxv?=
 =?utf-8?B?cGN1YXV5WFVRMStHNWRQRFZJZTh6TWJBL0RqSmRld3NFWEdIYnM3aDFFVkZH?=
 =?utf-8?B?VlpNVmNFSWljZWpwYzlIUlRyV3RtUHllQ0M3Uis5VkpCWFBVYUtBcmhyZEtW?=
 =?utf-8?B?c2tuVnhORTc3OHNOREtUeSs3eHNsc3E4aGFxTlluOEJPcm5majNQaGZuWFhq?=
 =?utf-8?B?YzlINkVZYjg3dHNKNlUzWFE4a056WjFiMElwUnhlZEdjazl4YXdSUVh3SlEy?=
 =?utf-8?B?azU2RDRRSzIwTmNqNE1NU0xZTzRPbnlxUnNIQ1JpZy8rT0JCTC84UkEvNXpH?=
 =?utf-8?B?czRxcVA1Qm5QNnNZYkJMZGtmdkp2dkhmTFhROWdwbnJ6azNGL3NIdTk4M3pm?=
 =?utf-8?B?ZGdOK0RaeStISzFwT0hWdEVQVDhSM2xKakYyTWtLZEdIWXdoL1hrak1TT2M3?=
 =?utf-8?B?MkRZSEYvZloydmpjd0hDZmhVbHFFNlZLa0Z2TzF5M3ZVV2NpQmhRSmdXVCtO?=
 =?utf-8?B?ZUp3ZUkrRWlROHBOa2g5UXp1d2hoR0JCZUFaREhvNk0wbVRCR3N5OGlGZlhV?=
 =?utf-8?B?NUpiWjZyeE1KZmNPTElaK21jVkxSNmxLelF4ZkZUSHRBYkFIZGxrNEJmcjAr?=
 =?utf-8?B?RUs1NC9MREhEWnQ3VVQ0ZUk4ZmF2a2dPbzRDOXk4ckt1UVdwaDdPRU5wMzRE?=
 =?utf-8?B?S2s0ZnB5SkZqcmg2bjRBUDJseFBrV2tRc3JnVVJtOGx2eVZZWnNtQ2V0dVc5?=
 =?utf-8?B?M2IxNDRqWU1jRmhTWXV4Qys1QTFsNEkvQnpyOGFnSmV6TFVYTEpXb3lqUVFO?=
 =?utf-8?B?VFRnZUxmNmFER0hOa3JHV05UblVBVFdjdDB3Q096ZTdZZnhzMWdqYi9Md3Iz?=
 =?utf-8?B?eVFzVU5BbUVRVTZZWHFGV1NPdURBTDJlU2RZblJYQ1hBNkxnSnZZcDh1N3Bv?=
 =?utf-8?B?ZFQzdUUyaWlOR2ZGVVFiQmRMbCtPdU1Dbm04Z01xWGh0b2hRSGkxenQwNXhP?=
 =?utf-8?B?VWhmdXU2b2hncDJiWG5JWnRRN3FLUmRaaWZGVEdiUlV4K2JWSXk3L1hhd1p5?=
 =?utf-8?B?M2YzNW50Z0NBTFFIbkQwekVLOHg5VUw3SElvMFN0R1RlSTZaSkozT3FwcDZK?=
 =?utf-8?B?RVNXaW8yclRtZ25XamZYUXRkZk5tQXdDRWhhbXRYV3I3UGkrV3pBZk55WUc1?=
 =?utf-8?B?UVJ5TkxieGhmdXUvQndyWTF1emNiWnpFa1NiZnFjWDRtNjB4SkRCUmloNHdV?=
 =?utf-8?B?anhVdnVsZnprRzdUWTVxcUJIb1hvNGNCOVVCVWhIVjhsZG0xWFI5dmNnUFpn?=
 =?utf-8?B?Ly9RR3EzT2o3dWF5UDBJNnJJbHU1MU1xbmlYemN0WWc4WEQzaTdSekRyS1Vr?=
 =?utf-8?B?bzAwTkVlV0d4ZDV3MGxTdlRUMVF3MUJ6dlNSUmhWdkhBa2ZvTUFFSmgydSt2?=
 =?utf-8?B?dmlPS25PbUVFeCt2eXZtSXNHY09vckxkaUZWdmEvUCt6cXpaRkFrMjZ6TGln?=
 =?utf-8?B?V1FUVTkrV3Q4VTFOT1BrT1BFWUhaa0pSZkt1MVhiMXkxSGptcmc0UUdTUXhU?=
 =?utf-8?B?Sjh6M2cwUFRLZStXM3hkWGRWTkxMMUZoSjJoaVpULzBGM3dZVVhyRWNvVGVo?=
 =?utf-8?B?K255OTB5OC9JWW4xMUtOcHMxS0dkUUlBVHhCZGhTVG84c0p2Q3ZmR1NhWTUr?=
 =?utf-8?B?N0MzRm5td2FHbHNubzFpTnNwOXZDKzk2UGxhYnpjVUhaNVJNcUV2TWREQW45?=
 =?utf-8?B?a0d1dXMxM2FBcGx5NlJVeURVendPV2I3dWEyb2NiUldMVms2cEVVSkdUcDha?=
 =?utf-8?B?VmhpV2pCQVdDYzlPZzRtQ05NdUdRPT0=?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 8bd8fdae-7ae6-4a5f-1e6f-08dabe7bb25d
X-MS-Exchange-CrossTenant-AuthSource: CO6PR12MB5444.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 04 Nov 2022 15:46:11.1860
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 43083d15-7273-40c1-b7db-39efd9ccc17a
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: 2Uj3tZt6cYzGb+GMtosy5p9XFTVJ1V/0cFiINE6Wg8ZDDyq3U1XNagxq1Jt9naSkHxT/DGQDl/8Mxp22Y8f27w==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY5PR12MB6552
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_HIGH,
        DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,FORGED_SPF_HELO,
        NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,
        SPF_NONE autolearn=ham autolearn_force=no version=3.4.6
X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on
        lindbergh.monkeyblade.net
Precedence: bulk
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org


On 04/11/2022 15:35, Krzysztof Kozlowski wrote:
> On 04/11/2022 11:33, Jon Hunter wrote:
>> Hi Thierry, Krzysztof,
>>
>> On 24/10/2022 14:15, Thierry Reding wrote:
>>> On Tue, Sep 20, 2022 at 11:11:56AM +0300, Mikko Perttunen wrote:
>>>> From: Mikko Perttunen <mperttunen@nvidia.com>
>>>>
>>>> On Tegra234 NVDEC firmware is loaded from a secure carveout, where it
>>>> has been loaded by a bootloader. When booting NVDEC, we need to tell it
>>>> the address of this firmware, which we can determine by checking the
>>>> starting address of the carveout. As such, add an MC API to query the
>>>> bounds of carveouts, and add related information on Tegra234.
>>>>
>>>> Signed-off-by: Mikko Perttunen <mperttunen@nvidia.com>
>>>> ---
>>>> v2:
>>>> - Add check for 64-bit phys_addr_t. In practice phys_addr_t
>>>>     is always 64 bits where this runs, but it avoids warnings in
>>>>     compile test.
>>>> ---
>>>>    drivers/memory/tegra/mc.c       | 25 +++++++++++++++++++++++++
>>>>    drivers/memory/tegra/tegra234.c |  5 +++++
>>>>    include/soc/tegra/mc.h          | 11 +++++++++++
>>>>    3 files changed, 41 insertions(+)
>>>
>>> Krzysztof,
>>>
>>> I've applied this to the same tree as the patch that uses it for now.
>>> Let me know if you want me to put this on a separate stable branch for
>>> you to pull in.
>>
>> Any update on this?
> 
> What kind of update do you expect?

Ha! I guess I should be more explicit :-)

Well, I would like to see this change in -next and so I was hoping that 
you would respond to the above to indicate how you would like to pull 
this in.

Cheers!
Jon

-- 
nvpublic