In message: Re: [linux-yocto][linux-yocto v5.15/standard/ti-sdk-5.10/ti-j72xx &
v5.15/standard/preempt-rt/ti-sdk-5.10/ti-j72xx][PATCH 1/2] drm/bridge:
ti-sn65dsi86: correct wrong macro and undeclared variable
on 12/03/2023 Xulin Sun wrote:
> On 3/1/23 23:29, Bruce Ashfield wrote:
> > Both patches are now merged.
>
> Hi Bruce,
>
> It seems the both patches have only been merged into the standard branch
> "v5.15/standard/ti-sdk-5.10/ti-j72xx", they are not in the rt branch
> "v5.15/standard/preempt-rt/ti-sdk-5.10/ti-j72xx".
>
> Could you please also help merge these two patches to the rt branch
> "v5.15/standard/preempt-rt/ti-sdk-5.10/ti-j72xx"?
I just did the cherry picks, they should be there now.
Bruce
>
> [PATCH 1/2] drm/bridge: ti-sn65dsi86: correct wrong macro and
> undeclared variable
>
> [PATCH 2/2] drm/bridge: ti-sn65dsi86: replace gpiod_set_value() with
> gpiod_set_value_cansleep()
>
>
> Thanks
>
> Xulin
>
> >
> > Bruce
> >
> > In message: [linux-yocto][linux-yocto v5.15/standard/ti-sdk-5.10/ti-j72xx &
> > v5.15/standard/preempt-rt/ti-sdk-5.10/ti-j72xx][PATCH 1/2] drm/bridge:
> > ti-sn65dsi86: correct wrong macro and undeclared variable
> > on 27/02/2023 Xulin Sun wrote:
> >
> > > To fix below building error:
> > > drivers/gpu/drm/bridge/ti-sn65dsi86.c:1146:60: error: 'client' undeclared
> > > (first use in this function)
> > > | 1146 | .node =
> > > client->dev.of_node,
> > > | |
> > > ^~~~~~
> > >
> > > drivers/gpu/drm/bridge/ti-sn65dsi86.c:1190:28: error:
> > > 'MIPI_DSI_MODE_EOT_PACKET' undeclared (first use in this function);
> > > did you mean 'MIPI_DSI_MODE_NO_EOT_PACKET'?
> > > | 1190 | dsi->mode_flags |= MIPI_DSI_MODE_EOT_PACKET |
> > > MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
> > > | | ^~~~~~~~~~~~~~~~~~~~~~~~
> > > | | MIPI_DSI_MODE_NO_EOT_PACKET
> > >
> > > Signed-off-by: Xulin Sun <xulin....@windriver.com>
> > > ---
> > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > index de261a052abd..c5b28739cdb1 100644
> > > --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > @@ -1143,7 +1143,7 @@ static int ti_sn_bridge_probe(struct
> > > auxiliary_device *adev,
> > > struct mipi_dsi_device *dsi;
> > > const struct mipi_dsi_device_info info = { .type =
> > > "ti_sn_bridge",
> > > .channel = 0,
> > > - .node = client->dev.of_node,
> > > + .node = np,
> > > };
> > > ret = drm_of_find_panel_or_bridge(np, 1, 0, &panel, NULL);
> > > @@ -1187,7 +1187,7 @@ static int ti_sn_bridge_probe(struct
> > > auxiliary_device *adev,
> > > dsi->lanes = 2;
> > > dsi->format = MIPI_DSI_FMT_RGB888;
> > > dsi->mode_flags = MIPI_DSI_MODE_VIDEO;
> > > - dsi->mode_flags |= MIPI_DSI_MODE_EOT_PACKET |
> > > MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
> > > + dsi->mode_flags |= MIPI_DSI_MODE_NO_EOT_PACKET |
> > > MIPI_DSI_MODE_VIDEO_SYNC_PULSE;
> > > /* check if continuous dsi clock is required or not */
> > > pm_runtime_get_sync(pdata->dev);
> > > --
> > > 2.36.1
> > >
> > In message: [linux-yocto][linux-yocto v5.15/standard/ti-sdk-5.10/ti-j72xx &
> > v5.15/standard/preempt-rt/ti-sdk-5.10/ti-j72xx][PATCH 2/2] drm/bridge:
> > ti-sn65dsi86: replace gpiod_set_value() with gpiod_set_value_cansleep()
> > on 27/02/2023 Xulin Sun wrote:
> >
> > > To avoid below call trace:
> > > [ 630.025726] WARNING: CPU: 6 PID: 342 at drivers/gpio/gpiolib.c:3051
> > > gpiod_set_value+0x64/0xd4
> > > [ 630.025799] CPU: 6 PID: 342 Comm: Xorg Tainted: G W
> > > 5.15.94-yocto-standard #1
> > > [ 630.025804] Hardware name: Texas Instruments J784S4 EVM (DT)
> > > [ 630.025806] pstate: 80000005 (Nzcv daif -PAN -UAO -TCO -DIT -SSBS
> > > BTYPE=--)
> > > [ 630.025811] pc : gpiod_set_value+0x64/0xd4
> > > [ 630.025814] lr : ti_sn65dsi86_suspend+0x6c/0xc0
> > > [ 630.025822] sp : fffffe001276f770
> > > [ 630.025824] x29: fffffe001276f770 x28: fffffe0008f84ad8 x27:
> > > fffffc080d62e000
> > > [ 630.025830] x26: fffffc0801835200 x25: 00000000000f4240 x24:
> > > fffffe0008098d80
> > > [ 630.025836] x23: 0000000000000000 x22: fffffc080402e904 x21:
> > > 0000000000000000
> > > [ 630.025841] x20: 0000000000000000 x19: fffffc0801a46050 x18:
> > > 0000000000000000
> > > [ 630.025847] x17: 0000000000000001 x16: 0000000000000001 x15:
> > > 000a7362abf69530
> > > [ 630.025852] x14: 0008a9a0b5cf2ce0 x13: fffffe0008ea1e28 x12:
> > > 00000000fa83b2da
> > > [ 630.025857] x11: 00000000000000aa x10: 0000000000000950 x9 :
> > > fffffe000889c4cc
> > > [ 630.025863] x8 : fffffe001276f6e0 x7 : 0000000000000000 x6 :
> > > 0000000000000000
> > > [ 630.025868] x5 : 0000000000000000 x4 : 0000000000000000 x3 :
> > > fffffc0804185280
> > > [ 630.025873] x2 : 0000000000000000 x1 : fffffc0804017220 x0 :
> > > 0000000000000001
> > > [ 630.025879] Call trace:
> > > [ 630.025881] gpiod_set_value+0x64/0xd4
> > > [ 630.025884] ti_sn65dsi86_suspend+0x6c/0xc0
> > > [ 630.025888] __rpm_callback+0x50/0x150
> > > [ 630.025894] rpm_callback+0x74/0x80
> > > [ 630.025898] rpm_suspend+0x11c/0x700
> > > [ 630.025903] rpm_idle+0x13c/0x440
> > > [ 630.025907] __pm_runtime_idle+0x68/0x190
> > > [ 630.025911] ti_sn_bridge_post_disable+0x74/0xdc
> > > [ 630.025915] drm_atomic_bridge_chain_post_disable+0x90/0xd0
> > > [ 630.025921] disable_outputs+0x114/0x2c4
> > > [ 630.025926] drm_atomic_helper_commit_modeset_disables+0x24/0x50
> > > [ 630.025929] tidss_atomic_commit_tail+0x34/0x90
> > > [ 630.025933] commit_tail+0xac/0x180
> > > [ 630.025937] drm_atomic_helper_commit+0x154/0x180
> > > [ 630.025940] drm_atomic_commit+0x54/0x6c
> > > [ 630.025944] drm_client_modeset_commit_atomic+0x214/0x254
> > > [ 630.025949] drm_client_modeset_dpms+0x90/0x220
> > > [ 630.025952] drm_fb_helper_blank+0x64/0x110
> > > [ 630.025955] fb_blank+0x5c/0xb0
> > > [ 630.025960] do_fb_ioctl+0x410/0x640
> > > [ 630.025964] fb_ioctl+0x40/0x5c
> > > [ 630.025968] __arm64_sys_ioctl+0xb0/0xf4
> > > [ 630.025974] invoke_syscall+0x5c/0x130
> > > [ 630.025980] el0_svc_common.constprop.0+0x4c/0x100
> > > [ 630.025983] do_el0_svc+0x4c/0xb4
> > > [ 630.025987] el0_svc+0x28/0x80
> > > [ 630.025992] el0t_64_sync_handler+0xa4/0x130
> > > [ 630.025995] el0t_64_sync+0x1a0/0x1a4
> > >
> > > Signed-off-by: Xulin Sun <xulin....@windriver.com>
> > > ---
> > > drivers/gpu/drm/bridge/ti-sn65dsi86.c | 4 ++--
> > > 1 file changed, 2 insertions(+), 2 deletions(-)
> > >
> > > diff --git a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > index c5b28739cdb1..a4c618a0047c 100644
> > > --- a/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > +++ b/drivers/gpu/drm/bridge/ti-sn65dsi86.c
> > > @@ -311,7 +311,7 @@ static int __maybe_unused ti_sn65dsi86_resume(struct
> > > device *dev)
> > > /* td2: min 100 us after regulators before enabling the GPIO */
> > > usleep_range(100, 110);
> > > - gpiod_set_value(pdata->enable_gpio, 1);
> > > + gpiod_set_value_cansleep(pdata->enable_gpio, 1);
> > > /*
> > > * If we have a reference clock we can enable communication w/
> > > the
> > > @@ -334,7 +334,7 @@ static int __maybe_unused ti_sn65dsi86_suspend(struct
> > > device *dev)
> > > if (pdata->refclk)
> > > ti_sn65dsi86_disable_comms(pdata);
> > > - gpiod_set_value(pdata->enable_gpio, 0);
> > > + gpiod_set_value_cansleep(pdata->enable_gpio, 0);
> > > ret = regulator_bulk_disable(SN_REGULATOR_SUPPLY_NUM,
> > > pdata->supplies);
> > > if (ret)
> > > --
> > > 2.36.1
> > >
-=-=-=-=-=-=-=-=-=-=-=-
Links: You receive all messages sent to this group.
View/Reply Online (#12252):
https://lists.yoctoproject.org/g/linux-yocto/message/12252
Mute This Topic: https://lists.yoctoproject.org/mt/97317352/21656
Group Owner: linux-yocto+ow...@lists.yoctoproject.org
Unsubscribe:
https://lists.yoctoproject.org/g/linux-yocto/leave/6687884/21656/624485779/xyzzy
[arch...@mail-archive.com]
-=-=-=-=-=-=-=-=-=-=-=-
