[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <20160920104855.GJ13620@mwanda>
Date: Tue, 20 Sep 2016 13:48:55 +0300
From: Dan Carpenter <dan.carpenter@...cle.com>
To: Jani Nikula <jani.nikula@...ux.intel.com>
Cc: SF Markus Elfring <elfring@...rs.sourceforge.net>,
dri-devel@...ts.freedesktop.org, Daniel Vetter <daniel@...ll.ch>,
David Airlie <airlied@...ux.ie>,
Patrik Jakobsson <patrik.r.jakobsson@...il.com>,
Julia Lawall <julia.lawall@...6.fr>,
kernel-janitors@...r.kernel.org,
LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH 5/6] GPU-DRM-GMA500: One error message less for a GCT
revision mismatch in mid_get_vbt_data()
On Tue, Sep 20, 2016 at 01:07:35PM +0300, Jani Nikula wrote:
> On Tue, 20 Sep 2016, SF Markus Elfring <elfring@...rs.sourceforge.net> wrote:
> > From: Markus Elfring <elfring@...rs.sourceforge.net>
> > Date: Tue, 20 Sep 2016 10:36:19 +0200
> >
> > A single error message should be sufficient to inform about
> > the detection of an unknown GCT revision at the end.
> > Thus return after the logging call in this case directly.
>
> Did you test this?
>
Don't be a dummy... This is easy to review an it fixes a bug.
I'm fine with you NAKing all these patches based on who they are from.
I mostly just delete these without responding because the guy has
history of introducing bugs and never listens to feedback. But asking
pointless rhetorical questions is not helpful.
A lot of people are CC'd and you're wasting everyone's time by asking
questions where you know the answer.
regards,
dan carpenter
Powered by blists - more mailing lists