| Message ID | 20191003165539.50318-1-tony@atomide.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | ARM: omap2plus_defconfig: Fix selected panels after generic panel changes | expand |
On 03/10/2019 19:55, Tony Lindgren wrote: > The old omapdrm panels got removed for v5.4 in favor of generic panels, > and the Kconfig options changed. Let's update omap2plus_defconfig > accordingly so the same panels are still enabled. > > Cc: Jyri Sarha <jsarha@ti.com> > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > Signed-off-by: Tony Lindgren <tony@atomide.com> > --- > arch/arm/configs/omap2plus_defconfig | 12 ++++++------ > 1 file changed, 6 insertions(+), 6 deletions(-) > > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig > --- a/arch/arm/configs/omap2plus_defconfig > +++ b/arch/arm/configs/omap2plus_defconfig > @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m > CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m > CONFIG_DRM_OMAP_PANEL_DPI=m > CONFIG_DRM_OMAP_PANEL_DSI_CM=m > -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m > -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m > -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m > -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m > -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m > -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m > CONFIG_DRM_TILCDC=m > CONFIG_DRM_PANEL_SIMPLE=m > +CONFIG_DRM_PANEL_LG_LB035Q02=m > +CONFIG_DRM_PANEL_NEC_NL8048HL11=m > +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m > +CONFIG_DRM_PANEL_SONY_ACX565AKM=m > +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m > +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m > CONFIG_FB=y > CONFIG_FIRMWARE_EDID=y > CONFIG_FB_MODE_HELPERS=y Sorry, I didn't remember to update these. Some additions: These can be dropped, they no longer exist: CONFIG_DRM_OMAP_ENCODER_TFP410=m CONFIG_DRM_OMAP_CONNECTOR_DVI=m CONFIG_DRM_OMAP_PANEL_DPI=m This can be added to get the DVI output working on many of the boards: CONFIG_DRM_TI_TFP410=m Tomi
* Tomi Valkeinen <tomi.valkeinen@ti.com> [191004 08:42]: > On 03/10/2019 19:55, Tony Lindgren wrote: > > The old omapdrm panels got removed for v5.4 in favor of generic panels, > > and the Kconfig options changed. Let's update omap2plus_defconfig > > accordingly so the same panels are still enabled. > > > > Cc: Jyri Sarha <jsarha@ti.com> > > Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > > Signed-off-by: Tony Lindgren <tony@atomide.com> > > --- > > arch/arm/configs/omap2plus_defconfig | 12 ++++++------ > > 1 file changed, 6 insertions(+), 6 deletions(-) > > > > diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig > > --- a/arch/arm/configs/omap2plus_defconfig > > +++ b/arch/arm/configs/omap2plus_defconfig > > @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m > > CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m > > CONFIG_DRM_OMAP_PANEL_DPI=m > > CONFIG_DRM_OMAP_PANEL_DSI_CM=m > > -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m > > -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m > > -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m > > -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m > > -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m > > -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m > > CONFIG_DRM_TILCDC=m > > CONFIG_DRM_PANEL_SIMPLE=m > > +CONFIG_DRM_PANEL_LG_LB035Q02=m > > +CONFIG_DRM_PANEL_NEC_NL8048HL11=m > > +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m > > +CONFIG_DRM_PANEL_SONY_ACX565AKM=m > > +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m > > +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m > > CONFIG_FB=y > > CONFIG_FIRMWARE_EDID=y > > CONFIG_FB_MODE_HELPERS=y > > Sorry, I didn't remember to update these. Some additions: > > These can be dropped, they no longer exist: > > CONFIG_DRM_OMAP_ENCODER_TFP410=m > CONFIG_DRM_OMAP_CONNECTOR_DVI=m > CONFIG_DRM_OMAP_PANEL_DPI=m > > This can be added to get the DVI output working on many of the boards: > > CONFIG_DRM_TI_TFP410=m We have patches already posted for these by Adam and me so we should be good to go with just $subject patch missing now. Regards, Tony
Hi Tomi, > Am 04.10.2019 um 10:41 schrieb Tomi Valkeinen <tomi.valkeinen@ti.com>: > > On 03/10/2019 19:55, Tony Lindgren wrote: >> The old omapdrm panels got removed for v5.4 in favor of generic panels, >> and the Kconfig options changed. Let's update omap2plus_defconfig >> accordingly so the same panels are still enabled. >> Cc: Jyri Sarha <jsarha@ti.com> >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> >> Signed-off-by: Tony Lindgren <tony@atomide.com> >> --- >> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ >> 1 file changed, 6 insertions(+), 6 deletions(-) >> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig >> --- a/arch/arm/configs/omap2plus_defconfig >> +++ b/arch/arm/configs/omap2plus_defconfig >> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m >> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m >> CONFIG_DRM_OMAP_PANEL_DPI=m >> CONFIG_DRM_OMAP_PANEL_DSI_CM=m >> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m >> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m >> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m >> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m >> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m >> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m >> CONFIG_DRM_TILCDC=m >> CONFIG_DRM_PANEL_SIMPLE=m >> +CONFIG_DRM_PANEL_LG_LB035Q02=m >> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m >> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m >> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m >> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m >> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m >> CONFIG_FB=y >> CONFIG_FIRMWARE_EDID=y >> CONFIG_FB_MODE_HELPERS=y > > Sorry, I didn't remember to update these. Some additions: > > These can be dropped, they no longer exist: > > CONFIG_DRM_OMAP_ENCODER_TFP410=m > CONFIG_DRM_OMAP_CONNECTOR_DVI=m > CONFIG_DRM_OMAP_PANEL_DPI=m After being hit by the __always_inline issue [1] I am now back with being able to test v5.4-rc1. Now this patch fixes the build problem, but we still have one: panel drivers are no longer automatically loaded at boot. We now need to manually modprobe them. This has been observed with GTA04 (TD028TTEC1) and OpenPandora (TD043MTEA1). So moving the drivers seems to have changed/broken something with DT based driver matching. BR and thanks, Nikolaus [1]: https://lkml.org/lkml/2019/10/2/201
Hi Nikolaus, On Sat, Oct 05, 2019 at 12:50:37PM +0200, H. Nikolaus Schaller wrote: > > Am 04.10.2019 um 10:41 schrieb Tomi Valkeinen <tomi.valkeinen@ti.com>: > > > > On 03/10/2019 19:55, Tony Lindgren wrote: > >> The old omapdrm panels got removed for v5.4 in favor of generic panels, > >> and the Kconfig options changed. Let's update omap2plus_defconfig > >> accordingly so the same panels are still enabled. > >> Cc: Jyri Sarha <jsarha@ti.com> > >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > >> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > >> Signed-off-by: Tony Lindgren <tony@atomide.com> > >> --- > >> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ > >> 1 file changed, 6 insertions(+), 6 deletions(-) > >> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig > >> --- a/arch/arm/configs/omap2plus_defconfig > >> +++ b/arch/arm/configs/omap2plus_defconfig > >> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m > >> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m > >> CONFIG_DRM_OMAP_PANEL_DPI=m > >> CONFIG_DRM_OMAP_PANEL_DSI_CM=m > >> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m > >> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m > >> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m > >> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m > >> CONFIG_DRM_TILCDC=m > >> CONFIG_DRM_PANEL_SIMPLE=m > >> +CONFIG_DRM_PANEL_LG_LB035Q02=m > >> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m > >> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m > >> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m > >> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m > >> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m > >> CONFIG_FB=y > >> CONFIG_FIRMWARE_EDID=y > >> CONFIG_FB_MODE_HELPERS=y > > > > Sorry, I didn't remember to update these. Some additions: > > > > These can be dropped, they no longer exist: > > > > CONFIG_DRM_OMAP_ENCODER_TFP410=m > > CONFIG_DRM_OMAP_CONNECTOR_DVI=m > > CONFIG_DRM_OMAP_PANEL_DPI=m > > After being hit by the __always_inline issue [1] I am now > back with being able to test v5.4-rc1. > > Now this patch fixes the build problem, but we still have > one: panel drivers are no longer automatically loaded at > boot. We now need to manually modprobe them. > > This has been observed with GTA04 (TD028TTEC1) and OpenPandora > (TD043MTEA1). So moving the drivers seems to have changed/broken > something with DT based driver matching. That's really weird, the driver name, the SPI device table and the OF device table are the same (except for the "omapdss," prefix that we don't add anymore in omapdss-boot-init.c). Would you be able to investigate what broke ? > [1]: https://lkml.org/lkml/2019/10/2/201
Hi Laurent, > Am 06.10.2019 um 16:53 schrieb Laurent Pinchart <laurent.pinchart@ideasonboard.com>: > > Hi Nikolaus, > > On Sat, Oct 05, 2019 at 12:50:37PM +0200, H. Nikolaus Schaller wrote: >>> Am 04.10.2019 um 10:41 schrieb Tomi Valkeinen <tomi.valkeinen@ti.com>: >>> >>> On 03/10/2019 19:55, Tony Lindgren wrote: >>>> The old omapdrm panels got removed for v5.4 in favor of generic panels, >>>> and the Kconfig options changed. Let's update omap2plus_defconfig >>>> accordingly so the same panels are still enabled. >>>> Cc: Jyri Sarha <jsarha@ti.com> >>>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >>>> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> >>>> Signed-off-by: Tony Lindgren <tony@atomide.com> >>>> --- >>>> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ >>>> 1 file changed, 6 insertions(+), 6 deletions(-) >>>> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig >>>> --- a/arch/arm/configs/omap2plus_defconfig >>>> +++ b/arch/arm/configs/omap2plus_defconfig >>>> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m >>>> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m >>>> CONFIG_DRM_OMAP_PANEL_DPI=m >>>> CONFIG_DRM_OMAP_PANEL_DSI_CM=m >>>> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m >>>> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m >>>> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m >>>> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m >>>> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m >>>> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m >>>> CONFIG_DRM_TILCDC=m >>>> CONFIG_DRM_PANEL_SIMPLE=m >>>> +CONFIG_DRM_PANEL_LG_LB035Q02=m >>>> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m >>>> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m >>>> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m >>>> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m >>>> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m >>>> CONFIG_FB=y >>>> CONFIG_FIRMWARE_EDID=y >>>> CONFIG_FB_MODE_HELPERS=y >>> >>> Sorry, I didn't remember to update these. Some additions: >>> >>> These can be dropped, they no longer exist: >>> >>> CONFIG_DRM_OMAP_ENCODER_TFP410=m >>> CONFIG_DRM_OMAP_CONNECTOR_DVI=m >>> CONFIG_DRM_OMAP_PANEL_DPI=m >> >> After being hit by the __always_inline issue [1] I am now >> back with being able to test v5.4-rc1. >> >> Now this patch fixes the build problem, but we still have >> one: panel drivers are no longer automatically loaded at >> boot. We now need to manually modprobe them. >> >> This has been observed with GTA04 (TD028TTEC1) and OpenPandora >> (TD043MTEA1). So moving the drivers seems to have changed/broken >> something with DT based driver matching. > > That's really weird, the driver name, the SPI device table and the OF > device table are the same Yes, that is what I wonder as well. The panels are child nodes of the SPI nodes and should be probed through the SPI bus. > (except for the "omapdss," prefix that we > don't add anymore in omapdss-boot-init.c). Would you be able to > investigate what broke ? Yes, they have been removed from the omapdss_of_fixups_whitelist but that shouldn't make a difference... And other generic dpi panels work on other omap3 boards. Most likely something with deferred probing? I'll try to find clues in the next days. BR, Nikolaus
On Sun, 6 Oct 2019 17:53:48 +0300 Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > Hi Nikolaus, > > On Sat, Oct 05, 2019 at 12:50:37PM +0200, H. Nikolaus Schaller wrote: > > > Am 04.10.2019 um 10:41 schrieb Tomi Valkeinen <tomi.valkeinen@ti.com>: > > > > > > On 03/10/2019 19:55, Tony Lindgren wrote: > > >> The old omapdrm panels got removed for v5.4 in favor of generic panels, > > >> and the Kconfig options changed. Let's update omap2plus_defconfig > > >> accordingly so the same panels are still enabled. > > >> Cc: Jyri Sarha <jsarha@ti.com> > > >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > >> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > > >> Signed-off-by: Tony Lindgren <tony@atomide.com> > > >> --- > > >> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ > > >> 1 file changed, 6 insertions(+), 6 deletions(-) > > >> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig > > >> --- a/arch/arm/configs/omap2plus_defconfig > > >> +++ b/arch/arm/configs/omap2plus_defconfig > > >> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m > > >> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m > > >> CONFIG_DRM_OMAP_PANEL_DPI=m > > >> CONFIG_DRM_OMAP_PANEL_DSI_CM=m > > >> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m > > >> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m > > >> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m > > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m > > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m > > >> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m > > >> CONFIG_DRM_TILCDC=m > > >> CONFIG_DRM_PANEL_SIMPLE=m > > >> +CONFIG_DRM_PANEL_LG_LB035Q02=m > > >> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m > > >> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m > > >> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m > > >> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m > > >> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m > > >> CONFIG_FB=y > > >> CONFIG_FIRMWARE_EDID=y > > >> CONFIG_FB_MODE_HELPERS=y > > > > > > Sorry, I didn't remember to update these. Some additions: > > > > > > These can be dropped, they no longer exist: > > > > > > CONFIG_DRM_OMAP_ENCODER_TFP410=m > > > CONFIG_DRM_OMAP_CONNECTOR_DVI=m > > > CONFIG_DRM_OMAP_PANEL_DPI=m > > > > After being hit by the __always_inline issue [1] I am now > > back with being able to test v5.4-rc1. > > > > Now this patch fixes the build problem, but we still have > > one: panel drivers are no longer automatically loaded at > > boot. We now need to manually modprobe them. > > > > This has been observed with GTA04 (TD028TTEC1) and OpenPandora > > (TD043MTEA1). So moving the drivers seems to have changed/broken > > something with DT based driver matching. > > That's really weird, the driver name, the SPI device table and the OF > device table are the same (except for the "omapdss," prefix that we > don't add anymore in omapdss-boot-init.c). Would you be able to > investigate what broke ? > in earlier times: root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias spi:tpo,td028ttec1 now in 5.4-rc1: root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias spi:td028ttec1 root@gta04:~# modinfo /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko filename: /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko license: GPL description: Toppoly TD028TTEC1 panel driver author: H. Nikolaus Schaller <hns@goldelico.com> srcversion: 6B3E224BCD3D76253CF361C alias: of:N*T*Ctoppoly,td028ttec1C* alias: of:N*T*Ctoppoly,td028ttec1 alias: of:N*T*Ctpo,td028ttec1C* alias: of:N*T*Ctpo,td028ttec1 alias: spi:toppoly,td028ttec1 alias: spi:tpo,td028ttec1 depends: drm intree: Y name: panel_tpo_td028ttec1 vermagic: 5.4.0-rc1-letux+ SMP preempt mod_unload ARMv7 p2v8 That alias is not in the list. Regards, Andreas
On Sun, 6 Oct 2019 22:39:58 +0200 Andreas Kemnade <andreas@kemnade.info> wrote: > On Sun, 6 Oct 2019 17:53:48 +0300 > Laurent Pinchart <laurent.pinchart@ideasonboard.com> wrote: > > > Hi Nikolaus, > > > > On Sat, Oct 05, 2019 at 12:50:37PM +0200, H. Nikolaus Schaller wrote: > > > > Am 04.10.2019 um 10:41 schrieb Tomi Valkeinen <tomi.valkeinen@ti.com>: > > > > > > > > On 03/10/2019 19:55, Tony Lindgren wrote: > > > >> The old omapdrm panels got removed for v5.4 in favor of generic panels, > > > >> and the Kconfig options changed. Let's update omap2plus_defconfig > > > >> accordingly so the same panels are still enabled. > > > >> Cc: Jyri Sarha <jsarha@ti.com> > > > >> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> > > > >> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> > > > >> Signed-off-by: Tony Lindgren <tony@atomide.com> > > > >> --- > > > >> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ > > > >> 1 file changed, 6 insertions(+), 6 deletions(-) > > > >> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig > > > >> --- a/arch/arm/configs/omap2plus_defconfig > > > >> +++ b/arch/arm/configs/omap2plus_defconfig > > > >> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m > > > >> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m > > > >> CONFIG_DRM_OMAP_PANEL_DPI=m > > > >> CONFIG_DRM_OMAP_PANEL_DSI_CM=m > > > >> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m > > > >> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m > > > >> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m > > > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m > > > >> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m > > > >> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m > > > >> CONFIG_DRM_TILCDC=m > > > >> CONFIG_DRM_PANEL_SIMPLE=m > > > >> +CONFIG_DRM_PANEL_LG_LB035Q02=m > > > >> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m > > > >> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m > > > >> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m > > > >> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m > > > >> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m > > > >> CONFIG_FB=y > > > >> CONFIG_FIRMWARE_EDID=y > > > >> CONFIG_FB_MODE_HELPERS=y > > > > > > > > Sorry, I didn't remember to update these. Some additions: > > > > > > > > These can be dropped, they no longer exist: > > > > > > > > CONFIG_DRM_OMAP_ENCODER_TFP410=m > > > > CONFIG_DRM_OMAP_CONNECTOR_DVI=m > > > > CONFIG_DRM_OMAP_PANEL_DPI=m > > > > > > After being hit by the __always_inline issue [1] I am now > > > back with being able to test v5.4-rc1. > > > > > > Now this patch fixes the build problem, but we still have > > > one: panel drivers are no longer automatically loaded at > > > boot. We now need to manually modprobe them. > > > > > > This has been observed with GTA04 (TD028TTEC1) and OpenPandora > > > (TD043MTEA1). So moving the drivers seems to have changed/broken > > > something with DT based driver matching. > > > > That's really weird, the driver name, the SPI device table and the OF > > device table are the same (except for the "omapdss," prefix that we > > don't add anymore in omapdss-boot-init.c). Would you be able to > > investigate what broke ? > > > in earlier times: > > root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > spi:tpo,td028ttec1 > > now in 5.4-rc1: > root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > spi:td028ttec1 > > root@gta04:~# modinfo /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > filename: /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > license: GPL > description: Toppoly TD028TTEC1 panel driver > author: H. Nikolaus Schaller <hns@goldelico.com> > srcversion: 6B3E224BCD3D76253CF361C > alias: of:N*T*Ctoppoly,td028ttec1C* > alias: of:N*T*Ctoppoly,td028ttec1 > alias: of:N*T*Ctpo,td028ttec1C* > alias: of:N*T*Ctpo,td028ttec1 > alias: spi:toppoly,td028ttec1 > alias: spi:tpo,td028ttec1 > depends: drm > intree: Y > name: panel_tpo_td028ttec1 > vermagic: 5.4.0-rc1-letux+ SMP preempt mod_unload ARMv7 p2v8 > > That alias is not in the list. > some more research: in former times of_modalias_node() has stripped the omapdss prefix, result: modalias=spi:tpo,td028ttec1 now it strips the tpo prefix because there is no omapdss prefix anymore. Regards, Andreas
On 04/10/2019 18:54, Tony Lindgren wrote: > * Tomi Valkeinen <tomi.valkeinen@ti.com> [191004 08:42]: >> On 03/10/2019 19:55, Tony Lindgren wrote: >>> The old omapdrm panels got removed for v5.4 in favor of generic panels, >>> and the Kconfig options changed. Let's update omap2plus_defconfig >>> accordingly so the same panels are still enabled. >>> >>> Cc: Jyri Sarha <jsarha@ti.com> >>> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> >>> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> >>> Signed-off-by: Tony Lindgren <tony@atomide.com> >>> --- >>> arch/arm/configs/omap2plus_defconfig | 12 ++++++------ >>> 1 file changed, 6 insertions(+), 6 deletions(-) >>> >>> diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig >>> --- a/arch/arm/configs/omap2plus_defconfig >>> +++ b/arch/arm/configs/omap2plus_defconfig >>> @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m >>> CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m >>> CONFIG_DRM_OMAP_PANEL_DPI=m >>> CONFIG_DRM_OMAP_PANEL_DSI_CM=m >>> -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m >>> -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m >>> -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m >>> -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m >>> -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m >>> -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m >>> CONFIG_DRM_TILCDC=m >>> CONFIG_DRM_PANEL_SIMPLE=m >>> +CONFIG_DRM_PANEL_LG_LB035Q02=m >>> +CONFIG_DRM_PANEL_NEC_NL8048HL11=m >>> +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m >>> +CONFIG_DRM_PANEL_SONY_ACX565AKM=m >>> +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m >>> +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m >>> CONFIG_FB=y >>> CONFIG_FIRMWARE_EDID=y >>> CONFIG_FB_MODE_HELPERS=y >> >> Sorry, I didn't remember to update these. Some additions: >> >> These can be dropped, they no longer exist: >> >> CONFIG_DRM_OMAP_ENCODER_TFP410=m >> CONFIG_DRM_OMAP_CONNECTOR_DVI=m >> CONFIG_DRM_OMAP_PANEL_DPI=m >> >> This can be added to get the DVI output working on many of the boards: >> >> CONFIG_DRM_TI_TFP410=m > > We have patches already posted for these by Adam and me so we > should be good to go with just $subject patch missing now. Ok. Reviewed-by: Tomi Valkeinen <tomi.valkeinen@ti.com> Tomi
On 07/10/2019 07:40, Andreas Kemnade wrote: >>> That's really weird, the driver name, the SPI device table and the OF >>> device table are the same (except for the "omapdss," prefix that we >>> don't add anymore in omapdss-boot-init.c). Would you be able to >>> investigate what broke ? >>> >> in earlier times: >> >> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias >> spi:tpo,td028ttec1 >> >> now in 5.4-rc1: >> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias >> spi:td028ttec1 >> >> root@gta04:~# modinfo /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko >> filename: /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko >> license: GPL >> description: Toppoly TD028TTEC1 panel driver >> author: H. Nikolaus Schaller <hns@goldelico.com> >> srcversion: 6B3E224BCD3D76253CF361C >> alias: of:N*T*Ctoppoly,td028ttec1C* >> alias: of:N*T*Ctoppoly,td028ttec1 >> alias: of:N*T*Ctpo,td028ttec1C* >> alias: of:N*T*Ctpo,td028ttec1 >> alias: spi:toppoly,td028ttec1 >> alias: spi:tpo,td028ttec1 >> depends: drm >> intree: Y >> name: panel_tpo_td028ttec1 >> vermagic: 5.4.0-rc1-letux+ SMP preempt mod_unload ARMv7 p2v8 >> >> That alias is not in the list. >> > some more research: > in former times > of_modalias_node() has stripped the omapdss prefix, result: modalias=spi:tpo,td028ttec1 > now it strips the tpo prefix because there is no omapdss prefix anymore. I haven't studied this more yet, but yes, I can see of_modalias_node stripping the vendor prefix on purpose. I do wonder how this is supposed to work, it would make more sense to me to keep the vendor prefix. Is the spi_device_id supposed to be without vendor prefix? With a quick grep, this seems to be the case. Tomi
On Mon, 7 Oct 2019 17:29:20 +0300 Tomi Valkeinen <tomi.valkeinen@ti.com> wrote: > On 07/10/2019 07:40, Andreas Kemnade wrote: > > >>> That's really weird, the driver name, the SPI device table and the OF > >>> device table are the same (except for the "omapdss," prefix that we > >>> don't add anymore in omapdss-boot-init.c). Would you be able to > >>> investigate what broke ? > >>> > >> in earlier times: > >> > >> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > >> spi:tpo,td028ttec1 > >> > >> now in 5.4-rc1: > >> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > >> spi:td028ttec1 > >> > >> root@gta04:~# modinfo /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > >> filename: /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > >> license: GPL > >> description: Toppoly TD028TTEC1 panel driver > >> author: H. Nikolaus Schaller <hns@goldelico.com> > >> srcversion: 6B3E224BCD3D76253CF361C > >> alias: of:N*T*Ctoppoly,td028ttec1C* > >> alias: of:N*T*Ctoppoly,td028ttec1 > >> alias: of:N*T*Ctpo,td028ttec1C* > >> alias: of:N*T*Ctpo,td028ttec1 > >> alias: spi:toppoly,td028ttec1 > >> alias: spi:tpo,td028ttec1 > >> depends: drm > >> intree: Y > >> name: panel_tpo_td028ttec1 > >> vermagic: 5.4.0-rc1-letux+ SMP preempt mod_unload ARMv7 p2v8 > >> > >> That alias is not in the list. > >> > > some more research: > > in former times > > of_modalias_node() has stripped the omapdss prefix, result: modalias=spi:tpo,td028ttec1 > > now it strips the tpo prefix because there is no omapdss prefix anymore. > > I haven't studied this more yet, but yes, I can see of_modalias_node > stripping the vendor prefix on purpose. I do wonder how this is supposed > to work, it would make more sense to me to keep the vendor prefix. > > Is the spi_device_id supposed to be without vendor prefix? With a quick > grep, this seems to be the case. > as it is done in drivers/spi/spi.c we have no influence from drm side. So we must live with it. Also checked several drivers, no vendor-prefix there. Regards, Andreas
On Mon, Oct 07, 2019 at 06:09:57PM +0200, Andreas Kemnade wrote: > On Mon, 7 Oct 2019 17:29:20 +0300 Tomi Valkeinen wrote: > > On 07/10/2019 07:40, Andreas Kemnade wrote: > > > >>>> That's really weird, the driver name, the SPI device table and the OF > >>>> device table are the same (except for the "omapdss," prefix that we > >>>> don't add anymore in omapdss-boot-init.c). Would you be able to > >>>> investigate what broke ? > >>>> > >>> in earlier times: > >>> > >>> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > >>> spi:tpo,td028ttec1 > >>> > >>> now in 5.4-rc1: > >>> root@gta04:/sys/bus/spi/devices/spi4.0# cat modalias > >>> spi:td028ttec1 > >>> > >>> root@gta04:~# modinfo /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > >>> filename: /lib/modules/5.4.0-rc1-letux+/kernel/drivers/gpu/drm/panel/panel-tpo-td028ttec1.ko > >>> license: GPL > >>> description: Toppoly TD028TTEC1 panel driver > >>> author: H. Nikolaus Schaller <hns@goldelico.com> > >>> srcversion: 6B3E224BCD3D76253CF361C > >>> alias: of:N*T*Ctoppoly,td028ttec1C* > >>> alias: of:N*T*Ctoppoly,td028ttec1 > >>> alias: of:N*T*Ctpo,td028ttec1C* > >>> alias: of:N*T*Ctpo,td028ttec1 > >>> alias: spi:toppoly,td028ttec1 > >>> alias: spi:tpo,td028ttec1 > >>> depends: drm > >>> intree: Y > >>> name: panel_tpo_td028ttec1 > >>> vermagic: 5.4.0-rc1-letux+ SMP preempt mod_unload ARMv7 p2v8 > >>> > >>> That alias is not in the list. > >>> > >> some more research: > >> in former times > >> of_modalias_node() has stripped the omapdss prefix, result: modalias=spi:tpo,td028ttec1 > >> now it strips the tpo prefix because there is no omapdss prefix anymore. > > > > I haven't studied this more yet, but yes, I can see of_modalias_node > > stripping the vendor prefix on purpose. I do wonder how this is supposed > > to work, it would make more sense to me to keep the vendor prefix. > > > > Is the spi_device_id supposed to be without vendor prefix? With a quick > > grep, this seems to be the case. > > as it is done in drivers/spi/spi.c we have no influence from drm side. > So we must live with it. Also checked several drivers, no vendor-prefix > there. I *think* SPI drivers are not supposed to include any prefix. The fix would then simply be to strip the prefix from the SPI table (leaving it there for the OF table of course). I've submitted a patch series to do so.
diff --git a/arch/arm/configs/omap2plus_defconfig b/arch/arm/configs/omap2plus_defconfig --- a/arch/arm/configs/omap2plus_defconfig +++ b/arch/arm/configs/omap2plus_defconfig @@ -356,14 +356,14 @@ CONFIG_DRM_OMAP_CONNECTOR_HDMI=m CONFIG_DRM_OMAP_CONNECTOR_ANALOG_TV=m CONFIG_DRM_OMAP_PANEL_DPI=m CONFIG_DRM_OMAP_PANEL_DSI_CM=m -CONFIG_DRM_OMAP_PANEL_SONY_ACX565AKM=m -CONFIG_DRM_OMAP_PANEL_LGPHILIPS_LB035Q02=m -CONFIG_DRM_OMAP_PANEL_SHARP_LS037V7DW01=m -CONFIG_DRM_OMAP_PANEL_TPO_TD028TTEC1=m -CONFIG_DRM_OMAP_PANEL_TPO_TD043MTEA1=m -CONFIG_DRM_OMAP_PANEL_NEC_NL8048HL11=m CONFIG_DRM_TILCDC=m CONFIG_DRM_PANEL_SIMPLE=m +CONFIG_DRM_PANEL_LG_LB035Q02=m +CONFIG_DRM_PANEL_NEC_NL8048HL11=m +CONFIG_DRM_PANEL_SHARP_LS037V7DW01=m +CONFIG_DRM_PANEL_SONY_ACX565AKM=m +CONFIG_DRM_PANEL_TPO_TD028TTEC1=m +CONFIG_DRM_PANEL_TPO_TD043MTEA1=m CONFIG_FB=y CONFIG_FIRMWARE_EDID=y CONFIG_FB_MODE_HELPERS=y
The old omapdrm panels got removed for v5.4 in favor of generic panels, and the Kconfig options changed. Let's update omap2plus_defconfig accordingly so the same panels are still enabled. Cc: Jyri Sarha <jsarha@ti.com> Cc: Laurent Pinchart <laurent.pinchart@ideasonboard.com> Cc: Tomi Valkeinen <tomi.valkeinen@ti.com> Signed-off-by: Tony Lindgren <tony@atomide.com> --- arch/arm/configs/omap2plus_defconfig | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-)