From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1758832AbcG1Owg (ORCPT ); Thu, 28 Jul 2016 10:52:36 -0400 Received: from esa6.dell-outbound.iphmx.com ([68.232.149.229]:41760 "EHLO esa6.dell-outbound.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1758713AbcG1Ow1 (ORCPT ); Thu, 28 Jul 2016 10:52:27 -0400 X-Greylist: delayed 572 seconds by postgrey-1.27 at vger.kernel.org; Thu, 28 Jul 2016 10:52:27 EDT X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd04.lss.emc.com u6SEgnuT009350 X-DKIM: OpenDKIM Filter v2.4.3 mailuogwprd04.lss.emc.com u6SEgnuT009350 From: "Allen Hubbe" To: "'Serge Semin'" , Cc: , , , , Subject: RE: [PATCH v2 0/3] ntb: Asynchronous NTB devices support Date: Thu, 28 Jul 2016 10:42:30 -0400 Message-ID: <000001d1e8de$463b4c10$d2b1e430$@emc.com> MIME-Version: 1.0 Content-Type: text/plain; charset="Windows-1252" Content-Transfer-Encoding: 7bit X-Mailer: Microsoft Outlook 14.0 Thread-Index: AdHo3jbvSpyJfiHeTum5BYmykCRoPw== Content-Language: en-us X-RSA-Classifications: public, GIS Solicitation X-Sentrion-Hostname: mailuogwprd04.lss.emc.com Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Serge Semin > Please, find the general patchset description in the cover letter of the first > patchset (see the very first message in thread). > > Changes in v2: > - Fix sparc64 compilation warning in drivers/ntb/hw/idt/ntb_hw_idt.c : > warning: right shift count >= width of type > - Fix sparc64 compilation warnings in drivers/ntb/test/ntb_mw_test.c : > warning: right shift count >= width of type > warning: cast to pointer from integer of different size Thanks for reacting to the test robot so quickly. Since nobody else has responded yet, I would like to assure you that the patches are not being ignored. Please be patient. The IDT driver will be a valuable contribution to the ntb subsystem. I am working carefully through patch 1/3 first, since it affects existing drivers and interface. A word of caution regarding your statement, "There are a some types of checkpatch warnings I left unfixed." Coding style can be a touchy subject, leading to some recent rants^H^H^H^H^Hdiscussion on some of the same topics that are included in that list of unfixed warnings. Be prepared to adhere to the style guide, even if it is inconvenient and against your own logic, because that is almost always the easier and more practical approach than asking for changes or exceptions, and better for your mental health not to be on the To: list of something like https://lkml.org/lkml/2016/7/8/625. "Of course all of these warnings are discussable, except the last one." Be prepared, even if it will require significant changes to the code. For really inconvenient changes, we can talk about other more readily acceptable approaches to keep the code short and elegant, as is obviously your intent. Please be patient with the review.