Commit 5ea9add6 authored Dec 06, 2011 by Rohit Vaswani Committed by Kyle Yan Jul 05, 2016

checkpatch: Add check for gpiomux usage in msm board files



MSM has a board-*-gpiomux file where all the gpiomux configs reside.
Warn if a non gpiomux board file tries to add gpiomux configs.
The camera board file is an exception to this rule.

Change-Id: Ibab190dcbd7ea78e7ca150142c68c5ae881e4e06
Signed-off-by: Rohit Vaswani <rvaswani@codeaurora.org>
(cherry picked from commit 60d78bb9809e7d4d1c3dc1425cbfd9e649e87c1c)
Signed-off-by: Stepan Moskovchenko <stepanm@codeaurora.org>

parent 41aaf689

scripts/checkpatch.pl

+8 −0

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

		# MSM - check if a non board-gpiomux file has any gpiomux declarations
		if ($realfile =~ /\/mach-msm\/board-[0-9]+/ &&
		$realfile !~ /camera/ && $realfile !~ /gpiomux/ &&
		$line =~ /\sstruct msm_gpiomux_config\s/ ) {
		WARN("GPIOMUX_IN_BOARD",
		"Non gpiomux board file cannot have a gpiomux config declarations. Please declare gpiomux configs in board-*-gpiomux.c file.\n" . $herecurr);
		}

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