All of lore.kernel.org
 help / color / mirror / Atom feed
From: dennis@ausil.us (Dennis Gilmore)
To: linux-arm-kernel@lists.infradead.org
Subject: bug in autoloading of imx-ipuv3-crtc
Date: Thu, 21 Apr 2016 14:39:19 -0500	[thread overview]
Message-ID: <4016678.ygHnHqLjmM@anubis.ausil.us> (raw)
In-Reply-To: <1668410.ik7sLCmBP7@anubis.ausil.us>

On Thursday, April 21, 2016 12:37:28 PM CDT Dennis Gilmore wrote:
> On Thursday, April 21, 2016 1:03:17 AM CDT Uwe Kleine-K?nig wrote:
> > Hello,
> > 
> > On Tue, Apr 19, 2016 at 03:16:01PM -0500, Dennis Gilmore wrote:
> > > On Tuesday, April 19, 2016 2:27:17 PM CDT Dennis Gilmore wrote:
> > > > On Tuesday, April 19, 2016 7:50:49 PM CDT Russell King - ARM Linux
> 
> wrote:
> > > > > On Tue, Apr 19, 2016 at 01:34:23PM -0500, Dennis Gilmore wrote:
> > > > > > on all of my i.MX6 systems imx-ipuv3-crtc ius not getting
> > > > > > automatically
> > > > > > loaded.  Everything is built as a module
> > > > > > 
> > > > > > CONFIG_DRM_IMX=m
> > > > > > CONFIG_DRM_IMX_FB_HELPER=m
> > > > > > CONFIG_DRM_IMX_HDMI=m
> > > > > > CONFIG_DRM_IMX_IPUV3=m
> > > > > > CONFIG_DRM_IMX_LDB=m
> > > > > > CONFIG_DRM_IMX_PARALLEL_DISPLAY=m
> > > > > > CONFIG_DRM_IMX_TVE=m
> > > > > > CONFIG_IMX_IPUV3_CORE=m
> > > > > > 
> > > > > > The result is that until I log in via serial or ssh and modprobe
> > > > > > the
> > > > > > module there is no display.  I suspect that there is some
> > > > > > devicetree
> > > > > > glue missing 4.4 and 4.5 seem to both be effected.
> > > > > 
> > > > > DT doesn't come into it for imx-ipuv3-crtc - these platform devices
> > > > > are
> > > > > created by drivers/gpu/ipu-v3/ipu-common.c itself.
> > > > > 
> > > > > drivers/gpu/drm/imx/ipuv3-crtc.c contains the proper module alias
> > > > > which
> > > > > should result in the module loaded at boot time when the
> > > > > imx-ipuv3-crtc
> > > > > devices are created.
> > > > > 
> > > > > Could the problem be that imx-ipu-v3 isn't being loaded?  However,
> > > > > again,
> > > > > it looks to me like everything is correct there.
> > > > > 
> > > > > Are you saying that this used to work in older kernel versions like
> > > > > 4.3,
> > > > > but stopped in 4.4?
> > > > 
> > > > yers it used to work and stopped working. I would need to go back and
> > > > test
> > > > old kernels to figure out where it broke.
> > > 
> > > after installing some old kernels it broke with 4.4-rc4 which included a
> > > patch with teh subject of "drm/imx: Remove of_node assignment from
> > > ipuv3-crtc driver probe"
> > 
> > Just to be sure: 4.4-rc4 with 407c9eba7897 ("drm/imx: Remove of_node
> > assignment from ipuv3-crtc driver probe") reverted works fine for you?
> 
> I can confirm that reverting just that patch makes things work again

Sorry I missed that due to an old kernel package bug the dtb was not in place 
and a different than expected kernel booted.  reverting the one patch is not 
sufficient

Dennis

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.infradead.org/pipermail/linux-arm-kernel/attachments/20160421/0af68342/attachment-0001.sig>

WARNING: multiple messages have this Message-ID (diff)
From: Dennis Gilmore <dennis@ausil.us>
To: "Uwe Kleine-König" <u.kleine-koenig@pengutronix.de>
Cc: kernel@pengutronix.de,
	Russell King - ARM Linux <linux@arm.linux.org.uk>,
	linux-arm-kernel@lists.infradead.org,
	dri-devel@lists.freedesktop.org
Subject: Re: bug in autoloading of imx-ipuv3-crtc
Date: Thu, 21 Apr 2016 14:39:19 -0500	[thread overview]
Message-ID: <4016678.ygHnHqLjmM@anubis.ausil.us> (raw)
In-Reply-To: <1668410.ik7sLCmBP7@anubis.ausil.us>


[-- Attachment #1.1: Type: text/plain, Size: 2658 bytes --]

On Thursday, April 21, 2016 12:37:28 PM CDT Dennis Gilmore wrote:
> On Thursday, April 21, 2016 1:03:17 AM CDT Uwe Kleine-König wrote:
> > Hello,
> > 
> > On Tue, Apr 19, 2016 at 03:16:01PM -0500, Dennis Gilmore wrote:
> > > On Tuesday, April 19, 2016 2:27:17 PM CDT Dennis Gilmore wrote:
> > > > On Tuesday, April 19, 2016 7:50:49 PM CDT Russell King - ARM Linux
> 
> wrote:
> > > > > On Tue, Apr 19, 2016 at 01:34:23PM -0500, Dennis Gilmore wrote:
> > > > > > on all of my i.MX6 systems imx-ipuv3-crtc ius not getting
> > > > > > automatically
> > > > > > loaded.  Everything is built as a module
> > > > > > 
> > > > > > CONFIG_DRM_IMX=m
> > > > > > CONFIG_DRM_IMX_FB_HELPER=m
> > > > > > CONFIG_DRM_IMX_HDMI=m
> > > > > > CONFIG_DRM_IMX_IPUV3=m
> > > > > > CONFIG_DRM_IMX_LDB=m
> > > > > > CONFIG_DRM_IMX_PARALLEL_DISPLAY=m
> > > > > > CONFIG_DRM_IMX_TVE=m
> > > > > > CONFIG_IMX_IPUV3_CORE=m
> > > > > > 
> > > > > > The result is that until I log in via serial or ssh and modprobe
> > > > > > the
> > > > > > module there is no display.  I suspect that there is some
> > > > > > devicetree
> > > > > > glue missing 4.4 and 4.5 seem to both be effected.
> > > > > 
> > > > > DT doesn't come into it for imx-ipuv3-crtc - these platform devices
> > > > > are
> > > > > created by drivers/gpu/ipu-v3/ipu-common.c itself.
> > > > > 
> > > > > drivers/gpu/drm/imx/ipuv3-crtc.c contains the proper module alias
> > > > > which
> > > > > should result in the module loaded at boot time when the
> > > > > imx-ipuv3-crtc
> > > > > devices are created.
> > > > > 
> > > > > Could the problem be that imx-ipu-v3 isn't being loaded?  However,
> > > > > again,
> > > > > it looks to me like everything is correct there.
> > > > > 
> > > > > Are you saying that this used to work in older kernel versions like
> > > > > 4.3,
> > > > > but stopped in 4.4?
> > > > 
> > > > yers it used to work and stopped working. I would need to go back and
> > > > test
> > > > old kernels to figure out where it broke.
> > > 
> > > after installing some old kernels it broke with 4.4-rc4 which included a
> > > patch with teh subject of "drm/imx: Remove of_node assignment from
> > > ipuv3-crtc driver probe"
> > 
> > Just to be sure: 4.4-rc4 with 407c9eba7897 ("drm/imx: Remove of_node
> > assignment from ipuv3-crtc driver probe") reverted works fine for you?
> 
> I can confirm that reverting just that patch makes things work again

Sorry I missed that due to an old kernel package bug the dtb was not in place 
and a different than expected kernel booted.  reverting the one patch is not 
sufficient

Dennis


[-- Attachment #1.2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

[-- Attachment #2: Type: text/plain, Size: 176 bytes --]

_______________________________________________
linux-arm-kernel mailing list
linux-arm-kernel@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/linux-arm-kernel

  reply	other threads:[~2016-04-21 19:39 UTC|newest]

Thread overview: 22+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-19 18:34 bug in autoloading of imx-ipuv3-crtc Dennis Gilmore
2016-04-19 18:34 ` Dennis Gilmore
2016-04-19 18:50 ` Russell King - ARM Linux
2016-04-19 18:50   ` Russell King - ARM Linux
2016-04-19 19:27   ` Dennis Gilmore
2016-04-19 19:27     ` Dennis Gilmore
2016-04-19 20:16     ` Dennis Gilmore
2016-04-19 20:16       ` Dennis Gilmore
2016-04-20 23:03       ` Uwe Kleine-König
2016-04-20 23:03         ` Uwe Kleine-König
2016-04-21 17:37         ` Dennis Gilmore
2016-04-21 17:37           ` Dennis Gilmore
2016-04-21 19:39           ` Dennis Gilmore [this message]
2016-04-21 19:39             ` Dennis Gilmore
2016-04-22 13:11         ` Dennis Gilmore
2016-04-22 13:11           ` Dennis Gilmore
2016-04-26 22:59           ` Dave Airlie
2016-04-26 22:59             ` Dave Airlie
2016-04-27 15:21             ` Philipp Zabel
2016-04-27 15:21               ` Philipp Zabel
2016-04-27  8:30           ` Philipp Zabel
2016-04-27  8:30             ` Philipp Zabel

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4016678.ygHnHqLjmM@anubis.ausil.us \
    --to=dennis@ausil.us \
    --cc=linux-arm-kernel@lists.infradead.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.