diff options
| author | Behan Webster <behanw@converseincode.com> | 2014-09-26 22:11:45 -0300 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-10-30 10:46:17 -0200 |
| commit | 61110fbab1f083249ac0ac7090526798d8815210 (patch) | |
| tree | b675a9cb672729965f36c7f9d2c2e72f5c0e2321 /drivers/media/platform/ti-vpe/sc.c | |
| parent | 678fa12fb8e75c6dc1e781a02e3ddbbba7e1a904 (diff) | |
| download | linux-61110fbab1f083249ac0ac7090526798d8815210.tar.bz2 | |
[media] ti-fpe: LLVMLinux: Remove nested function from ti-vpe
Replace the use of nested functions where a normal function will suffice.
Nested functions are not liked by upstream kernel developers in general. Their
use breaks the use of clang as a compiler, and doesn't make the code any
better.
This code now works for both gcc and clang.
Suggested-by: Arnd Bergmann <arnd@arndb.de>
Signed-off-by: Behan Webster <behanw@converseincode.com>
Signed-off-by: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
Diffstat (limited to 'drivers/media/platform/ti-vpe/sc.c')
| -rw-r--r-- | drivers/media/platform/ti-vpe/sc.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/media/platform/ti-vpe/sc.c b/drivers/media/platform/ti-vpe/sc.c index 6314171ffe9b..1088381bd349 100644 --- a/drivers/media/platform/ti-vpe/sc.c +++ b/drivers/media/platform/ti-vpe/sc.c @@ -24,12 +24,8 @@ void sc_dump_regs(struct sc_data *sc) { struct device *dev = &sc->pdev->dev; - u32 read_reg(struct sc_data *sc, int offset) - { - return ioread32(sc->base + offset); - } - -#define DUMPREG(r) dev_dbg(dev, "%-35s %08x\n", #r, read_reg(sc, CFG_##r)) +#define DUMPREG(r) dev_dbg(dev, "%-35s %08x\n", #r, \ + ioread32(sc->base + CFG_##r)) DUMPREG(SC0); DUMPREG(SC1); |