Messages in this thread | | | Date | Wed, 8 Nov 2023 09:46:47 -0800 | Subject | Re: net: bcmasp: Use common error handling code in bcmasp_probe() | From | Florian Fainelli <> |
| |
On 11/7/23 10:48, Justin Chen wrote: > On Mon, Nov 6, 2023 at 10:38 PM Markus Elfring <Markus.Elfring@web.de> wrote: >> >>>> Add a jump target so that a bit of exception handling can be better >>>> reused at the end of this function. >> … >>>> --- >>>> drivers/net/ethernet/broadcom/asp2/bcmasp.c | 10 ++++++---- >>>> 1 file changed, 6 insertions(+), 4 deletions(-) >>> >>> The diffstat proves otherwise. >>> Please don't send such patches to networking. >> >> How does this feedback fit to a change possibility which was reviewed by >> Wojciech Drewek yesterday? >> >> Regards, >> Markus > > We are making the code harder to follow with these changes. Also > adding more lines than removing. Don't think this patch is an > improvement IMHO. NAK on my end.
Likewise, at the very least, why not have the remove_intfs label immediately above the of_put_exit one so then it just falls through, and then obviously update the return path to drop the reference count and return success? -- Florian
[unhandled content-type:application/pkcs7-signature] | |