Hi All, On 24 March 2016 at 20:04, Adrian Hunter <adrian.hunter@xxxxxxxxx> wrote: > On 24/03/16 16:03, Ludovic Desroches wrote: >> Hi, >> >> On Thu, Mar 24, 2016 at 03:11:25PM +0200, Adrian Hunter wrote: >>> On 24/03/16 10:42, Krzysztof Kozlowski wrote: >>>> On 24.03.2016 17:24, Jisheng Zhang wrote: >>>>> Hi, >>>>> >>>>> On Thu, 24 Mar 2016 17:09:27 +0900 Jaehoon Chung wrote: >>>>> >>>>>> Hi, >>>>>> >>>>>> On 03/24/2016 04:58 PM, Jisheng Zhang wrote: >>>>>>> Hi, >>>>>>> >>>>>>> On Thu, 24 Mar 2016 16:28:56 +0900 Krzysztof Kozlowski wrote: >>>>>>> >>>>>>>> Hi, >>>>>>>> >>>>>>>> After 918f4cbd4340 ("mmc: sdhci: restore behavior when setting VDD via >>>>>>>> external regulator") On Trats2 board I see warnings for invalid VDD >>>>>>>> value (2.8V): >>>>>>>> >>>>>>>> [ 3.119656] ------------[ cut here ]------------ >>>>>>>> [ 3.119666] WARNING: CPU: 3 PID: 90 at >>>>>>>> ../drivers/mmc/host/sdhci.c:1234 sdhci_do_set_ios+0x4cc/0x5e0 >>>>>>>> [ 3.119669] mmc0: Invalid vdd 0x10 >>>>>>> >>>>>>> Per my understanding, the wrong vdd indicates a wrong ocr, what's the voltage of >>>>>>> this host's vmmc regulator? >>>>>> >>>>>> As i know, it's fixed-voltage with gpio on trats2. It's 2.8V. >>>>>> I didn't check this entirely..need to check ocr value. >>>>>> >>>>> >>>>> I may know the reason. the vmmc is 2.8v, then mmc_regulator_get_supply() convert >>>>> the value to a ocr as 0x10. The key here is that the 2.8v is invalid in SDHCI >>>>> case and isn't accepted by current sdhci driver. >>>> >>>> Yeah, I already wrote that. It is the part of the warning and my email. >>>> Our regulator is fixed at 2.8 which is 0x10. :) >>>> >>>>> I dunno the elegant solution to handle this case, let's wait for sdhci maintainers >>>>> idea. >>>> >>>> Hmm... >>> >>> I haven't tested it, but what about this: >>> >>> From: Adrian Hunter <adrian.hunter@xxxxxxxxx> >>> Date: Thu, 24 Mar 2016 14:29:24 +0200 >>> Subject: [PATCH] mmc: sdhci: Fix regression setting power on Trats2 board >>> >>> Several commits relating to setting power have been introducing >>> problems by putting driver-specific rules into generic SDHCI code. >>> >> >> I remember that I have acked this patch and I still believe it is not >> driver-specific code. > > It depends how you look at it. I decided that since the previous code had > been there for a while (since 3.19), it should stay the default. > > I agree you could look at it differently. > >> >> Without this patch, if a regulator is used, we set this regulator and we >> write SDHCI_POWER_ON to SDHCI_POWER_CONTROL so 0x01. In the >> specification, it is said: >> 'If an unsupported voltage is selected, the Host System shall not supply >> SD Bus voltage'. >> Valid voltage are 111b for 3.3V, 110b for 3.0V and 101b for 1.8V. 0 is >> seen as an invalid voltage. > > Again, it depends how you look at it. From my point of view, the > specification doesn't cover external regulators so you can't expect to know > what to put in the power control register. > >> >> If the controller strictly applies the specification then it won't perform >> the power on request. It's what happens with sdhci-pxav3 and >> sdhci-of-at91. > > So if I make the same change to sdhci-of-at91 as sdhci-pxav3, it will work > for you? > >> >> I don't care if the patch is reverted because I no more describe >> the regulators but I think the behavior with Jisheng's patch is the right >> one. Maybe more cases should be managed in the switch statement as >> suggested by Tobias. > > But always the possibility it won't work for some driver. > >> >> >>> Fix by adding a 'set_power' callback and restoring the default >>> behaviour prior to commit 918f4cbd4340. The desired behaviour >>> of commit 918f4cbd4340 is gotten by having sdhci-pxav3 provide >>> its own set_power callback. >>> >> >> Adding callbacks is the only way to solve all these quirks or >> interpretations of the specification. >> > > At least it means we can go forward. On My Odroid U3 with debug flags enable I am observing bellow deadlock. --------------------------------------------------------------------------------------------------------- [ 202.519524] BUG: sleeping function called from invalid context at kernel/locking/mutex.c:617 [ 202.522364] in_atomic(): 1, irqs_disabled(): 128, pid: 100, name: mmcqd/0 [ 202.529129] 1 lock held by mmcqd/0/100: [ 202.529150] #0: (&(&host->lock)->rlock#2){-.-...}, at: [<c059db68>] sdhci_do_set_ios+0x1c/0x484 [ 202.529271] irq event stamp: 703530 [ 202.529291] hardirqs last enabled at (703529): [<c076d108>] _raw_spin_unlock_irqrestore+0x6c/0x74 [ 202.529343] hardirqs last disabled at (703530): [<c076cea8>] _raw_spin_lock_irqsave+0x1c/0x84 [ 202.529384] softirqs last enabled at (703456): [<c0127518>] __do_softirq+0x244/0x2c0 [ 202.529438] softirqs last disabled at (703445): [<c0127938>] irq_exit+0xec/0x128 [ 202.529472] Preemption disabled at:[< (null)>] (null) [ 202.534415] [ 202.534449] CPU: 0 PID: 100 Comm: mmcqd/0 Not tainted 4.6.0-rc1-u3s #38 [ 202.534473] Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) [ 202.534544] [<c010ee2c>] (unwind_backtrace) from [<c010bab8>] (show_stack+0x10/0x14) [ 202.534594] [<c010bab8>] (show_stack) from [<c037e0c0>] (dump_stack+0x98/0xc4) [ 202.534639] [<c037e0c0>] (dump_stack) from [<c0768934>] (mutex_lock_nested+0x2c/0x4dc) [ 202.534685] [<c0768934>] (mutex_lock_nested) from [<c05c00a0>] (clk_prepare_lock+0x50/0xf8) [ 202.534726] [<c05c00a0>] (clk_prepare_lock) from [<c05c14b0>] (clk_round_rate+0x1c/0x58) [ 202.534773] [<c05c14b0>] (clk_round_rate) from [<c05a00a0>] (sdhci_s3c_set_clock+0x18c/0x1b0) [ 202.534819] [<c05a00a0>] (sdhci_s3c_set_clock) from [<c05a00e8>] (sdhci_cmu_set_clock+0x24/0x17c) [ 202.534860] [<c05a00e8>] (sdhci_cmu_set_clock) from [<c059dbc4>] (sdhci_do_set_ios+0x78/0x484) [ 202.534904] [<c059dbc4>] (sdhci_do_set_ios) from [<c059e1a8>] (sdhci_runtime_resume_host+0x60/0x114) [ 202.534957] [<c059e1a8>] (sdhci_runtime_resume_host) from [<c04728c8>] (__rpm_callback+0x2c/0x60) [ 202.535000] [<c04728c8>] (__rpm_callback) from [<c0472950>] (rpm_callback+0x54/0x80) [ 202.535041] [<c0472950>] (rpm_callback) from [<c0473830>] (rpm_resume+0x364/0x558) [ 202.535081] [<c0473830>] (rpm_resume) from [<c0473a84>] (__pm_runtime_resume+0x60/0x8c) [ 202.535125] [<c0473a84>] (__pm_runtime_resume) from [<c0588f50>] (__mmc_claim_host+0x1b4/0x1f8) [ 202.535176] [<c0588f50>] (__mmc_claim_host) from [<c059174c>] (mmc_sd_runtime_resume+0x20/0xac) [ 202.535220] [<c059174c>] (mmc_sd_runtime_resume) from [<c04728c8>] (__rpm_callback+0x2c/0x60) [ 202.535259] [<c04728c8>] (__rpm_callback) from [<c0472950>] (rpm_callback+0x54/0x80) [ 202.535299] [<c0472950>] (rpm_callback) from [<c0473830>] (rpm_resume+0x364/0x558) [ 202.535340] [<c0473830>] (rpm_resume) from [<c0473a84>] (__pm_runtime_resume+0x60/0x8c) [ 202.535379] [<c0473a84>] (__pm_runtime_resume) from [<c0588fa8>] (mmc_get_card+0x14/0x24) [ 202.535420] [<c0588fa8>] (mmc_get_card) from [<c0598b50>] (mmc_blk_issue_rq+0x258/0x4f0) [ 202.535461] [<c0598b50>] (mmc_blk_issue_rq) from [<c059a4b0>] (mmc_queue_thread+0xd0/0x1d8) [ 202.535513] [<c059a4b0>] (mmc_queue_thread) from [<c0142c6c>] (kthread+0xf4/0x10c) [ 202.535561] [<c0142c6c>] (kthread) from [<c0107950>] (ret_from_fork+0x14/0x24) [ 202.535624] [ 202.535893] ====================================================== [ 202.542059] [ INFO: HARDIRQ-safe -> HARDIRQ-unsafe lock order detected ] [ 202.548742] 4.6.0-rc1-u3s #38 Not tainted [ 202.552732] ------------------------------------------------------ [ 202.558902] mmcqd/0/100 [HC0[0]:SC0[0]:HE0:SE1] is trying to acquire: [ 202.565317] (prepare_lock){+.+...}, at: [<c05c00a0>] clk_prepare_lock+0x50/0xf8 [ 202.572695] [ 202.572695] and this task is already holding: [ 202.578510] (&(&host->lock)->rlock#2){-.-...}, at: [<c059db68>] sdhci_do_set_ios+0x1c/0x484 [ 202.586930] which would create a new lock dependency: [ 202.591964] (&(&host->lock)->rlock#2){-.-...} -> (prepare_lock){+.+...} [ 202.598650] [ 202.598650] but this new dependency connects a HARDIRQ-irq-safe lock: [ 202.606546] (&(&host->lock)->rlock#2){-.-...} [ 202.606546] ... which became HARDIRQ-irq-safe at: [ 202.614359] [<c076cc5c>] _raw_spin_lock+0x3c/0x74 [ 202.619219] [<c059f44c>] sdhci_irq+0x1c/0x814 [ 202.623732] [<c01810e8>] handle_irq_event_percpu+0x9c/0x150 [ 202.629461] [<c01811d4>] handle_irq_event+0x38/0x5c [ 202.634495] [<c0184504>] handle_fasteoi_irq+0xd0/0x1a8 [ 202.639790] [<c0180704>] generic_handle_irq+0x24/0x34 [ 202.644998] [<c0180a18>] __handle_domain_irq+0x7c/0xec [ 202.650293] [<c0101514>] gic_handle_irq+0x54/0x94 [ 202.655154] [<c010c5b8>] __irq_svc+0x58/0x98 [ 202.659581] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 202.664354] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 202.669128] [<c0167100>] cpu_startup_entry+0x1c8/0x24c [ 202.674423] [<c0b00c88>] start_kernel+0x39c/0x3a8 [ 202.679284] [<4000807c>] 0x4000807c [ 202.682933] [ 202.682933] to a HARDIRQ-irq-unsafe lock: [ 202.688399] (prepare_lock){+.+...} [ 202.688399] ... which became HARDIRQ-irq-unsafe at: [ 202.695429] ... [<c07687e8>] mutex_trylock+0x130/0x250 [ 202.700637] [<c05c0060>] clk_prepare_lock+0x10/0xf8 [ 202.705672] [<c05c2a0c>] __clk_create_clk.part.13+0x4c/0x80 [ 202.711400] [<c05c33e8>] __of_clk_get_from_provider+0x88/0xf8 [ 202.717303] [<c05bfb08>] __of_clk_get_by_name+0xfc/0x114 [ 202.722771] [<c05bfb68>] clk_get+0x2c/0x5c [ 202.727024] [<c05bf62c>] devm_clk_get+0x3c/0x78 [ 202.731712] [<c0b1d8a4>] exynos_sysmmu_probe+0xe8/0x310 [ 202.737093] [<c046baa4>] platform_drv_probe+0x4c/0xb0 [ 202.742301] [<c046a2a4>] driver_probe_device+0x20c/0x2b8 [ 202.747769] [<c046863c>] bus_for_each_drv+0x60/0x94 [ 202.752804] [<c0469fb8>] __device_attach+0xb4/0x118 [ 202.757838] [<c046945c>] bus_probe_device+0x88/0x90 [ 202.762873] [<c0467794>] device_add+0x370/0x570 [ 202.767560] [<c05babe0>] of_platform_device_create_pdata+0x84/0xb8 [ 202.773896] [<c0b1d784>] exynos_iommu_of_setup+0x120/0x158 [ 202.779538] [<c0b1d59c>] of_iommu_init+0x44/0x78 [ 202.784312] [<c0b03594>] customize_machine+0x8/0x44 [ 202.789347] [<c01017f4>] do_one_initcall+0x90/0x1dc [ 202.794381] [<c0b00dec>] kernel_init_freeable+0x158/0x1e8 [ 202.799936] [<c076527c>] kernel_init+0x8/0x114 [ 202.804537] [<c0107950>] ret_from_fork+0x14/0x24 [ 202.809313] [ 202.809313] other info that might help us debug this: [ 202.809313] [ 202.817299] Possible interrupt unsafe locking scenario: [ 202.817299] [ 202.824068] CPU0 CPU1 [ 202.828581] ---- ---- [ 202.833094] lock(prepare_lock); [ 202.836394] local_irq_disable(); [ 202.842295] lock(&(&host->lock)->rlock#2); [ 202.849065] lock(prepare_lock); [ 202.854881] <Interrupt> [ 202.857484] lock(&(&host->lock)->rlock#2); [ 202.861912] [ 202.861912] *** DEADLOCK *** [ 202.861912] [ 202.867820] 1 lock held by mmcqd/0/100: [ 202.871634] #0: (&(&host->lock)->rlock#2){-.-...}, at: [<c059db68>] sdhci_do_set_ios+0x1c/0x484 [ 202.880489] [ 202.880489] the dependencies between HARDIRQ-irq-safe lock and the holding lock: [ 202.888106] -> (&(&host->lock)->rlock#2){-.-...} ops: 69588 { [ 202.893768] IN-HARDIRQ-W at: [ 202.896893] [<c076cc5c>] _raw_spin_lock+0x3c/0x74 [ 202.903316] [<c059f44c>] sdhci_irq+0x1c/0x814 [ 202.909392] [<c01810e8>] handle_irq_event_percpu+0x9c/0x150 [ 202.916683] [<c01811d4>] handle_irq_event+0x38/0x5c [ 202.923280] [<c0184504>] handle_fasteoi_irq+0xd0/0x1a8 [ 202.930137] [<c0180704>] generic_handle_irq+0x24/0x34 [ 202.936908] [<c0180a18>] __handle_domain_irq+0x7c/0xec [ 202.943765] [<c0101514>] gic_handle_irq+0x54/0x94 [ 202.950188] [<c010c5b8>] __irq_svc+0x58/0x98 [ 202.956177] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 202.962513] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 202.968850] [<c0167100>] cpu_startup_entry+0x1c8/0x24c [ 202.975707] [<c0b00c88>] start_kernel+0x39c/0x3a8 [ 202.982130] [<4000807c>] 0x4000807c [ 202.987340] IN-SOFTIRQ-W at: [ 202.990463] [<c076ced4>] _raw_spin_lock_irqsave+0x48/0x84 [ 202.997581] [<c059d71c>] sdhci_tasklet_finish+0x18/0x1f4 [ 203.004611] [<c01271b4>] tasklet_action+0xac/0x164 [ 203.011122] [<c012743c>] __do_softirq+0x168/0x2c0 [ 203.017544] [<c0127938>] irq_exit+0xec/0x128 [ 203.023534] [<c0180a1c>] __handle_domain_irq+0x80/0xec [ 203.030391] [<c0101514>] gic_handle_irq+0x54/0x94 [ 203.036814] [<c010c5b8>] __irq_svc+0x58/0x98 [ 203.042804] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 203.049140] [<c01083b8>] arch_cpu_idle+0x24/0x3c [ 203.055476] [<c0167100>] cpu_startup_entry+0x1c8/0x24c [ 203.062334] [<c0b00c88>] start_kernel+0x39c/0x3a8 [ 203.068756] [<4000807c>] 0x4000807c [ 203.073966] INITIAL USE at: [ 203.077003] [<c076ced4>] _raw_spin_lock_irqsave+0x48/0x84 [ 203.084033] [<c059db68>] sdhci_do_set_ios+0x1c/0x484 [ 203.090630] [<c059e1a8>] sdhci_runtime_resume_host+0x60/0x114 [ 203.098008] [<c04728c8>] __rpm_callback+0x2c/0x60 [ 203.104345] [<c047291c>] rpm_callback+0x20/0x80 [ 203.110507] [<c0473830>] rpm_resume+0x364/0x558 [ 203.116670] [<c0473a84>] __pm_runtime_resume+0x60/0x8c [ 203.123440] [<c0588f50>] __mmc_claim_host+0x1b4/0x1f8 [ 203.130124] [<c058b9fc>] mmc_start_host+0x34/0xa8 [ 203.136461] [<c058cb00>] mmc_add_host+0x5c/0x80 [ 203.142624] [<c059eb20>] sdhci_add_host+0x8c4/0xdec [ 203.149133] [<c05a0704>] sdhci_s3c_probe+0x4c4/0x568 [ 203.155730] [<c046baa4>] platform_drv_probe+0x4c/0xb0 [ 203.162414] [<c046a2a4>] driver_probe_device+0x20c/0x2b8 [ 203.169358] [<c046a410>] __driver_attach+0xc0/0xc4 [ 203.175781] [<c0468594>] bus_for_each_dev+0x68/0x9c [ 203.182291] [<c046970c>] bus_add_driver+0x1a0/0x218 [ 203.188801] [<c046ab24>] driver_register+0x78/0xf8 [ 203.195224] [<c01017f4>] do_one_initcall+0x90/0x1dc [ 203.201734] [<c0b00dec>] kernel_init_freeable+0x158/0x1e8 [ 203.208765] [<c076527c>] kernel_init+0x8/0x114 [ 203.214841] [<c0107950>] ret_from_fork+0x14/0x24 [ 203.221091] } [ 203.222740] ... key at: [<c1617248>] __key.32364+0x0/0x8 [ 203.228556] ... acquired at: [ 203.231506] [<c0174a48>] lock_acquire+0xa8/0xd0 [ 203.236280] [<c0768980>] mutex_lock_nested+0x78/0x4dc [ 203.241575] [<c05c00a0>] clk_prepare_lock+0x50/0xf8 [ 203.246696] [<c05c14b0>] clk_round_rate+0x1c/0x58 [ 203.251643] [<c05a00a0>] sdhci_s3c_set_clock+0x18c/0x1b0 [ 203.257199] [<c05a00e8>] sdhci_cmu_set_clock+0x24/0x17c [ 203.262667] [<c059dbc4>] sdhci_do_set_ios+0x78/0x484 [ 203.267875] [<c059e1a8>] sdhci_runtime_resume_host+0x60/0x114 [ 203.273864] [<c04728c8>] __rpm_callback+0x2c/0x60 [ 203.278812] [<c0472950>] rpm_callback+0x54/0x80 [ 203.283586] [<c0473830>] rpm_resume+0x364/0x558 [ 203.288359] [<c0473a84>] __pm_runtime_resume+0x60/0x8c [ 203.293741] [<c0588f50>] __mmc_claim_host+0x1b4/0x1f8 [ 203.299036] [<c059174c>] mmc_sd_runtime_resume+0x20/0xac [ 203.304591] [<c04728c8>] __rpm_callback+0x2c/0x60 [ 203.309539] [<c0472950>] rpm_callback+0x54/0x80 [ 203.314313] [<c0473830>] rpm_resume+0x364/0x558 [ 203.319087] [<c0473a84>] __pm_runtime_resume+0x60/0x8c [ 203.324468] [<c0588fa8>] mmc_get_card+0x14/0x24 [ 203.329243] [<c0598b50>] mmc_blk_issue_rq+0x258/0x4f0 [ 203.334538] [<c059a4b0>] mmc_queue_thread+0xd0/0x1d8 [ 203.339745] [<c0142c6c>] kthread+0xf4/0x10c [ 203.344172] [<c0107950>] ret_from_fork+0x14/0x24 [ 203.349033] [ 203.350509] [ 203.350509] the dependencies between the lock to be acquired and HARDIRQ-irq-unsafe lock: [ 203.358904] -> (prepare_lock){+.+...} ops: 5285 { [ 203.363531] HARDIRQ-ON-W at: [ 203.366654] [<c07687e8>] mutex_trylock+0x130/0x250 [ 203.373164] [<c05c0060>] clk_prepare_lock+0x10/0xf8 [ 203.379761] [<c05c2a0c>] __clk_create_clk.part.13+0x4c/0x80 [ 203.387052] [<c05c33e8>] __of_clk_get_from_provider+0x88/0xf8 [ 203.394517] [<c05bfb08>] __of_clk_get_by_name+0xfc/0x114 [ 203.401548] [<c05bfb68>] clk_get+0x2c/0x5c [ 203.407363] [<c05bf62c>] devm_clk_get+0x3c/0x78 [ 203.413613] [<c0b1d8a4>] exynos_sysmmu_probe+0xe8/0x310 [ 203.420557] [<c046baa4>] platform_drv_probe+0x4c/0xb0 [ 203.427327] [<c046a2a4>] driver_probe_device+0x20c/0x2b8 [ 203.434358] [<c046863c>] bus_for_each_drv+0x60/0x94 [ 203.440955] [<c0469fb8>] __device_attach+0xb4/0x118 [ 203.447552] [<c046945c>] bus_probe_device+0x88/0x90 [ 203.454149] [<c0467794>] device_add+0x370/0x570 [ 203.460399] [<c05babe0>] of_platform_device_create_pdata+0x84/0xb8 [ 203.468297] [<c0b1d784>] exynos_iommu_of_setup+0x120/0x158 [ 203.475501] [<c0b1d59c>] of_iommu_init+0x44/0x78 [ 203.481838] [<c0b03594>] customize_machine+0x8/0x44 [ 203.488434] [<c01017f4>] do_one_initcall+0x90/0x1dc [ 203.495031] [<c0b00dec>] kernel_init_freeable+0x158/0x1e8 [ 203.502149] [<c076527c>] kernel_init+0x8/0x114 [ 203.508312] [<c0107950>] ret_from_fork+0x14/0x24 [ 203.514650] SOFTIRQ-ON-W at: [ 203.517773] [<c07687e8>] mutex_trylock+0x130/0x250 [ 203.524283] [<c05c0060>] clk_prepare_lock+0x10/0xf8 [ 203.530880] [<c05c2a0c>] __clk_create_clk.part.13+0x4c/0x80 [ 203.538171] [<c05c33e8>] __of_clk_get_from_provider+0x88/0xf8 [ 203.545636] [<c05bfb08>] __of_clk_get_by_name+0xfc/0x114 [ 203.552666] [<c05bfb68>] clk_get+0x2c/0x5c [ 203.558482] [<c05bf62c>] devm_clk_get+0x3c/0x78 [ 203.564732] [<c0b1d8a4>] exynos_sysmmu_probe+0xe8/0x310 [ 203.571676] [<c046baa4>] platform_drv_probe+0x4c/0xb0 [ 203.578446] [<c046a2a4>] driver_probe_device+0x20c/0x2b8 [ 203.585477] [<c046863c>] bus_for_each_drv+0x60/0x94 [ 203.592074] [<c0469fb8>] __device_attach+0xb4/0x118 [ 203.598671] [<c046945c>] bus_probe_device+0x88/0x90 [ 203.605267] [<c0467794>] device_add+0x370/0x570 [ 203.611517] [<c05babe0>] of_platform_device_create_pdata+0x84/0xb8 [ 203.619416] [<c0b1d784>] exynos_iommu_of_setup+0x120/0x158 [ 203.626620] [<c0b1d59c>] of_iommu_init+0x44/0x78 [ 203.632956] [<c0b03594>] customize_machine+0x8/0x44 [ 203.639553] [<c01017f4>] do_one_initcall+0x90/0x1dc [ 203.646150] [<c0b00dec>] kernel_init_freeable+0x158/0x1e8 [ 203.653268] [<c076527c>] kernel_init+0x8/0x114 [ 203.659431] [<c0107950>] ret_from_fork+0x14/0x24 [ 203.665768] INITIAL USE at: [ 203.668805] [<c07687b0>] mutex_trylock+0xf8/0x250 [ 203.675142] [<c05c0060>] clk_prepare_lock+0x10/0xf8 [ 203.681651] [<c05c2a0c>] __clk_create_clk.part.13+0x4c/0x80 [ 203.688856] [<c05c2d2c>] clk_register+0x1b8/0x5f0 [ 203.695192] [<c05c4564>] clk_register_fixed_rate_with_accuracy+0x94/0xc4 [ 203.703525] [<c05c45ac>] clk_register_fixed_rate+0x18/0x20 [ 203.710642] [<c0b249fc>] samsung_clk_register_fixed_rate+0x4c/0xb8 [ 203.718455] [<c0b24af8>] samsung_clk_of_register_fixed_ext+0x90/0x98 [ 203.726440] [<c0b25800>] exynos4_clk_init+0x88/0x5b0 [ 203.733037] [<c0b24650>] of_clk_init+0x14c/0x1d8 [ 203.739287] [<c0b04488>] time_init+0x24/0x2c [ 203.745189] [<c0b00b4c>] start_kernel+0x260/0x3a8 [ 203.751526] [<4000807c>] 0x4000807c [ 203.756647] } [ 203.758296] ... key at: [<c0d3bf80>] prepare_lock+0x3c/0x54 [ 203.764373] ... acquired at: [ 203.767322] [<c0174a48>] lock_acquire+0xa8/0xd0 [ 203.772096] [<c0768980>] mutex_lock_nested+0x78/0x4dc [ 203.777391] [<c05c00a0>] clk_prepare_lock+0x50/0xf8 [ 203.782512] [<c05c14b0>] clk_round_rate+0x1c/0x58 [ 203.787460] [<c05a00a0>] sdhci_s3c_set_clock+0x18c/0x1b0 [ 203.793015] [<c05a00e8>] sdhci_cmu_set_clock+0x24/0x17c [ 203.798484] [<c059dbc4>] sdhci_do_set_ios+0x78/0x484 [ 203.803691] [<c059e1a8>] sdhci_runtime_resume_host+0x60/0x114 [ 203.809680] [<c04728c8>] __rpm_callback+0x2c/0x60 [ 203.814628] [<c0472950>] rpm_callback+0x54/0x80 [ 203.819402] [<c0473830>] rpm_resume+0x364/0x558 [ 203.824176] [<c0473a84>] __pm_runtime_resume+0x60/0x8c [ 203.829558] [<c0588f50>] __mmc_claim_host+0x1b4/0x1f8 [ 203.834852] [<c059174c>] mmc_sd_runtime_resume+0x20/0xac [ 203.840408] [<c04728c8>] __rpm_callback+0x2c/0x60 [ 203.845355] [<c0472950>] rpm_callback+0x54/0x80 [ 203.850129] [<c0473830>] rpm_resume+0x364/0x558 [ 203.854903] [<c0473a84>] __pm_runtime_resume+0x60/0x8c [ 203.860285] [<c0588fa8>] mmc_get_card+0x14/0x24 [ 203.865059] [<c0598b50>] mmc_blk_issue_rq+0x258/0x4f0 [ 203.870354] [<c059a4b0>] mmc_queue_thread+0xd0/0x1d8 [ 203.875562] [<c0142c6c>] kthread+0xf4/0x10c [ 203.879988] [<c0107950>] ret_from_fork+0x14/0x24 [ 203.884850] [ 203.886326] [ 203.886326] stack backtrace: [ 203.890679] CPU: 0 PID: 100 Comm: mmcqd/0 Not tainted 4.6.0-rc1-u3s #38 [ 203.897266] Hardware name: SAMSUNG EXYNOS (Flattened Device Tree) [ 203.903374] [<c010ee2c>] (unwind_backtrace) from [<c010bab8>] (show_stack+0x10/0x14) [ 203.911089] [<c010bab8>] (show_stack) from [<c037e0c0>] (dump_stack+0x98/0xc4) [ 203.918296] [<c037e0c0>] (dump_stack) from [<c017098c>] (check_usage+0x49c/0x658) [ 203.925757] [<c017098c>] (check_usage) from [<c0170ba8>] (check_irq_usage+0x60/0xb8) [ 203.933509] [<c0170ba8>] (check_irq_usage) from [<c0173404>] (__lock_acquire+0x15c8/0x201c) [ 203.941844] [<c0173404>] (__lock_acquire) from [<c0174a48>] (lock_acquire+0xa8/0xd0) [ 203.949598] [<c0174a48>] (lock_acquire) from [<c0768980>] (mutex_lock_nested+0x78/0x4dc) [ 203.957639] [<c0768980>] (mutex_lock_nested) from [<c05c00a0>] (clk_prepare_lock+0x50/0xf8) [ 203.965969] [<c05c00a0>] (clk_prepare_lock) from [<c05c14b0>] (clk_round_rate+0x1c/0x58) [ 203.974085] [<c05c14b0>] (clk_round_rate) from [<c05a00a0>] (sdhci_s3c_set_clock+0x18c/0x1b0) [ 203.982551] [<c05a00a0>] (sdhci_s3c_set_clock) from [<c05a00e8>] (sdhci_cmu_set_clock+0x24/0x17c) [ 203.991403] [<c05a00e8>] (sdhci_cmu_set_clock) from [<c059dbc4>] (sdhci_do_set_ios+0x78/0x484) [ 203.999993] [<c059dbc4>] (sdhci_do_set_ios) from [<c059e1a8>] (sdhci_runtime_resume_host+0x60/0x114) [ 204.009094] [<c059e1a8>] (sdhci_runtime_resume_host) from [<c04728c8>] (__rpm_callback+0x2c/0x60) [ 204.017938] [<c04728c8>] (__rpm_callback) from [<c0472950>] (rpm_callback+0x54/0x80) [ 204.025661] [<c0472950>] (rpm_callback) from [<c0473830>] (rpm_resume+0x364/0x558) [ 204.033214] [<c0473830>] (rpm_resume) from [<c0473a84>] (__pm_runtime_resume+0x60/0x8c) [ 204.041201] [<c0473a84>] (__pm_runtime_resume) from [<c0588f50>] (__mmc_claim_host+0x1b4/0x1f8) [ 204.049885] [<c0588f50>] (__mmc_claim_host) from [<c059174c>] (mmc_sd_runtime_resume+0x20/0xac) [ 204.058563] [<c059174c>] (mmc_sd_runtime_resume) from [<c04728c8>] (__rpm_callback+0x2c/0x60) [ 204.067063] [<c04728c8>] (__rpm_callback) from [<c0472950>] (rpm_callback+0x54/0x80) [ 204.074788] [<c0472950>] (rpm_callback) from [<c0473830>] (rpm_resume+0x364/0x558) [ 204.082340] [<c0473830>] (rpm_resume) from [<c0473a84>] (__pm_runtime_resume+0x60/0x8c) [ 204.090326] [<c0473a84>] (__pm_runtime_resume) from [<c0588fa8>] (mmc_get_card+0x14/0x24) [ 204.098487] [<c0588fa8>] (mmc_get_card) from [<c0598b50>] (mmc_blk_issue_rq+0x258/0x4f0) [ 204.106559] [<c0598b50>] (mmc_blk_issue_rq) from [<c059a4b0>] (mmc_queue_thread+0xd0/0x1d8) [ 204.114903] [<c059a4b0>] (mmc_queue_thread) from [<c0142c6c>] (kthread+0xf4/0x10c) [ 204.122452] [<c0142c6c>] (kthread) from [<c0107950>] (ret_from_fork+0x14/0x24) --------------------------------------------------------------------------------------------------------- Best Regards -Anand Moon > -- > To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in > the body of a message to majordomo@xxxxxxxxxxxxxxx > More majordomo info at http://vger.kernel.org/majordomo-info.html -- To unsubscribe from this list: send the line "unsubscribe linux-samsung-soc" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html