Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18;
Date:   Wed, 24 Jun 2020 15:11:39 +0100
From:   Mark Brown <broonie@kernel.org>
To:     Andrzej Hajda <a.hajda@samsung.com>
Cc:     Jernej Skrabec <jernej.skrabec@siol.net>,
        "Rafael J. Wysocki" <rafael@kernel.org>,
        Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
        Bartlomiej Zolnierkiewicz <b.zolnierkie@samsung.com>,
        linux-kernel@vger.kernel.org,
        "open list:DRM DRIVERS" <dri-devel@lists.freedesktop.org>,
        Russell King - ARM Linux <linux@armlinux.org.uk>,
        Neil Armstrong <narmstrong@baylibre.com>,
        Jonas Karlman <jonas@kwiboo.se>, andy.shevchenko@gmail.com,
        Laurent Pinchart <Laurent.pinchart@ideasonboard.com>,
        linux-arm-kernel@lists.infradead.org,
        Marek Szyprowski <m.szyprowski@samsung.com>
Subject: Re: [RESEND PATCH v5 4/5] drm/bridge/sii8620: fix resource
 acquisition error handling
Message-ID: <20200624141139.GF5472@sirena.org.uk>
References: <20200624114127.3016-1-a.hajda@samsung.com>
 <CGME20200624114137eucas1p13599d33a0c4a9abf7708bf8c8e77264b@eucas1p1.samsung.com>
 <20200624114127.3016-5-a.hajda@samsung.com>
 <20200624132532.GC5472@sirena.org.uk>
 <cf95aac3-fef5-29d0-d94e-ce6cce4ccdb4@samsung.com>
MIME-Version: 1.0
Content-Type: multipart/signed; micalg=pgp-sha512;
        protocol="application/pgp-signature"; boundary="27ZtN5FSuKKSZcBU"
Content-Disposition: inline
In-Reply-To: <cf95aac3-fef5-29d0-d94e-ce6cce4ccdb4@samsung.com>
User-Agent: Mutt/1.10.1 (2018-07-13)
Sender: linux-kernel-owner@vger.kernel.org
Precedence: bulk


--27ZtN5FSuKKSZcBU
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
Content-Transfer-Encoding: quoted-printable

On Wed, Jun 24, 2020 at 03:43:10PM +0200, Andrzej Hajda wrote:
>=20
> On 24.06.2020 15:25, Mark Brown wrote:

> > If we silently ignore all deferred probe errors we make it hard for
> > anyone who is experiencing issues with deferred probe to figure out what
> > they're missing.  We should at least be logging problems at debug level
> > so there's something for people to go on without having to hack the
> > kernel source.

> But you can always do:

> cat /sys/kernel/debug/devices_deferred

> And you will find there deferred probe reason (thanks to patch 2/5).

Right, my point is more that we shouldn't be promoting discarding the
diagnostics entirely but rather saying that we want to redirect those
somewhere else.

> Eventually if you want it in dmesg anyway, one can adjust probe_err funct=
ion
> to log probe error on debug level as well.

That would most likely be very useful as a boot option for problems that
occur before we get a console up.

--27ZtN5FSuKKSZcBU
Content-Type: application/pgp-signature; name="signature.asc"

-----BEGIN PGP SIGNATURE-----

iQEzBAABCgAdFiEEreZoqmdXGLWf4p/qJNaLcl1Uh9AFAl7zXxsACgkQJNaLcl1U
h9C/2wf/YBbKSbShg3LyXqrQ3TXwZGSL43JPPExpduhQPiAEn9LJIa1bgE640xzI
0v5GUg3chRw9YilP9SA5RZ9ODZNo1fw9TwXaIhTVJgbeqc8PVadTm96IOEJj9Tgg
VSxnkPB4Az6RoG1462+Aov9q7/CaAfRmedwIECmapT16oJx+yh1cwzEkxez8koQM
QyQHrYH4IrDb8W160DbdiDuW2hYQia+4BybDU5MJ6B85QmAq++aqb77ocgfL2mIs
GYbKKyBTaPKw8VukABV8X5ZEBOM6xh5yGRJ8KIQmBQQi8cCM6HdEq0eOvwe30FaX
nVihsz6PcEAZZwx/wBTXKxgmzZv3hQ==
=iEvE
-----END PGP SIGNATURE-----

--27ZtN5FSuKKSZcBU--