Re: [PATCH] drm/i915/gvt: Clean up dead code in cmd_parser

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On 16/10/17 12:08:19 +0300, Jani Nikula wrote:
> On Sun, 15 Oct 2017, Christos Gkekas <chris.gekas@xxxxxxxxx> wrote:
> > Delete variables 'gma_bottom' that are set but never used.
> 
> Please use common sense in choosing the mailing lists and maintainers
> you send patches to. In this case, intel-gvt-dev would have been
> sufficient, with intel-gfx more than enough. Including Dave or dri-devel
> or LKML is excessive. Arguably including Joonas, Rodrigo and myself is
> also too much.
> 
> Please at least put it in context. This is a non-functional change
> removing some dead code, not an urgent fix to some hair rising jaw
> dropping bug.
> 
> Thanks,
> Jani.
> 
> -- 
> Jani Nikula, Intel Open Source Technology Center

Hi Jani,

Thanks for your feedback and sorry for any inconvenience caused.

I was trying to follow the good patch practices outlined in the link
below and used the get_maintainer.pl script to identify who to send the
patches to. I am sorry but I have no idea of who-is-who within intel and
who would be the best person or list to contact. Also intel-gvt-dev is a
members-only list.

https://kernelnewbies.org/FirstKernelPatch#head-dc6a8aa0be0d0e8ed9dc03726d0b5a1fb0f65e1f

Regarding your second point, I believe the title of this patch is pretty
clear. Dead code is _dead_ and thus by definition not a functional
change or bug fix. There was nothing in my email to indicate this is an
'urgent fix to some hair rising jaw dropping bug'. I am not sure what is
the source of confusion?

Thanks
Christos.

_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux