Commit 41aaf689 authored Jun 15, 2011 by Gregory Bean Committed by Kyle Yan Jul 05, 2016

checkpatch: complain about the use of dsb().



Now that mb() does what we want, dsb() should be discouraged.

Change-Id: Ib8fe8f44f669753c3d91fac3c6e598e117d6d90e
Signed-off-by: Gregory Bean <gbean@codeaurora.org>
(cherry picked from commit 9c0619be7b93ad114d6f33a749d905ddff93df7d)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>

parent 13611e09

scripts/checkpatch.pl

+7 −0

Original line number	Diff line number	Diff line
		@@ -5014,6 +5014,13 @@ sub process {
		$herecurr);
		}

		# dsb is too ARMish, and should usually be mb.
		if ($line =~ /\bdsb\b/) {
		WARN("ARM_BARRIER",
		"Use of dsb is discouranged: prefer mb.\n" .
		$herecurr);
		}

		# unbounded string functions are overflow risks
		my %str_fns = (
		"sprintf" => "snprintf",