linux-kernel - Re: [PATCH] imx-drm: Fix probe failure

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <CAOMZO5DMBh++42AQJPAA+tcuC_c=Mzdw5N_oJGyVDvt1JRF4_A@mail.gmail.com>
Date:	Wed, 28 Aug 2013 00:24:35 -0300
From:	Fabio Estevam <festevam@...il.com>
To:	Greg KH <gregkh@...uxfoundation.org>
Cc:	Sascha Hauer <s.hauer@...gutronix.de>,
	Philipp Zabel <p.zabel@...gutronix.de>,
	Shawn Guo <shawn.guo@...aro.org>,
	linux-kernel <linux-kernel@...r.kernel.org>,
	Fabio Estevam <fabio.estevam@...escale.com>
Subject: Re: [PATCH] imx-drm: Fix probe failure

Hi Greg,

On Wed, Aug 28, 2013 at 12:11 AM, Greg KH <gregkh@...uxfoundation.org> wrote:
> On Tue, Aug 27, 2013 at 11:32:43PM -0300, Fabio Estevam wrote:
>> From: Fabio Estevam <fabio.estevam@...escale.com>
>>
>> Since commit b5dc0d10 (drm/imx: kill firstopen callback) the following probe
>> failure is seen:
>
> I don't have that commit in any tree I control, so I can't apply this
> patch.  What tree is this commit in?

Ok, understood.

Just realized that this commit reached linux-next via Dave Airlie's tree:

commit b5dc0d108cd3c0b50ddcb6f6c54be1
bea4c39e01
Author: Daniel Vetter <daniel.vetter@...ll.ch>
Date:   Thu Aug 8 15:41:13 2013 +0200

    drm/imx: kill firstopen callback

    This thing seems to do some kind of delayed setup. Really, real kms
    drivers shouldn't do that at all. Either stuff needs to be dynamically
    hotplugged or the driver setup sequence needs to be fixed.

    This patch here just moves the setup at the very end of the driver
    load callback, with the locking adjusted accordingly.

    v2: Also move the corresponding put from ->lastclose to ->unload.

    Cc: Sascha Hauer <s.hauer@...gutronix.de>
    Cc: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
    Signed-off-by: Daniel Vetter <daniel.vetter@...ll.ch>
    Acked-by: Greg Kroah-Hartman <gregkh@...uxfoundation.org>
    Signed-off-by: Dave Airlie <airlied@...hat.com>

Will resend it to Dave then.

Thanks,

Fabio Estevam
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/