lkml.org 
[lkml]   [2021]   [May]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH V10] i2c: i2c-qcom-geni: Add shutdown callback for i2c
On 2021-05-20 13:45, Stephen Boyd wrote:
> Quoting rojay@codeaurora.org (2021-05-16 23:32:50)
>> Hi Stephen,
>>
>> Now, I have made the changes, calling i2c_mark_adapter_suspended() in
>> shutdown() and i2c_mark_adapter_suspended()/_resumed() from runtime
>> suspend/resume also and validated the changes. I have also picked
>> your patch [1] for this validation.
>>
>> During the device boot up I am seeing multiple traces shown below.
>> Are these expected now and needs to be fixed from rt5682/respective
>> client driver?
>>
>> Trace1:
>> [ 11.709477] i2c i2c-9: Transfer while suspended
>> [ 11.905595] Call trace:
>> [ 11.908124] __i2c_transfer+0xb8/0x38c
>> [ 11.911984] i2c_transfer+0xa0/0xf4
>> [ 11.915569] i2c_transfer_buffer_flags+0x68/0x9c
>> [ 11.920314] regmap_i2c_write+0x34/0x64
>> [ 11.924255] _regmap_raw_write_impl+0x4e8/0x7bc
>> [ 11.928911] _regmap_bus_raw_write+0x70/0x8c
>> [ 11.933301] _regmap_write+0x100/0x150
>> [ 11.937152] regmap_write+0x54/0x78
>> [ 11.940744] soc_component_write_no_lock+0x34/0xa8
>> [ 11.945666] snd_soc_component_write+0x3c/0x5c
>> [ 11.950242] rt5682_set_component_pll+0x1e4/0x2b4 [snd_soc_rt5682]
>> [ 11.956588] snd_soc_component_set_pll+0x50/0xa8
>> [ 11.961328] snd_soc_dai_set_pll+0x74/0xc8
>> [ 11.965542] sc7180_snd_startup+0x9c/0x120 [snd_soc_sc7180]
>> [ 11.971262] snd_soc_link_startup+0x34/0x88
>> [ 11.975557] soc_pcm_open+0x100/0x538
>> [ 11.979323] snd_pcm_open_substream+0x530/0x704
>> [ 11.983980] snd_pcm_open+0xc8/0x210
>> [ 11.987653] snd_pcm_playback_open+0x50/0x80
>> [ 11.992049] snd_open+0x120/0x150
>> [ 11.995462] chrdev_open+0xb8/0x1a4
>> [ 11.999056] do_dentry_open+0x238/0x358
>> [ 12.003001] vfs_open+0x34/0x40
>> [ 12.006235] path_openat+0x9e8/0xd60
>> [ 12.009913] do_filp_open+0x90/0x10c
>> [ 12.013587] do_sys_open+0x148/0x314
>> [ 12.017260] __arm64_compat_sys_openat+0x28/0x34
>> [ 12.022009] el0_svc_common+0xa4/0x16c
>> [ 12.025860] el0_svc_compat_handler+0x2c/0x40
>> [ 12.030337] el0_svc_compat+0x8/0x10
>> [ 12.034018] ---[ end trace 745ead557fcbb5dc ]---
>
> Ah I see. Maybe it isn't correct to mark the device as suspended in
> runtime PM operations because the bus will be resumed during the
> transfer? So only mark it suspended during system wide suspend/resume
> transitions?
>
> -Stephen
>

Yes, we cannot mark device as suspended/resumed during
runtime PM operations. Bus will be resumed during i2c
transfers and before transfer initiation, in __i2c_transfer()
from i2c-core-base.c there is a check to see whether the device
is marked as suspended with "__i2c_check_suspended(adap)" call,
which is "true" in this case and returning from there.

To mark it only suspended during system wide suspend/resume
transitions, currently our geni i2c driver has only
system_suspend implemented (geni_i2c_suspend_noirq()) and
does not have system_resume implemented, which again causes i2c
transfers to fail during system_resume after system_suspend.

Shall I go ahead with marking device suspended during
shutdown() only?

-Roja

>> [ 12.040151] rt5682 9-001a: ASoC: error at
>> soc_component_write_no_lock
>> on rt5682.9-001a: -108
>> [ 12.049055] rt5682 9-001a: ASoC: error at
>> soc_component_write_no_lock
>> on rt5682.9-001a: -108
>> [ 12.057742] rt5682 9-001a: ASoC: error at
>> snd_soc_component_update_bits on rt5682.9-001a: -108
>>
>> Trace2:
>> [ 3.515390] i2c i2c-2: Transfer while suspended
>> [ 3.606749] Call trace:
>> [ 3.606751] __i2c_transfer+0xb8/0x38c
>> [ 3.606752] i2c_transfer+0xa0/0xf4
>> [ 3.606754] i2c_transfer_buffer_flags+0x68/0x9c
>> [ 3.639599] hub 2-1.4:1.0: USB hub found
>> [ 3.644375] regmap_i2c_write+0x34/0x64
>> [ 3.644376] _regmap_raw_write_impl+0x4e8/0x7bc
>> [ 3.644378] _regmap_bus_raw_write+0x70/0x8c
>> [ 3.644379] _regmap_write+0x100/0x150
>> [ 3.644381] regmap_write+0x54/0x78
>> [ 3.644383] ti_sn_aux_transfer+0x90/0x244
>> [ 3.650695] hub 2-1.4:1.0: 4 ports detected
>> [ 3.655288] drm_dp_dpcd_access+0x8c/0x11c
>> [ 3.655289] drm_dp_dpcd_read+0x64/0x10c
>> [ 3.655290] ti_sn_bridge_enable+0x5c/0x824
>> [ 3.655292] drm_atomic_bridge_chain_enable+0x78/0xa0
>> [ 3.655294] drm_atomic_helper_commit_modeset_enables+0x198/0x238
>> [ 3.655295] msm_atomic_commit_tail+0x324/0x714
>> [ 3.655297] commit_tail+0xa4/0x108
>> [ 3.664985] usb 1-1.4: new high-speed USB device number 4 using
>> xhci-hcd
>> [ 3.666204] drm_atomic_helper_commit+0xf4/0xfc
>> [ 3.666205] drm_atomic_commit+0x50/0x5c
>> [ 3.666206] drm_atomic_helper_set_config+0x64/0x98
>> [ 3.666208] drm_mode_setcrtc+0x26c/0x590
>> [ 3.666209] drm_ioctl_kernel+0x9c/0x114
>> [ 3.701074] hub 2-1.4:1.0: USB hub found
>> [ 3.703347] drm_ioctl+0x288/0x420
>> [ 3.703349] drm_compat_ioctl+0xd0/0xe0
>> [ 3.703351] __arm64_compat_sys_ioctl+0x100/0x2108
>> [ 3.703354] el0_svc_common+0xa4/0x16c
>> [ 3.708499] hub 2-1.4:1.0: 4 ports detected
>> [ 3.711588] el0_svc_compat_handler+0x2c/0x40
>> [ 3.711590] el0_svc_compat+0x8/0x10
>> [ 3.711591] ---[ end trace 745ead557fcbb5db ]---
>> [ 3.772120] usb 1-1.4: New USB device found, idVendor=0bda,
>> idProduct=5411, bcdDevice= 1.04
>> [ 3.794990] ti_sn65dsi86 2-002d: [drm:ti_sn_bridge_enable] *ERROR*
>> Can't read lane count (-108); assuming 4
>>
>> [1]
>> https://lore.kernel.org/r/20210508075151.1626903-2-swboyd@chromium.org
>>

\
 
 \ /
  Last update: 2021-05-21 18:12    [W:0.043 / U:0.284 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site