Received-SPF: pass (google.com: domain of linux-kernel+bounces-52772-linux.lists.archive=gmail.com@vger.kernel.org designates 147.75.48.161 as permitted sender) client-ip=147.75.48.161;
Message-ID: <f27d7b45-89d9-4b5c-9bde-806b57bbe8e7@arm.com>
Date: Mon, 5 Feb 2024 14:00:57 +0000
Precedence: bulk
MIME-Version: 1.0
User-Agent: Mozilla Thunderbird
Subject: Re: iommu/ipmmu-vmsa: Use devm_platform_get_and_ioremap_resource() in
 ipmmu_probe()
Content-Language: en-GB
To: Markus Elfring <Markus.Elfring@web.de>, iommu@lists.linux.dev,
 kernel-janitors@vger.kernel.org, =?UTF-8?B?SsO2cmcgUsO2ZGVs?=
 <joro@8bytes.org>, Will Deacon <will@kernel.org>
Cc: LKML <linux-kernel@vger.kernel.org>
References: <e01cdb99-8497-4fec-b423-a0bfb483ac75@web.de>
 <324d4e02-6a5a-4112-a3a7-d7aeb5876acc@arm.com>
 <49fc6a59-2c07-4366-b32f-0599c2418916@web.de>
 <51315925-21ac-427c-abea-4d652ed5280f@arm.com>
 <0c757d69-ec60-477c-a978-a94118a571a2@web.de>
From: Robin Murphy <robin.murphy@arm.com>
In-Reply-To: <0c757d69-ec60-477c-a978-a94118a571a2@web.de>
Content-Type: text/plain; charset=UTF-8; format=flowed
Content-Transfer-Encoding: 7bit

On 2024-02-05 1:02 pm, Markus Elfring wrote:
>>>>> Thus reuse existing functionality instead of keeping duplicate source code.
>>>>
>>>> Much as I detest the get_and_ioremap_resource obfuscator, it's not even appropriate here since nothing else is using "res".
>>>
>>> I got the impression that this local variable is needed to perform
>>> a desired function call.
>>
>> Yes, the call to devm_ioremap_resource(). Which you're proposing to remove...
> 
> I propose to replace a specific function call combination
> by an other single call for a known wrapper function.
> The mentioned variable is preserved for this purpose.

No. If you believe in cleaning up code, please apply your brain and 
clean up code *meaningfully* - others seem to have managed it just fine 
(e.g. [1][2] at random from a quick search on lore). Hell, I think even 
the original now-deleted script could auto-generate a proper patch for 
situations where devm_platform_ioremap_resource() was appropriate - 
there is zero excuse for doing a *worse* job 4 years later.

If on the other hand your interest is not to actually clean up the code 
for the good of the community, but to stoke your own ego as a 
"contributor" of shitty patches that annoy people and reinforce the 
popular impression that you are in fact an AI, please stop, or at the 
very least please refrain from doing so in any areas I review or maintain.

Thanks,
Robin.

[1] https://lore.kernel.org/all/20191013152716.1830911-1-jic23@kernel.org/
[2] 
https://lore.kernel.org/all/20200412135023.3831-1-aishwaryarj100@gmail.com/