[PATCH 3/3] block: brd: Staticize 'rd_size'

2013-08-08 Thread Jingoo Han
'rd_size' is used only in this file. Fix the following sparse warning: drivers/block/brd.c:432:5: warning: symbol 'rd_size' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/block/brd.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-)

[PATCH 2/3] block: mg_disk: Staticize mg_times_out()

2013-08-08 Thread Jingoo Han
mg_times_out() is used only in this file. Fix the following sparse warning: drivers/block/mg_disk.c:639:6: warning: symbol 'mg_times_out' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/block/mg_disk.c |2 +- 1 file changed, 1 insertion(+), 1 deletio

[PATCH 1/3] block: rbd: use NULL instead of 0

2013-08-08 Thread Jingoo Han
inter Signed-off-by: Jingoo Han --- drivers/block/rbd.c |4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/block/rbd.c b/drivers/block/rbd.c index 9e00010..fa198de 100644 --- a/drivers/block/rbd.c +++ b/drivers/block/rbd.c @@ -2163,9 +2163,9 @@ static int rbd_img

Re: [PATCH V5] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-08-08 Thread Jingoo Han
On Thursday, August 08, 2013 2:05 AM, Bjorn Helgaas wrote: > On Tue, Aug 6, 2013 at 10:13 PM, Jingoo Han wrote: > > On Wednesday, July 31, 2013 5:14 PM, Jingoo Han wrote: > >> Exynos PCIe IP consists of Synopsys specific part and Exynos > >> specific part. On

Re: [PATCH v3] backlight: lm3630: apply chip revision

2013-08-07 Thread Jingoo Han
u can find more information about LM3630A(datasheet, evm etc) > at http://www.ti.com/product/lm3630a > > Signed-off-by: Daniel Jeong Acked-by: Jingoo Han Best regards, Jingoo Han > --- > drivers/video/backlight/Kconfig |6 +- > drivers/video/backlight/Makefile

Re: [PATCH V5] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-08-06 Thread Jingoo Han
On Wednesday, July 31, 2013 5:14 PM, Jingoo Han wrote: > Exynos PCIe IP consists of Synopsys specific part and Exynos > specific part. Only core block is a Synopsys designware part; > other parts are Exynos specific. > Also, the Synopsys designware part can be shared with other >

[PATCH 3/3] dma: sirf: use NULL instead of 0

2013-08-06 Thread Jingoo Han
sirfsoc_dma_prep_cyclic() returns pointer, thus NULL should be used instead of 0 in order to fix the following sparse warning: drivers/dma/sirf-dma.c:598:24: warning: Using plain integer as NULL pointer Signed-off-by: Jingoo Han --- drivers/dma/sirf-dma.c |2 +- 1 file changed, 1 insertion

[PATCH 2/3] dma: mv_xor: use NULL instead of 0

2013-08-06 Thread Jingoo Han
%p is used, thus NULL should be used instead of 0 in order to fix the following sparse warning: drivers/dma/mv_xor.c:648:9: warning: Using plain integer as NULL pointer Signed-off-by: Jingoo Han --- drivers/dma/mv_xor.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a

[PATCH 1/3] dma: mmp_pdma: Staticize mmp_pdma_alloc_descriptor()

2013-08-06 Thread Jingoo Han
mmp_pdma_alloc_descriptor() is used only in this file. Fix the following sparse warning: drivers/dma/mmp_pdma.c:359:25: warning: symbol 'mmp_pdma_alloc_descriptor' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/dma/mmp_pdma.c |3 ++- 1 file

[PATCH 3/3] pinctrl: sirf: add missing __iomem annotation

2013-08-06 Thread Jingoo Han
/pinctrl-sirf.c:909:17: warning: incorrect type in argument 1 (different address spaces) drivers/pinctrl/sirf/pinctrl-sirf.c:909:17:expected void volatile [noderef] *addr drivers/pinctrl/sirf/pinctrl-sirf.c:909:17:got void *regs Signed-off-by: Jingoo Han --- drivers/pinctrl/sirf/pinctrl

[PATCH 2/3] pinctrl: st: Staticize local symbols

2013-08-06 Thread Jingoo Han
7; was not declared. Should it be static? drivers/pinctrl/pinctrl-st.c:327:14: warning: symbol 'stih416_delays' was not declared. Should it be static? drivers/pinctrl/pinctrl-st.c:814:12: warning: symbol 'st_pmx_get_fname' was not declared. Should it be static? Signed-off-by:

[PATCH 1/3] pinctrl: pinconf: fix comparison of different types

2013-08-06 Thread Jingoo Han
Fix the following sparse warning: drivers/pinctrl/pinconf.c:521:20: error: incompatible types in comparison expression (different type sizes) Signed-off-by: Jingoo Han --- drivers/pinctrl/pinconf.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/pinctrl

Re: [PATCH RESEND] ARM: dts: Add USB host node for Exynos4

2013-08-05 Thread Jingoo Han
On Tuesday, August 06, 2013 3:09 AM, Dongjin Kim wrote: > > This patch adds EHCI and OHCI host device nodes for Exynos4. > > CC: Jingoo Han Acked-by: Jingoo Han > Signed-off-by: Dongjin Kim > --- > arch/arm/boot/dts/exynos4.dtsi | 18 ++ > 1 fil

[PATCH V2] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Jingoo Han
'mmc_callback_data' and 'vtg' are pointers. Fix the following sparse warning: drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer Signed-off-by: Jingoo Han --- drivers/m

Re: [PATCH] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Jingoo Han
On Friday, August 02, 2013 2:53 PM, Sachin Kamat wrote: > On 2 August 2013 11:20, Jingoo Han wrote: [...] > > > > mutex_lock(&the_menelaus->lock); > > - if (vtg == 0) > > + if (vtg == NULL) > > Instead you could just make this: >

[PATCH] mfd: menelaus: use NULL instead of 0

2013-08-01 Thread Jingoo Han
'mmc_callback_data' and 'vtg' are pointers. Fix the following sparse warning: drivers/mfd/menelaus.c:445:43: warning: Using plain integer as NULL pointer drivers/mfd/menelaus.c:469:20: warning: Using plain integer as NULL pointer Signed-off-by: Jingoo Han --- drivers/m

[PATCH] MFD: ucb1x00-core: add CONFIG_PM_SLEEP to suspend/resume

2013-08-01 Thread Jingoo Han
spend' defined but not used [-Wunused-function] drivers/mfd/ucb1x00-core.c:704:12: warning: 'ucb1x00_resume' defined but not used [-Wunused-function] Signed-off-by: Jingoo Han --- drivers/mfd/ucb1x00-core.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/mfd/ucb1x00-

[PATCH 2/2] mfd: ab8500-gpadc: add CONFIG_PM_SLEEP to suspend/resume

2013-08-01 Thread Jingoo Han
: 'ab8500_gpadc_suspend' defined but not used [-Wunused-function] drivers/mfd/ab8500-gpadc.c:903:12: warning: 'ab8500_gpadc_resume' defined but not used [-Wunused-function] Signed-off-by: Jingoo Han --- drivers/mfd/ab8500-gpadc.c |2 ++ 1 file changed, 2 insertions(+) diff --git a/drivers/

[PATCH 1/2] mfd: ab8500-gpadc: add CONFIG_PM_RUNTIME to runtime_suspend/runtime_resume

2013-08-01 Thread Jingoo Han
: 'ab8500_gpadc_runtime_suspend' defined but not used [-Wunused-function] drivers/mfd/ab8500-gpadc.c:878:12: warning: 'ab8500_gpadc_runtime_resume' defined but not used [-Wunused-function] Signed-off-by: Jingoo Han --- drivers/mfd/ab8500-gpadc.c |2 ++ 1 file changed, 2

[PATCH] mfd: max8997: cast void pointer to data of max8997_pmic_dt_match

2013-07-31 Thread Jingoo Han
Casting (void *) data of max8997_pmic_dt_match is necessary, because variable 'data' of struct 'of_device_id' is defined as 'const void *data'. Thus, pointer should be used instead of value. Signed-off-by: Jingoo Han --- drivers/mfd/max8997.c |2 +- 1 f

[PATCH] mfd: lpc_ich: Staticize struct 'lpc_chipset_info'

2013-07-31 Thread Jingoo Han
'lpc_chipset_info' is used only in this file. Fix the following sparse warning: drivers/mfd/lpc_ich.c:216:21: warning: symbol 'lpc_chipset_info' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/mfd/lpc_ich.c |2 +- 1 file changed, 1 inse

[PATCH] mfd: twl4030-power: Staticize local functions

2013-07-31 Thread Jingoo Han
rivers/mfd/twl4030-power.c:512:5: warning: symbol 'twl4030_power_configure_resources' was not declared. Should it be static? drivers/mfd/twl4030-power.c:556:5: warning: symbol 'twl4030_power_probe' was not declared. Should it be static? Signed-off-by: Jingoo Han --- driv

[PATCH 1/3] backlight: ld9040: Staticize local variable gamma_table

2013-07-31 Thread Jingoo Han
Fix the following sparse warnings: drivers/video/backlight/ld9040_gamma.h:172:3: warning: symbol 'gamma_table' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/video/backlight/ld9040_gamma.h |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH 3/3] backlight: lp8788: Staticize 'default_bl_config'

2013-07-31 Thread Jingoo Han
Fix the following sparse warnings: drivers/video/backlight/lp8788_bl.c:55:25: warning: symbol 'default_bl_config' was not declared. Should it be static? Signed-off-by: Jingoo Han --- drivers/video/backlight/lp8788_bl.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) di

[PATCH 2/3] backlight: lm3639: Don't mix different enum types

2013-07-31 Thread Jingoo Han
ivers/video/backlight/lm3639_bl.c:82:51: warning: mixing different enum types drivers/video/backlight/lm3639_bl.c:82:51: int enum lm3639_fleds versus drivers/video/backlight/lm3639_bl.c:82:51: int enum lm3639_bleds Signed-off-by: Jingoo Han --- drivers/video/backlight/lm3639_bl.c |

Re: [PATCH] Driver core: Add accessor for device tree node

2013-07-30 Thread Jingoo Han
On Tuesday, July 30, 2013 10:57 PM, Greg Kroah-Hartman wrote: > On Tue, Jul 30, 2013 at 06:20:47PM +0900, Jingoo Han wrote: > > Add the wrapper function for retrieving device tree node instead of > > accessing dev->of_node directly. > > Why? Sorry, I cannot find the spec

[PATCH V2 03/35] watchdog: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/watchdog/iTCO_wdt.c |2 +- drivers/watchdog/mv64x60_wdt.c |2 +- drivers/watchdog/omap_wdt.c |2 +- drivers/watch

[PATCH] Driver core: Add accessor for device tree node

2013-07-30 Thread Jingoo Han
Add the wrapper function for retrieving device tree node instead of accessing dev->of_node directly. Signed-off-by: Jingoo Han --- include/linux/device.h |5 + 1 file changed, 5 insertions(+) diff --git a/include/linux/device.h b/include/linux/device.h index 22b546a..4faaa51 100

[PATCH 34/35] regulator: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/regulator/88pm800.c|2 +- drivers/regulator/88pm8607.c |2 +- drivers/regulator/aat2870-regulator.c |2 +- driv

[PATCH 33/35] power: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/power/88pm860x_battery.c|2 +- drivers/power/ab8500_btemp.c|2 +- drivers/power/ab8500_charger.c |2 +- drivers/po

[PATCH 19/35] extcon: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/extcon/extcon-adc-jack.c |2 +- drivers/extcon/extcon-gpio.c |2 +- drivers/extcon/extcon-palmas.c |2 +- 3 files changed

[PATCH 18/35] mfd: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/mfd/88pm800.c |2 +- drivers/mfd/88pm805.c |2 +- drivers/mfd/88pm860x-core.c |2 +- drivers/mfd/aat2870-core.c |

[PATCH 17/35] dma: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/dma/imx-sdma.c|2 +- drivers/dma/iop-adma.c|6 +++--- drivers/dma/mv_xor.c |2 +- drivers/dma/pl330.c |

[PATCH 03/35] watchdog: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/watchdog/iTCO_wdt.c |2 +- drivers/watchdog/mv64x60_wdt.c |2 +- drivers/watchdog/omap_wdt.c |3 ++- driv

[PATCH 02/35] rtc: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/rtc/rtc-88pm80x.c |8 drivers/rtc/rtc-88pm860x.c|2 +- drivers/rtc/rtc-cmos.c|2 +- drivers/rtc/

[PATCH 01/35] backlight: use dev_get_platdata()

2013-07-30 Thread Jingoo Han
Use the wrapper function for retrieving the platform data instead of accessing dev->platform_data directly. Signed-off-by: Jingoo Han --- drivers/video/backlight/88pm860x_bl.c|2 +- drivers/video/backlight/aat2870_bl.c |2 +- drivers/video/backlight/adp5520_b

Re: [PATCH v1] backlight-lm3630-apply chip revision

2013-07-28 Thread Jingoo Han
ng Please use upper case as below: Signed-off-by: Daniel Jeong ^^^ ^^^ In my opinion, you have to fix your .gitconfig file as below: [user] email = daniel.je...@ti.com name = Daniel Jeong Daniel Jeong, Please, read the './Documentation/SubmittingPatch

Re: [PATCH] video: remove unused variable dev

2013-07-28 Thread Jingoo Han
deletion(-) CC'ed Luis Henriques. Hi Kefeng Wang, The same patch was submitted by Luis Henriques 2 weeks ago. Also, the patch was already applied by Tomi Valkeinen. Best regards, Jingoo Han > > diff --git a/drivers/video/vga16fb.c b/drivers/video/vga16fb.c > index 830ded4..2827333

[PATCH V2] driver core: replace strict_strto*() with kstrto*()

2013-07-25 Thread Jingoo Han
The usage of strict_strto*() is not preferred, because strict_strto*() is obsolete. Thus, kstrto*() should be used. Signed-off-by: Jingoo Han --- Changes since v1: - mechanically replaced strict_strto*() with kstrto*(). drivers/base/core.c |2 +- drivers/base/memory.c

Re: [PATCH] driver core: replace strict_strto*() with kstrto*()

2013-07-25 Thread Jingoo Han
On Fri, Jul 19, 2013 at 04:21:28PM +0900, Jingoo Han wrote:, Jingoo Han wrote: > > The usage of strict_strto*() is not preferred, because > strict_strto*() is obsolete. Thus, kstrto*() should be > used. > > Signed-off-by: Jingoo Han > --- > drivers/base/core.c

Re: [PATCHv2 3/3] fb: backlight: HX8357: Add HX8369 support

2013-07-25 Thread Jingoo Han
On Thursday, July 25, 2013 10:05 PM, Maxime Ripard wrote: > > From: Alexandre Belloni > > Add support for the Himax HX8369 controller as it is quite similar to the > hx8357. > > Signed-off-by: Alexandre Belloni > Signed-off-by: Maxime Ripard > Acked-by: Jingoo

Re: [PATCHv2 2/3] video: hx8357: Make IM pins optional

2013-07-25 Thread Jingoo Han
quest gpio %d: %d\n", > - lcd->im_pins[i], ret); > - return -EINVAL; > - } > - } > + } else > + lcd->use_im_pins = 0; According to the 'Documentation/CodingStyle', braces are necessary

Re: [PATCH] mfd: max8925: fix dt code for backlight

2013-07-25 Thread Jingoo Han
On Friday, July 26, 2013 6:01 AM, Olof Johansson wrote: > On Wed, Jul 24, 2013 at 6:13 PM, Jingoo Han wrote: > > On Thursday, July 25, 2013 12:42 AM, Olof Johansson wrote: > >> > >> The device-tree enablement for max8925 has several problems, but besides > >&g

Re: [PATCH v3] video: xilinxfb: Fix compilation warning

2013-07-25 Thread Jingoo Han
c3e23c3e32ccbeeabc587f81) > > Signed-off-by: Michal Simek It looks good. Reviewed-by: Jingoo Han Best regards, Jingoo Han > --- > Changes in v3: > - Fix commit message reported by Geert > > Changes in v2: > - Use %pa instead of casting > > ppc44x_defconfig >

Re: [PATCH re-send] backlight: lp855x_bl: support new LP8555 device

2013-07-24 Thread Jingoo Han
On Monday, July 15, 2013 10:10 AM, Jingoo Han wrote: > On Tuesday, July 09, 2013 6:12 PM, Kim, Milo wrote: > > > > LP8555 is one of LP855x family device. > > This device needs pre_init_device() and post_init_device() driver structure. > > It's same as LP8557, so

Re: [PATCH 27/36] video: backlight: convert class code to use dev_groups

2013-07-24 Thread Jingoo Han
On Thursday, July 25, 2013 7:06 AM, Greg Kroah-Hartman wrote: > > The dev_attrs field of struct class is going away soon, dev_groups > should be used instead. This converts the video backlight class code to > use the correct field. > > Cc: Richard Purdie > Cc: Jingoo Han

Re: [PATCH 28/36] video: backlight: lcd: convert class code to use dev_groups

2013-07-24 Thread Jingoo Han
On Thursday, July 25, 2013 7:06 AM, Greg Kroah-Hartman wrote: > > The dev_attrs field of struct class is going away soon, dev_groups > should be used instead. This converts the video backlight lcd class > code to use the correct field. > > Cc: Richard Purdie > Cc: Jingoo

Re: [PATCH] mfd: max8925: fix dt code for backlight

2013-07-24 Thread Jingoo Han
acklight subsystem. I hope that this would go into 3.11 through one of other trees with my Ack. If not, it will go into 3.12-rc1 through mm-tree. Acked-by: Jingoo Han Best regards, Jingoo Han > > > -Olof > > drivers/video/backlight/max8925_bl.c | 41 > +

Re: [PATCH] video: xilinxfb: Fix compilation warning

2013-07-24 Thread Jingoo Han
On Wednesday, July 24, 2013 4:42 PM, Michal Simek wrote: > On 07/24/2013 09:18 AM, Michal Simek wrote: > > On 07/24/2013 08:00 AM, Jingoo Han wrote: > >> On Wednesday, July 24, 2013 2:32 PM, Michal Simek wrote: > >>> > >>> regs_phys is phys_addr_t

Re: [PATCH] ARM: dts: Add USB host node for Exynos4

2013-07-23 Thread Jingoo Han
On Wednesday, July 24, 2013 3:29 PM, Dongjin Kim wrote: > > +Jingoo Han > > Sorry, its my bad. Typo problem. > > I found another patch same with mine, > https://patchwork.kernel.org/patch/2628451, submitted by Jingoo Han > and tested on Exynos4210. Mine is tested on E

Re: [PATCH] video: xilinxfb: Fix compilation warning

2013-07-23 Thread Jingoo Han
dev_dbg(dev, "regs: phys=%pa, virt=%p\n", drvdata->regs_phys, drvdata->regs); Best regards, Jingoo Han > > Signed-off-by: Michal Simek > --- > ppc44x_defconfig > Fixes regressions in v3.11-rc2 > > --

Re: [PATCH 13/27] drivers/pci/host: don't check resource with devm_ioremap_resource

2013-07-23 Thread Jingoo Han
gt; > Acked-by: Bjorn Helgaas > > Jingoo, I can apply this, or if you have other pcie-designware.c > updates this cycle, you can fold it into those. Just let me know what > you'd prefer. Hi Bjorn Helgaas, I will send other updates for pcie-designware.c. I will fold it into th

Re: [PATCH V3] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-23 Thread Jingoo Han
On Tuesday, July 23, 2013 3:30 PM, Kishon Vijay Abraham I wrote: > On Tuesday 23 July 2013 06:44 AM, Jingoo Han wrote: > > On Tuesday, July 23, 2013 12:04 AM, Kishon Vijay Abraham I wrote: > >> On Thursday 18 July 2013 10:51 AM, Jingoo Han wrote: > >>> Exynos PCIe IP

Re: [PATCH V3] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-23 Thread Jingoo Han
On Tuesday, July 23, 2013 3:30 PM, Kishon Vijay Abraham I wrote: > On Tuesday 23 July 2013 06:44 AM, Jingoo Han wrote: > > On Tuesday, July 23, 2013 12:04 AM, Kishon Vijay Abraham I wrote: > >> On Thursday 18 July 2013 10:51 AM, Jingoo Han wrote: > >>> Exynos PCIe IP

Re: [PATCH V3] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-22 Thread Jingoo Han
On Tuesday, July 23, 2013 12:04 AM, Kishon Vijay Abraham I wrote: > On Thursday 18 July 2013 10:51 AM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exyn

[PATCH] block: blk-sysfs: replace strict_strtoul() with kstrtoul()

2013-07-22 Thread Jingoo Han
The usage of strict_strtoul() is not preferred, because strict_strtoul() is obsolete. Thus, kstrtoul() should be used. Signed-off-by: Jingoo Han --- block/blk-sysfs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/block/blk-sysfs.c b/block/blk-sysfs.c index 5efc5a6

[PATCH] Documentation: replace strict_strtoul() with kstrtoul()

2013-07-22 Thread Jingoo Han
The usage of strict_strtoul() is not preferred, because strict_strtoul() is obsolete. Thus, kstrtoul() should be used. Signed-off-by: Jingoo Han --- Documentation/target/tcm_mod_builder.py |2 +- Documentation/trace/ftrace.txt |4 ++-- 2 files changed, 3 insertions(+), 3

[PATCH] kernel: replace strict_strto*() with kstrto*()

2013-07-22 Thread Jingoo Han
The usage of strict_strto*() is not preferred, because strict_strto*() is obsolete. Thus, kstrto*() should be used. Signed-off-by: Jingoo Han --- kernel/gcov/fs.c |2 +- kernel/ksysfs.c |2 +- kernel/params.c | 14 +++--- 3 files changed, 9 insertions(+), 9 deletions

Re: [PATCH 2/3] video: hx8357: Make IM pins optional

2013-07-22 Thread Jingoo Han
On Friday, July 19, 2013 5:36 PM, Maxime Ripard wrote: > On Tue, Jul 16, 2013 at 05:29:40AM +0200, Mike Galbraith wrote: > > On Tue, 2013-07-16 at 09:49 +0900, Jingoo Han wrote: > > > On Tuesday, July 16, 2013 12:27 AM, Maxime Ripard wrote: > > > > > > +

Re: [PATCH] backlight-lm3630-apply chip revision

2013-07-21 Thread Jingoo Han
S; > + pchip->pdata->ledb_init_brt = LM3630_MAX_BRIGHTNESS; > + } else > + pchip->pdata = pdata; According to Document/CodingStyle, Please, add braces as below: } else { pchip->pdata = pdata; } > > /* chi

[PATCH] driver core: replace strict_strto*() with kstrto*()

2013-07-19 Thread Jingoo Han
The usage of strict_strto*() is not preferred, because strict_strto*() is obsolete. Thus, kstrto*() should be used. Signed-off-by: Jingoo Han --- drivers/base/core.c |2 +- drivers/base/memory.c| 10 ++ drivers/base/power/sysfs.c |7

[PATCH] w1: replace strict_strtol() with kstrtol()

2013-07-19 Thread Jingoo Han
The usage of strict_strtol() is not preferred, because strict_strtol() is obsolete. Thus, kstrtol() should be used. Signed-off-by: Jingoo Han --- drivers/w1/w1.c | 12 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/drivers/w1/w1.c b/drivers/w1/w1.c index 0459df8

[PATCH] firmware: google: replace strict_strtoul() with kstrtoul()

2013-07-19 Thread Jingoo Han
The usage of strict_strtoul() is not preferred, because strict_strtoul() is obsolete. Thus, kstrtoul() should be used. Signed-off-by: Jingoo Han --- drivers/firmware/google/gsmi.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/firmware/google/gsmi.c b/drivers

[PATCH] power: power: replace strict_strtol() with kstrtol()

2013-07-19 Thread Jingoo Han
The usage of strict_strtol() is not preferred, because strict_strtol() is obsolete. Thus, kstrtol() should be used. Signed-off-by: Jingoo Han --- drivers/power/power_supply_sysfs.c |2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/power/power_supply_sysfs.c b

[PATCH] block: replace strict_strtoul() with kstrtoul()

2013-07-19 Thread Jingoo Han
The usage of strict_strtoul() is not preferred, because strict_strtoul() is obsolete. Thus, kstrtoul() should be used. Signed-off-by: Jingoo Han --- drivers/block/osdblk.c |2 +- drivers/block/rbd.c|2 +- drivers/block/xen-blkback/xenbus.c |2 +- 3 files

Re: [PATCH 3/3] fb: backlight: HX8357: Add HX8369 support

2013-07-15 Thread Jingoo Han
reviewed this patch with Himax HX8369 datasheet. I cannot find any problems. It looks good. However, if possible, please add comment on huge delays such as msleep(120), msleep(100), etc. Acked-by: Jingoo Han Best regards, Jingoo Han > --- > drivers/

Re: [PATCH 2/3] video: hx8357: Make IM pins optional

2013-07-15 Thread Jingoo Han
lcd->im_pins[i], ret); > + return -EINVAL; > + } > } > - > - ret = devm_gpio_request_one(&spi->dev, lcd->im_pins[i], > - GPIOF_OUT_INIT_LOW,

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-15 Thread Jingoo Han
On Saturday, July 13, 2013 2:16 AM, Thierry Reding wrote: > On Fri, Jul 12, 2013 at 08:28:05AM +0900, Jingoo Han wrote: > > On Friday, July 12, 2013 12:41 AM, Thierry Reding wrote: > > > On Thu, Jul 11, 2013 at 02:49:43PM +0900, Jingoo Han wrote: > > > [...] > >

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-15 Thread Jingoo Han
On Friday, July 12, 2013 7:01 PM, Kishon Vijay Abraham I wrote: > On Thursday 11 July 2013 11:19 AM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exyn

Re: [PATCH] video: remove unused variable "dev" in vga16fb_destroy()

2013-07-15 Thread Jingoo Han
On Monday, July 15, 2013 3:29 PM, Yijing Wang wrote: > > Delete unused variable "dev" to fix build warning in > drivers/video/vga16fb.c > > Signed-off-by: Yijing Wang > Cc: Jingoo Han It looks good. Reviewed-by: Jingoo Han Best regards, Jingoo Han > --- &g

Re: [PATCH re-send] backlight: lp855x_bl: support new LP8555 device

2013-07-14 Thread Jingoo Han
generated from dual DC-DC boost converters. > It's configurable EPROM settings which are defined in the platform data. > > Driver documentation and device tree bindings are updated. > > Signed-off-by: Milo Kim It looks good. Acked-by: Jingoo Han Best regards, Jingoo

Re: [PATCH] usb: phy: samsung-usb2: Toggle HSIC GPIO from device tree

2013-07-11 Thread Jingoo Han
erged yet. (http://permalink.gmane.org/gmane.linux.drivers.devicetree/36830) I think that 'phy-samsung-usb*.c' files are not a good place. However, Julius Werner's comment looks reasonable enough. If you have a comment, please feel free to share it. :) Thank you. Best regards, Jingo

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-11 Thread Jingoo Han
On Thursday, July 11, 2013 5:55 PM, Kishon Vijay Abraham I wrote: > On Thursday 11 July 2013 11:19 AM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exyn

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-11 Thread Jingoo Han
On Friday, July 12, 2013 12:41 AM, Thierry Reding wrote: > On Thu, Jul 11, 2013 at 02:49:43PM +0900, Jingoo Han wrote: > [...] > > diff --git a/drivers/pci/host/Makefile b/drivers/pci/host/Makefile > [...] > > obj-$(CONFIG_PCI_MVEBU) += pci-mvebu.o > > obj-$(CONFIG_PC

Re: [PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-11 Thread Jingoo Han
On Thursday, July 11, 2013 3:40 PM, Kishon Vijay Abraham I wrote: > On Thursday 11 July 2013 11:19 AM, Jingoo Han wrote: > > > > drivers/pci/host/Makefile |1 + > > drivers/pci/host/pcie-designware.c | 963 > > +---

[PATCH V2] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-10 Thread Jingoo Han
Exynos specific part. Signed-off-by: Jingoo Han Cc: Pratyush Anand Cc: Mohit KUMAR --- Changes since v1: - moved the configuration, I/O, memory space handling to dw_pcie_host_init() - removed exynos_pcie_abort() - replaced 'purple_base' with 'block_base' - replaced '

Re: [PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-10 Thread Jingoo Han
On Wednesday, July 10, 2013 11:02 PM, Kishon Vijay Abraham I: > On Friday 05 July 2013 01:59 PM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exynos sp

Re: [PATCH] usb: phy: samsung-usb2: Toggle HSIC GPIO from device tree

2013-07-10 Thread Jingoo Han
27;drivers/platform/arm/' would be a good alternative; thus, USB hub reset code should be moved to 'drivers/platform/arm/'. Please refer to the discussion. (http://patches.linaro.org/16856/) Best regards, Jingoo Han > > Change-Id: Ieecac52c27daa7a17a7ed3b2863ddb

Re: [PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-08 Thread Jingoo Han
On Friday, July 05, 2013 7:46 PM, Arnd Bergmann wrote: > On Friday 05 July 2013, Jingoo Han wrote: > > > --- /dev/null > > +++ b/drivers/pci/host/pcie-exynos.c > > > + > > +/* PCIe ELBI registers */ > > +#define PCIE_IRQ_PULSE

Re: [PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-07 Thread Jingoo Han
On Friday, July 05, 2013 7:44 PM, Pratyush Anand wrote: > On 7/5/2013 1:59 PM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exynos specific. > > Al

Re: [PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-07 Thread Jingoo Han
On Monday, July 08, 2013 2:06 PM, Jingoo Han wrote: > On Friday, July 05, 2013 7:44 PM, Pratyush Anand wrote: > > On 7/5/2013 1:59 PM, Jingoo Han wrote: > > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > > specific part. Only core block is

Re: [PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-07 Thread Jingoo Han
On Friday, July 05, 2013 7:44 PM, Pratyush Anand wrote: > On 7/5/2013 1:59 PM, Jingoo Han wrote: > > Exynos PCIe IP consists of Synopsys specific part and Exynos > > specific part. Only core block is a Synopsys designware part; > > other parts are Exynos specific. > > Al

Re: [PATCH 8/8] ARM: exynos: select PM_GENERIC_DOMAINS only when used

2013-07-07 Thread Jingoo Han
On Friday, July 05, 2013 9:25 AM, Arnd Bergmann wrote: Hi Arnd, This bug fix patch was already submitted on May 27th, as below: https://patchwork.kernel.org/patch/2617191/ Best regards, Jingoo Han > This fixes building exynos kernels with CONFIG_PM disabled. > > Signed-off-by: Arnd

[PATCH] pci: exynos: split into two parts such as Synopsys part and Exynos part

2013-07-05 Thread Jingoo Han
Exynos specific part. Signed-off-by: Jingoo Han Cc: Pratyush Anand Cc: Mohit KUMAR --- drivers/pci/host/Makefile |1 + drivers/pci/host/pcie-designware.c | 907 +++- drivers/pci/host/pcie-designware.h | 72 +++ drivers/pci/host/pcie-exynos.c

Re: [PATCH] backlight: lp855x: set zero brightness at FBBLANK

2013-07-02 Thread Jingoo Han
go Nakao > Cc: Milo Kim > Cc: Richard Purdie > Signed-off-by: Oskar Andero Acked-by: Jingoo Han Best regards, Jingoo Han > --- > drivers/video/backlight/lp855x_bl.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/video/backlight/lp85

Re: [PATCH v9 1/8] drivers: phy: add generic PHY framework

2013-06-27 Thread Jingoo Han
t and the documentation for dt binding can be found at > Documentation/devicetree/bindings/phy/phy-bindings.txt > > Signed-off-by: Kishon Vijay Abraham I > Tested-by: Sylwester Nawrocki Tested-by: Jingoo Han It looks great to me! I tested this General PHY framework with Exynos525

Re: [PATCH V10 0/4] PCIe support for Samsung Exynos5440 SoC

2013-06-23 Thread Jingoo Han
can > > > > make sure the ordering is correct with regard to the of/pci changes and > > > > the mvebu/pci driver. Yes, right. That is the reason why I based on 'linu-next' tree, instead of 'PCI' -next branch. Bjorn Helgaas, Could you give your ACK? Thank

[PATCH V10 4/4] ARM: dts: Add pcie controller node for exynos5440-ssdk5440

2013-06-21 Thread Jingoo Han
This patch adds pcie controller node for exynos5440-ssdk5440, and also adds a phandle for pin controller node. Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- arch/arm/boot/dts/exynos5440-ssdk5440.dts |8 arch/arm/boot/dts/exynos5440.dtsi |2 +- 2 files changed

[PATCH V10 3/4] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC

2013-06-21 Thread Jingoo Han
Exynos5440 has two PCIe controllers which can be used as root complex for PCIe interface. Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- arch/arm/boot/dts/exynos5440.dtsi | 38 + 1 file changed, 38 insertions(+) diff --git a/arch/arm/boot/dts

[PATCH V10 2/4] ARM: EXYNOS: Enable PCIe support for Exynos5440

2013-06-21 Thread Jingoo Han
Enable PCIe support for Exynos5440 which has two PCIe controllers. Signed-off-by: Jingoo Han --- arch/arm/Kconfig |1 + arch/arm/mach-exynos/Kconfig |2 ++ 2 files changed, 3 insertions(+) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index ca30d75..421ce68 100644 --- a

[PATCH V10 1/4] pci: Add PCIe driver for Samsung Exynos

2013-06-21 Thread Jingoo Han
Exynos5440 has a PCIe controller which can be used as Root Complex. This driver supports a PCIe controller as Root Complex mode. Signed-off-by: Surendranath Gurivireddy Balla Signed-off-by: Siva Reddy Kallam Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- .../devicetree/bindings/pci

[PATCH V10 0/4] PCIe support for Samsung Exynos5440 SoC

2013-06-21 Thread Jingoo Han
Hi, This series of patches introduces PCIe support for Samsung Exynos5440, and is based on the latest 'linux-next' tree (20130620). These patches was tested with Intel e1000e LAN card on Exynos5440. This PATCH v10 follows: * PATCH v9, sent on June, 21st 2013 * PATCH v8, sent on June, 20th 2013

Re: [PATCH V9 1/4] pci: Add PCIe driver for Samsung Exynos

2013-06-20 Thread Jingoo Han
On Friday, June 21, 2013 1:23 PM, Pratyush Anand wrote: > On 6/21/2013 8:51 AM, Jingoo Han wrote: > > diff --git a/drivers/pci/host/pci-designware.c > > b/drivers/pci/host/pci-designware.c > > new file mode 100644 > > index 000..f8558ff > > --- /dev/nu

Re: [PATCH V9 1/4] pci: Add PCIe driver for Samsung Exynos

2013-06-20 Thread Jingoo Han
On Friday, June 21, 2013 1:11 PM, Sachin Kamat wrote: > > Hi Jingoo, > > Some small corrections inline. > > On 21 June 2013 08:51, Jingoo Han wrote: > > Exynos5440 has a PCIe controller which can be used as Root Complex. > > This driver supports a PCIe c

[PATCH V9 4/4] ARM: dts: Add pcie controller node for exynos5440-ssdk5440

2013-06-20 Thread Jingoo Han
This patch adds pcie controller node for exynos5440-ssdk5440, and also adds a phandle for pin controller node. Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- arch/arm/boot/dts/exynos5440-ssdk5440.dts |8 arch/arm/boot/dts/exynos5440.dtsi |2 +- 2 files changed

[PATCH V9 3/4] ARM: dts: Add pcie controller node for Samsung EXYNOS5440 SoC

2013-06-20 Thread Jingoo Han
Exynos5440 has two PCIe controllers which can be used as root complex for PCIe interface. Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- arch/arm/boot/dts/exynos5440.dtsi | 38 + 1 file changed, 38 insertions(+) diff --git a/arch/arm/boot/dts

[PATCH V9 2/4] ARM: EXYNOS: Enable PCIe support for Exynos5440

2013-06-20 Thread Jingoo Han
Enable PCIe support for Exynos5440 which has two PCIe controllers. Signed-off-by: Jingoo Han --- arch/arm/Kconfig |1 + arch/arm/mach-exynos/Kconfig |2 ++ 2 files changed, 3 insertions(+) diff --git a/arch/arm/Kconfig b/arch/arm/Kconfig index ca30d75..421ce68 100644 --- a

[PATCH V9 1/4] pci: Add PCIe driver for Samsung Exynos

2013-06-20 Thread Jingoo Han
Exynos5440 has a PCIe controller which can be used as Root Complex. This driver supports a PCIe controller as Root Complex mode. Signed-off-by: Surendranath Gurivireddy Balla Signed-off-by: Siva Reddy Kallam Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- .../devicetree/bindings/pci

[PATCH V9 0/4] PCIe support for Samsung Exynos5440 SoC

2013-06-20 Thread Jingoo Han
Hi, This series of patches introduces PCIe support for Samsung Exynos5440, and is based on the latest 'linux-next' tree (20130620). These patches was tested with Intel e1000e LAN card on Exynos5440. This PATCH v9 follows: * PATCH v8, sent on June, 20th 2013 * PATCH v7, sent on June, 20th 2013

[PATCH V8 4/4] ARM: dts: Add pcie controller node for exynos5440-ssdk5440

2013-06-20 Thread Jingoo Han
This patch adds pcie controller node for exynos5440-ssdk5440, and also adds a phandle for pin controller node. Signed-off-by: Jingoo Han Acked-by: Arnd Bergmann --- arch/arm/boot/dts/exynos5440-ssdk5440.dts |8 arch/arm/boot/dts/exynos5440.dtsi |2 +- 2 files changed

<    5   6   7   8   9   10   11   12   13   14   >