[PATCH v3 2/2] PCI: j721e: Use inline reset GPIO assignment and drop local variable
Anand Moon
linux.amoon at gmail.com
Tue Oct 28 00:06:39 PDT 2025
Hi Markus,
Thanks for your review comments.
On Mon, 27 Oct 2025 at 19:13, Markus Elfring <Markus.Elfring at web.de> wrote:
>
> > The result of devm_gpiod_get_optional() is now assigned directly
> > assigned to pcie->reset_gpio. This removes a superfluous local gpiod
> > variable, improving code readability and compactness. The functionality
> > remains unchanged, but the code is now more streamlined
>
> Would a corresponding imperative wording become helpful for an improved change description?
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/submitting-patches.rst?h=v6.18-rc3#n94
>
Assign the result of devm_gpiod_get_optional() directly to pcie->reset_gpio.
This removes a superfluous local variable, improving code clarity without
altering the driver's behavior.
Is this ok with you?
> Regards,
> Markus
Thanks
-Anand
More information about the linux-arm-kernel
mailing list