Messages in this thread |  | | Date | Wed, 17 May 2023 21:11:10 +0100 | | From | "Russell King (Oracle)" <> | | Subject | Re: [PATCH] net: mvpp2: Fix error checking |
| |
On Thu, May 18, 2023 at 03:08:11AM +0800, Yeqi Fu wrote: > The function debugfs_create_dir returns ERR_PTR if an error occurs, > and the appropriate way to verify for errors is to use the inline > function IS_ERR. The patch will substitute the null-comparison with > IS_ERR.
Exactly as I said to a very similar patch received a few days ago from SikkiLadho:
"The modern wisdom for debugfs is not to check for any errors, so if we're going to touch this, that's the route that any patch should be taking.
Thanks."
Your patch seems to have the same Suggested-by: which suggests to me that you probably know SikkiLadho and are working together with the person who suggested the change, so it would be good that when a patch from one of you is commented upon, those comments are taken into account rather than someone else sending an identical patch to the first.
Thanks.
-- RMK's Patch system: https://www.armlinux.org.uk/developer/patches/ FTTP is here! 80Mbps down 10Mbps up. Decent connectivity at last!
|  |