From: Taku Izumi <izumi.t...@jp.fujitsu.com> Date: Thu, 20 Aug 2015 17:46:08 +0900
> + > +err_register: > + fjes_hw_exit(&adapter->hw); > +err_hw_init: > +err_sw_init: > + free_netdev(netdev); > +err_alloc_netdev: > + return err; Having multiple code labels in the same exact spot is suboptimal. Instead, name the labels such that they describe the first cleanup action they will perform, instead of the context in which they are jumped to from. So "err_hw_exit:", "err_free_netdev", and "err_out:" would be appropriate. -- To unsubscribe from this list: send the line "unsubscribe netdev" in the body of a message to majord...@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
