On Tue, Aug 02, 2016 at 06:50:01PM +0800, Baole Ni wrote: > I find that the developers often just specified the numeric value > when calling a macro which is defined with a parameter for access permission. > As we know, these numeric value for access permission have had the > corresponding macro, > and that using macro can improve the robustness and readability of the code, > thus, I suggest replacing the numeric parameter with the macro. > > Signed-off-by: Chuansheng Liu <chuansheng.liu at intel.com> > Signed-off-by: Baole Ni <baolex.ni at intel.com> > --- > drivers/gpu/drm/qxl/qxl_drv.c | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-)
I got almost 1000 patches out of your mail bomb. For all of them: NACK. Other people pointed actual reasons (title, readability, cc-list). The commit msg should be also written in a more readable way. Anyway, in Intel there are a lot of people having kernel contributions so it is a shame that you didn't ask them for advice how to do it... BR, Krzysztof