Re: [GIT PULL] Samsung fixes-2 for v3.8

2013-01-07 Thread Olof Johansson
Hi,

On Mon, Jan 7, 2013 at 3:33 PM, Kukjin Kim  wrote:
> Hi Arnd, Olof
>
> Here is Samsung-fixes-2 for v3.8.
> Most of them are EXYNOS5440 fixes which are for changing uart console, cpu
> id (typo)  and silent complaining gpio error in kernel boot.
>
> If any problems, please kindly let me know.
>
> Thanks and a very happy new year.
>
> - Kukjin
>
> The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:
>
>   Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)
>
> are available in the git repository at:
>
>   git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
> v3.8-samsung-fixes-2
>
> for you to fetch changes up to 61bcbc2af8be4cafb5723d4bd306053b70f52a7a:
>
>   ARM: EXYNOS: skip the clock initialization for exynos5440 (2013-01-05
> 08:32:55 -0800)
>
> 
> Kukjin Kim (5):
>   ARM: SAMSUNG: fix the cpu id for EXYNOS5440
>   ARM: dts: use uart port0 for console on exynos5440-ssdk5440
>   ARM: dts: use uart port1 for console on exynos4210-smdkv310
>   ARM: EXYNOS: enable PINCTRL for EXYNOS5440
>   ARM: EXYNOS: skip the clock initialization for exynos5440

See comment I just sent to the posted patch for this one.


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] ARM: EXYNOS: skip the clock initialization for exynos5440

2013-01-07 Thread Olof Johansson
Hi,

On Mon, Dec 31, 2012 at 3:33 PM, Kukjin Kim  wrote:
> Since exynos5440 can support only common clk stuff, so this
> patch skips legacy exynos5 clock initialization.
>
> Signed-off-by: Kukjin Kim 
> ---
>  arch/arm/mach-exynos/common.c |7 +++
>  1 file changed, 7 insertions(+)
>
> diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c
> index d6d0dc6..1a89824 100644
> --- a/arch/arm/mach-exynos/common.c
> +++ b/arch/arm/mach-exynos/common.c
> @@ -424,11 +424,18 @@ static void __init exynos5_init_clocks(int xtal)
>  {
> printk(KERN_DEBUG "%s: initializing clocks\n", __func__);
>
> +   /* EXYNOS5440 can support only common clock framework */
> +
> +   if (soc_is_exynos5440())
> +   return;
> +
> +#ifdef CONFIG_SOC_EXYNOS5250

Why are you adding an ifdef here, when the return above will take care
of returning early for 5440?


-Olof
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[GIT PULL] Samsung fixes-2 for v3.8

2013-01-07 Thread Kukjin Kim
Hi Arnd, Olof

Here is Samsung-fixes-2 for v3.8.
Most of them are EXYNOS5440 fixes which are for changing uart console, cpu
id (typo)  and silent complaining gpio error in kernel boot.

If any problems, please kindly let me know.

Thanks and a very happy new year.

- Kukjin

The following changes since commit a49f0d1ea3ec94fc7cf33a7c36a16343b74bd565:

  Linux 3.8-rc1 (2012-12-21 17:19:00 -0800)

are available in the git repository at:

  git://git.kernel.org/pub/scm/linux/kernel/git/kgene/linux-samsung.git
v3.8-samsung-fixes-2

for you to fetch changes up to 61bcbc2af8be4cafb5723d4bd306053b70f52a7a:

  ARM: EXYNOS: skip the clock initialization for exynos5440 (2013-01-05
08:32:55 -0800)


Kukjin Kim (5):
  ARM: SAMSUNG: fix the cpu id for EXYNOS5440
  ARM: dts: use uart port0 for console on exynos5440-ssdk5440
  ARM: dts: use uart port1 for console on exynos4210-smdkv310
  ARM: EXYNOS: enable PINCTRL for EXYNOS5440
  ARM: EXYNOS: skip the clock initialization for exynos5440

Sean Paul (1):
  ARM: EXYNOS: Revise HDMI resource size

 arch/arm/boot/dts/exynos4210-smdkv310.dts |2 +-
 arch/arm/boot/dts/exynos5250.dtsi |2 +-
 arch/arm/boot/dts/exynos5440-ssdk5440.dts |2 +-
 arch/arm/mach-exynos/Kconfig  |2 ++
 arch/arm/mach-exynos/common.c |7 +++
 arch/arm/plat-samsung/include/plat/cpu.h  |2 +-
 6 files changed, 13 insertions(+), 4 deletions(-)

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] ARM: dts: exynos5250: Set HDMI version to v1.4

2013-01-07 Thread Sean Paul
Set the HDMI version to v1.4 for all exynos5250 targets. This property
was previously being inferred by the existence of the hdmi node.

Signed-off-by: Sean Paul 
---
 arch/arm/boot/dts/exynos5250.dtsi |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/arch/arm/boot/dts/exynos5250.dtsi 
b/arch/arm/boot/dts/exynos5250.dtsi
index 2e3b6ef..9fb6845 100644
--- a/arch/arm/boot/dts/exynos5250.dtsi
+++ b/arch/arm/boot/dts/exynos5250.dtsi
@@ -576,6 +576,7 @@
compatible = "samsung,exynos5-hdmi";
reg = <0x1453 0x10>;
interrupts = <0 95 0>;
+   samsung,supports-hdmi-1.4;
};
 
mixer {
-- 
1.7.7.3

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] ARM: dts: correct the dw-mshc timing properties as per binding

2013-01-07 Thread Doug Anderson
Alim,

On Mon, Jan 7, 2013 at 3:19 AM, Alim Akhtar  wrote:
> As per the current exynos-dw-mshc bindings, dw-mshc-sdr-timing and
> dw-mshc-ddr-timing properties are having only two cells, these properties
> are wrongly set for exynos5250 based cros5250 and smdk5250 platfroms. This 
> patch
> corrects above timing propreties for above platfroms

Thanks for catching this.  Given that this just removes a number from
the dts file that isn't read by the code, I don't expect any
functional change and I don't see any.

I also double-checked usage and compared to the current Chrome OS
tree.  The third number (that you removed) is already correctly
specified in the dts as "samsung,dw-mshc-ciu-div".  :)


> Signed-off-by: Alim Akhtar 

Tested-by: Doug Anderson 
Acked-by: Doug Anderson 
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH v2] arch/arm/mach-s5pv210: adding ifdef CONFIG_CPU_S5PV210 to fix compiling issue

2013-01-07 Thread Chen Gang

  if not define CONFIG_CPU_S5PV210

it will define NULL in common.h for:
  s5pv210_map_io, s5pv210_init_clocks, s5pv210_init, s5pv210_init_uarts

so need using #ifdef CONFIG_CPU_S5PV210 for them too, in common.c.
  or compiling issue will occur.


Signed-off-by: Chen Gang 
---
 arch/arm/mach-s5pv210/common.c |9 +
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/mach-s5pv210/common.c b/arch/arm/mach-s5pv210/common.c
index 9dfe93e..5c88f88 100644
--- a/arch/arm/mach-s5pv210/common.c
+++ b/arch/arm/mach-s5pv210/common.c
@@ -167,6 +167,8 @@ void __init s5pv210_init_io(struct map_desc *mach_desc, int 
size)
s3c_init_cpu(samsung_cpu_id, cpu_ids, ARRAY_SIZE(cpu_ids));
 }
 
+#ifdef CONFIG_CPU_S5PV210
+
 void __init s5pv210_map_io(void)
 {
/* initialise device information early */
@@ -209,6 +211,9 @@ void __init s5pv210_init_clocks(int xtal)
s5pv210_setup_clocks();
 }
 
+#endif /* CONFIG_CPU_S5PV210 */
+
+
 void __init s5pv210_init_irq(void)
 {
u32 vic[4]; /* S5PV210 supports 4 VIC */
@@ -227,6 +232,8 @@ struct bus_type s5pv210_subsys = {
.dev_name   = "s5pv210-core",
 };
 
+#ifdef CONFIG_CPU_S5PV210
+
 static struct device s5pv210_dev = {
.bus= &s5pv210_subsys,
 };
@@ -249,3 +256,5 @@ void __init s5pv210_init_uarts(struct s3c2410_uartcfg *cfg, 
int no)
 {
s3c24xx_init_uartdevs("s5pv210-uart", s5p_uart_resources, cfg, no);
 }
+
+#endif /* CONFIG_CPU_S5PV210 */
-- 
1.7.10.4
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] i2c: samsung: resume race fix

2013-01-07 Thread Jean Delvare
On Mon, 7 Jan 2013 17:35:25 +0530, Naveen Krishna Ch wrote:
> On 7 November 2012 16:14, Jean Delvare  wrote:
> > On Wed, 07 Nov 2012 15:58:26 +0530, Naveen Krishna Chatradhi wrote:
> >> Don't unmark the device as suspended until after it's been re-setup.
> >>
> >> The main race would be w.r.t. an i2c driver that gets resumed at the same
> >> time (asyncronously), that is allowed to do a transfer since suspended
> >> is set to 0 before reinit, but really should have seen the -EIO return
> >> instead.
> >>
> >> Signed-off-by: Olof Johansson 
> >> Signed-off-by: Naveen Krishna Chatradhi 
> >> ---
> >>  drivers/i2c/busses/i2c-s3c2410.c |2 +-
> >>  1 file changed, 1 insertion(+), 1 deletion(-)
> >>
> >> diff --git a/drivers/i2c/busses/i2c-s3c2410.c 
> >> b/drivers/i2c/busses/i2c-s3c2410.c
> >> index 3e0335f..dbaf920 100644
> >> --- a/drivers/i2c/busses/i2c-s3c2410.c
> >> +++ b/drivers/i2c/busses/i2c-s3c2410.c
> >> @@ -1134,10 +1134,10 @@ static int s3c24xx_i2c_resume(struct device *dev)
> >>   struct platform_device *pdev = to_platform_device(dev);
> >>   struct s3c24xx_i2c *i2c = platform_get_drvdata(pdev);
> >>
> >> - i2c->suspended = 0;
> >>   clk_prepare_enable(i2c->clk);
> >>   s3c24xx_i2c_init(i2c);
> >>   clk_disable_unprepare(i2c->clk);
> >> + i2c->suspended = 0;
> >>
> >>   return 0;
> >>  }
> >
> > Acked-by: Jean Delvare 
> I don't see this patch landed any where in linux-i2c tree, Though it was 
> acked.
> Was it missed or should i be doing something for this to be merged ??

Nothing needed from your side AFAIK, Wolfram should pick patches when I
ack them, maybe this one was simply overlooked.

-- 
Jean Delvare
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] ARM: exynos5: Add gate clocks for HS-I2C

2013-01-07 Thread Naveen Krishna Ch
On 28 December 2012 17:10, Naveen Krishna Chatradhi
 wrote:
> Adds clock gating bits for High Speed I2C channels 0, 1, 2 and 3.
>
> Signed-off-by: Naveen Krishna Chatradhi 
> ---
>  arch/arm/mach-exynos/clock-exynos5.c |   24 
>  1 file changed, 24 insertions(+)
>
> diff --git a/arch/arm/mach-exynos/clock-exynos5.c 
> b/arch/arm/mach-exynos/clock-exynos5.c
> index 0208c3a..f9fa0c7 100644
> --- a/arch/arm/mach-exynos/clock-exynos5.c
> +++ b/arch/arm/mach-exynos/clock-exynos5.c
> @@ -753,6 +753,30 @@ static struct clk exynos5_init_clocks_off[] = {
> .enable = exynos5_clk_ip_peric_ctrl,
> .ctrlbit= (1 << 27),
> }, {
> +   .name   = "hsi2c",
> +   .devname= "exynos5-hsi2c.0",
> +   .enable = exynos5_clk_ip_peric_ctrl,
> +   .parent = &exynos5_clk_aclk_66.clk,
> +   .ctrlbit= (1 << 28),
> +   }, {
> +   .name   = "hsi2c",
> +   .devname= "exynos5-hsi2c.1",
> +   .enable = exynos5_clk_ip_peric_ctrl,
> +   .parent = &exynos5_clk_aclk_66.clk,
> +   .ctrlbit= (1 << 29),
> +   }, {
> +   .name   = "hsi2c",
> +   .devname= "exynos5-hsi2c.2",
> +   .enable = exynos5_clk_ip_peric_ctrl,
> +   .parent = &exynos5_clk_aclk_66.clk,
> +   .ctrlbit= (1 << 30),
> +   }, {
> +   .name   = "hsi2c",
> +   .devname= "exynos5-hsi2c.3",
> +   .enable = exynos5_clk_ip_peric_ctrl,
> +   .parent = &exynos5_clk_aclk_66.clk,
> +   .ctrlbit= (1 << 31),
> +   }, {
> .name   = "usbhost",
> .enable = exynos5_clk_ip_fsys_ctrl ,
> .ctrlbit= (1 << 18),
> --
> 1.7.9.5

Any comments on this patch please.
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" 
> in
> the body of a message to majord...@vger.kernel.org
> More majordomo info at  http://vger.kernel.org/majordomo-info.html



-- 
Shine bright,
(: Nav :)
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH] i2c: samsung: resume race fix

2013-01-07 Thread Naveen Krishna Ch
On 7 November 2012 16:14, Jean Delvare  wrote:
> On Wed, 07 Nov 2012 15:58:26 +0530, Naveen Krishna Chatradhi wrote:
>> Don't unmark the device as suspended until after it's been re-setup.
>>
>> The main race would be w.r.t. an i2c driver that gets resumed at the same
>> time (asyncronously), that is allowed to do a transfer since suspended
>> is set to 0 before reinit, but really should have seen the -EIO return
>> instead.
>
> I thought that the suspend order was children first and the resume
> order was parent first?
>
> If this can really happen then I am afraid this is an issue for more
> than just i2c-s3c2410. The proposed solution is also not really
> satisfactory, as the i2c client will certainly still fail to resume
> properly (the only improvement is that now the failure is no longer
> silent.)
>
>>
>> Signed-off-by: Olof Johansson 
>> Signed-off-by: Naveen Krishna Chatradhi 
>> ---
>>  drivers/i2c/busses/i2c-s3c2410.c |2 +-
>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/drivers/i2c/busses/i2c-s3c2410.c 
>> b/drivers/i2c/busses/i2c-s3c2410.c
>> index 3e0335f..dbaf920 100644
>> --- a/drivers/i2c/busses/i2c-s3c2410.c
>> +++ b/drivers/i2c/busses/i2c-s3c2410.c
>> @@ -1134,10 +1134,10 @@ static int s3c24xx_i2c_resume(struct device *dev)
>>   struct platform_device *pdev = to_platform_device(dev);
>>   struct s3c24xx_i2c *i2c = platform_get_drvdata(pdev);
>>
>> - i2c->suspended = 0;
>>   clk_prepare_enable(i2c->clk);
>>   s3c24xx_i2c_init(i2c);
>>   clk_disable_unprepare(i2c->clk);
>> + i2c->suspended = 0;
>>
>>   return 0;
>>  }
>
> Acked-by: Jean Delvare 
I don't see this patch landed any where in linux-i2c tree, Though it was acked.
Was it missed or should i be doing something for this to be merged ??
>
> (Not perfect but still better than before.)
>
> --
> Jean Delvare



-- 
Shine bright,
(: Nav :)
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 1/2] i2c-s3c2410: Leave the bus disabled unless it is in use

2013-01-07 Thread Naveen Krishna Ch
On 29 November 2012 20:14, Kyungmin Park  wrote:
> Acked-by: Kyungmin Park 

I don't see this patch landed any where in linux-i2c tree, Though it was acked.
Was it missed or should i be doing something for this to be merged ??

>
> On Thu, Nov 29, 2012 at 2:05 PM, Naveen Krishna Chatradhi
>  wrote:
>> From: Simon Glass 
>>
>> There is a rather odd feature of the exynos i2c controller that if it
>> is left enabled, it can lock itself up with the clk line held low.
>> This makes the bus unusable.
>>
>> Unfortunately, the s3c24xx_i2c_set_master() function does not notice
>> this, and reports a timeout. From then on the bus cannot be used until
>> the AP is rebooted.
>>
>> The problem happens when any sort of interrupt occurs (e.g. due to a
>> bus transition) when we are not in the middle of a transaction. We
>> have seen many instances of this when U-Boot leaves the bus apparently
>> happy, but Linux cannot access it.
>>
>> The current code is therefore pretty fragile.
>>
>> This fixes things by leaving the bus disabled unless we are actually
>> in a transaction. We enable the bus at the start of the transaction and
>> disable it at the end. That way we won't get interrupts and will not
>> lock up the bus.
>>
>> It might be possible to clear pending interrupts on start-up, but this
>> seems to be a more robust solution. We can't service interrupts when
>> we are not in a transaction, and anyway would rather not lock up the
>> bus while we try.
>>
>> Signed-off-by: Simon Glass 
>> Cc: Grant Grundler 
>> Signed-off-by: Naveen Krishna Chatradhi 
>> ---
>>  drivers/i2c/busses/i2c-s3c2410.c |   37 
>> +
>>  1 file changed, 33 insertions(+), 4 deletions(-)
>>
>> diff --git a/drivers/i2c/busses/i2c-s3c2410.c 
>> b/drivers/i2c/busses/i2c-s3c2410.c
>> index e93e7d6..2fd346d 100644
>> --- a/drivers/i2c/busses/i2c-s3c2410.c
>> +++ b/drivers/i2c/busses/i2c-s3c2410.c
>> @@ -186,6 +186,31 @@ static inline void s3c24xx_i2c_enable_irq(struct 
>> s3c24xx_i2c *i2c)
>> writel(tmp | S3C2410_IICCON_IRQEN, i2c->regs + S3C2410_IICCON);
>>  }
>>
>> +/*
>> + * Disable the bus so that we won't get any interrupts from now on, or try
>> + * to drive any lines. This is the default state when we don't have
>> + * anything to send/receive.
>> + *
>> + * If there is an event on the bus, or we have a pre-existing event at
>> + * kernel boot time, we may not notice the event and the I2C controller
>> + * will lock the bus with the I2C clock line low indefinitely.
>> + */
>> +static inline void s3c24xx_i2c_disable_bus(struct s3c24xx_i2c *i2c)
>> +{
>> +   unsigned long tmp;
>> +
>> +   /* Stop driving the I2C pins */
>> +   tmp = readl(i2c->regs + S3C2410_IICSTAT);
>> +   tmp &= ~S3C2410_IICSTAT_TXRXEN;
>> +   writel(tmp, i2c->regs + S3C2410_IICSTAT);
>> +
>> +   /* We don't expect any interrupts now, and don't want send acks */
>> +   tmp = readl(i2c->regs + S3C2410_IICCON);
>> +   tmp &= ~(S3C2410_IICCON_IRQEN | S3C2410_IICCON_IRQPEND |
>> +   S3C2410_IICCON_ACKEN);
>> +   writel(tmp, i2c->regs + S3C2410_IICCON);
>> +}
>> +
>>
>>  /* s3c24xx_i2c_message_start
>>   *
>> @@ -646,7 +671,11 @@ static int s3c24xx_i2c_doxfer(struct s3c24xx_i2c *i2c,
>>
>> s3c24xx_i2c_wait_idle(i2c);
>>
>> +   s3c24xx_i2c_disable_bus(i2c);
>> +
>>   out:
>> +   i2c->state = STATE_IDLE;
>> +
>> return ret;
>>  }
>>
>> @@ -912,7 +941,6 @@ static void s3c24xx_i2c_dt_gpio_free(struct s3c24xx_i2c 
>> *i2c)
>>
>>  static int s3c24xx_i2c_init(struct s3c24xx_i2c *i2c)
>>  {
>> -   unsigned long iicon = S3C2410_IICCON_IRQEN | S3C2410_IICCON_ACKEN;
>> struct s3c2410_platform_i2c *pdata;
>> unsigned int freq;
>>
>> @@ -926,12 +954,12 @@ static int s3c24xx_i2c_init(struct s3c24xx_i2c *i2c)
>>
>> dev_info(i2c->dev, "slave address 0x%02x\n", pdata->slave_addr);
>>
>> -   writel(iicon, i2c->regs + S3C2410_IICCON);
>> +   writel(0, i2c->regs + S3C2410_IICCON);
>> +   writel(0, i2c->regs + S3C2410_IICSTAT);
>>
>> /* we need to work out the divisors for the clock... */
>>
>> if (s3c24xx_i2c_clockrate(i2c, &freq) != 0) {
>> -   writel(0, i2c->regs + S3C2410_IICCON);
>> dev_err(i2c->dev, "cannot meet bus frequency required\n");
>> return -EINVAL;
>> }
>> @@ -939,7 +967,8 @@ static int s3c24xx_i2c_init(struct s3c24xx_i2c *i2c)
>> /* todo - check that the i2c lines aren't being dragged anywhere */
>>
>> dev_info(i2c->dev, "bus frequency set to %d KHz\n", freq);
>> -   dev_dbg(i2c->dev, "S3C2410_IICCON=0x%02lx\n", iicon);
>> +   dev_dbg(i2c->dev, "S3C2410_IICCON=0x%02x\n",
>> +   readl(i2c->regs + S3C2410_IICCON));
>>
>> return 0;
>>  }
>> --
>> 1.7.9.5
>>
>>
>> ___
>> linux-arm-kernel mailing list
>> linux-arm-ker...@lists.infradead.org
>> http://lists.infradead.org/mailman/listin

Re: [PATCH] arch/arm/mach-s5pv210: adding ifdef CONFIG_CPU_S5PV210 to fix compiling issue

2013-01-07 Thread Chen Gang
于 2013年01月07日 17:36, Chen Gang 写道:
> 
>  
> +#ifdef CONFIG_CPU_S5PV210
> +
>  struct bus_type s5pv210_subsys = {
>   .name   = "s5pv210-core",
>   .dev_name   = "s5pv210-core",

  sorry, it is my fault.
should let s5pv210_subsys outside of #ifdef CONFIG_CPU_S5PV210.
it is used in pm.c

  I need send patch v2.


-- 
Chen Gang

Asianux Corporation
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] ARM: dts: correct the dw-mshc timing properties as per binding

2013-01-07 Thread Alim Akhtar
As per the current exynos-dw-mshc bindings, dw-mshc-sdr-timing and
dw-mshc-ddr-timing properties are having only two cells, these properties
are wrongly set for exynos5250 based cros5250 and smdk5250 platfroms. This patch
corrects above timing propreties for above platfroms

Signed-off-by: Alim Akhtar 
---
 arch/arm/boot/dts/cros5250-common.dtsi| 12 ++--
 arch/arm/boot/dts/exynos5250-smdk5250.dts |  8 
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/arch/arm/boot/dts/cros5250-common.dtsi 
b/arch/arm/boot/dts/cros5250-common.dtsi
index fddd174..46c0980 100644
--- a/arch/arm/boot/dts/cros5250-common.dtsi
+++ b/arch/arm/boot/dts/cros5250-common.dtsi
@@ -96,8 +96,8 @@
fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
-   samsung,dw-mshc-sdr-timing = <2 3 3>;
-   samsung,dw-mshc-ddr-timing = <1 2 3>;
+   samsung,dw-mshc-sdr-timing = <2 3>;
+   samsung,dw-mshc-ddr-timing = <1 2>;
 
slot@0 {
reg = <0>;
@@ -120,8 +120,8 @@
fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
-   samsung,dw-mshc-sdr-timing = <2 3 3>;
-   samsung,dw-mshc-ddr-timing = <1 2 3>;
+   samsung,dw-mshc-sdr-timing = <2 3>;
+   samsung,dw-mshc-ddr-timing = <1 2>;
 
slot@0 {
reg = <0>;
@@ -141,8 +141,8 @@
fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
-   samsung,dw-mshc-sdr-timing = <2 3 3>;
-   samsung,dw-mshc-ddr-timing = <1 2 3>;
+   samsung,dw-mshc-sdr-timing = <2 3>;
+   samsung,dw-mshc-ddr-timing = <1 2>;
 
slot@0 {
reg = <0>;
diff --git a/arch/arm/boot/dts/exynos5250-smdk5250.dts 
b/arch/arm/boot/dts/exynos5250-smdk5250.dts
index 942d576..e05b18f 100644
--- a/arch/arm/boot/dts/exynos5250-smdk5250.dts
+++ b/arch/arm/boot/dts/exynos5250-smdk5250.dts
@@ -115,8 +115,8 @@
fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
-   samsung,dw-mshc-sdr-timing = <2 3 3>;
-   samsung,dw-mshc-ddr-timing = <1 2 3>;
+   samsung,dw-mshc-sdr-timing = <2 3>;
+   samsung,dw-mshc-ddr-timing = <1 2>;
 
slot@0 {
reg = <0>;
@@ -139,8 +139,8 @@
fifo-depth = <0x80>;
card-detect-delay = <200>;
samsung,dw-mshc-ciu-div = <3>;
-   samsung,dw-mshc-sdr-timing = <2 3 3>;
-   samsung,dw-mshc-ddr-timing = <1 2 3>;
+   samsung,dw-mshc-sdr-timing = <2 3>;
+   samsung,dw-mshc-ddr-timing = <1 2>;
 
slot@0 {
reg = <0>;
-- 
1.8.0

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] arch/arm/mach-s5pv210: adding ifdef CONFIG_CPU_S5PV210 to fix compiling issue

2013-01-07 Thread Chen Gang

  if not define CONFIG_CPU_S5PV210

it will define NULL in common.h for:
  s5pv210_map_io, s5pv210_init_clocks, s5pv210_init, s5pv210_init_uarts

so need using #ifdef CONFIG_CPU_S5PV210 for them too, in common.c.
  or compiling issue will occur.


Signed-off-by: Chen Gang 
---
 arch/arm/mach-s5pv210/common.c |9 +
 1 file changed, 9 insertions(+)

diff --git a/arch/arm/mach-s5pv210/common.c b/arch/arm/mach-s5pv210/common.c
index 9dfe93e..ad56485 100644
--- a/arch/arm/mach-s5pv210/common.c
+++ b/arch/arm/mach-s5pv210/common.c
@@ -167,6 +167,8 @@ void __init s5pv210_init_io(struct map_desc *mach_desc, int 
size)
s3c_init_cpu(samsung_cpu_id, cpu_ids, ARRAY_SIZE(cpu_ids));
 }
 
+#ifdef CONFIG_CPU_S5PV210
+
 void __init s5pv210_map_io(void)
 {
/* initialise device information early */
@@ -209,6 +211,9 @@ void __init s5pv210_init_clocks(int xtal)
s5pv210_setup_clocks();
 }
 
+#endif /* CONFIG_CPU_S5PV210 */
+
+
 void __init s5pv210_init_irq(void)
 {
u32 vic[4]; /* S5PV210 supports 4 VIC */
@@ -222,6 +227,8 @@ void __init s5pv210_init_irq(void)
s5p_init_irq(vic, ARRAY_SIZE(vic));
 }
 
+#ifdef CONFIG_CPU_S5PV210
+
 struct bus_type s5pv210_subsys = {
.name   = "s5pv210-core",
.dev_name   = "s5pv210-core",
@@ -249,3 +256,5 @@ void __init s5pv210_init_uarts(struct s3c2410_uartcfg *cfg, 
int no)
 {
s3c24xx_init_uartdevs("s5pv210-uart", s5p_uart_resources, cfg, no);
 }
+
+#endif /* CONFIG_CPU_S5PV210 */
-- 
1.7.10.4
--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH] cpufreq: exynos: Get booting freq value in exynos_cpufreq_init

2013-01-07 Thread Jonghwan Choi
Boot_freq is for saving booting freq. But exynos_cpufreq_cpu_init
is called in hotplug. If boot_freq is existed in exynos_cpufreq_cpu_init,
boot_freq could be changed.

Signed-off-by: Jonghwan Choi 
---
 drivers/cpufreq/exynos-cpufreq.c |4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/cpufreq/exynos-cpufreq.c
b/drivers/cpufreq/exynos-cpufreq.c
index 7012ea8..41fc550 100644
--- a/drivers/cpufreq/exynos-cpufreq.c
+++ b/drivers/cpufreq/exynos-cpufreq.c
@@ -222,8 +222,6 @@ static int exynos_cpufreq_cpu_init(struct cpufreq_policy
*policy)
 
cpufreq_frequency_table_get_attr(exynos_info->freq_table,
policy->cpu);
 
-   locking_frequency = exynos_getspeed(0);
-
/* set the transition latency value */
policy->cpuinfo.transition_latency = 10;
 
@@ -288,6 +286,8 @@ static int __init exynos_cpufreq_init(void)
goto err_vdd_arm;
}
 
+   locking_frequency = exynos_getspeed(0);
+
register_pm_notifier(&exynos_cpufreq_nb);
 
if (cpufreq_register_driver(&exynos_driver)) {
-- 
1.7.4.1

--
To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html