On Sun, Aug 10, 2014 at 1:28 PM, Dave Airlie wrote:
>>
>
> Been on holidays, but ack for anything DT related,
>
> Look I wishI knew how DT worked, and what tree is canonical for changes to
> it, I'm not sure its well defined enough or sub maintaniers know enough,
> so I do just trust the exynos gu
ue with ehci-exynos and ohci-exynos, wherein
> in the absence of SAMSUNG_USB2PHY config symbol, we end up getting
> the NOP_USB_XCEIV phy when the same is enabled. And thus the PHYs
> are not configured properly.
>
> Reported-by: Sachin Kamat
> Signed-off-by: Vivek Gautam
>
On Fri, Jul 11, 2014 at 3:41 PM, Tomasz Figa wrote:
> On 11.07.2014 11:57, Sachin Kamat wrote:
>> + Tomasz
>>
>> Sorry for missing the maintainer himself :)
>>
>> On Fri, Jul 11, 2014 at 2:29 PM, Sachin Kamat
>> wrote:
>>> Update to ref
the error lables and instead releases resources and returns
> after failures
>
> CC: Dmitry Torokhov
> CC: Lejun Zhu
> CC: Sachin Kamat
>
> Signed-off-by: Pramod Gurav
> ---
>
> Changes since v1:
> - Removes de_err logs on OOM cases which is not nee
from err_free_mem to err_mem.
> - Adds couple of dev_err on failure to allocate memory
>
> CC: Dmitry Torokhov
> CC: Lejun Zhu
> CC: Sachin Kamat
>
> Signed-off-by: Pramod Gurav
> ---
> drivers/input/misc/soc_button_array.c | 19 ++-
> 1 file chang
‘rx_buf’ is not used in this function.
Signed-off-by: Sachin Kamat
---
drivers/mfd/tps65912-spi.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/mfd/tps65912-spi.c b/drivers/mfd/tps65912-spi.c
index 69a5178bf152..de60ad98bd9f 100644
--- a/drivers/mfd/tps65912
Removed code unused in this function.
Signed-off-by: Sachin Kamat
Cc: Cory Maccarrone
---
Compile tested.
---
drivers/mfd/htc-i2cpld.c |5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/mfd/htc-i2cpld.c b/drivers/mfd/htc-i2cpld.c
index d7b2a75aca3e..b44f0203983b 100644
--- a
Hi Naveen,
On Thu, Jul 17, 2014 at 4:49 PM, Naveen Krishna Chatradhi
wrote:
> This patch updates the IIO based ADC driver to use syscon and regmap
> APIs to access and use PMU registers instead of remapping the PMU
> registers in the driver.
>
> Signed-off-by: Naveen Krishna Chatradhi
> To: linu
that they are meant to work with. This
will make life easier for end users who do not have any way knowing the
dependencies.
Signed-off-by: Sachin Kamat
Reviewed-by: Jingoo Han
---
Changes since v3:
* Rebased the series on top of latest PHY tree [1].
[1] git://git.kernel.org/pub/scm/linux/kernel
USB DWC3 driver on Exynos platform does not work without its
corresponding phy driver. Hence make the PHY driver depend on
Exynos DWC3 driver and default it to yes to make things easier
for the end user.
Signed-off-by: Sachin Kamat
Reviewed-by: Jingoo Han
---
drivers/phy/Kconfig |2 ++
1
On Mon, Jul 14, 2014 at 2:39 PM, Kishon Vijay Abraham I wrote:
>
>
> On Monday 14 July 2014 02:36 PM, Sachin Kamat wrote:
>> On Mon, Jul 14, 2014 at 2:32 PM, Kishon Vijay Abraham I
>> wrote:
>>>
>>>
>>> On Monday 14 July 2014 02:29 PM, Sachin Kama
On Mon, Jul 14, 2014 at 2:32 PM, Kishon Vijay Abraham I wrote:
>
>
> On Monday 14 July 2014 02:29 PM, Sachin Kamat wrote:
>> Hi,
>>
>> On Mon, Jul 14, 2014 at 2:26 PM, Kishon Vijay Abraham I
>> wrote:
>>> Hi,
>>>
>>> On Wednesday 09 Ju
Hi,
On Mon, Jul 14, 2014 at 2:26 PM, Kishon Vijay Abraham I wrote:
> Hi,
>
> On Wednesday 09 July 2014 05:19 PM, Sachin Kamat wrote:
>> Since the USB 2.0 PHYs are required with EHCI/OHCI USB drivers and
>> USB gadget controller supported by the DWC2 gadget driver, make it
‘rx_buf’ is not used in this function.
Signed-off-by: Sachin Kamat
---
drivers/mfd/tps65912-spi.c |3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/mfd/tps65912-spi.c b/drivers/mfd/tps65912-spi.c
index 69a5178bf152..de60ad98bd9f 100644
--- a/drivers/mfd/tps65912
Removed code unused in this function.
Signed-off-by: Sachin Kamat
---
Compile tested.
---
drivers/mfd/htc-i2cpld.c |5 -
1 file changed, 5 deletions(-)
diff --git a/drivers/mfd/htc-i2cpld.c b/drivers/mfd/htc-i2cpld.c
index d7b2a75aca3e..b44f0203983b 100644
--- a/drivers/mfd/htc
+ Tomasz
Sorry for missing the maintainer himself :)
On Fri, Jul 11, 2014 at 2:29 PM, Sachin Kamat wrote:
> Update to reflect the recent file movement to a sub-directory.
>
> Signed-off-by: Sachin Kamat
> ---
> MAINTAINERS |4 +---
> 1 file changed, 1 insertion(+), 3 del
Hi Naveen,
On Fri, Jul 11, 2014 at 2:36 PM, Naveen Krishna Chatradhi
wrote:
> The DT bindings in exynos-adc.txt applies to the ADC
> driver (exynos-adc.c) developed based on IIO framework.
>
> The bindings are more appropriate to be under
> Documentation/devicetree/bindings/iio/adc/
>
> Signed-of
Update to reflect the recent file movement to a sub-directory.
Signed-off-by: Sachin Kamat
---
MAINTAINERS |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/MAINTAINERS b/MAINTAINERS
index e3a517635eb4..898cf76f8837 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -6943,9
t;adap.retries = 2;
> - i2c->adap.class = I2C_CLASS_HWMON | I2C_CLASS_SPD |
> I2C_CLASS_DEPRECATED;
> - i2c->tx_setup = 50;
> + i2c->adap.class = I2C_CLASS_DEPRECATED;
> + i2c->tx_setup = 50;
>
> init_waitqueue_head(&i2c->wait);
Group all pin control drivers of Samsung platform together in
a sub-directory for easy maintenance.
Signed-off-by: Sachin Kamat
---
This patch has been inspired by a similar patch [1] from Linus Walleij for
Qualcomm drivers. I have boot tested this patch on Exynos boards and
compile tested for
Hi Joonyoung,
On Thu, Jul 10, 2014 at 9:53 AM, Joonyoung Shim wrote:
> This fixes below build warning.
>
> drivers/usb/misc/usb3503.c: In function ‘usb3503_probe’:
> drivers/usb/misc/usb3503.c:195:11: warning: ‘err’ may be used uninitialized
> in this function [-Wmaybe-uninitialized]
> dev_e
that they are meant to work with. This
will make life easier for end users who do not have any way knowing the
dependencies.
Signed-off-by: Sachin Kamat
---
Changes since v1:
* Added dependency on DWC2 gadget driver as suggested by Tomasz Figa.
---
drivers/phy/Kconfig | 35
USB DWC3 driver on Exynos platform does not work without its
corresponding phy driver. Hence make the PHY driver depend on
Exynos DWC3 driver and default it to yes to make things easier
for the end user.
Signed-off-by: Sachin Kamat
---
No changes since v1.
---
drivers/phy/Kconfig |2 ++
1
Hi Tomasz,
On Tue, Jul 8, 2014 at 9:15 PM, Tomasz Figa wrote:
> On 30.06.2014 10:56, Sachin Kamat wrote:
>> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
>
> That's not true. They are also required for USB gadget controller
> supported by the DWC2
On Tue, Jul 8, 2014 at 10:49 AM, Shawn Guo wrote:
> On Tue, Jul 08, 2014 at 09:18:56AM +0530, Sachin Kamat wrote:
>> Hi Shawn,
>>
>> On Mon, Jul 7, 2014 at 11:25 AM, Shawn Guo wrote:
>> > On Fri, Jul 04, 2014 at 09:03:10PM +0200, Fabian Frederick wrote
On Mon, Jun 30, 2014 at 2:26 PM, Sachin Kamat wrote:
> Since the USB 2.0 PHYs are required only with EHCI/OHCI USB drivers,
> make it depend on them and default to ARCH_EXYNOS as they are meant
> for Exynos platforms. Also, make the sub-drivers silent options enabling
> them base
Hi Shawn,
On Mon, Jul 7, 2014 at 11:25 AM, Shawn Guo wrote:
> On Fri, Jul 04, 2014 at 09:03:10PM +0200, Fabian Frederick wrote:
>> replace IS_ERR/PTR_ERR
>>
>> Cc: Shawn Guo
>> Cc: Sascha Hauer
>> Cc: linux-arm-ker...@lists.infradead.org
>> Signed-off-by: Fabian Frederick
>
> Applied, thanks.
On Sat, Jul 5, 2014 at 6:18 AM, J. Bruce Fields wrote:
> On Sat, Jul 05, 2014 at 01:10:58AM +1000, Stephen Rothwell wrote:
>> Hi Sachin,
>>
>> On Fri, 4 Jul 2014 14:12:11 +0530 Sachin Kamat wrote:
>> >
>> > Was bisecting a kernel crash on Arndale octa
On Fri, Jul 4, 2014 at 8:40 PM, Stephen Rothwell wrote:
> Hi Sachin,
>
> On Fri, 4 Jul 2014 14:12:11 +0530 Sachin Kamat wrote:
>>
>> Was bisecting a kernel crash on Arndale octa board (Exynos5420). It
>> points to a merge
>> commit:
>> 40556a4c485d12d324
IOMMU Support"
> depends on ARCH_EXYNOS
> select IOMMU_API
> + select ARM_DMA_USE_IOMMU
> help
> Support for the IOMMU (System MMU) of Samsung Exynos application
> processor family. This enables H/W multimedia accelerato
Hi Stephen,
Was bisecting a kernel crash on Arndale octa board (Exynos5420). It
points to a merge
commit:
40556a4c485d12d324f1ea196cc30f590e564237 is the first bad commit
("Merge remote-tracking branch 'nfsd/nfsd-next'").
How do I proceed with this?
Bisect log as follows:
git bisect start
# goo
n Warren
>>
>> [with the s/static// already mentioned in this thread, obviously:-)]
>
> Right; I pushed out a fixed version right before loosing my tubes at the
> airport :-)
>
> https://git.kernel.org/cgit/linux/kernel/git/peterz/queue.git/commit/?h=timers/nohz&id=921d8b81281ecdca686369f52165d04fa350
On Fri, Jul 4, 2014 at 5:23 AM, Sangbeom Kim wrote:
> On Friday, July 04, 2014 7:08 AM, Joe Perches wrote:
>
>> F: drivers/regulator/s2m*.c
>> F: drivers/regulator/s5m*.c
>> -F: drivers/rtc/rtc-sec.c
>> F: include/linux/mfd/samsung/
>>
>
> Yes, You are right.
> There is no rtc-sec.c
> I
On Thu, Jun 12, 2014 at 3:27 PM, Kishon Vijay Abraham I wrote:
>
>
> On Thursday 12 June 2014 03:19 PM, Sachin Kamat wrote:
>> Hi Kishon,
>>
>> On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han wrote:
>>> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote:
>
On Wed, Jul 2, 2014 at 5:22 AM, Chanwoo Choi wrote:
> On 07/01/2014 01:10 PM, Sachin Kamat wrote:
>> Hi Chanwoo,
>>
>> On Tue, Jul 1, 2014 at 9:34 AM, Chanwoo Choi wrote:
>>> Hi Sachin,
>>>
>>> On 07/01/2014 12:33 PM, Sachin Kamat wrote:
>>&
d-variable]
drivers/clk/clk.c:46:27: warning: ‘orphan_list’ defined but not used
[-Wunused-variable]
Signed-off-by: Sachin Kamat
Cc: Peter De Schrijver
---
Changes since v1:
Moved the definitions under already existing #ifdefs.
---
drivers/clk/clk.c | 22 +++---
1 file c
On Mon, Jun 30, 2014 at 7:07 PM, Peter De Schrijver
wrote:
> On Mon, Jun 30, 2014 at 11:40:54AM +0200, Sachin Kamat wrote:
>> ‘all_lists’ and ‘orphan_list’ is accessed only when DEBUG_FS is defined.
>> Thus, make their compilation conditional to fix the below warnings introduc
Hi Chanwoo,
On Tue, Jul 1, 2014 at 9:34 AM, Chanwoo Choi wrote:
> Hi Sachin,
>
> On 07/01/2014 12:33 PM, Sachin Kamat wrote:
>> Hi Chanwoo,
>>
>> On Tue, Jul 1, 2014 at 6:10 AM, Chanwoo Choi wrote:
>>> This patch add TMU (Thermal Management Unit) dt node to
Hi Chanwoo,
On Tue, Jul 1, 2014 at 6:10 AM, Chanwoo Choi wrote:
> This patch add TMU (Thermal Management Unit) dt node to monitor the high
> temperature for Exynos3250.
>
> Signed-off-by: Chanwoo Choi
> Acked-by: Kyungmin Park
> ---
> This patch has a dependency on following patch [1]:
> [1] h
d-variable]
drivers/clk/clk.c:46:27: warning: ‘orphan_list’ defined but not used
[-Wunused-variable]
Signed-off-by: Sachin Kamat
Cc: Peter De Schrijver
---
drivers/clk/clk.c |2 ++
1 file changed, 2 insertions(+)
diff --git a/drivers/clk/clk.c b/drivers/clk/clk.c
index 7dfb2f308b35..a9
USB DWC3 driver on Exynos platform does not work without its
corresponding phy driver. Hence make the PHY driver depend on
Exynos DWC3 driver and default it to yes to make things easier
for the end user.
Signed-off-by: Sachin Kamat
---
drivers/phy/Kconfig |2 ++
1 file changed, 2 insertions
easier for end users who do not have any way knowing the
dependencies.
Signed-off-by: Sachin Kamat
---
drivers/phy/Kconfig | 35 +++
1 file changed, 11 insertions(+), 24 deletions(-)
diff --git a/drivers/phy/Kconfig b/drivers/phy/Kconfig
index 16a2f067c242
_boot_reg()
> again.
>
> Signed-off-by: Krzysztof Kozlowski
> ---
Thanks for the patch.
Reviewed-by: Sachin Kamat
Regards,
Sachin.
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo
Hi Naveen,
On Wed, Jun 25, 2014 at 4:08 PM, Naveen Krishna Chatradhi
wrote:
> This patch removes an extra line and fixes a styling nit
> in exynos5_i2c_message_start()
>
> Signed-off-by: Naveen Krishna Chatradhi
> ---
> drivers/i2c/busses/i2c-exynos5.c |3 +--
> 1 file changed, 1 insertion(
‘irq_bit’ is unused in the function. Remove it.
Signed-off-by: Sachin Kamat
---
Compile tested.
---
drivers/mfd/pm8921-core.c |4
1 file changed, 4 deletions(-)
diff --git a/drivers/mfd/pm8921-core.c b/drivers/mfd/pm8921-core.c
index 959513803542..39904f77c049 100644
--- a/drivers/mfd
'size' is not used in the function. Remove it.
Signed-off-by: Sachin Kamat
---
drivers/regulator/s5m8767.c |4 +---
1 file changed, 1 insertion(+), 3 deletions(-)
diff --git a/drivers/regulator/s5m8767.c b/drivers/regulator/s5m8767.c
index c79af943a5c0..0ab5cbeeb797 100644
---
ned
expression < 0 is always false [-Wtype-limits]
Signed-off-by: Sachin Kamat
---
Series compile tested.
---
drivers/regulator/tps65217-regulator.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/tps65217-regulator.c
b/drivers/regulator/tps65217-regul
ned
expression < 0 is always false [-Wtype-limits]
Signed-off-by: Sachin Kamat
---
drivers/regulator/tps65218-regulator.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/tps65218-regulator.c
b/drivers/regulator/tps65218-regulator.c
index 69b4b7
'name' field was re-initialized and getting overwritten in some
cases possibly due to a typo. Code inspection says the second time
it should be 'sname' instead of 'name'. Replace it.
Signed-off-by: Sachin Kamat
---
Compile tested only.
---
drivers/regulator/as3
Hi Sherman,
On Mon, Jun 23, 2014 at 11:35 PM, Sherman Yin wrote:
> Hi Sachin,
>
> Looks good, thanks. How did you come across the need for this change?
Thanks for the review. Sparse threw up a warning while doing a regular build.
Regards,
Sachin.
--
To unsubscribe from this list: send the line
bcm281xx_pinctrl_probe is local to this file. Make it static.
Signed-off-by: Sachin Kamat
---
drivers/pinctrl/pinctrl-bcm281xx.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-bcm281xx.c
b/drivers/pinctrl/pinctrl-bcm281xx.c
index 3bed792b2c03
module.h was included twice.
Signed-off-by: Sachin Kamat
---
drivers/spmi/spmi.c |1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/spmi/spmi.c b/drivers/spmi/spmi.c
index 3b5780710d50..1d92f5103ebf 100644
--- a/drivers/spmi/spmi.c
+++ b/drivers/spmi/spmi.c
@@ -18,7 +18,6
3c24xx_uart_drv);
> @@ -1335,9 +1335,6 @@ static int s3c24xx_serial_probe(struct platform_device
> *pdev)
> dev_err(&pdev->dev, "failed to add cpufreq notifier\n");
>
> return 0;
> -
> - probe_err:
> - return ret;
> }
>
> s
s2m_osc: clocks {
> compatible = "samsung,s2mps11-clk";
> - #clock-cells = 1;
> + #clock-cells = <1>;
> clock-output-names = "xx", "yy", "zz";
; - valid values for n are 1 to 28
> - - Example: LDO1, LD02, LDO28
> + - Example: LDO1, LDO2, LDO28
> - BUCKn
> - valid values for n are 1 to 9.
> - Example: BUCK1, BUCK2, BUCK9
> --
'ltc3589_reg_defaults' is local to this file. Make it static.
Signed-off-by: Sachin Kamat
---
drivers/regulator/ltc3589.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/ltc3589.c b/drivers/regulator/ltc3589.c
index c8105182b8b8..c756955bf
: Sachin Kamat
---
drivers/thunderbolt/switch.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/thunderbolt/switch.c b/drivers/thunderbolt/switch.c
index 0d50e7e7b29b..26e76e4aa835 100644
--- a/drivers/thunderbolt/switch.c
+++ b/drivers/thunderbolt/switch.c
@@ -5,6 +5,7 @@
*/
#include
'nhi_ids' is local to this file.
Signed-off-by: Sachin Kamat
---
drivers/thunderbolt/nhi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thunderbolt/nhi.c b/drivers/thunderbolt/nhi.c
index 0fc137af89f5..2054fbf8b382 100644
--- a/drivers/thunderbolt/n
The function returns a pointer. Hence return NULL instead of 0.
Signed-off-by: Sachin Kamat
---
drivers/thunderbolt/ctl.c |4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/thunderbolt/ctl.c b/drivers/thunderbolt/ctl.c
index d04fee4acb2e..4c6da92edcb4 100644
The function returns a pointer. Hence return NULL instead of 0.
Signed-off-by: Sachin Kamat
---
drivers/thunderbolt/switch.c |6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/thunderbolt/switch.c b/drivers/thunderbolt/switch.c
index 26e76e4aa835..aeb982969629
'descriptors' is a pointer. Use NULL isntead of 0.
Signed-off-by: Sachin Kamat
---
drivers/thunderbolt/nhi.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/thunderbolt/nhi.c b/drivers/thunderbolt/nhi.c
index 346b41e7d5d1..0fc137af89f5 100644
---
Fixes the below error:
drivers/thunderbolt/eeprom.c:407:2: error: implicit declaration of function
‘kzalloc’ [-Werror=implicit-function-declaration]
drivers/thunderbolt/eeprom.c:444:2: error: implicit declaration of function
‘kfree’ [-Werror=implicit-function-declaration]
Signed-off-by: Sachin
On Wed, Jun 18, 2014 at 5:50 PM, Russell King - ARM Linux
wrote:
> On Wed, Jun 18, 2014 at 05:38:14PM +0530, Sachin Kamat wrote:
>> On Wed, Jun 18, 2014 at 5:26 PM, Russell King - ARM Linux
>> wrote:
>> > On Wed, Jun 18, 2014 at 04:50:43PM +0530, Sachin Kamat wrote:
On Wed, Jun 18, 2014 at 5:26 PM, Russell King - ARM Linux
wrote:
> On Wed, Jun 18, 2014 at 04:50:43PM +0530, Sachin Kamat wrote:
>> Hi,
>>
>> I observe the following build error while building uImage for ARM.
>>
>> Kernel: arch/arm/boot/Image is ready
>&
Hi,
I observe the following build error while building uImage for ARM.
Kernel: arch/arm/boot/Image is ready
make[2]: *** No rule to make target
`arch/arm/boot/compressed/vmlinux.lds', needed by
`arch/arm/boot/compressed/vmlinux'. Stop.
make[1]: *** [arch/arm/boot/compressed/vmlinux] Error 2
ma
Hi Srivatsa,
On Tue, Jun 17, 2014 at 3:24 PM, Srivatsa S. Bhat
wrote:
> On 06/17/2014 03:03 PM, Sachin Kamat wrote:
>
>>> Below is an updated patch, please let me know how it goes. (You'll have to
>>> revert c47a9d7cca first, and then 56e692182, before trying this
Hi Srivatsa,
Thanks for your prompt reply.
On Tue, Jun 17, 2014 at 2:48 PM, Srivatsa S. Bhat
wrote:
> Hi Sachin,
>
> On 06/17/2014 01:39 PM, Sachin Kamat wrote:
>> Hi,
>>
>> I observe the below warnings while trying to boot Exynos5420 based boards
>> since yest
Hi,
I observe the below warnings while trying to boot Exynos5420 based boards
since yesterday's linux-next (next-20140616) using multi_v7_defconfig. Looks
like it is triggered by the commit 56e6921829 ("CPU hotplug, smp:
flush any pending IPI callbacks before CPU offline"). Any ideas?
**
On Tue, Jun 17, 2014 at 8:55 AM, Joe Perches wrote:
> On Tue, 2014-06-17 at 08:46 +0530, Sachin Kamat wrote:
>> On Tue, Jun 17, 2014 at 7:30 AM, Joe Perches wrote:
>> > On Mon, 2014-06-16 at 17:44 -0700, Anish Bhatt wrote:
>> >> My code has multiple exit la
On Tue, Jun 17, 2014 at 7:30 AM, Joe Perches wrote:
> On Mon, 2014-06-16 at 17:44 -0700, Anish Bhatt wrote:
>> My code has multiple exit lables:
>> void function(void)
>> {
>> ...
>>
>> if (err1)
>> goto exit1;
>> ...
>> if (err2)
>> goto exit2;
On Sun, Jun 15, 2014 at 2:03 AM, Joe Perches wrote:
> Greg? Can you please apply this soon?
A similar patch has already been posted [1]. Probably Greg will apply it
once 3.16-rc1 is out.
[1] http://www.spinics.net/lists/linux-serial/msg12843.html
Regards,
Sachin.
>
> Thanks.
>
> On Sat, 2014
On Thu, Jun 12, 2014 at 3:27 PM, Kishon Vijay Abraham I wrote:
>
>
> On Thursday 12 June 2014 03:19 PM, Sachin Kamat wrote:
>> Hi Kishon,
>>
>> On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han wrote:
>>> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote:
>
Hi Kishon,
On Mon, Jun 2, 2014 at 7:56 AM, Jingoo Han wrote:
> On Thursday, May 29, 2014 3:31 PM, Sachin Kamat wrote:
>>
>> PTR_ERR_OR_ZERO simplifies the code.
>>
>> Signed-off-by: Sachin Kamat
>> Cc: Jingoo Han
>
> Acked-by: Jingoo Han
>
> Best r
Hi Guillaume,
On Thu, Jun 12, 2014 at 1:48 PM, Guillaume Gardet
wrote:
> Hi,
>
> please find in attachment a patch which fixes a typo in
> drivers/phy/phy-exynos5250-sata.c module license.
> It fixes the following build error:
> FATAL: modpost: GPL-incompatible module phy-exynos5250-sata.ko
We previously assumed 'mem_sdio' could be null but it is
dereferenced in ioremap(). Add a check to avoid a potential
null pointer dereference error.
Signed-off-by: Sachin Kamat
---
drivers/mfd/asic3.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/d
'ltc3589_reg_defaults' is local to this file. Make it static.
Signed-off-by: Sachin Kamat
---
drivers/regulator/ltc3589.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/ltc3589.c b/drivers/regulator/ltc3589.c
index 110a99e..217d10d6 100644
---
Hi Arnd,
On Tue, Jun 3, 2014 at 3:23 PM, Arnd Bergmann wrote:
> commit e4ac92df2791 ("serial: samsung: Neaten dbg uses") introduced
> a regression in the conversion from vsprintf to vsnprintf.
>
> This fixes the build error by passing the correct variable name.
I submitted a similar patch someti
On Tue, Jun 3, 2014 at 3:37 AM, Stephen Boyd wrote:
> This config exists entirely to hide the cpufreq menu from the
> kernel configuration unless a platform has selected it. Nothing
> is actually built if this config is 'Y' and it just leads to
> more patches that add a select under a platform Kco
kzalloc can fail. Add a null check to avoid null pointer
dereference error while accessing the pointer later.
Signed-off-by: Sachin Kamat
---
drivers/pinctrl/sunxi/pinctrl-sunxi.c |4
1 file changed, 4 insertions(+)
diff --git a/drivers/pinctrl/sunxi/pinctrl-sunxi.c
b/drivers/pinctrl
We previously assumed 'mem_sdio' could be null but it is
dereferenced in ioremap(). Add a check to avoid a potential
null pointer dereference error.
Signed-off-by: Sachin Kamat
---
drivers/mfd/asic3.c | 12 +++-
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/d
Hi Viresh,
On 27 May 2014 17:20, Viresh Kumar wrote:
> All callers of of_init_opp_table() are required to take reference of
> dev->of_node, by initiating calls to of_node_{get|put}(), before and after
> calling of_init_opp_table().
>
> Its better to call these from within of_init_opp_table(), no
module_i2c_driver simplifies the code by eliminating
boilerplate code.
Signed-off-by: Sachin Kamat
---
drivers/mfd/menelaus.c | 23 +--
1 file changed, 1 insertion(+), 22 deletions(-)
diff --git a/drivers/mfd/menelaus.c b/drivers/mfd/menelaus.c
index ad25bfa3fb02
PTR_ERR_OR_ZERO simplifies the code.
Signed-off-by: Sachin Kamat
Cc: Hans de Goede
Cc: Maxime Ripard
---
drivers/phy/phy-sun4i-usb.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-sun4i-usb.c b/drivers/phy/phy-sun4i-usb.c
index 115d8d5190d5
PTR_ERR_OR_ZERO simplifies the code.
Signed-off-by: Sachin Kamat
Cc: Jingoo Han
---
drivers/phy/phy-exynos-dp-video.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-exynos-dp-video.c
b/drivers/phy/phy-exynos-dp-video.c
index 0786fef842e7
PTR_ERR_OR_ZERO simplifies the code.
Signed-off-by: Sachin Kamat
Cc: Sylwester Nawrocki
---
drivers/phy/phy-exynos-mipi-video.c |5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-exynos-mipi-video.c
b/drivers/phy/phy-exynos-mipi-video.c
index
Symbols local to this file are made static.
Signed-off-by: Sachin Kamat
---
arch/arm/mach-ux500/cpu-db8500.c |8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/arch/arm/mach-ux500/cpu-db8500.c b/arch/arm/mach-ux500/cpu-db8500.c
index fa308f07fae5..6f63954c8bde 100644
'ab8505_regulators' is used only in this file.
Signed-off-by: Sachin Kamat
---
arch/arm/mach-ux500/board-mop500-regulators.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-ux500/board-mop500-regulators.c
b/arch/arm/mach-ux500/board-mop500-re
Hi Lee,
On 28 May 2014 14:06, Lee Jones wrote:
> On Wed, 28 May 2014, Sachin Kamat wrote:
>
>> 'ux500_soc_attr' is local to this file. While at it also make it
>> const to match the argument list of device_create_file.
>
> What's going on Sachin? I
'ux500_soc_attr' is local to this file. While at it also make it
const to match the argument list of device_create_file.
Signed-off-by: Sachin Kamat
---
arch/arm/mach-ux500/cpu.c |2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/arch/arm/mach-ux500/cpu.c b/arc
Export this symbol to avoid build error when built as module.
ERROR: "devm_regmap_init_vexpress_config" [drivers/regulator/vexpress.ko]
undefined!
Signed-off-by: Sachin Kamat
---
drivers/bus/vexpress-config.c |1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/bus/vexpres
ead to updating of ramp delay of buck3 when setting buck1
> and actually the ramp delay of buck1 was never set.
>
> Cc:
> Fixes: f18792714608 ("regulator: Add support for S2MPA01 regulator")
> Signed-off-by: Krzysztof Kozlowski
Reviewed-by: Sachin Kamat
--
With warm reg
Hi Krzysztof
Apologies for the delay. I was on vacation during the early part of
this month and
got busy with some other stuff later and this mail fell through the cracks.
On 23 May 2014 19:49, Krzysztof Kozlowski wrote:
> Hi Sachin,
>
> The s2mpa01 regulator driver uses wrong registers for ram
Hi Chanwoo,
On 26 May 2014 10:27, Chanwoo Choi wrote:
> This patch add IRQF_NO_SUSPEND flag before requesting interrupt handler
> if button->wakeup is true. If ther interrupt of gpio-keys hasn't included
> IRQF_NO_SUSPEND, wouldn't wake-up from freeze state when pressing button.
Please refer to
igned-off-by: Paul Bolle
Reviewed-by: Sachin Kamat
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Hi Paul,
On 22 May 2014 12:11, Paul Bolle wrote:
> Sachin,
>
> On Thu, 2014-05-22 at 08:52 +0530, Sachin Kamat wrote:
>> The config symbol itself is not defined and hence could be removed.
>> However I feel, the
>> error message could be retained after trimming down
On 22 May 2014 09:44, Viresh Kumar wrote:
> On 22 May 2014 08:52, Sachin Kamat wrote:
>> On 22 May 2014 01:23, Paul Bolle wrote:
>>> A call of pr_err() was added in v3.1. It was guarded by a check for
>>> CONFIG_PM_VERBOSE. The Kconfig symbol PM_VERBOSE was removed i
On 22 May 2014 01:23, Paul Bolle wrote:
> A call of pr_err() was added in v3.1. It was guarded by a check for
> CONFIG_PM_VERBOSE. The Kconfig symbol PM_VERBOSE was removed in v3.0. So
> this call of pr_err() has never been used. Remove it.
>
> Signed-off-by: Paul Bolle
> ---
> Untested.
>
> dri
On 30 April 2014 14:25, Vivek Gautam wrote:
> Add required device node for ehci and ohci controllers to
> enable USB 2.0 support.
>
> Signed-off-by: Vivek Gautam
> ---
> arch/arm/boot/dts/exynos5420.dtsi | 36 +++-
> 1 file changed, 35 insertions(+), 1 deletion(
Hi Vivek,
On 30 April 2014 14:25, Vivek Gautam wrote:
> From: Kamil Debski
>
> Add support to PHY of USB2 of the Exynos 5250 SoC.
>
> Signed-off-by: Kamil Debski
> [gautam.vi...@samsung.com: Split the usb phy entries from
> syscon entries from earlier patch: dts: Add usb2phy to Exynos 5250]
> [
Hi Krzysztof,
On 30 April 2014 18:56, Krzysztof Kozlowski wrote:
> Dear Sachin,
>
>
> The S2MPA01 and S2MPS1X PMICs regulator drivers look quite similar to
> each other. Most of the S2MPA01 driver is the same as S2MPS11 (except
> changes added for supporting S2MPS14).
>
> I would like to merge th
1 - 100 of 679 matches
Mail list logo
