diff options
| author | Paul Mundt <lethal@linux-sh.org> | 2012-06-21 00:03:41 +0900 |
|---|---|---|
| committer | Paul Mundt <lethal@linux-sh.org> | 2012-06-21 00:03:41 +0900 |
| commit | 06d5631f56460917af3d9417ef63811cf0cad9ce (patch) | |
| tree | a25eee33f3198c4ba75c79b51a3bc85fd10a5f50 /drivers/sh | |
| parent | 9f65b2b60c42c1e2d885acede4443b53f141c987 (diff) | |
| download | op-kernel-dev-06d5631f56460917af3d9417ef63811cf0cad9ce.zip op-kernel-dev-06d5631f56460917af3d9417ef63811cf0cad9ce.tar.gz | |
sh: pfc: Verify pin type encoding size at build time.
The encoding is tightly packed, and future changes (such as
pinconf-generic support) can easily lead to a situation where we violate
the encoding constraints and trample data bit/reg bits. This plugs in
some sanity checks by way of a BUILD_BUG_ON() to blow up if we fail to
fit.
Signed-off-by: Paul Mundt <lethal@linux-sh.org>
Diffstat (limited to 'drivers/sh')
| -rw-r--r-- | drivers/sh/pfc.c | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/drivers/sh/pfc.c b/drivers/sh/pfc.c index 8a9ae09..ce4579e 100644 --- a/drivers/sh/pfc.c +++ b/drivers/sh/pfc.c @@ -549,6 +549,11 @@ int register_sh_pfc(struct sh_pfc *pfc) int (*initroutine)(struct sh_pfc *) = NULL; int ret; + /* + * Ensure that the type encoding fits + */ + BUILD_BUG_ON(PINMUX_FLAG_TYPE > ((1 << PINMUX_FLAG_DBIT_SHIFT) - 1)); + if (sh_pfc) return -EBUSY; |
