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: <07d92fbf-7573-dde0-958e-20845b8b1089@amd.com>
Date:   Mon, 9 Jan 2023 16:07:49 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101
 Thunderbird/102.4.2
Subject: Re: [PATCH] drm/amdkfd: Use resource_size() helper function
Content-Language: en-US
To:     Deepak R Varma <drv@mailo.com>,
        Alex Deucher <alexander.deucher@amd.com>,
        =?UTF-8?Q?Christian_K=c3=b6nig?= <christian.koenig@amd.com>,
        "Pan, Xinhui" <Xinhui.Pan@amd.com>,
        David Airlie <airlied@gmail.com>,
        Daniel Vetter <daniel@ffwll.ch>, amd-gfx@lists.freedesktop.org,
        dri-devel@lists.freedesktop.org, linux-kernel@vger.kernel.org
Cc:     Saurabh Singh Sengar <ssengar@microsoft.com>,
        Praveen Kumar <kumarpraveen@linux.microsoft.com>
References: <Y6TI1O9adSP/OrnT@qemulion>
 <Y7nRdaDtptxE8Rwd@ubun2204.myguest.virtualbox.org>
From:   Felix Kuehling <felix.kuehling@amd.com>
In-Reply-To: <Y7nRdaDtptxE8Rwd@ubun2204.myguest.virtualbox.org>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 8bit
MIME-Version: 1.0
X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1
X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?YlRQUlg4ZkhlbzJHcllVQjljdDFiemlHME9uS0pzaDRLQ3RYdW5RT21HOEUy?=
 =?utf-8?B?dVR5WWVUL3RGRHJ1anRjdFBUOUswNE15c0hyUGlOU0VEK2xicFpXSWo5UVl0?=
 =?utf-8?B?VDl5NXl0NzNuSTNja2kxbmhnVEhIamJZTlBwWFlYbllFT2lhd0g1SHRobnlo?=
 =?utf-8?B?ZlZ2cmo5d0U3OHp5YXl0RjFSUzNEUG9jKzBnOFVkUVUrY0tHK1pTZ3hHbjNa?=
 =?utf-8?B?RzROWURYbFU4WVpJNmtRc1huVTdLa0tnVFUwcFF6N3FuYlF2UUNZVzJoU1JU?=
 =?utf-8?B?blZ1Qmx0UVVZbGFIT1VVY2RaSk5aNDFUSmxQUmMvUmRTdk8yQXZaRGZsNVJi?=
 =?utf-8?B?UmRSVTFsbVY1bTJaa0NSeTFWUW5RNVdia29lVEJkSnlWQ3kzbGdiNlJiU0lI?=
 =?utf-8?B?eTh0L084bXE4VkJ5YTlJbzZsSmV0bXFQOVZmQUNCZVhyYysraGh1eGprdUhG?=
 =?utf-8?B?T3R0TTdkbndiY0tJV3hPT0pDQzdaUUVQb1laY2lBVkNIVDdmU1RUQ0xRNDIr?=
 =?utf-8?B?dnE3NVkwZHNjaGpieXFNOTNodkhuUWIxSnYzSWZzR05WL3ErUkpLcWJacmZ4?=
 =?utf-8?B?YnZZdDFMWVJEd0dPQng4UnB0NElFaFMyUk1YYU5GNDJvekhiSER5QWNqVkkr?=
 =?utf-8?B?c1plTnJ5ck5haEd1Rzh6aUZhYlE5ZTdNWDRJOVZvRG5sWG1DQndDTkw0a1R6?=
 =?utf-8?B?WWR1dHBiTWZYZDlERzRSRjNIWHpxV2lxVkRNUW41d29UT3NqSk42WEtqTDhK?=
 =?utf-8?B?NVl1MDl2UTZtV1l4NFlsYzRocXp3dGlaY2wydWxtM1NCbTZTYVY1RWQ1czlK?=
 =?utf-8?B?Ukh1QTFLbTdOTmVqZnRjQUlIbU9oQmg1RWYySE5NMk9mbmt1N09QQmtla3ZQ?=
 =?utf-8?B?WVNjNHR4M3pZODJONmtBa2RMaHY3MU0zSTBpT2ViM2hXUmcwMmJ1SmxtM081?=
 =?utf-8?B?ZUFva2ZlNHdVRTNNWkl0NlczYTg5NUN0S3pZRVhJUHQ1QXVnOW1QN3pNaDVj?=
 =?utf-8?B?dzRMbS80czg5bXVhQThBS0NoK3RzUnY5UVZXOEZJT2MzOVNHUlFQeFIxdnUy?=
 =?utf-8?B?NjdVZlROU0dlVHQ1OTBJL0NkNDBVTHYxNExuekhsSjExMEhxMDJWSlFJSTF1?=
 =?utf-8?B?L0hSZDZNRHdMSnpxeFg3cDhIUys3b200aHJFUFc0bi8wU01SMVpXRXZHc0M4?=
 =?utf-8?B?TjBJYUk0b25vV1hqbzhKUWQ5dklNSlhIWFUybG9iTndHMVpyaUZSNGxxay9L?=
 =?utf-8?B?dWJhNzlJQXRJTktzSUZDdEJmR2xsemNUZGg4N2lCZnkrNW9sZGxrZHNPVWxv?=
 =?utf-8?B?cWRxaE9QZXAxeU9QTmJsMzN2YlBNU204WUNPeXpDbVM5anlZeE5ucFY0WTR3?=
 =?utf-8?B?WHFwbXZqV2dkM1BRMlpBUFVmeVZsSW96aTI1RncxWWgrMDVsaEpmRjZzNFhB?=
 =?utf-8?B?VDFFNEdOU1VsRTdIQXdES0JsVjZlSGU5ZUlqUk5UWVptL1RLY3ZucUI4Vm9p?=
 =?utf-8?B?d3lZTS9Da1ZKeE5jSGNJTFhFd0t6SVRUNDZsMm5ML0VOL2dsV0FkU1lLQ2w3?=
 =?utf-8?B?ZmZONTh6eWlKNFNDWFlKRm03bHlzd2VmdzBjWnorSWhhempWckdyR1o2cjVm?=
 =?utf-8?B?UlIwRXNVSmtpSEpLTDNBQTB2V0pJbDVFdTBRbnBNL2pxL1BITEJ0cjNBVzBQ?=
 =?utf-8?B?K0J5VTN0STQ4RVZVVUJwS0FKa3hWdnYyTU4wUkdFOVZUWlFPam93TXFLMytP?=
 =?utf-8?B?TGZVekpJOEF2SUdDVWhBcDRvWHZNNEd5QVhWU0ZMQy8ySEJrWk9TcSs4bkEy?=
 =?utf-8?B?NnNqcnZwM01weHcwb0w0OW1aV2FsdlJRbURSMUNkWk01V0Y1eFMxc0tyaHJp?=
 =?utf-8?B?eW5FS0V5ejNtUXc1YXY4YUdRVU5oOU90dlpFMTFubnVZalJlaFZPV3B6M1dF?=
 =?utf-8?B?STVpL0t5OGgwSUxoQjdNOEsrbTczR3AwYWpPSTd2NXhTdTYrRThRUE05ZE5D?=
 =?utf-8?B?K1BnN2lvLzd1aDhwdzVOVUpvQUpLcmhPRmg5b2MzMFp1NkJPVmtOV2pnSk1z?=
 =?utf-8?B?YVlHc3RtTnE3emNVbmJPTDNOL21LNjVqOHFaYzR1d1B4azM0czg5MnM3U0ZF?=
 =?utf-8?Q?mxzy61/KXOu2x3m0FgVxktYG2?=
