From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754716AbdGNRce (ORCPT ); Fri, 14 Jul 2017 13:32:34 -0400 Received: from mail-yb0-f177.google.com ([209.85.213.177]:36416 "EHLO mail-yb0-f177.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753382AbdGNRcc (ORCPT ); Fri, 14 Jul 2017 13:32:32 -0400 MIME-Version: 1.0 In-Reply-To: <87eftjl6xg.fsf@nikula.org> References: <20170420215605.176722-1-mka@chromium.org> <20170505172636.GA128305@google.com> <20170505174043.GK12629@intel.com> <20170713101351.GS12629@intel.com> <20170713174232.GK12629@intel.com> <87eftjl6xg.fsf@nikula.org> From: Grant Grundler Date: Fri, 14 Jul 2017 10:32:30 -0700 X-Google-Sender-Auth: U1t0pL8KyVm4zaNvwqASj1bo4iw Message-ID: Subject: Re: [Intel-gfx] [PATCH RESEND] drm/i915: Fix pipe/transcoder enum mismatches To: Jani Nikula Cc: =?UTF-8?Q?St=C3=A9phane_Marchesin?= , =?UTF-8?B?VmlsbGUgU3lyasOkbMOk?= , Grant Grundler , intel-gfx , Linux Kernel Mailing List , "dri-devel@lists.freedesktop.org" , Michael Davidson , Matthias Kaehlcke , Daniel Vetter , =?UTF-8?Q?St=C3=A9phane_Marchesin?= Content-Type: text/plain; charset="UTF-8" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by nfs id v6EHWwpC026861 On Fri, Jul 14, 2017 at 2:11 AM, Jani Nikula wrote: > On Thu, 13 Jul 2017, Stéphane Marchesin wrote: >> So, if you think this is wrong, can you fix this warning in a way that >> you'd like? > > As I replied previously [1], with more background, fixing the warnings > properly, in a way that actually improves the code instead of making it > worse, would mean a bunch of churn that's not just purely mechanical > conversion. That's fair. > Unless you can point out a bug which is actually caused by mixing the > types (which is mostly intentional, see the background) I have a hard > time telling people this should be a priority. This feels like "can't see the forest because of the trees". The original patch was submitted in order to compile cleanly using clang and the above suggests using clang is not important. Using clang is important to Matthias and the Chrome OS organization for many good reasons - including better warnings. The original patch message was clear that clang was generating the warning. This isn't the only patch mka has sent to kernel devs. What one can infer is Chrome OS is trying to move to clang (like other Google products _already_ have.) My impression is all these products are a priority to Intel - but it would be good to know otherwise. > Definitely something we'd > like to do in the long run and pedantically correct (and I tend to > prefer code that way) but we certainly have more important things to do. The long run is now. Everyone agrees the code should change and you don't have to do it. Matthias submitted an unacceptable patch and giving him some concrete guidance on what would be acceptable would enable him to implement/test it (or anyone else could for that matter). Can you do that? Just give an example of what the "right" API looks like and see where it goes. cheers, grant > > BR, > Jani. > > [1] http://mid.mail-archive.com/87wp9rahjy.fsf@intel.com > > > -- > Jani Nikula, Intel Open Source Technology Center