This patch converts this driver to use the regmap helper functions provided by
regulator core.
Signed-off-by: Axel Lin axel@ingics.com
Acked-by: Balaji T K balaj...@ti.com
---
v2: Add Balaji's Ack.
drivers/regulator/pbias-regulator.c | 74 ++---
1 file
000.
2) The is_enable implementation is wrong in some cases:
e.g. for pbias_mmc_omap5: emable_mask is : BIT(27) | BIT(25) | BIT(26)
However, pbias_regulator_enable() only sets BIT(26) | BIT(22) bits.
So is_enable always return false in this case.
Signed-off-by: Axel Lin
---
Hi Balaji,
I do
) The is_enable implementation is wrong in some cases:
e.g. for pbias_mmc_omap5: emable_mask is : BIT(27) | BIT(25) | BIT(26)
However, pbias_regulator_enable() only sets BIT(26) | BIT(22) bits.
So is_enable always return false in this case.
Signed-off-by: Axel Lin axel@ingics.com
Signed-off-by: Axel Lin
---
drivers/regulator/Kconfig | 6 +++---
drivers/regulator/pfuze100-regulator.c | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 4ddfb6c..eb36d8b 100644
--- a/drivers/regulator
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/Kconfig | 6 +++---
drivers/regulator/pfuze100-regulator.c | 2 +-
2 files changed, 4 insertions(+), 4 deletions(-)
diff --git a/drivers/regulator/Kconfig b/drivers/regulator/Kconfig
index 4ddfb6c..eb36d8b 100644
for
'pfuze_device_id') [enabled by default]
drivers/regulator/pfuze100-regulator.c:93:2: warning: excess elements in array
initializer [enabled by default]
drivers/regulator/pfuze100-regulator.c:93:2: warning: (near initialization for
'pfuze_dt_ids') [enabled by default]
Signed-off-by: Axel Lin
for
'pfuze_device_id') [enabled by default]
drivers/regulator/pfuze100-regulator.c:93:2: warning: excess elements in array
initializer [enabled by default]
drivers/regulator/pfuze100-regulator.c:93:2: warning: (near initialization for
'pfuze_dt_ids') [enabled by default]
Signed-off-by: Axel Lin axel
to remove testing if control_reg is 0.
Signed-off-by: Axel Lin
---
drivers/regulator/anatop-regulator.c | 25 ++---
1 file changed, 2 insertions(+), 23 deletions(-)
diff --git a/drivers/regulator/anatop-regulator.c
b/drivers/regulator/anatop-regulator.c
index 38e8122
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65217-regulator.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff --git a/drivers
to remove testing if control_reg is 0.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/anatop-regulator.c | 25 ++---
1 file changed, 2 insertions(+), 23 deletions(-)
diff --git a/drivers/regulator/anatop-regulator.c
b/drivers/regulator/anatop-regulator.c
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65217-regulator.c | 15 ++-
1 file changed, 2 insertions(+), 13 deletions(-)
diff
This is a DT-only driver, so make it depend on OF and remove of_match_ptr in
the code.
Signed-off-by: Axel Lin
---
drivers/regulator/Kconfig | 2 +-
drivers/regulator/tps65218-regulator.c | 14 ++
2 files changed, 7 insertions(+), 9 deletions(-)
diff --git a/drivers
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps->rdev[id].
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 18 --
1 file changed,
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Signed-off-by: Axel Lin
---
I'm sorry that I just found I CC wrong developers in my previous mail.
So here is a resend.
drivers/regulator/tps65218-regulator.c | 1 +
1
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps->rdev[id].
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 18 --
1 file changed,
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/tps65218-regulator.c
b
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/tps65218
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps-rdev[id].
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 18 --
1
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Signed-off-by: Axel Lin axel@ingics.com
---
I'm sorry that I just found I CC wrong developers in my previous mail.
So here is a resend.
drivers/regulator/tps65218
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps-rdev[id].
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 18 --
1
This is a DT-only driver, so make it depend on OF and remove of_match_ptr in
the code.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/Kconfig | 2 +-
drivers/regulator/tps65218-regulator.c | 14 ++
2 files changed, 7 insertions(+), 9 deletions
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65910-regulator.c | 13 ++---
1 file changed, 2 insertions(+), 11 deletions(-)
diff --git a/drivers
Use regulator_set_voltage_time_sel() instead of open-coded.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 37 +-
1 file changed, 10 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/tps65218-regulator.c
b/drivers/regulator
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps->rdev[id].
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 18 --
1 file changed,
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Also remove of_match_ptr as this is a DT-only driver.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65218-regulator.c | 15 +++
1 file changed, 7
Fixes below build error:
FATAL: drivers/regulator/tps65218-regulator: struct of_device_id is not
terminated with a NULL entry!
Also remove of_match_ptr as this is a DT-only driver.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 15 +++
1
Current code uses devm_regulator_register() so the we don't need to explicitly
call regulator_unregister() in .remove.
And then we don't need to save rdev pointer to tps-rdev[id].
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 18 --
1
Use regulator_set_voltage_time_sel() instead of open-coded.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65218-regulator.c | 37 +-
1 file changed, 10 insertions(+), 27 deletions(-)
diff --git a/drivers/regulator/tps65218-regulator.c
b
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65910-regulator.c | 13 ++---
1 file changed, 2 insertions(+), 11 deletions(-)
diff
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin
---
drivers/regulator/rc5t583-regulator.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/regulator
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin
---
drivers/regulator/as3711-regulator.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git a/drivers/regulator/as3711
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/as3711-regulator.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git
The regulator core supports this to allow the configuration to be inspected
at runtime even if no software management is enabled.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/rc5t583-regulator.c | 9 +
1 file changed, 1 insertion(+), 8 deletions(-)
diff --git
Signed-off-by: Axel Lin
---
drivers/pinctrl/pinctrl-capri.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-capri.c b/drivers/pinctrl/pinctrl-capri.c
index 4669c53..b4aa8e3 100644
--- a/drivers/pinctrl/pinctrl-capri.c
+++ b/drivers/pinctrl/pinctrl
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/pinctrl/pinctrl-capri.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-capri.c b/drivers/pinctrl/pinctrl-capri.c
index 4669c53..b4aa8e3 100644
--- a/drivers/pinctrl/pinctrl-capri.c
+++ b/drivers
: *** [drivers] Error 2
Reported-by: Stephen Rothwell
Signed-off-by: Axel Lin
---
drivers/regulator/act8865-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers/regulator/act8865-regulator.c
index 5e3e1d2..084cc08 100644
make: *** [drivers] Error 2
Reported-by: Stephen Rothwell s...@canb.auug.org.au
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/act8865-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers/regulator/act8865
The current mode setting is stored in mode field of struct clock_event_device.
So we can just remove the mxs_clockevent_mode variable.
Signed-off-by: Axel Lin
---
drivers/clocksource/mxs_timer.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git a/drivers/clocksource
The current mode setting is stored in mode field of struct clock_event_device.
So we can just remove the mxs_clockevent_mode variable.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/clocksource/mxs_timer.c | 8 ++--
1 file changed, 2 insertions(+), 6 deletions(-)
diff --git
Hi,
I just found this commit after pull regulator updates.
So why to use 'platform' prefix in MODULE_ALIAS for i2c driver?
Does it really work?
Regards,
Axel
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majord...@vger.kernel.org
More
Hi,
I just found this commit after pull regulator updates.
So why to use 'platform' prefix in MODULE_ALIAS for i2c driver?
Does it really work?
Regards,
Axel
--
To unsubscribe from this list: send the line unsubscribe linux-kernel in
the body of a message to majord...@vger.kernel.org
More
BBCH_BBCHEN_MASK is equivalent to TPS65910_SUPPLY_STATE_ENABLED.
So all regulators have the same enable_mask setting.
BBCH_BBCHEN_MASK and BBCH_BBCHEN_SHIFT are not used now, remove them.
Signed-off-by: Axel Lin
---
v2: (Base on Markus' comment) remove unused BBCH_BBCHEN_* in tps65910.h
BBCH_BBCHEN_MASK is equivalent to TPS65910_SUPPLY_STATE_ENABLED.
So all regulators have the same enable_mask setting.
BBCH_BBCHEN_MASK and BBCH_BBCHEN_SHIFT are not used now, remove them.
Signed-off-by: Axel Lin axel@ingics.com
---
v2: (Base on Markus' comment) remove unused BBCH_BBCHEN_
BBCH_BBCHEN_MASK is equivalent to TPS65910_SUPPLY_STATE_ENABLED.
So all regulators have the same enable_mask setting.
Signed-off-by: Axel Lin
---
drivers/regulator/tps65910-regulator.c | 16 +---
1 file changed, 1 insertion(+), 15 deletions(-)
diff --git a/drivers/regulator
BBCH_BBCHEN_MASK is equivalent to TPS65910_SUPPLY_STATE_ENABLED.
So all regulators have the same enable_mask setting.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/tps65910-regulator.c | 16 +---
1 file changed, 1 insertion(+), 15 deletions(-)
diff --git
There is no suspend enable/disable settings mentioned in datasheet,
so just don't implement .set_suspend_[en|dis]able callbacks.
Signed-off-by: Axel Lin
---
drivers/regulator/act8865-regulator.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b
This is not required because current code uses devm_regulator_register() to
register regulators.
Signed-off-by: Axel Lin
---
v2: Fix typo in subject line s/Rmove/Remove/
drivers/regulator/act8865-regulator.c | 12
1 file changed, 12 deletions(-)
diff --git a/drivers/regulator
This is not required because current code use devm_regulator_register() to
register regulators.
Signed-off-by: Axel Lin
---
drivers/regulator/act8865-regulator.c | 12
1 file changed, 12 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers/regulator/act8865
Add missing .owner field in regulator_desc, which is used for refcounting.
Signed-off-by: Axel Lin
---
drivers/regulator/max77693.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index feb20bf..5fb899f 100644
--- a/drivers
Add missing .owner field in regulator_desc, which is used for refcounting.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/max77693.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/max77693.c b/drivers/regulator/max77693.c
index feb20bf..5fb899f 100644
This is not required because current code use devm_regulator_register() to
register regulators.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/act8865-regulator.c | 12
1 file changed, 12 deletions(-)
diff --git a/drivers/regulator/act8865-regulator.c
b/drivers
This is not required because current code uses devm_regulator_register() to
register regulators.
Signed-off-by: Axel Lin axel@ingics.com
---
v2: Fix typo in subject line s/Rmove/Remove/
drivers/regulator/act8865-regulator.c | 12
1 file changed, 12 deletions(-)
diff --git
There is no suspend enable/disable settings mentioned in datasheet,
so just don't implement .set_suspend_[en|dis]able callbacks.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/act8865-regulator.c | 2 --
1 file changed, 2 deletions(-)
diff --git a/drivers/regulator/act8865
Signed-off-by: Axel Lin
---
drivers/regulator/max14577.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/max14577.c b/drivers/regulator/max14577.c
index 516e8af..b1078ba3 100644
--- a/drivers/regulator/max14577.c
+++ b/drivers/regulator/max14577.c
@@ -270,3 +270,4
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/max14577.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/drivers/regulator/max14577.c b/drivers/regulator/max14577.c
index 516e8af..b1078ba3 100644
--- a/drivers/regulator/max14577.c
+++ b/drivers/regulator/max14577.c
Hi,
I got below hangup with current Linus' tree.
[ 166.378906] [ cut here ]
[ 166.385742] WARNING: CPU: 0 PID: 0 at kernel/rcu/update.c:107
__rcu_read_unlock+0x70/0x84()
[ 166.394531] CPU: 0 PID: 0 Comm: Not tainted 3.13.0-rc4-00291-g002c96b-dirty
#2135
[
These settings are not used when using linear_ranges and it makes the code
looks confusing. Thus remove them.
Signed-off-by: Axel Lin
---
drivers/regulator/as3722-regulator.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/regulator/as3722-regulator.c
b/drivers/regulator/as3722
These settings are not used when using linear_ranges and it makes the code
looks confusing. Thus remove them.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/as3722-regulator.c | 3 ---
1 file changed, 3 deletions(-)
diff --git a/drivers/regulator/as3722-regulator.c
b/drivers
Hi,
I got below hangup with current Linus' tree.
[ 166.378906] [ cut here ]
[ 166.385742] WARNING: CPU: 0 PID: 0 at kernel/rcu/update.c:107
__rcu_read_unlock+0x70/0x84()
[ 166.394531] CPU: 0 PID: 0 Comm: Not tainted 3.13.0-rc4-00291-g002c96b-dirty
#2135
[
Set g->out_bit bit for gpio output high, clear g->out_bit bit for gpio output
low.
Signed-off-by: Axel Lin
---
Hi Bjorn,
I don't have the datasheet, just found current code does not make sense
because current code unconditionally set BIT(g->out_bit) in msm_gpio_set.
I assume setting g
Set g-out_bit bit for gpio output high, clear g-out_bit bit for gpio output
low.
Signed-off-by: Axel Lin axel@ingics.com
---
Hi Bjorn,
I don't have the datasheet, just found current code does not make sense
because current code unconditionally set BIT(g-out_bit) in msm_gpio_set.
I assume
2013/11/25 Axel Lin :
> I'm testing on a nommu platform (arm7tdmi SoC).
> Using current Linus' tree + out-of-tree patches for this SoC.
> I got below hang while executing ls (busybox) after boot.
>
> / # ls
> [ 51.036191] [ cut here ]
> [ 51.042242
2013/12/9 Axel Lin :
> 2013/12/9 Greg Kroah-Hartman :
>> On Mon, Dec 09, 2013 at 04:54:29PM +0800, Axel Lin wrote:
>>> 2013/12/9 Greg Kroah-Hartman :
>>> > On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
>>> >> 2013/12/4 Rob Landley :
>&
2013/12/9 Greg Kroah-Hartman :
> On Mon, Dec 09, 2013 at 04:54:29PM +0800, Axel Lin wrote:
>> 2013/12/9 Greg Kroah-Hartman :
>> > On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
>> >> 2013/12/4 Rob Landley :
>> >> > On 11/16/2013 02:15:23 AM,
2013/12/9 Greg Kroah-Hartman :
> On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
>> 2013/12/4 Rob Landley :
>> > On 11/16/2013 02:15:23 AM, Axel Lin wrote:
>> >>
>> >> The deleted variable is always 1 in current code.
>> >&g
2013/12/9 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
2013/12/4 Rob Landley r...@landley.net:
On 11/16/2013 02:15:23 AM, Axel Lin wrote:
The deleted variable is always 1 in current code.
Initialize deleted variable to be 0, so
2013/12/9 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Mon, Dec 09, 2013 at 04:54:29PM +0800, Axel Lin wrote:
2013/12/9 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
2013/12/4 Rob Landley r...@landley.net:
On 11/16/2013 02:15
2013/12/9 Axel Lin axel@ingics.com:
2013/12/9 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Mon, Dec 09, 2013 at 04:54:29PM +0800, Axel Lin wrote:
2013/12/9 Greg Kroah-Hartman gre...@linuxfoundation.org:
On Wed, Dec 04, 2013 at 02:44:14PM +0800, Axel Lin wrote:
2013/12/4 Rob Landley
2013/11/25 Axel Lin axel@ingics.com:
I'm testing on a nommu platform (arm7tdmi SoC).
Using current Linus' tree + out-of-tree patches for this SoC.
I got below hang while executing ls (busybox) after boot.
/ # ls
[ 51.036191] [ cut here ]
[ 51.042242] WARNING
According to the datasheet, the address of FABID is 0x4. Fix it.
Signed-off-by: Axel Lin
---
drivers/regulator/pfuze100-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/pfuze100-regulator.c
b/drivers/regulator/pfuze100-regulator.c
index 032df37
According to the datasheet, the address of FABID is 0x4. Fix it.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/regulator/pfuze100-regulator.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/regulator/pfuze100-regulator.c
b/drivers/regulator/pfuze100
於 三，2013-12-04 於 13:32 -0800，Andrew Morton 提到：
> On Wed, 04 Dec 2013 16:59:38 +0800 Axel Lin wrote:
>
> > >
> > > Please add a lot more printk's so we can narrow it down further? I'd
> > > use something like
> > >
> > > printk(%d: %d
cess+0x20/0x4c)
[<83c4>] (try_to_run_init_process+0x0/0x4c) from [<002c74f0>]
(kernel_init+0x98/0x104)
r5:002c7458 r4:003e8300
[<002c7458>] (kernel_init+0x0/0x104) from [<9070>] (ret_from_fork+0x14/0x24)
r4: r3:
於 日，2013-12-01 於 18:53 +0800，Axel L
)
[83c4] (try_to_run_init_process+0x0/0x4c) from [002c74f0]
(kernel_init+0x98/0x104)
r5:002c7458 r4:003e8300
[002c7458] (kernel_init+0x0/0x104) from [9070] (ret_from_fork+0x14/0x24)
r4: r3:
於 日，2013-12-01 於 18:53 +0800，Axel Lin 提到：
Hi,
I got below messages while booting
於 三，2013-12-04 於 13:32 -0800，Andrew Morton 提到：
On Wed, 04 Dec 2013 16:59:38 +0800 Axel Lin axel@ingics.com wrote:
Please add a lot more printk's so we can narrow it down further? I'd
use something like
printk(%d: %d\n, __LINE__, preempt_count());
(note
2013/12/4 Rob Landley :
> On 11/16/2013 02:15:23 AM, Axel Lin wrote:
>>
>> The deleted variable is always 1 in current code.
>> Initialize deleted variable to be 0, so delete_path() will be called only
>> when
>> necessary.
>>
>> Signed-off-by: Axel
2013/12/4 Rob Landley r...@landley.net:
On 11/16/2013 02:15:23 AM, Axel Lin wrote:
The deleted variable is always 1 in current code.
Initialize deleted variable to be 0, so delete_path() will be called only
when
necessary.
Signed-off-by: Axel Lin axel@ingics.com
I'm not seeing
2013/11/13 Ming Lei :
> Hi Axel,
>
> On Wed, Nov 13, 2013 at 5:58 PM, Axel Lin wrote:
>>
>> Hi Ming,
>> You missed "; then" in the end of if statement in your patch.
>>
>> So I got below error with your patch:
>> scripts/link-vmlinux.sh: l
Fix a trivial typo.
Signed-off-by: Axel Lin
---
drivers/pinctrl/pinctrl-abx500.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-abx500.h b/drivers/pinctrl/pinctrl-abx500.h
index eeca8f9..8229380 100644
--- a/drivers/pinctrl/pinctrl-abx500.h
+++ b
Hi,
I got below messages while booting.
Testing on a nommu platform with 3.13.0-rc2 + patches for this SoC.
VFS: Mounted root (jffs2 filesystem) on device 31:1.
devtmpfs: mounted
Freeing unused kernel memory: 92K (003a4000 - 003bb000)
BUG: sleeping function called from invalid context at
Hi,
I got below messages while booting.
Testing on a nommu platform with 3.13.0-rc2 + patches for this SoC.
VFS: Mounted root (jffs2 filesystem) on device 31:1.
devtmpfs: mounted
Freeing unused kernel memory: 92K (003a4000 - 003bb000)
BUG: sleeping function called from invalid context at
Fix a trivial typo.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/pinctrl/pinctrl-abx500.h | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/drivers/pinctrl/pinctrl-abx500.h b/drivers/pinctrl/pinctrl-abx500.h
index eeca8f9..8229380 100644
--- a/drivers/pinctrl/pinctrl
2013/11/13 Ming Lei tom.leim...@gmail.com:
Hi Axel,
On Wed, Nov 13, 2013 at 5:58 PM, Axel Lin axel@ingics.com wrote:
Hi Ming,
You missed ; then in the end of if statement in your patch.
So I got below error with your patch:
scripts/link-vmlinux.sh: line 87: syntax error near
Change the device name of the regulator function to the one chosen for
MODULE_ALIAS. This fixes kernel auto-module loading for the regulator function.
Signed-off-by: Axel Lin
---
Hi,
I don't have the hardware to test, but I think this needs fix.
We have a similar fix in
commit ec8da805c43a &quo
Change the device name of the regulator function to the one chosen for
MODULE_ALIAS. This fixes kernel auto-module loading for the regulator function.
Signed-off-by: Axel Lin axel@ingics.com
---
Hi,
I don't have the hardware to test, but I think this needs fix.
We have a similar fix in
commit
ndefined reference to `might_fault'
arch/arm/kernel/built-in.o:arch/arm/kernel/signal.c:177: more undefined
references to `might_fault' follow
make: *** [vmlinux] Error 1
Signed-off-by: Axel Lin
Cc: Michael S. Tsirkin
Cc: Peter Zijlstra
---
include/linux/kernel.h | 3 ++-
1 file changed, 2 insertio
to `might_fault'
arch/arm/kernel/built-in.o:arch/arm/kernel/signal.c:177: more undefined
references to `might_fault' follow
make: *** [vmlinux] Error 1
Signed-off-by: Axel Lin axel@ingics.com
Cc: Michael S. Tsirkin m...@redhat.com
Cc: Peter Zijlstra pet...@infradead.org
---
include/linux
2013/11/27 Andrew Morton :
> On Tue, 26 Nov 2013 17:29:29 +0800 Axel Lin wrote:
>
>> Hi,
>> I got below error messages while starting mdev (busybox).
>>
>> ...
>>
>> [ 108.537109] chmod: page allocation failure: order:8, mode:0xd0
>
> It w
2013/11/27 Andrew Morton a...@linux-foundation.org:
On Tue, 26 Nov 2013 17:29:29 +0800 Axel Lin axel@ingics.com wrote:
Hi,
I got below error messages while starting mdev (busybox).
...
[ 108.537109] chmod: page allocation failure: order:8, mode:0xd0
It wants to allocate 2^8
Both num_regulators and **rdev are no longer required after this conversion,
thus remove them from struct lp3971.
Signed-off-by: Axel Lin
---
Hi Mark,
This patch was sent on https://lkml.org/lkml/2013/11/13/34 .
Although you replied applied, it still not in your tree.
So here is a resend.
Axel
Hi,
I got below error messages while starting mdev (busybox).
/ # free
total used free shared buffers
Mem: 23628 8924147040 72
-/+ buffers: 885214776
/ # exec /sbin/init
Starting logging:
Hi,
I got below error messages while starting mdev (busybox).
/ # free
total used free shared buffers
Mem: 23628 8924147040 72
-/+ buffers: 885214776
/ # exec /sbin/init
Starting logging:
Both num_regulators and **rdev are no longer required after this conversion,
thus remove them from struct lp3971.
Signed-off-by: Axel Lin axel@ingics.com
---
Hi Mark,
This patch was sent on https://lkml.org/lkml/2013/11/13/34 .
Although you replied applied, it still not in your tree.
So here
2013/11/26 Uwe Kleine-König :
> Hello,
>
> On Mon, Nov 25, 2013 at 11:26:00PM +0800, Axel Lin wrote:
>> I got hangup at boot after "VFS: Mounted root (ext2 filesystem) on device
>> 1:0.".
>> I'm using ext2 initrd as root.
> which mainline version are you u
Hi,
I got hangup at boot after "VFS: Mounted root (ext2 filesystem) on device 1:0.".
I'm using ext2 initrd as root.
It seems hangup after mount root, but in the progress of mounting devtmpfs.
After debug it, I found it hangs at calling page_cache_release.
Note, I'm testing on an nommu platform
Hi,
I got hangup at boot after VFS: Mounted root (ext2 filesystem) on device 1:0..
I'm using ext2 initrd as root.
It seems hangup after mount root, but in the progress of mounting devtmpfs.
After debug it, I found it hangs at calling page_cache_release.
Note, I'm testing on an nommu platform
2013/11/26 Uwe Kleine-König u.kleine-koe...@pengutronix.de:
Hello,
On Mon, Nov 25, 2013 at 11:26:00PM +0800, Axel Lin wrote:
I got hangup at boot after VFS: Mounted root (ext2 filesystem) on device
1:0..
I'm using ext2 initrd as root.
which mainline version are you using? I don't know your
I'm testing on a nommu platform (arm7tdmi SoC).
Using current Linus' tree + out-of-tree patches for this SoC.
I got below hang while executing ls (busybox) after boot.
/ # ls
[ 51.036191] [ cut here ]
[ 51.042242] WARNING: CPU: 0 PID: 1 at kernel/locking/lockdep.c:3312
bcm_timer_ids is no longer used after converting to CLOCKSOURCE_OF_DECLARE.
Signed-off-by: Axel Lin
---
drivers/clocksource/bcm_kona_timer.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/clocksource/bcm_kona_timer.c
b/drivers/clocksource/bcm_kona_timer.c
index 0d7d8c3
bcm_timer_ids is no longer used after converting to CLOCKSOURCE_OF_DECLARE.
Signed-off-by: Axel Lin axel@ingics.com
---
drivers/clocksource/bcm_kona_timer.c | 6 --
1 file changed, 6 deletions(-)
diff --git a/drivers/clocksource/bcm_kona_timer.c
b/drivers/clocksource/bcm_kona_timer.c
901 - 1000 of 2364 matches
Mail list logo
