diff options
| author | Behan Webster <behanw@converseincode.com> | 2014-09-27 03:11:45 +0200 |
|---|---|---|
| committer | Mauro Carvalho Chehab <mchehab@osg.samsung.com> | 2014-10-30 13:46:17 +0100 |
| commit | 61110fbab1f083249ac0ac7090526798d8815210 (patch) | |
| tree | b675a9cb672729965f36c7f9d2c2e72f5c0e2321 /drivers/media/platform/ti-vpe/csc.c | |
| parent | [media] sound: Update au0828 quirks table (diff) | |
| download | linux-61110fbab1f083249ac0ac7090526798d8815210.tar.xz linux-61110fbab1f083249ac0ac7090526798d8815210.zip | |
[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/csc.c')
| -rw-r--r-- | drivers/media/platform/ti-vpe/csc.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/drivers/media/platform/ti-vpe/csc.c b/drivers/media/platform/ti-vpe/csc.c index 940df4000c42..44fbf41fba7f 100644 --- a/drivers/media/platform/ti-vpe/csc.c +++ b/drivers/media/platform/ti-vpe/csc.c @@ -93,12 +93,8 @@ void csc_dump_regs(struct csc_data *csc) { struct device *dev = &csc->pdev->dev; - u32 read_reg(struct csc_data *csc, int offset) - { - return ioread32(csc->base + offset); - } - -#define DUMPREG(r) dev_dbg(dev, "%-35s %08x\n", #r, read_reg(csc, CSC_##r)) +#define DUMPREG(r) dev_dbg(dev, "%-35s %08x\n", #r, \ + ioread32(csc->base + CSC_##r)) DUMPREG(CSC00); DUMPREG(CSC01); |