Received-SPF: pass (google.com: domain of linux-kernel+bounces-184845-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <40621383-dfd5-4438-8bfb-75edb92abba6@nvidia.com>
Date: Tue, 21 May 2024 16:28:49 +0530
User-Agent: Mozilla Thunderbird
Subject: Re: [PATCH 1/2] dt-bindings: dma: Add reg-names to
 nvidia,tegra210-adma
To: Krzysztof Kozlowski <krzk@kernel.org>, vkoul@kernel.org,
 thierry.reding@gmail.com, dmaengine@vger.kernel.org
Cc: jonathanh@nvidia.com, linux-tegra@vger.kernel.org,
 linux-kernel@vger.kernel.org, mkumard@nvidia.com, ldewangan@nvidia.com
References: <20240520122351.1691058-1-spujar@nvidia.com>
 <20240520122351.1691058-2-spujar@nvidia.com>
 <918ca28c-2498-4c98-a257-33e828134087@kernel.org>
Content-Language: en-US
From: Sameer Pujar <spujar@nvidia.com>
Autocrypt: addr=spujar@nvidia.com; keydata=
 xsDNBGYqYXcBDADHIGvXQYflJq+W1ox2LXyMVw8W+FMrjsPc+amt7am0SqOs++ujNsmxKUU5
 R1qfkCq6fdoyu6wivKJsVENYYuJxBzqwrD8JbgXdgio+ErjINbdgekRvrhgYLgR++MbqWHMz
 Qddn68X70uqj/DcCeYtZ9WfMkxfUjR6bdNZijkT0OSlNJ1GkJpEk+9vKj+C5CxqyBt7bn1xm
 bIU/Nc4wTfwKAeVB3CqeyuTjKJhQpayu3g5jxGx7ymZ89gikGSq42PukSC2C37N/+QLZM/lj
 YbIesiPsQklBvVG3KFXT215L+Y33SBNdHShVw0mx1V2KIrfaXmKgcTH9xV9BX1TuMTe+rOw1
 Qk93vhmdZzeXbfd6A9sIl2wv43xaiVRyvVEgwB2WI3juY6plNvpm+1xvx0cGKaFqKl/ZOi7H
 Nvoi5BnXPYDojIVEh3kI4CNWKDS5IvOKToygoiwStsTdIPUgTr/ZUwBZ2mqRSqG0k2jpXBsr
 /f8xFL4UZDQH/MShEDMeRS8AEQEAAc0gU2FtZWVyIFB1amFyIDxzcHVqYXJAbnZpZGlhLmNv
 bT7CwQ0EEwEIADcWIQSnDI1YueUy/3lua0zaG1jDPHbs6wUCZipheAUJBaOagAIbAwQLCQgH
 BRUICQoLBRYCAwEAAAoJENobWMM8duzrE0ML/jKV+rtJqc2ILKRnaU21nPRfVH5q/QetC3ZX
 uhVBziTJSJinFN/00Iz2DCVc6htkNtWDE006SH7HeoyBKyaScwFic74AHQylG+QCW7GkyY8N
 TFpbJ1jQpWeBQmB8fGVoT3rdBsOzwQCHD5TqICnj7Vgi4lqx+NgzUqrX0QTbtgyQC/UGPBWR
 zK2N4tK/ypcR8cYfarCpbkcTpBPnUOG+I1Xy11Hyqo44JCGazf8YBZLi45UaGctbqGZgxaKW
 E+KXTi996aSSNEjUaqdUslLnma1nINmG1zHGprmFFfxiuFKeke2iJ7YkSc5iTDWpTYO8EKws
 2xw1GbCspXYP81slNNYEQsbV+DHrJevzVW5fdUWLKkOtbnNoo9+D4r6VuhveXP4wpmo9ZjU2
 tneavy3uzt0E6HL69b0ZlQdHfu88trH+oNrnZ7tozTCPKawtkeeePajQnZ0pG4zWsJ5O59pU
 tgH4h0fMAp2NMY1dwv4UJK9fC8ouTcF8moIUs9RgyQBA487AzQRmKmF4AQwAwFFPRVrDpCWx
 kHk5ONNBqdbUu6M/SXh2U76NZp2BUb79dqlc0FF/lgKgvCDqSvgW69R+ET5vP8flfccd96Jx
 7GIVVBJ4WSurIgKpq1t07amWAR+21h37/XLUgbeEqEoyLsgvzpJ8cFH6spq3FvCB/zXTGCVQ
 KgJEkLrKdvMnu0s04cuZH1edM9VxYOMmJkm3JodOKUqgmwcrFcCWW9lSmLSiMnL1QNH3PpNz
 yeqLvuDvn7sohH6QNFfpP4gKLMyU1gRZERvjycbROnEhRAujV1sXyV0fRKpxRmvAnPQtQYNn
 6GzCsPP6XPjHFMxoKvnPBECfBoGeAzpsDV1/a9Eu9dVMe38ndtZYzKSidJfoFs4X5Au8+ieG
 NXCZMSWB85Xb2DAR2Qmsxe2KOOp+oKFE3WZS0dtdocWKysUVE4uxtSpaym34cq6N3XioHoez
 ze9zqcF8TSA0kOJVFJfcqmKdf+TzwQ3JeXRguD1OcpRRq4zEFO0r7kQ1cixh74xXlp2LABEB
 AAHCwPwEGAEIACYWIQSnDI1YueUy/3lua0zaG1jDPHbs6wUCZipheAUJBaOagAIbDAAKCRDa
 G1jDPHbs60FXC/9fCHL8/ZeP5ckL50CTkeiiY6yjWMvbtsr+B0lYXMz1ljPcGLExaqxUN8KP
 aQFcJQNR8npPDlQMBY39OlzbKXh+nIq9NVfbm0hrgOsBhtksXGFVOiKVQCXIOk/ntNT0NVpH
 iAmgfLPXBEzmiuoFWH+1XTCCfQOWtPFFuKfsqT3y0HJla6k/6+UV6jCD1d1Mlo8ricjfeW6h
 85+/dxdiiGPYPcVwa4c1iBbrD5RDkpHHNDCYsBvcweBwZu0b5T9wjPCba1K7TujQGT+ItQL7
 9UUdlMWj4GNVcRqNYdUJ3LPYefWymdM+W5/fF+QPrLHSxS4B4BBTjk90Hj/rIh0AdCpW4gko
 0G/IyfzJ26SaKmMk9SFYZBMthYgoEqDjxPfvFKZNB8q9zv34zQ5j73YbQjE6NctcpOBNRQIH
 5vZ6eC7Qly8qPznjRV9MF2DIT4D2J1s6ncJVcuULnSDqgNaayYmNsi1aZFQUWBYL0SIP0vFi
 tFh1Kv06N2eJoLklRzTKy6I=
In-Reply-To: <918ca28c-2498-4c98-a257-33e828134087@kernel.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit
Precedence: bulk
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0:
	=?utf-8?B?RFhxUFJQM0J3Mk9pdTBFTHFFUTBpUVBiT3lSVUZmRHlTN29EV3ZRZGEydldD?=
 =?utf-8?B?R1FabmVsMVN0b1M1a29hS3R1a0I4N0VtSHoyMjlMdGdNb3BTR1NBNjMxK2NC?=
 =?utf-8?B?bFhrOHF0emVoN0xRUXgyV3hMVWxtemEzMk1ETG9DUHF1Z21wTU93Q2E2SHFM?=
 =?utf-8?B?SWVoeHZIZmZ4NTdwSStSc05JS003THFrWEwxTmk1R3cwRlBEbGtGU1BiNmUx?=
 =?utf-8?B?RGVOQkozdHdRRmRLZDFKV2MwRyswTktPREtpeXZwejNvTTlyNEpGTmNaOWE5?=
 =?utf-8?B?QnVIbGZ2L21zZXRURzg3RVRBREYza1F0WXUzc3RFOTFpUTVRWTg1K1NLVXVZ?=
 =?utf-8?B?VnRaenZzZ25DTWVSSEp5ZG85RXliaElFRlJvWFBKQVhzK09DMmZqd2NkY0NW?=
 =?utf-8?B?VFRKdHNMc3IvLzdpVHYvU3RNR1lKQTZjS1NvMk13eWpjdnAyWEYyWWFEcjVh?=
 =?utf-8?B?UWQvd0loQXhoelJKOEdyTDM4b3UrRHlWY3poL2R6STZsZmZOSXpsdHV3a1NX?=
 =?utf-8?B?VG5NRlVtbjhwVnJ2dUozYTRrMDVLLzQ2MTNHaXh2ckFCT2loUWJBMEVnV2dh?=
 =?utf-8?B?dVdIbjdxbUR6NldsZXJpbEtTVkVWWlRuOWp2NnVGdkJDMVREbVdZZjcxeXJk?=
 =?utf-8?B?OW9XQjMvUGlhbkw4ZWw5NWFzS0ZOekFheGtGbWFJbFlncnNES2FySTNtbEE0?=
 =?utf-8?B?QXJLZENGMVZxR1cwOWVNSHRYdzcyendxbG9TWFZpc05mNU04K1RGMGZObHNj?=
 =?utf-8?B?SzMzbFR5cHdFV2NEbzVlVXFKV2FWaXhlZHFQOE9pUWhWekJ5enBudEx6cnNN?=
 =?utf-8?B?RFlEWm9IbmRNTHlaM3hFYzhJOWV4SkpzbkdlWFgyN0w5VkQ2MXBrNG1oVkx4?=
 =?utf-8?B?SGpJMUxMbGhEUERMSDBMRUlzRUNZODZMZzY1MlpCVHk3enhYY0Y0SkFHMmc5?=
 =?utf-8?B?eGxMWDMrdjhhS3ZVSXVKa01jMVk3K25xZkRBeC9BOHdsOFM1R05MVTBwdjBi?=
 =?utf-8?B?N3V1NUZUZ0V3aFFlSlpiNHJYbXVDQlg1NWRJTG9jcnU3dFFEazkrL295bUNS?=
 =?utf-8?B?dVZKRml2dWNQb0hneWRoeEtSQ2VQQTEzUkR3MnVvbjBvVUI2MEJRK2lFQXN6?=
 =?utf-8?B?WERRektQKy8yMTJtTFd1NHJkWFBESG85TUlwSzdZbmpYWFZxV3JtUlQwUStB?=
 =?utf-8?B?MG8wMlljSTdXaVU1a1JQVE8zdGh4aTM1dTBJTkZ3Z3ZOVlQweXFOa201cVpI?=
 =?utf-8?B?K1E2NVB4VTFSLys3K2c4Z285aHpmRUtPb1lEZUNVdkdNZjdLYjZWOEQxVmVy?=
 =?utf-8?B?bW9leFU4c0owY29jOGgwRWNrZEpMTzlIenhOU3lqR0lVbkU2QU8rRG5DanJm?=
 =?utf-8?B?UlpYU3NKQjlhM3RBcUFVVWNzMUdHenY5V1RXdWhJZGdrdi82QmF0dFhIaDRp?=
 =?utf-8?B?YkJEVFpBYkF2d3RPMUdoYnNoWHVGUXdqNzM2QmR4Snp0T1pwYkQwS1A3WElW?=
 =?utf-8?B?Q2psYVVXR0tBdWJWSkVZL0NONUJIcE1yUWlaa1FYWERqZ1hnbkcxam5IOGJk?=
 =?utf-8?B?M0ZIc2Mya1U4RFJpbzFQLzErUkVPWm56a3lVRnB3RVpLbk41UExXd0hJNWdK?=
 =?utf-8?B?MnV0K2s1U2tLNkYxaTNway9hQ3FzdXIzaGJNVGNVOXpKQkdjbTdNZ2JWQmxm?=
 =?utf-8?B?SEhZc01sUTliQ2ltY3hsYWlvRmZYR3doOTFzZ2ZqUlRZeWJhNUY3ZEtFRGJS?=
 =?utf-8?B?R081Q0xSQkVhb3p6d0RsSkJHajVMU05hdzNFSkQ4bnFUQXFxMnhkYW9jZkJp?=
 =?utf-8?B?QVhjME5uRnVLWWgyRm44eWtnM1g3NHFOV0laMUJmNDladEZNK3JrSUNzV2Vl?=
 =?utf-8?B?T0lMQkRMM1RwKzVwNHA4RlhvNFZGYTBKd1ZyaUIxRjc4SEEyZUZWNitrZE9j?=
 =?utf-8?B?R3lzUy9nU2ZNd0Mydm4yQVVuNE81YURGdmZwVStQK3JGQWV6VzlrWHlYMExF?=
 =?utf-8?B?OXVlejBuMEd5YkZBN0xCYzYyVnRaZzJtVmpqcU5RejdneUFDcGFBSzQzWlFm?=
 =?utf-8?B?bWZrOTV5MG5kT2FZTkplcTBqekNHem5XNEF6eXhDa0pMSDluZVVnbThYTERu?=
 =?utf-8?Q?Aac7OmCCq5URH09Y/zpQ8lka5?=
X-OriginatorOrg: Nvidia.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 4b3e1cfa-4577-4c57-5ebe-08dc798502dd
X-MS-Exchange-CrossTenant-AuthSource: CY8PR12MB7706.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 21 May 2024 10:58:56.8959
 (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: kE5VAxSs6fPfUdsFLFr1sthU7prjT0DXWFVacTSofQuNtY+SmV7rvezUuGmrwEdBjaKCH3M1/krTDmx9UUhpaQ==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: CY8PR12MB8241


On 21-05-2024 14:54, Krzysztof Kozlowski wrote:
> On 20/05/2024 14:23, Sameer Pujar wrote:
>> From: Mohan Kumar <mkumard@nvidia.com>
>>
>> For Non-Hypervisor mode, Tegra ADMA driver requires the register
>> resource range to include both global and channel page in the reg
>> entry. For Hypervisor more, Tegra ADMA driver requires only the
>> channel page and global page range is not allowed for access.
>>
>> Add reg-names DT binding for Hypervisor mode to help driver to
>> differentiate the config between Hypervisor and Non-Hypervisor
>> mode of execution.
>>
>> Signed-off-by: Mohan Kumar <mkumard@nvidia.com>
>> Signed-off-by: Sameer Pujar <spujar@nvidia.com>
> Please use scripts/get_maintainers.pl to get a list of necessary people
> and lists to CC. It might happen, that command when run on an older
> kernel, gives you outdated entries. Therefore please be sure you base
> your patches on recent Linux kernel.
>
> Tools like b4 or scripts/get_maintainer.pl provide you proper list of
> people, so fix your workflow. Tools might also fail if you work on some
> ancient tree (don't, instead use mainline), work on fork of kernel
> (don't, instead use mainline) or you ignore some maintainers (really
> don't). Just use b4 and everything should be fine, although remember
> about `b4 prep --auto-to-cc` if you added new patches to the patchset.
>
> You missed at least devicetree list (maybe more), so this won't be
> tested by automated tooling. Performing review on untested code might be
> a waste of time, thus I will skip this patch entirely till you follow
> the process allowing the patch to be tested.
>
> Please kindly resend and include all necessary To/Cc entries.

Sorry about this, I will resend the series. Thanks.