mirror of https://lore.kernel.org/lkml/
 help / color / mirror / Atom feed
* [PATCH 0/2] vt6656: Cleanup of checkpatch problems in bssdb.c
@ 2013-12-19 15:27 Sebastian Rachuj
  2013-12-19 15:27 ` [PATCH 1/2] vt6656: Make checkpatch happier Sebastian Rachuj
  2013-12-19 15:27 ` [PATCH 2/2] vt6656: Combined nested conditions Sebastian Rachuj
  0 siblings, 2 replies; 4+ messages in thread
From: Sebastian Rachuj @ 2013-12-19 15:27 UTC (permalink / raw)
  To: sebastian.rachuj
  Cc: linux, linux-kernel, linux-kernel, devel, tvboxspy, more.andres,
	gregkh, forest

This patch series reformats the bssdb.c file of the vt6656 driver in
staging. It resolves all errors and some warnings which checkpatch.pl
discovered.

However some occurrences of the following warnings are not patched:
- Lines over 80 characters
- Too many indentation levels
- printk should include KERN_ facility level

The first two noted types of warnings require more code modification
to reduce the indentation level. The latter should be fixed along with
the removal of the custom printk macro in
drivers/staging/vt6656/device.h.

NOTE: As the first patch retabs the complete file, you might consider
using "git diff --ignore-all-space"

No functional changes should be introduced by these patches.

This file was already changed in linux-next to some degree, but still
does not conform to the linux coding style. Nevertheless, we submit
this patch series since we fix more coding style problems. However
we welcome any recommendations concerning this conflict.


^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2013-12-19 16:32 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-12-19 15:27 [PATCH 0/2] vt6656: Cleanup of checkpatch problems in bssdb.c Sebastian Rachuj
2013-12-19 15:27 ` [PATCH 1/2] vt6656: Make checkpatch happier Sebastian Rachuj
2013-12-19 16:31   ` Dan Carpenter
2013-12-19 15:27 ` [PATCH 2/2] vt6656: Combined nested conditions Sebastian Rachuj

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox

all inboxes | Powered by JetHome®