On Tue, Oct 08, 2019 at 07:44:15PM -0700, Nachammai Karuppiah wrote: > Remove NULL comparison. Issue found using checkpatch.pl > > Signed-off-by: Nachammai Karuppiah <nachukannan@xxxxxxxxx> > > --- > > Changes in V2 > - Remove all NULL comparisons in vc04_services/interface directory. > --- > > changes in V3 > - Fixed warnings. Reported-by: kbuild test robot <lkp@xxxxxxxxx> > --- > > Signed-off-by: Nachammai Karuppiah <nachukannan@xxxxxxxxx> > --- > .../interface/vchiq_arm/vchiq_2835_arm.c | 4 ++-- Only one --- cut off line is needed. You have two Signed-of-by lines as well. It should just be: Remove NULL comparison. Issue found using checkpatch.pl Signed-off-by: Nachammai Karuppiah <nachukannan@xxxxxxxxx> --- Changes in V2 - Remove all NULL comparisons in vc04_services/interface directory. changes in V3 - Fixed warnings. Reported-by: kbuild test robot <lkp@xxxxxxxxx> .../interface/vchiq_arm/vchiq_2835_arm.c | 4 ++-- But this doesn't affect the final patch, and we can see all the information so it's fine. No need to resend. Reviewed-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx> regards, dan carpenter _______________________________________________ devel mailing list devel@xxxxxxxxxxxxxxxxxxxxxx http://driverdev.linuxdriverproject.org/mailman/listinfo/driverdev-devel