X-OriginatorOrg: amd.com
X-MS-Exchange-CrossTenant-Network-Message-Id: 94801a28-4d12-4d72-8a4d-08daf28591d1
X-MS-Exchange-CrossTenant-AuthSource: BN9PR12MB5115.namprd12.prod.outlook.com
X-MS-Exchange-CrossTenant-AuthAs: Internal
X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Jan 2023 21:07:51.9222
 (UTC)
X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted
X-MS-Exchange-CrossTenant-Id: 3dd8961f-e488-4e60-8e11-a82d994e183d
X-MS-Exchange-CrossTenant-MailboxType: HOSTED
X-MS-Exchange-CrossTenant-UserPrincipalName: rR1Kp3xEzOlu5kQ7Rvq30bDeDX6YbjOouwrVMZesy0HJyGkxpMZ5emZOxokYXUg5sjSL/IRlT5pEyEEu6Fpv+A==
X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7PR12MB6767
X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED,
        DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,RCVD_IN_DNSWL_NONE,
        RCVD_IN_MSPIKE_H2,SPF_HELO_PASS,SPF_PASS 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


Am 2023-01-07 um 15:09 schrieb Deepak R Varma:
> On Fri, Dec 23, 2022 at 02:45:00AM +0530, Deepak R Varma wrote:
>> Use the resource_size() function instead of a open coded computation
>> resource size. It makes the code more readable.
>>
>> Issue identified using resource_size.cocci coccinelle semantic patch.
>>
>> Signed-off-by: Deepak R Varma <drv@mailo.com>
>> ---
> Hello,
> Requesting a review and feedback on this patch proposal?

I applied the patch to amd-staging-drm-next.

Thanks,
   Felix


>
> Thank you,
> ./drv
>
>
>