Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754562Ab3IZWQh (ORCPT ); Thu, 26 Sep 2013 18:16:37 -0400 Received: from mail-oa0-f45.google.com ([209.85.219.45]:65053 "EHLO mail-oa0-f45.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1754512Ab3IZWQd convert rfc822-to-8bit (ORCPT ); Thu, 26 Sep 2013 18:16:33 -0400 MIME-Version: 1.0 In-Reply-To: References: <1379611214-1089-1-git-send-email-grundler@chromium.org> <87fvss2uh9.fsf@octavius.laptop.org> <8738os2tqt.fsf@octavius.laptop.org> Date: Thu, 26 Sep 2013 15:16:32 -0700 X-Google-Sender-Auth: NL4gbamISjPA7zDXnoAJwx3yaKU Message-ID: Subject: Re: [PATCH] mmc: core: remove issue_fn indirect function call From: Grant Grundler To: Grant Grundler Cc: Chris Ball , Ulf Hansson , Seungwon Jeon , linux-mmc , "linux-kernel@vger.kernel.org" Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8BIT Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1912 Lines: 50 On Thu, Sep 26, 2013 at 2:56 PM, Grant Grundler wrote: > On Wed, Sep 25, 2013 at 7:37 PM, Chris Ball wrote: >> Hi, >> >> On Wed, Sep 25 2013, Chris Ball wrote: >>> Hi, >>> >>> On Fri, Sep 20 2013, Ulf Hansson wrote: >>>> On 19 September 2013 19:20, Grant Grundler wrote: >>>>> struct mmc_queue defines issue_fn as an indirect function call. >>>>> issue_fn field only gets set to mmc_blk_issue_rq and only gets >>>>> invoked immediately after calling blk_fetch_request(). >>>>> Don't bother with indirect function call - it's pointless and just >>>>> obfuscates the code. >>>>> >>>>> Signed-off-by: Grant Grundler >>>> >>>> Acked-by: Ulf Hansson >>> >>> Thanks, pushed to mmc-next for 3.13. >> >> Have dropped this, it's breaking my build: >> >> /home/cjb/git/mmc/drivers/mmc/card/block.c:1955:12: warning: ‘mmc_blk_issue_rq’ defined but not used [-Wunused-function] > > The function is declared static. :( Let me respin to remove the > static and add a function prototype to a header file. block.o and queue.o are linked together into one .ko all the time: obj-$(CONFIG_MMC_BLOCK) += mmc_block.o mmc_block-objs := block.o queue.o Two ways to handle this: I can 1) add a local function prototype of mmc_blk_issue_rq() to queue.c 2) move mmc_init_queue() and mmc_queue_thread() from queue.c to block.c (2) actually makes sense since both functions are block IO specific. Thoughts? Preference? Other ideas? thanks, grant ps. It's more obvious now that the return value from mmc_blk_issue_rq() is getting ignored. *sigh* -- 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/