On Sunday 21 of July 2013 23:51:24 Tomasz Figa wrote:
> Hi Sylwester,
>
> On Sunday 21 of July 2013 21:46:09 Sylwester Nawrocki wrote:
> > Hi Tomasz,
> >
> > On 07/20/2013 02:04 AM, Tomasz Figa wrote:
> > > This patch adds new samsung_device_pwm platform device that
> > > represents
> > > the whole PWM/timer block and includes memory and IRQ resources.
> > >
> > > Signed-off-by: Tomasz Figa<tomasz.figa@xxxxxxxxx>
> > > ---
> > >
> > > arch/arm/plat-samsung/devs.c | 17
> > > +++++++++++++++++
> > > arch/arm/plat-samsung/include/plat/devs.h | 1 +
> > > arch/arm/plat-samsung/include/plat/pwm-core.h | 24
> > > ++++++++++++++++++++++++ 3 files changed, 42 insertions(+)
> > > create mode 100644 arch/arm/plat-samsung/include/plat/pwm-core.h
> > >
> > > diff --git a/arch/arm/plat-samsung/devs.c
> > > b/arch/arm/plat-samsung/devs.c index 0f9c3f4..bba6d78 100644
> > > --- a/arch/arm/plat-samsung/devs.c
> > > +++ b/arch/arm/plat-samsung/devs.c
> > > @@ -58,6 +58,7 @@
> > >
> > > #include<plat/keypad.h>
> > > #include<linux/platform_data/mmc-s3cmci.h>
> > > #include<linux/platform_data/mtd-nand-s3c2410.h>
> > >
> > > +#include<plat/pwm-core.h>
> > >
> > > #include<plat/sdhci.h>
> > > #include<linux/platform_data/touchscreen-s3c2410.h>
> > > #include<linux/platform_data/usb-s3c2410_udc.h>
> > >
> > > @@ -1127,6 +1128,22 @@ struct platform_device s3c_device_timer[] = {
> > >
> > > [3] = { DEFINE_S3C_TIMER(3, IRQ_TIMER3) },
> > > [4] = { DEFINE_S3C_TIMER(4, IRQ_TIMER4) },
> > >
> > > };
> > >
> > > +
> > > +static struct resource samsung_pwm_resource[] = {
> > > + DEFINE_RES_MEM(SAMSUNG_PA_TIMER, SZ_4K),
> > > +};
> > > +
> > > +struct platform_device samsung_device_pwm = {
> > > + .name = "samsung-pwm",
> > > + .id = -1,
> > > + .num_resources = ARRAY_SIZE(samsung_pwm_resource),
> > > + .resource = samsung_pwm_resource,
> > > +};
> > > +
> > > +void __init samsung_pwm_set_platdata(struct samsung_pwm_variant
> > > *pd)
> > > +{
> > > + samsung_device_pwm.dev.platform_data = pd;
> > > +}
> > >
> > > #endif /* CONFIG_SAMSUNG_DEV_PWM */
> > >
> > > /* RTC */
> > >
> > > diff --git a/arch/arm/plat-samsung/include/plat/devs.h
> > > b/arch/arm/plat-samsung/include/plat/devs.h index 87d501f..0dc4ac4
> > > 100644
> > > --- a/arch/arm/plat-samsung/include/plat/devs.h
> > > +++ b/arch/arm/plat-samsung/include/plat/devs.h
> > > @@ -134,6 +134,7 @@ extern struct platform_device
> > > exynos4_device_spdif;
> > >
> > > extern struct platform_device samsung_asoc_idma;
> > > extern struct platform_device samsung_device_keypad;
> > >
> > > +extern struct platform_device samsung_device_pwm;
> > >
> > > /* s3c2440 specific devices */
> > >
> > > diff --git a/arch/arm/plat-samsung/include/plat/pwm-core.h
> > > b/arch/arm/plat-samsung/include/plat/pwm-core.h new file mode 100644
> > > index 0000000..df50f5c
> > > --- /dev/null
> > > +++ b/arch/arm/plat-samsung/include/plat/pwm-core.h
> > > @@ -0,0 +1,24 @@
> > > +/*
> > > + * linux/arch/arm/plat-samsung/onenand-core.h
> >
> > Are such full file path names really useful, especially we are never
> > sure they are up to date ? I guess this line could be removed.
>
> No, they are not. This is a copy-pasto I missed (notice
> onenand-core...).
> > > + * Copyright (c) 2013 Tomasz Figa<tomasz.figa@xxxxxxxxx>
> > > + *
> > > + * Samsung PWM Controller core functions
> > > + *
> > > + * This program is free software; you can redistribute it and/or
> > > modify + * it under the terms of the GNU General Public License
> > > version 2 as + * published by the Free Software Foundation.
> > > +*/
> > > +
> > > +#ifndef __ASM_ARCH_PWM_CORE_H
> > > +#define __ASM_ARCH_PWM_CORE_H __FILE__
> > > +
> > > +#include<clocksource/samsung_pwm.h>
> > > +
> > > +#ifdef CONFIG_SAMSUNG_DEV_PWM
> > > +extern void samsung_pwm_set_platdata(struct samsung_pwm_variant
> > > *pd);
> >
> > nit: 'extern' is implicit for function declarations, it could be
> > omitted here.
>
> The extern kernel is used widely over the kernel (including headers in
> plat-samsung) and I prefer adding it for clarity.
s/extern kernel/extern keyword/
I guess it's time to sleep.
Best regards,
Tomasz
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
