Hi, I reworked what Dan suggested. I also added another patch which outsources a chunk of code from the InterfaceRDM() and InterfaceRWM() functions, which was doubled. I missed that one in the first place. I hope you like the changes. Kind regards, Matthias Beyer Matthias Beyer (2): Staging: bcm: Shortened lines Staging: bcm: InterfaceMisc.c: Outsourced doubled error checking drivers/staging/bcm/InterfaceMisc.c | 124 +++++++++++++++++++++++------------- 1 file changed, 78 insertions(+), 46 deletions(-) -- 2.0.0 _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel