Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1161962AbbKEPZP (ORCPT ); Thu, 5 Nov 2015 10:25:15 -0500 Received: from smtp.codeaurora.org ([198.145.29.96]:47797 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1756303AbbKEPZN (ORCPT ); Thu, 5 Nov 2015 10:25:13 -0500 Subject: Re: [PATCH 3/4] scsi: fix compiler warning for sg To: Sinan Kaya , Andy Shevchenko Cc: linux-scsi , cov@codeaurora.org, jcm@redhat.com, Doug Gilbert , "James E.J. Bottomley" , "linux-kernel@vger.kernel.org" References: <1446698789-19308-1-git-send-email-okaya@codeaurora.org> <1446698789-19308-3-git-send-email-okaya@codeaurora.org> <563B7180.2000405@codeaurora.org> From: Timur Tabi Message-ID: <563B74D7.5090505@codeaurora.org> Date: Thu, 5 Nov 2015 09:25:11 -0600 User-Agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10.9; rv:41.0) Gecko/20100101 Firefox/41.0 SeaMonkey/2.38 MIME-Version: 1.0 In-Reply-To: <563B7180.2000405@codeaurora.org> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 753 Lines: 19 Sinan Kaya wrote: > > > #define MULDIV(X,MUL,DIV) mult_frac64(X, MUL, DIV) Why bother with the macro at all? Just change the code to use do_div() directly. It's possible that the original code was written before do_div() became standard, or the developer didn't know about, which is why we have this macro in the first place. -- Sent by an employee of the Qualcomm Innovation Center, Inc. The Qualcomm Innovation Center, Inc. is a member of the Code Aurora Forum, hosted by The Linux Foundation. -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/