lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Mon, 05 Jan 2015 13:14:07 +0530 From: Pankaj Dubey <pankaj.dubey@...sung.com> To: Rickard Strandqvist <rickard_strandqvist@...ctrumdigital.se>, Mauro Carvalho Chehab <m.chehab@...sung.com>, Kukjin Kim <kgene.kim@...sung.com> Cc: Jacek Anaszewski <j.anaszewski@...sung.com>, Kamil Debski <k.debski@...sung.com>, Sylwester Nawrocki <s.nawrocki@...sung.com>, linux-media@...r.kernel.org, linux-arm-kernel@...ts.infradead.org, linux-samsung-soc@...r.kernel.org, linux-kernel@...r.kernel.org, tony nadackal <tony.kn@...sung.com> Subject: Re: media: platform: s5p-jpeg: jpeg-hw-exynos4: Remove some unused functions +Tony Nadackal Hi Rickard, On Saturday 03 January 2015 02:07 AM, Rickard Strandqvist wrote: > Removes some functions that are not used anywhere: > exynos4_jpeg_set_timer_count() exynos4_jpeg_get_frame_size() exynos4_jpeg_set_sys_int_enable() exynos4_jpeg_get_fifo_status() > > This was partially found by using a static code analysis program called cppcheck. > > Signed-off-by: Rickard Strandqvist <rickard_strandqvist@...ctrumdigital.se> > > --- > drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c | 35 --------------------- > drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.h | 5 --- > 2 files changed, 40 deletions(-) > > diff --git a/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c b/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c > index ab6d6f4..5685577 100644 > --- a/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c > +++ b/drivers/media/platform/s5p-jpeg/jpeg-hw-exynos4.c > @@ -163,15 +163,6 @@ unsigned int exynos4_jpeg_get_int_status(void __iomem *base) > return int_status; > } > > -unsigned int exynos4_jpeg_get_fifo_status(void __iomem *base) > -{ > - unsigned int fifo_status; > - > - fifo_status = readl(base + EXYNOS4_FIFO_STATUS_REG); > - > - return fifo_status; > -} > - > void exynos4_jpeg_set_huf_table_enable(void __iomem *base, int value) > { > unsigned int reg; > @@ -186,18 +177,6 @@ void exynos4_jpeg_set_huf_table_enable(void __iomem *base, int value) > base + EXYNOS4_JPEG_CNTL_REG); > } > > -void exynos4_jpeg_set_sys_int_enable(void __iomem *base, int value) > -{ > - unsigned int reg; > - > - reg = readl(base + EXYNOS4_JPEG_CNTL_REG) & ~(EXYNOS4_SYS_INT_EN); > - > - if (value == 1) > - writel(reg | EXYNOS4_SYS_INT_EN, base + EXYNOS4_JPEG_CNTL_REG); > - else > - writel(reg & ~EXYNOS4_SYS_INT_EN, base + EXYNOS4_JPEG_CNTL_REG); > -} > - Above function will be needed for enabling JPEG support on Exynos7 SoC. There is already inflight patch [1] which will be using it. 1: https://patchwork.kernel.org/patch/5505391/ Thanks, Pankaj Dubey -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists