Commit-ID: e1a96fb861b0781c8ef66a76d0fc6fb6f628f84c
Gitweb: http://git.kernel.org/tip/e1a96fb861b0781c8ef66a76d0fc6fb6f628f84c
Author: Axel Lin
AuthorDate: Thu, 21 May 2015 22:57:34 +0800
Committer: Thomas Gleixner
CommitDate: Fri, 22 May 2015 11:06:47 +0200
irqchip: mtk-sysirq: Make
PTR_ERR(NULL) returns 0 so current code returns 0 if ioremap fails, fix it.
Signed-off-by: Axel Lin
---
drivers/irqchip/irq-mtk-sysirq.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/irqchip/irq-mtk-sysirq.c b/drivers/irqchip/irq-mtk-sysirq.c
index 04de2d4..15c1303
Remove extra space between platform prefix and driver name in MODULE_ALIAS.
Signed-off-by: Axel Lin
---
drivers/bus/omap-ocp2scp.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/bus/omap-ocp2scp.c b/drivers/bus/omap-ocp2scp.c
index 9f18569..bf500e0 100644
--- a
Commit-ID: 63781394c540dd9e666a6b21d70b64dd52bce76e
Gitweb: http://git.kernel.org/tip/63781394c540dd9e666a6b21d70b64dd52bce76e
Author: Axel Lin
AuthorDate: Mon, 11 May 2015 17:02:58 +0800
Committer: Thomas Gleixner
CommitDate: Wed, 13 May 2015 10:47:37 +0200
genirq: devres: Fix
().
Fixes: 0668d3065128 ("genirq: Add devm_request_any_context_irq()")
Signed-off-by: Axel Lin
---
v2: Also fixup return value of devm_request_any_context_irq
kernel/irq/devres.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/kernel/irq/devres.c b/kernel/ir
request_any_context_irq() returns a negative value on failure.
It returns either IRQC_IS_HARDIRQ or IRQC_IS_NESTED on success.
So fix testing return value of request_any_context_irq().
Fixes: 0668d3065128 ("genirq: Add devm_request_any_context_irq()")
Signed-off-by: Axel Lin
---
Use regulator_is_enabled_regmap() to replace max77843_reg_is_enabled().
Signed-off-by: Axel Lin
---
drivers/regulator/max77843.c | 18 ++
1 file changed, 2 insertions(+), 16 deletions(-)
diff --git a/drivers/regulator/max77843.c b/drivers/regulator/max77843.c
index 3ae2a9b
MAX77843_CHG_ENABLE is 0x05, so the enable_mask should be
MAX77843_CHG_MASK | MAX77843_CHG_BUCK_MASK.
Signed-off-by: Axel Lin
---
Hi,
I don't have this h/w, so please help to review and test this patch serial.
Thanks,
Axel
drivers/regulator/max77843.c | 2 +-
1 file changed, 1 insertion(
2015-04-09 16:46 GMT+08:00 :
> From: Axel Lin
>
> 3.4.107-rc1 review patch. If anyone has any objections, please let me know.
>
> --
>
>
> commit d297933cc7fcfbaaf2d37570baac73287bf0357d upstream.
>
> Current code tries to find the highest valid fi
2015-04-07 22:21 GMT+08:00 Kishon Vijay Abraham I :
> Hi,
>
> On Tuesday 07 April 2015 05:13 PM, Axel Lin wrote:
>>
>> Don't pass valid pointer to PTR_ERR, use PTR_ERR(phy) only when
>> IS_ERR(phy) is true.
>>
>> Signed-off-by: Axel Lin
>> ---
&g
Don't pass valid pointer to PTR_ERR, use PTR_ERR(phy) only when
IS_ERR(phy) is true.
Signed-off-by: Axel Lin
---
drivers/phy/phy-core.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index 3791838f..63bc12d 100644
Don't pass valid pointer to PTR_ERR, use PTR_ERR(phy) only when
IS_ERR(phy) is true.
Signed-off-by: Axel Lin
---
drivers/phy/phy-core.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/phy/phy-core.c b/drivers/phy/phy-core.c
index 3791838f..f112fff 1
This is a common checking in various drivers, so move the checking to
_of_phy_get().
Signed-off-by: Axel Lin
---
v2: Fixup the error patch. It needs to call module_put if Requested PHY is
disabled.
drivers/phy/phy-core.c | 12 ++--
drivers/phy/phy-miphy28lp.c | 5
This is a common checking in various drivers, so move the checking to
_of_phy_get().
Signed-off-by: Axel Lin
---
drivers/phy/phy-core.c | 6 ++
drivers/phy/phy-miphy28lp.c | 5 -
drivers/phy/phy-miphy365x.c | 5 -
drivers/phy/phy-rcar-gen2.c | 5 -
4 files changed, 6
The of_xlate callback should return ERR_PTR on error.
Signed-off-by: Axel Lin
---
v2: Return -ENODEV instead of -EINVAL so that phy_optional_get() doesn't fail.
drivers/phy/phy-spear1340-miphy.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-spea
The of_xlate callback should return ERR_PTR on error.
Signed-off-by: Axel Lin
---
v2: Return -ENODEV instead of -EINVAL so that phy_optional_get() doesn't fail.
drivers/phy/phy-spear1310-miphy.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-spea
The of_xlate callback should return ERR_PTR on error.
Signed-off-by: Axel Lin
---
drivers/phy/phy-spear1340-miphy.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-spear1340-miphy.c
b/drivers/phy/phy-spear1340-miphy.c
index e42bc20..210979b 100644
--- a
The of_xlate callback should return ERR_PTR on error.
Signed-off-by: Axel Lin
---
drivers/phy/phy-spear1310-miphy.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-spear1310-miphy.c
b/drivers/phy/phy-spear1310-miphy.c
index 9f47fae..4684b04 100644
--- a
"dev_ref_clk_ctrl_mem" is optional resource, so don't return error if fail to
get the resource. Since it's an optional resource, don't emit error if
fail to get dev_ref_clk_ctrl_mem.
Also remove redundant test for res, it's done by devm_ioremap_resource().
Signed-
vers/phy/phy-qcom-ufs-qmp-14nm.ko]
undefined!
ERROR: "ufs_qcom_phy_remove" [drivers/phy/phy-qcom-ufs-qmp-14nm.ko] undefined!
ERROR: "get_ufs_qcom_phy" [drivers/phy/phy-qcom-ufs-qmp-14nm.ko] undefined!
make[1]: *** [__modpost] Error 1
make: *** [modules] Error 2
Signed-off-by: A
The mutex lock is not used at all, remove it.
The *vmmc_regulator is not necessary, use a local variable in
stw481x_vmmc_regulator_probe() instead.
Signed-off-by: Axel Lin
Reviewed-by: Linus Walleij
Acked-by: Lee Jones
---
Change the prefix from mfd to regulator.
drivers/regulator/stw481x
dev_get_regulator() does not exist, fix the typo.
Signed-off-by: Axel Lin
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/regulator/driver.h b/include/linux/regulator/driver.h
index d4ad5b5..045f709 100644
--- a/include
The mutex lock is not used at all, remove it.
The *vmmc_regulator is not necessary, use a local variable in
stw481x_vmmc_regulator_probe() instead.
Signed-off-by: Axel Lin
Reviewed-by: Linus Walleij
Acked-by: Lee Jones
---
Hi Mark,
Lee prefers this patch to be applied via regulator tree,
so I
2015-02-16 19:51 GMT+08:00 Lee Jones :
> On Fri, 13 Feb 2015, Axel Lin wrote:
>
>> The mutex lock is not used at all, remove it.
>> The *vmmc_regulator is not necessary, use a local variable in
>> stw481x_vmmc_regulator_probe() instead.
>>
>> Signed-off-by: Ax
2015-03-03 9:05 GMT+08:00 Axel Lin :
> Current code does NULL test against return value of
> ufs_qcom_phy_generic_probe.
> However, in the case of devm_phy_create() failure, ufs_qcom_phy_generic_probe
> does not return NULL. Fix it.
Hi Kishon,
Seems this patch is missed.
Should
Ensure we have balanced clk_prepare_enable/clk_disable_unprepare calls if
.power_on or .power_off callbacks return error.
Signed-off-by: Axel Lin
---
drivers/phy/phy-samsung-usb2.c | 10 --
1 file changed, 8 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-samsung-usb2.c b
Current code does not call clk_prepare(phy->optclk) when using the old
usb_otg_ss_refclk960m name. Fix it.
Signed-off-by: Axel Lin
---
drivers/phy/phy-omap-usb2.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy-omap-usb
Remove the second lock on non-regmap path.
Fixes: 278270e39efa (phy: exynos-mipi-video: Use spin_lock to protct
state->regmap rmw operations)
Reported-by: Julia Lawall
Signed-off-by: Axel Lin
---
drivers/phy/phy-exynos-mipi-video.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/driv
At the context where we have pointer to struct phy, it's useful to call
phy_get_drvdata() to get the address of priv. With this change, we can
remove the to_phy_berlin_usb_priv() macro and remove *phy from struct
phy_berlin_usb_priv.
Signed-off-by: Axel Lin
---
drivers/phy/phy-berlin-usb.c
It's pointless to set twl->linkstat twice.
Signed-off-by: Axel Lin
---
Resend with linux-usb in Cc.
drivers/phy/phy-twl4030-usb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/phy/phy-twl4030-usb.c b/drivers/phy/phy-twl4030-usb.c
index 8e87f54..bc42d6a 100644
--- a/drivers
2015-03-09 19:44 GMT+08:00 Kishon Vijay Abraham I :
> Hi,
>
>
> On Thursday 05 March 2015 07:27 AM, Axel Lin wrote:
>>
>> The defines in phy-miphy365x.h are all covered in phy.h:
>>
>> MIPHY_TYPE_SATA == PHY_TYPE_STA
>> MIPHY_TYPE_PCIE == PHY_TYPE_PCIE
&g
Remove extra space in MODULE_ALIAS.
Signed-off-by: Axel Lin
---
drivers/phy/phy-omap-control.c | 2 +-
drivers/phy/phy-omap-usb2.c| 2 +-
drivers/phy/phy-ti-pipe3.c | 2 +-
3 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-omap-control.c b/drivers/phy/phy
When phy_pm_runtime_get_sync() returns -ENOTSUPP, phy_exit() also returns
-ENOTSUPP if !phy->ops->exit. Fix it.
Also move the code to override ret close to the code we got ret.
I think it is less error prone this way.
Signed-off-by: Axel Lin
---
drivers/phy/phy-core.c | 7 +++
2015-03-05 18:48 GMT+08:00 Roger Quadros :
> Hi Alex,
It's Axel.
>
> Since we're anyways going to fail probe for !CONFIG_OF, why do you want to
> add the unnecessary device table to the kernel?
>
> what are you benefiting from this change?
If the driver works for both dt case and non-dt case, usi
if !CONFIG_OF, the probe fails.
This is a dt-only driver, so the ifdef CONFIG_OF guard and of_match_ptr are
not needed.
Signed-off-by: Axel Lin
---
drivers/phy/phy-ti-pipe3.c | 9 ++---
1 file changed, 2 insertions(+), 7 deletions(-)
diff --git a/drivers/phy/phy-ti-pipe3.c b/drivers/phy
if !CONFIG_OF, the probe fails.
This is a dt-only driver, so the ifdef CONFIG_OF guard and of_match_ptr are
not needed.
Signed-off-by: Axel Lin
---
drivers/phy/phy-omap-usb2.c | 6 ++
1 file changed, 2 insertions(+), 4 deletions(-)
diff --git a/drivers/phy/phy-omap-usb2.c b/drivers/phy/phy
if !CONFIG_OF, the probe fails.
This is a dt-only driver, so the ifdef CONFIG_OF guard and of_match_ptr are
not needed.
Signed-off-by: Axel Lin
---
drivers/phy/phy-omap-control.c | 10 ++
1 file changed, 2 insertions(+), 8 deletions(-)
diff --git a/drivers/phy/phy-omap-control.c b
The defines in phy-miphy365x.h are all covered in phy.h:
MIPHY_TYPE_SATA == PHY_TYPE_STA
MIPHY_TYPE_PCIE == PHY_TYPE_PCIE
MIPPHY_TYPE_USB == PHY_TYPE_USB2
So covert to use phy.h and then delete phy-miphy365x.h.
Signed-off-by: Axel Lin
---
Documentation/devicetree/bindings/ata/ahci-st.txt
Prefer devm_kcalloc over devm_kzalloc with multiply.
In additional, use sizeof(phy) is incorrect, fix it.
Signed-off-by: Axel Lin
---
drivers/phy/phy-miphy365x.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-miphy365x.c b/drivers/phy/phy-miphy365x.c
PTR_ERR_OR_ZERO simplifies the code.
Signed-off-by: Axel Lin
---
drivers/phy/phy-miphy28lp.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/phy/phy-miphy28lp.c b/drivers/phy/phy-miphy28lp.c
index 9334352..c4cc11d 100644
--- a/drivers/phy/phy-miphy28lp.c
+++ b
Prefer devm_kcalloc over devm_kzalloc with multiply.
In additional, use sizeof(phy) is incorrect, fix it.
Signed-off-by: Axel Lin
---
This patch is on top of below patch:
[PATCH 1/2] phy: miphy28lp: Avoid calling of_get_child_count() multiple times
drivers/phy/phy-miphy28lp.c | 5 ++---
1 file
Prefer devm_kcalloc over devm_kzalloc with multiply.
Signed-off-by: Axel Lin
---
drivers/phy/phy-berlin-sata.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/phy/phy-berlin-sata.c b/drivers/phy/phy-berlin-sata.c
index 099eee8..6f3e06d 100644
--- a/drivers/phy/phy
Also remove unneeded goto and rc variable.
Signed-off-by: Axel Lin
---
drivers/phy/phy-xgene.c | 23 +--
1 file changed, 5 insertions(+), 18 deletions(-)
diff --git a/drivers/phy/phy-xgene.c b/drivers/phy/phy-xgene.c
index 2263cd0..385362e 100644
--- a/drivers/phy/phy
If rockchip_usb_phy_power() fails, we need to call clk_disable_unprepare()
before return. This is to ensure we have balanced clk_enable/disable calls.
Also remove unneeded ret checking in rockchip_usb_phy_power_off.
Signed-off-by: Axel Lin
---
drivers/phy/phy-rockchip-usb.c | 6 +++---
1 file
Code simplification. No functional change.
Signed-off-by: Axel Lin
---
drivers/phy/phy-ti-pipe3.c | 10 +++---
1 file changed, 3 insertions(+), 7 deletions(-)
diff --git a/drivers/phy/phy-ti-pipe3.c b/drivers/phy/phy-ti-pipe3.c
index 95c88f9..ed72b0d 100644
--- a/drivers/phy/phy-ti-pipe3.c
Current code uses num_phys settings to tell the number of entries in phys.
Thus remove the NULL terminating entry from phys array which is not necessary.
Signed-off-by: Axel Lin
---
drivers/phy/phy-exynos4210-usb2.c | 1 -
drivers/phy/phy-exynos4x12-usb2.c | 1 -
drivers/phy/phy-exynos5250-usb2
Current code does NULL test against return value of ufs_qcom_phy_generic_probe.
However, in the case of devm_phy_create() failure, ufs_qcom_phy_generic_probe
does not return NULL. Fix it.
Signed-off-by: Axel Lin
---
drivers/phy/phy-qcom-ufs.c | 1 +
1 file changed, 1 insertion(+)
diff --git a
This prevent NULL pointer dereference if res is NULL.
Signed-off-by: Axel Lin
---
This patch replaces [PATCH] phy: hix5hd2-sata: Convert to use
devm_ioremap_resource,
as zhangfei pinted out that the memory region is shared with sata controller so
devm_request_mem_region would fail.
drivers/phy
devm_ioremap_resource() also has sanity check for the res argument which is
missed in current code.
Signed-off-by: Axel Lin
---
drivers/phy/phy-hix5hd2-sata.c | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/drivers/phy/phy-hix5hd2-sata.c b/drivers/phy/phy-hix5hd2
PTR_ERR_OR_ZERO simplifies the code.
Signed-off-by: Axel Lin
---
drivers/phy/phy-berlin-usb.c | 5 +
1 file changed, 1 insertion(+), 4 deletions(-)
diff --git a/drivers/phy/phy-berlin-usb.c b/drivers/phy/phy-berlin-usb.c
index c8a8d53..9f7cc7e 100644
--- a/drivers/phy/phy-berlin-usb.c
It's pointless to set twl->linkstat twice.
Signed-off-by: Axel Lin
---
drivers/phy/phy-twl4030-usb.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/phy/phy-twl4030-usb.c b/drivers/phy/phy-twl4030-usb.c
index 8e87f54..bc42d6a 100644
--- a/drivers/phy/phy-twl4030-usb.c
+++ b
Current code uses args->args[0] as array subscript of phy_drd->phys[].
So the valid value range for args->args[0] is 0 ... EXYNOS5_DRDPHYS_NUM - 1.
Signed-off-by: Axel Lin
---
drivers/phy/phy-exynos5-usbdrd.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/driver
If IS_ERR(state->regs) the .probe fails.
So IS_ERR(state->regs) test in exynos_dp_video_phy_pwr_isol() is not necessary.
exynos_dp_video_phy_pwr_isol() simply does a regmap_update_bits() call now,
just call regmap_update_bits() instead and return proper return value.
Signed-off-by: Ax
The state->regmap is initialized by devm_regmap_init_mmio().
So it's fine to use spin_lock rather than mutex to protct state->regmap rmw
operations.
Signed-off-by: Axel Lin
---
Hi Sylwester,
I don't have this hardware handy, so can you test if this patch works.
This patch
syscon_regmap_lookup_by_phandle() returns ERR_PTR on error.
Thus don't use null test against state->regmap.
Signed-off-by: Axel Lin
---
drivers/phy/phy-exynos-mipi-video.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/phy/phy-exynos-mipi-video.c
b/drivers
At the context we have pointer to struct phy, it's useful to call
phy_get_drvdata() to get the address of cluster_phy. This has slightly
better readability than calling dev_get_drvdata(phy->dev.parent).
Signed-off-by: Axel Lin
---
v2: update commit log
drivers/phy/phy-armada375-us
Add missing .owner field in miphy28lp_ops, which is used for refcounting.
Signed-off-by: Axel Lin
---
drivers/phy/phy-miphy28lp.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/phy/phy-miphy28lp.c b/drivers/phy/phy-miphy28lp.c
index 9b2848e..44e2861 100644
--- a/drivers/phy/phy
Set it once is enough and it's done after devm_kzalloc().
Signed-off-by: Axel Lin
---
drivers/phy/phy-xgene.c | 1 -
1 file changed, 1 deletion(-)
diff --git a/drivers/phy/phy-xgene.c b/drivers/phy/phy-xgene.c
index 29214a3..2263cd0 100644
--- a/drivers/phy/phy-xgene.c
+++ b/drivers/ph
Signed-off-by: Axel Lin
---
drivers/phy/phy-armada375-usb2.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/phy/phy-armada375-usb2.c b/drivers/phy/phy-armada375-usb2.c
index 7c99ca2..8ccc395 100644
--- a/drivers/phy/phy-armada375-usb2.c
+++ b/drivers/phy/phy
2015-02-24 21:47 GMT+08:00 Kishon Vijay Abraham I :
> Hi,
>
> On Tuesday 24 February 2015 05:22 PM, Axel Lin wrote:
>>
>> Currently, of_get_child_count() is called in each iteration of the for
>> loop in
>> miphy28lp_xlate(). This patch stores the return value
Currently, of_get_child_count() is called in each iteration of the for loop in
miphy365x_xlate(). This patch stores the return value of of_get_child_count()
in miphy_dev->nphys and call of_get_child_count() once in miphy365x_probe().
Signed-off-by: Axel Lin
---
drivers/phy/phy-miphy365x.c |
Currently, of_get_child_count() is called in each iteration of the for loop in
miphy28lp_xlate(). This patch stores the return value of of_get_child_count()
in miphy_dev->nphys and call of_get_child_count() once in miphy28lp_probe().
Signed-off-by: Axel Lin
---
drivers/phy/phy-miphy28lp.c |
The mutex lock is not used at all, remove it.
The *vmmc_regulator is not necessary, use a local variable in
stw481x_vmmc_regulator_probe() instead.
Signed-off-by: Axel Lin
---
drivers/regulator/stw481x-vmmc.c | 8
include/linux/mfd/stw481x.h | 4
2 files changed, 4 insertions
Current code is using devm_regulator_register(), so we don't need to store *rdev
in struct lp872x for clean up.
Also clean up lp872x_probe() a bit to remove unnecessary goto and num_regulators
variable.
Signed-off-by: Axel Lin
---
drivers/regulator/lp872x.c | 24 +---
1
Signed-off-by: Axel Lin
---
include/linux/regulator/driver.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/include/linux/regulator/driver.h b/include/linux/regulator/driver.h
index d4ad5b5..045f709 100644
--- a/include/linux/regulator/driver.h
+++ b/include/linux/regulator
Ensure get_voltage return correct voltage if set_voltage fails.
Signed-off-by: Axel Lin
---
drivers/regulator/qcom_rpm-regulator.c | 12
1 file changed, 8 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/qcom_rpm-regulator.c
b/drivers/regulator/qcom_rpm-regulator.c
list_for_each_entry_safe() is necessary if list objects are deleted from
the list while traversing it.
Signed-off-by: Axel Lin
---
drivers/soc/ti/knav_qmss_queue.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti
>>
>> Arnd are you merging this directly into ARM SoC?
>>
>
> Applied to the fixes branch.
Thanks.
>
> Please remember to add a...@kernel.org when you want us to pick up the
> patches.
scripts/get_maintainer.pl does not suggest a...@kernel.org for this patch.
$ scripts/get_maintainer.pl
0001-soc-v
Return -ENOMEM if devm_kzalloc fails.
Signed-off-by: Axel Lin
---
drivers/soc/ti/knav_qmss_queue.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c
index d66aaf2..6f22d56 100644
--- a/drivers/soc/ti
Don't call mutex_unlock() in the error patch if the mutex_lock() is not called.
Signed-off-by: Axel Lin
---
drivers/soc/ti/knav_qmss_queue.c | 7 ---
1 file changed, 4 insertions(+), 3 deletions(-)
diff --git a/drivers/soc/ti/knav_qmss_queue.c b/drivers/soc/ti/knav_qmss_queue.c
Use list_first_entry_or_null() for first_region() and first_queue_range().
list_first_entry() expects the list is not empty, so first_region() and
first_queue_range() never return NULL.
Thus use list_first_entry_or_null() instead.
Signed-off-by: Axel Lin
---
drivers/soc/ti/knav_qmss.h | 8
The of_device_id table is supposed to be zero-terminated.
Signed-off-by: Axel Lin
---
drivers/soc/versatile/soc-realview.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/soc/versatile/soc-realview.c
b/drivers/soc/versatile/soc-realview.c
index cea8ea3..1a07bf5 100644
--- a/drivers
Also makes regulator_get_exclusive and devm_regulator_get_exclusive stub
functions return error pointer.
Signed-off-by: Axel Lin
---
Hi Mark,
I hit below build error and then found this patch is still not yet upstream.
So here is a resend.
CC [M] drivers/gpu/drm/msm/mdp/mdp4/mdp4_kms.o
The module version is unlikely to be updated, use kernel version should be
enough.
Signed-off-by: Axel Lin
---
Hi Gyungoh,
Seems you have added MODULE_VERSION for the sky81452 serial patches.
Do you really need that and will keep updating it?
Otherwise, you may want to drop the MODULE_VERSION in
Current code does not take the macro parameter, fix it.
This is not a problem at this moment because the only user actually passes
vreg to FORCE_MODE_IS_2_BITS().
Signed-off-by: Axel Lin
---
drivers/regulator/qcom_rpm-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git
Doing so generates a warning as the first field is a pointer but we use
0 to initialize it.
Signed-off-by: Axel Lin
---
drivers/regulator/qcom_rpm-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/qcom_rpm-regulator.c
b/drivers/regulator/qcom_rpm
Set di->regulator before dereference it.
Signed-off-by: Axel Lin
---
drivers/regulator/fan53555.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/fan53555.c b/drivers/regulator/fan53555.c
index 70b31bf..c82fb9e 100644
--- a/drivers/regulator/fan5355
Signed-off-by: Axel Lin
---
drivers/regulator/fan53555.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
diff --git a/drivers/regulator/fan53555.c b/drivers/regulator/fan53555.c
index f2f5535..70b31bf 100644
--- a/drivers/regulator/fan53555.c
+++ b/drivers/regulator/fan53555.c
@@ -267,8
Also makes regulator_get_exclusive and devm_regulator_get_exclusive stub
functions return error pointer.
Signed-off-by: Axel Lin
---
Now drivers/gpu/drm/msm/hdmi/hdmi.c is using devm_regulator_get
instead of devm_regulator_get_exclusive.
commit 3e87599b68e7929a84a32ab65ad17b79a3f271f6
"dr
us add missing of_node_put(reg_np).
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/drivers/regulator/rk808-regulator.c
b/drivers/regulator/rk808-regulator.c
index 9557428..07b0eb4 100644
--- a/drivers/regulator/rk808-reg
Also remove non-informative comment.
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 26 --
1 file changed, 26 deletions(-)
diff --git a/drivers/regulator/rk808-regulator.c
b/drivers/regulator/rk808-regulator.c
index d91f2b6..9557428 100644
--- a
2014-09-10 17:23 GMT+08:00 Guodong Xu :
>
>
> On 09/10/2014 12:23 PM, Axel Lin wrote:
>> 2014-09-10 12:20 GMT+08:00 Axel Lin :
>>> 2014-09-10 11:50 GMT+08:00 Guodong Xu :
>>>> These of_node_get() were added to balance refcount decrements inside of
>>&
2014-09-10 12:20 GMT+08:00 Axel Lin :
> 2014-09-10 11:50 GMT+08:00 Guodong Xu :
>> These of_node_get() were added to balance refcount decrements inside of
>> of_find_node_by_name().
>> See: commit c92f5dd2c42f ("regulator: Add missing of_node_put()")
>>
>&
2014-09-10 11:50 GMT+08:00 Guodong Xu :
> These of_node_get() were added to balance refcount decrements inside of
> of_find_node_by_name().
> See: commit c92f5dd2c42f ("regulator: Add missing of_node_put()")
>
> However of_find_node_by_name() was then replaced by of_get_child_by_name(),
> which doe
Fix ERROR: "devm_regmap_init_i2c" [drivers/regulator/isl9305.ko] undefined!
Signed-off-by: Axel Lin
---
drivers/regulator/Kconfig | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 2b98a34..2419fe4 100644
--- a/drivers
2014-09-09 19:07 GMT+08:00 Guodong Xu :
> mc13xxx_parse_regulators_dt() goes through dt tree to search for regulator
> sub-nodes. For each matched node, later on it will be used and registered
> into a regulator_dev.
>
> of_node_get() should be called to increase its refcount.
I think it's done by
2014-09-06 21:49 GMT+08:00 Mark Brown :
> On Sat, Sep 06, 2014 at 08:59:36PM +0800, Axel Lin wrote:
>> Signed-off-by: Axel Lin
>
> Applied, thanks. Please always send incremental patches.
Thanks, and sorry that I have limited internet access while traveling.
I did not realize th
Signed-off-by: Axel Lin
---
drivers/regulator/bcm590xx-regulator.c | 8
1 file changed, 8 deletions(-)
diff --git a/drivers/regulator/bcm590xx-regulator.c
b/drivers/regulator/bcm590xx-regulator.c
index 5d1fd6f..fe6ac69 100644
--- a/drivers/regulator/bcm590xx-regulator.c
+++ b/drivers
ays 0, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff --git a/dr
ays 0, so remove it.
_min_uV and _max_uV are not required, all required settings are set in
REGULATOR_LINEAR_RANGE macro.
Signed-off-by: Axel Lin
---
drivers/regulator/as3711-regulator.c | 61 +---
1 file changed, 28 insertions(+), 33 deletions(-)
diff --git a/dr
Signed-off-by: Axel Lin
---
drivers/regulator/hi6421-regulator.c | 8
1 file changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..156d0d1 100644
--- a/drivers/regulator/hi6421-regulator.c
+++ b
Signed-off-by: Axel Lin
---
drivers/regulator/max77802.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/drivers/regulator/max77802.c b/drivers/regulator/max77802.c
index 967e109..d89792b 100644
--- a/drivers/regulator/max77802.c
+++ b/drivers/regulator/max77802.c
Signed-off-by: Axel Lin
---
drivers/regulator/hi6421-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/hi6421-regulator.c
b/drivers/regulator/hi6421-regulator.c
index e389920..a8c362c 100644
--- a/drivers/regulator/hi6421-regulator.c
+++ b/drivers
Also move da9211_i2c_id and da9211_dt_ids close to the user for better
readability.
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 29 +++--
1 file changed, 15 insertions(+), 14 deletions(-)
diff --git a/drivers/regulator/da9211-regulator.c
b
Signed-off-by: Axel Lin
---
drivers/regulator/da9211-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/da9211-regulator.c
b/drivers/regulator/da9211-regulator.c
index 9722728..044c36c 100644
--- a/drivers/regulator/da9211-regulator.c
+++ b/drivers/regulator
2014-09-02 10:11 GMT+08:00 Greg Kroah-Hartman :
> On Tue, Sep 02, 2014 at 09:47:37AM +0800, Axel Lin wrote:
>> 2014-08-03 10:36 GMT+08:00 Axel Lin :
>> > This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP
>> > to
>> > avoid build error
2014-08-03 10:36 GMT+08:00 Axel Lin :
> This driver uses GENERIC_IRQ_CHIP, so it needs to select GENERIC_IRQ_CHIP to
> avoid build error.
>
> Fixes below build errors:
> ERROR: "irq_alloc_domain_generic_chips" [drivers/gpu/ipu-v3/imx-ipu-v3.ko]
> undefined!
> ERROR:
The min_sel is 0, max_sel is 15, so n_voltages should be 16.
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/rk808-regulator.c
b/drivers/regulator/rk808-regulator.c
index 0d11df1..0da8867
The memory allocated in rk808_regulator_probe() needs to be freed when the
module is unloaded. Thus pass &pdev->dev rather than &client->dev to
devm_kzalloc.
Signed-off-by: Axel Lin
---
drivers/regulator/rk808-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
The valid_modes_mask and *dev are not used in this driver, remove them.
Current code uses devm_regulator_register, so we don't need *regulator in
hi6421_regulator_info. Use a local variable instead.
Also removes a few unnecessary inclusion of header files.
Signed-off-by: Axel Lin
---
dr
401 - 500 of 1317 matches
Mail list logo