* Re: [PATCH] net: macb: fix an issue about leak related to system resources
@ 2020-04-25 17:00 Markus Elfring
2020-04-26 2:53 ` Dejin Zheng
0 siblings, 1 reply; 2+ messages in thread
From: Markus Elfring @ 2020-04-25 17:00 UTC (permalink / raw)
To: Dejin Zheng, netdev
Cc: linux-kernel, Andy Shevchenko, David S. Miller, Nicolas Ferre,
Palmer Dabbelt, Paul Walmsley, Yash Shah
> A call of the function macb_init() can fail in the function
> fu540_c000_init. The related system resources were not released
> then. use devm_ioremap() to replace ioremap() for fix it.
How do you think about a wording variant like the following?
Subject:
[PATCH v2] net: macb: Use devm_ioremap() in fu540_c000_init()
Change description:
A call of the macb_init() function can fail here.
The corresponding system resources were not released then.
Thus replace a call of the ioremap() function by devm_ioremap().
Regards,
Markus
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [PATCH] net: macb: fix an issue about leak related to system resources
2020-04-25 17:00 [PATCH] net: macb: fix an issue about leak related to system resources Markus Elfring
@ 2020-04-26 2:53 ` Dejin Zheng
0 siblings, 0 replies; 2+ messages in thread
From: Dejin Zheng @ 2020-04-26 2:53 UTC (permalink / raw)
To: Markus Elfring
Cc: netdev, linux-kernel, Andy Shevchenko, David S. Miller,
Nicolas Ferre, Palmer Dabbelt, Paul Walmsley, Yash Shah
On Sat, Apr 25, 2020 at 07:00:35PM +0200, Markus Elfring wrote:
> > A call of the function macb_init() can fail in the function
> > fu540_c000_init. The related system resources were not released
> > then. use devm_ioremap() to replace ioremap() for fix it.
>
> How do you think about a wording variant like the following?
>
Markus, I think my commit comments is a sufficiently clear description
for this patch. Someone has told me not to send commit comments again
and again when it is enough clear. Because it only wastes the precious
time of the maintainer and very very little help for patch improvement.
BTW, In the past week, you asked me to change the commit comments in my
6 patches like this one. Let me return to the essence of patch, point out
that the code problems and better solutions will be more popular.
> Subject:
> [PATCH v2] net: macb: Use devm_ioremap() in fu540_c000_init()
>
> Change description:
> A call of the macb_init() function can fail here.
> The corresponding system resources were not released then.
> Thus replace a call of the ioremap() function by devm_ioremap().
>
>
> Regards,
> Markus
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2020-04-26 2:53 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-25 17:00 [PATCH] net: macb: fix an issue about leak related to system resources Markus Elfring
2020-04-26 2:53 ` Dejin Zheng
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox