signal.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
Tested on arndale board on arm-soc next/soc branch.
arch/arm/mach-exynos/include/mach/regs-pmu.h |1 +
arch/arm/mach-exynos/pmu.c |5 ++---
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git
Hi Daniel,
On 16 March 2013 12:51, Padma Venkat padma@gmail.com wrote:
Hi,
On Fri, Mar 15, 2013 at 10:01 PM, Daniel Lezcano
daniel.lezc...@linaro.org wrote:
Using the exynos4_defconfig and compiling the kernel from the samsung
git tree at 3.9-rc1, I am stuck.
Does anyone have any
are disabled ?
It seems arch/arm/mach-s5pv210/clock.c might be missing apb_pclk clock
supply names, which I suspect may be required after commits:
commit 7c71b8eb268ee38235f7e924d943ea9d90e59469
Author: Inderpal Singh inderpal.si...@linaro.org
Date: Fri Sep 7 12:14:48 2012 +0530
DMA
Only cortex-a9 based samsung platforms have l2x0 cache controller. Hence check
the same before restoring the cache in resume.
This is needed for single kernel image.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
changes in v2:
- check processor midr instead of checking all
On 26 February 2013 15:32, Russell King - ARM Linux
li...@arm.linux.org.uk wrote:
On Tue, Feb 26, 2013 at 03:26:53PM +0530, Inderpal Singh wrote:
Only cortex-a9 based samsung platforms have l2x0 cache controller. Hence
check
the same before restoring the cache in resume.
Why is this patch
Hi Kukjin,
Thanks for reviewing the patch.
On 13 February 2013 00:57, Kukjin Kim kgene@samsung.com wrote:
Inderpal Singh wrote:
Only exynos4 based platforms have l2x0 cache controller. Hence check
the same before restoring the cache in resume.
I think, the code can determine
On 12 February 2013 06:42, amit kachhap amit.kach...@gmail.com wrote:
On Fri, Feb 8, 2013 at 8:49 AM, Inderpal Singh
inderpal.si...@linaro.org wrote:
On 8 February 2013 00:03, amit kachhap amit.kach...@gmail.com wrote:
On Wed, Feb 6, 2013 at 8:49 PM, Inderpal Singh
inderpal.si...@linaro.org
On 8 February 2013 00:03, amit kachhap amit.kach...@gmail.com wrote:
On Wed, Feb 6, 2013 at 8:49 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
On 7 February 2013 01:09, Amit Daniel Kachhap amit.dan...@samsung.com
wrote:
Add cpufreq controller device node for Exynos5440 SoC for passing
On 7 February 2013 16:49, Viresh Kumar viresh.ku...@linaro.org wrote:
On Thu, Feb 7, 2013 at 10:39 AM, Inderpal Singh
inderpal.si...@linaro.org wrote:
+#define DRIVER_NAMEexynos5440_dvfs
+static struct cpufreq_driver exynos_driver = {
+ .name = DRIVER_NAME
On 7 February 2013 01:09, Amit Daniel Kachhap amit.dan...@samsung.com wrote:
Add cpufreq controller device node for Exynos5440 SoC for passing
parameters like controller base address, interrupt and cpufreq
table.
Signed-off-by: Amit Daniel Kachhap amit.dan...@samsung.com
---
On 7 February 2013 01:09, Amit Daniel Kachhap amit.dan...@samsung.com wrote:
This patch adds dvfs support for exynos5440 SOC. The nature of exynos5440
clock controller is different from previous exynos controllers so not using
the common exynos cpufreq framework. Also, the device tree parsing
Add freq_attr attribute to show list of available frequencies.
Signed-off-by: Donggeun Kim dg77@samsung.com
Signed-off-by: MyungJoo Ham myungjoo@samsung.com
Signed-off-by: KyungMin Park kyungmin.p...@samsung.com
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/cpufreq
+CC:
dg77@samsung.com, myungjoo@samsung.com, kyungmin.p...@samsung.com
On 8 January 2013 16:20, Inderpal Singh inderpal.si...@linaro.org wrote:
Add freq_attr attribute to show list of available frequencies.
Signed-off-by: Donggeun Kim dg77@samsung.com
Signed-off-by: MyungJoo Ham
Set the gic arch extension callback to support rtc wakeup.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
arch/arm/mach-exynos/common.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/arch/arm/mach-exynos/common.c b/arch/arm/mach-exynos/common.c
index 578a610..bf2ee1d
i2c-gpios[] is being used to free_gpios but it's not getting initialized.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/i2c/busses/i2c-s3c2410.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/i2c/busses/i2c-s3c2410.c b/drivers/i2c/busses/i2c-s3c2410.c
While resuming the gpios are being requested again, hence we need
to free the gpios before going to suspend otherwise it gives
gpio request failed errors while resuming.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/i2c/busses/i2c-s3c2410.c |1 +
1 file changed, 1
On 27 December 2012 09:39, Inderpal Singh inderpal.si...@linaro.org wrote:
While resuming the gpios are being requested again, hence we need
to free the gpios before going to suspend otherwise it gives
gpio request failed errors while resuming.
Signed-off-by: Inderpal Singh inderpal.si
On 27 December 2012 09:39, Inderpal Singh inderpal.si...@linaro.org wrote:
i2c-gpios[] is being used to free_gpios but it's not getting initialized.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/i2c/busses/i2c-s3c2410.c |2 ++
1 file changed, 2 insertions(+)
diff
) from [c0297dd0]
(really_probe+0x68/0x1f4)
[c0297dd0] (really_probe+0x68/0x1f4) from [c0298070]
(driver_probe_device+0x30/0x48)
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/regulator/s5m8767.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git
Hi Vinod,
On 29 October 2012 10:15, Vinod Koul vk...@infradead.org wrote:
On Sat, 2012-10-27 at 15:50 +0530, Inderpal Singh wrote:
Hi Vinod,
On 26 October 2012 10:15, Vinod Koul vk...@infradead.org wrote:
On Thu, 2012-10-25 at 16:53 +0530, Inderpal Singh wrote:
This code will get
Hi Vinod,
On 26 October 2012 10:15, Vinod Koul vk...@infradead.org wrote:
On Thu, 2012-10-25 at 16:53 +0530, Inderpal Singh wrote:
This code will get executed only in case of force removal of the
module which was discussed in the first version of the patch at [1].
Now, if we do not have
Hi Vinod,
Thanks for reviewing.
On 24 October 2012 09:35, Vinod Koul vk...@infradead.org wrote:
On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
The allocated memory for peripheral channels is not being freed upon
failure in probe and in module's remove funtion. It will lead to memory
On 24 October 2012 09:40, Vinod Koul vk...@infradead.org wrote:
On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
In probe, memory for multiple DMA descriptors were being allocated at once
and then it was being split and added into DMA pool one by one. The address
of this memory
On 24 October 2012 09:49, Vinod Koul vk...@infradead.org wrote:
On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
unregister dma_device in module's remove function.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |2 ++
1 file changed, 2
Hi Vinod,
On 24 October 2012 09:44, Vinod Koul vk...@infradead.org wrote:
On Fri, 2012-10-05 at 15:47 +0530, Inderpal Singh wrote:
Since peripheral channel resources are not being allocated at probe,
no need to flush the channels and free the resources in remove function.
In case, the channel
reg_offset is offset of the status/mask registers. Now, since status_base
and mask_base are pointing to corresponding first registers, reg_offset
should start from 0 otheriwse regmap_add_irq_chip will fail during probe.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
It is based
On 13 October 2012 16:33, Jassi Brar jassisinghb...@gmail.com wrote:
On Fri, Oct 5, 2012 at 3:47 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
The first 2 patches of this series fix memory leaks because the memory
allocated for peripheral channels and DMA descriptors were not getting
and
free_chan_resources.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
This patch is based on slave-dma's next branch and on top
of the clean up patches at [1].
[1] https://lkml.org/lkml/2012/10/5/169
drivers/dma/pl330.c | 10 ++
1 file changed, 10 insertions(+)
diff --git a/drivers/dma
Hello,
On 5 October 2012 06:17, Inderpal Singh inderpal.si...@linaro.org wrote:
The first 2 patches of this series fix memory leaks because the memory
allocated for peripheral channels and DMA descriptors were not getting
freed.
The last 2 patches balance the module's remove function
from remove if channel is in use
- unregister dma_device in remove
Inderpal Singh (4):
DMA: PL330: Free memory allocated for peripheral channels
DMA: PL330: Change allocation method to properly free DMA
descriptors
DMA: PL330: Balance module remove function with probe
DMA: PL330
Since peripheral channel resources are not being allocated at probe,
no need to flush the channels and free the resources in remove function.
In case, the channel is in use by some client, return EBUSY.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c | 13
unregister dma_device in module's remove function.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/dma/pl330.c b/drivers/dma/pl330.c
index 4b7a34d..e7dc040 100644
--- a/drivers/dma/pl330.c
+++ b
The allocated memory for peripheral channels is not being freed upon
failure in probe and in module's remove funtion. It will lead to memory
leakage. Hence free the allocated memory.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |5 -
1 file changed, 4
descs can also be freed.
2. Free DMA descs in case of error in probe and in module's remove function
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c | 35 +--
1 file changed, 25 insertions(+), 10 deletions(-)
diff --git a/drivers
On 27 September 2012 15:18, Vinod Koul vinod.k...@linux.intel.com wrote:
On Wed, 2012-09-26 at 12:11 +0530, Inderpal Singh wrote:
If we fail pl330_remove while some client is queued, the force unload
will fail and the
force unload will lose its purpose.
How about conditionally
On 25 September 2012 18:47, Jassi Brar jassisinghb...@gmail.com wrote:
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
Since peripheral channel resources are not being allocated at probe,
no need to flush the channels and free the resources in remove function
On 26 September 2012 15:02, Jassi Brar jassisinghb...@gmail.com wrote:
On Wed, Sep 26, 2012 at 12:11 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
How about conditionally DMA_TERMINATE_ALL and free resources like below ?
@@ -3017,9 +3017,11 @@ static int __devexit pl330_remove(struct
On 26 September 2012 22:19, Jassi Brar jassisinghb...@gmail.com wrote:
On Wed, Sep 26, 2012 at 4:25 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
On 26 September 2012 15:02, Jassi Brar jassisinghb...@gmail.com wrote:
On Wed, Sep 26, 2012 at 12:11 PM, Inderpal Singh
inderpal.si
On 27 September 2012 10:35, Jassi Brar jassisinghb...@gmail.com wrote:
On Thu, Sep 27, 2012 at 9:43 AM, Inderpal Singh
inderpal.si...@linaro.org wrote:
Don't you think free_chan_resource should be done __only if__
alloc_chan_resource was successful ?
No, I don't think so. Thanks.
Thanks
descs can also be freed.
2. Free DMA descs in case of error in probe and in module's remove function
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c | 28 +---
1 file changed, 21 insertions(+), 7 deletions(-)
diff --git a/drivers/dma/pl330
Since peripheral channel resources are not being allocated at probe,
no need to flush the channels and free the resources in remove function.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |8 +---
1 file changed, 1 insertion(+), 7 deletions(-)
diff
patch at [1] to fix the build error.
[1] http://permalink.gmane.org/gmane.linux.kernel.next/24274
Inderpal Singh (3):
DMA: PL330: Free memory allocated for peripheral channels
DMA: PL330: Change allocation method to properly free DMA descriptors
DMA: PL330: Balance module remove function
The allocated memory for peripheral channels is not being freed upon
failure in probe and in module's remove funtion. It will lead to memory
leakage. Hence free the allocated memory.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |5 -
1 file changed, 4
On 25 September 2012 18:17, Jassi Brar jassisinghb...@gmail.com wrote:
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
The allocated memory for peripheral channels is not being freed upon
failure in probe and in module's remove funtion. It will lead to memory
On 25 September 2012 18:39, Jassi Brar jassisinghb...@gmail.com wrote:
On Tue, Sep 25, 2012 at 2:27 PM, Inderpal Singh
inderpal.si...@linaro.org wrote:
In probe, memory for multiple DMA descriptors were being allocated at once
and then it was being split and added into DMA pool one by one
Since 0 is not considered as error at dmaengine level, return ENOMEM
from pl330_alloc_chan_resources in case of failure.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/dma/pl330.c b
Hi Kukjin,
On 15 May 2012 10:01, Inderpal Singh inderpal.si...@linaro.org wrote:
Hi Kukjin,
On 14 May 2012 19:13, Kukjin Kim kgene@samsung.com wrote:
On 05/14/12 17:42, Inderpal Singh wrote:
Hi Kukjin,
On 14 May 2012 13:45, Kukjin Kimkgene@samsung.com wrote:
Inderpal Singh
the redundant clock enable/disable from the driver.
Inderpal Singh (2):
DMA: PL330: Remove controller clock enable/disable
DMA: PL330: Remove redundant runtime_suspend/resume functions
drivers/dma/pl330.c | 73 ---
1 file changed, 5 insertions
The controller clock is being enabled/disabled in AMBA bus
infrastructre in probe/remove functions. Hence, its not required
at driver level probe/remove.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma/pl330.c | 12
1 file changed, 12 deletions(-)
diff
The driver's runtime_suspend/resume functions just disable/enable
the clock which is already being managed at AMBA bus level
runtime_suspend/resume functions.
Hence, remove the driver's runtime_suspend/resume functions.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
drivers/dma
Hi Kukjin,
On 14 May 2012 13:45, Kukjin Kim kgene@samsung.com wrote:
Inderpal Singh wrote:
Upon wake-up, clear the sleep mode set in INFORM1 register.
Is this for wake-up or _really_ software reset? And is this available on all
of EXYNOS4 SoCs?
This is for soft reboot hang after one
Hi Kukjin,
On 14 May 2012 19:13, Kukjin Kim kgene@samsung.com wrote:
On 05/14/12 17:42, Inderpal Singh wrote:
Hi Kukjin,
On 14 May 2012 13:45, Kukjin Kimkgene@samsung.com wrote:
Inderpal Singh wrote:
Upon wake-up, clear the sleep mode set in INFORM1 register.
Is this for wake
Upon wake-up, clear the sleep mode set in INFORM1 register.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
arch/arm/mach-exynos/pm.c |4
1 files changed, 4 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-exynos/pm.c b/arch/arm/mach-exynos/pm.c
index f0bb467
This patch provides the suspend/resume support for EXYNOS4412.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
arch/arm/mach-exynos/include/mach/regs-pmu.h | 10 +-
arch/arm/mach-exynos/pm.c|2 +-
arch/arm/mach-exynos/pmu.c | 24
Hi Amit,
On 22 November 2011 15:05, Amit Daniel Kachhap amit.kach...@linaro.org wrote:
This patch adds support AFTR(ARM OFF TOP RUNNING) mode in
cpuidle driver. L2 cache keeps their data in this mode.
This patch ports the code to the latest interfaces to
save/restore CPU state inclusive of
-by: Inderpal Singh inderpal.si...@linaro.org
---
1. This change is applicable only for s5pc100, s5pv210 and Exynos
platforms. For all other platforms, the driver strength registers are
part of special port configuration register (SPCON) and these
registers are saved and restored separately from
Hi Kukjin,
Thanks for applying.
I would like to use inderpal.si...@linaro.org
Have updated my .gitconfig accordingly as well.
Thanks,
Inder
On 3 October 2011 08:57, Kukjin Kim kgene@samsung.com wrote:
Kukjin Kim wrote:
Inderpal Singh wrote:
From: Inderpal Singh inderpa...@samsung.com
From: Inderpal Singh inderpa...@samsung.com
Configure MAX8997 PMIC and provide platform specific data
for origen.
Signed-off-by: Inderpal Singh inderpal.si...@linaro.org
---
Changes from v1:
- Removed soc-audio
- Used macro for GPIO pin
- irq assigned statically
On 16 August 2011 20:49, Mark Brown broo...@opensource.wolfsonmicro.com wrote:
On Tue, Aug 16, 2011 at 04:38:46PM +0530, Inderpal Singh wrote:
On 14 August 2011 20:31, Mark Brown
broo...@opensource.wolfsonmicro.comwrote:
On Thu, Aug 11, 2011 at 09:26:05AM +0530, Inderpal Singh wrote
1. After wake-up, the system-wide flags register loses its value.
Hence, write the address of secondary startup function to
successfully boot the secondary CPU.
2. Changes SGI1 to SGI0 for secondary CPU boot up
Signed-off-by: Inderpal Singh inderpa...@samsung.com
---
1. The below patch
1. After wake-up, the system-wide flags register loses its value.
Hence, write the address of secondary startup function to
successfully boot the secondary CPU.
2. Fix to remove the Unknown IPI message 0x1 message when
secondary CPU boots.
Signed-off-by: Inderpal Singh inderpa
Adds device definition to enable SATA on SMDKV310
Signed-off-by: Inderpal Singh inderpa...@samsung.com
---
arch/arm/mach-exynos4/Kconfig |1 +
arch/arm/mach-exynos4/mach-smdkv310.c |1 +
2 files changed, 2 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-exynos4/Kconfig
controller registers conditional based
on whether it is already enabled or not.
Signed-off-by: Inderpal Singh inderpa...@samsung.com
Signed-off-by: Jaecheol Lee jc@samsung.com
---
arch/arm/mach-exynos4/pm.c | 14 +-
1 files changed, 13 insertions(+), 1 deletions(-)
diff --git
This patch adds the parent clock for watchdog timer for SMDKV310.
Signed-off-by: Inderpal Singh inderpa...@samsung.com
---
arch/arm/mach-exynos4/clock.c |1 +
1 files changed, 1 insertions(+), 0 deletions(-)
diff --git a/arch/arm/mach-exynos4/clock.c b/arch/arm/mach-exynos4/clock.c
index
64 matches
Mail list logo