From patchwork Fri Dec 30 12:47:25 2016 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Ulf Hansson X-Patchwork-Id: 89346 Delivered-To: patches@linaro.org Received: by 10.140.20.101 with SMTP id 92csp6424715qgi; Fri, 30 Dec 2016 04:48:21 -0800 (PST) X-Received: by 10.25.163.213 with SMTP id m204mr16931333lfe.77.1483102101055; Fri, 30 Dec 2016 04:48:21 -0800 (PST) Return-Path: Received: from mail-lf0-x232.google.com (mail-lf0-x232.google.com. [2a00:1450:4010:c07::232]) by mx.google.com with ESMTPS id y18si33028251lja.88.2016.12.30.04.48.20 for (version=TLS1_2 cipher=ECDHE-RSA-AES128-GCM-SHA256 bits=128/128); Fri, 30 Dec 2016 04:48:21 -0800 (PST) Received-SPF: pass (google.com: domain of ulf.hansson@linaro.org designates 2a00:1450:4010:c07::232 as permitted sender) client-ip=2a00:1450:4010:c07::232; Authentication-Results: mx.google.com; dkim=pass header.i=@linaro.org; spf=pass (google.com: domain of ulf.hansson@linaro.org designates 2a00:1450:4010:c07::232 as permitted sender) smtp.mailfrom=ulf.hansson@linaro.org; dmarc=pass (p=NONE dis=NONE) header.from=linaro.org Received: by mail-lf0-x232.google.com with SMTP id y21so236380705lfa.1 for ; Fri, 30 Dec 2016 04:48:20 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=linaro.org; s=google; h=from:to:cc:subject:date:message-id:in-reply-to:references; bh=rteKGLG9/0d43VOzMlV18XjESl+T3mAOCRRI3HBkDMQ=; b=klsRvnMEi1WxwpuJuaNWDx9YM10YMbdrn3Gs5Mj5fB60bsDaMFbUPKWGwCWLzz+yBO ZRQzkC3ddkqXtMHPwCNF6h1cUYEKzafpO69nkjbVYvV30jkZHqi+Uh4lm/VYCX7C5X2T 1hV/g00Pw05TcdF81lNVu70gk3wZsZ/T0wXI4= X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20161025; h=x-gm-message-state:from:to:cc:subject:date:message-id:in-reply-to :references; bh=rteKGLG9/0d43VOzMlV18XjESl+T3mAOCRRI3HBkDMQ=; b=AR/SSiqb4YU4lxbvyK+uhjjj4CQ8I3Ns/5aObZnwLzKHTbzREgvKOdd6R7OD1jWP3O vDBt6Ep1YZonoiVN8Cs2lrhCCJhU89G9DsqKukCspjYi1ckO1Tu/Wtrni25CiYkctswr nmxe2Fz9OpERqLQtBHqjBAu17Srlxrt7HFaYmSUqgxVFieyigv6AdTK/crF1StVYeT4M 9yV9DyLehb1W6/IvHg0L6DEi2lxU4wXOFxvDkNh5hsO6GH4yXubzkGQg1MLi8I7o1Ue/ RB6ajHIaegql2mZExVXvZkZTjG9KiZabNyPrkzLKh3qiyCj3UcyO5jfdS1xEolAKzsI4 nLCA== X-Gm-Message-State: AIkVDXKmtCLv26magew/fi//oBZaiqLywUUH1e+C5F5SZxQYVMRIM4URECzLcOr6mvoHIyZA8vc= X-Received: by 10.25.210.147 with SMTP id j141mr13246665lfg.78.1483102100587; Fri, 30 Dec 2016 04:48:20 -0800 (PST) Return-Path: Received: from localhost.localdomain (h-155-4-221-67.na.cust.bahnhof.se. [155.4.221.67]) by smtp.gmail.com with ESMTPSA id a79sm7450069ljb.46.2016.12.30.04.48.17 (version=TLS1_2 cipher=ECDHE-RSA-AES128-SHA bits=128/128); Fri, 30 Dec 2016 04:48:19 -0800 (PST) From: Ulf Hansson To: linux-mmc@vger.kernel.org, Ulf Hansson Cc: Jaehoon Chung , Adrian Hunter , Linus Walleij Subject: [PATCH 12/21] mmc: core: First step in cleaning up private mmc header files Date: Fri, 30 Dec 2016 13:47:25 +0100 Message-Id: <1483102054-1752-13-git-send-email-ulf.hansson@linaro.org> X-Mailer: git-send-email 1.9.1 In-Reply-To: <1483102054-1752-1-git-send-email-ulf.hansson@linaro.org> References: <1483102054-1752-1-git-send-email-ulf.hansson@linaro.org> This is the first step in cleaning up the private mmc header files. In this change we makes sure each header file builds standalone, as that helps to resolve dependencies. While changing this, it also seems reasonable to stop including other headers from inside a header itself which it don't depend upon. Additionally, in some cases such dependencies are better resolved by forward declaring the needed struct. Signed-off-by: Ulf Hansson --- drivers/mmc/core/block.h | 8 ++++++++ drivers/mmc/core/bus.h | 4 ++++ drivers/mmc/core/core.h | 4 ++++ drivers/mmc/core/host.h | 3 ++- drivers/mmc/core/mmc_ops.h | 5 +++++ drivers/mmc/core/pwrseq.h | 6 +++++- drivers/mmc/core/queue.h | 6 +++++- drivers/mmc/core/sd.h | 5 ++++- drivers/mmc/core/sd_ops.h | 5 +++++ drivers/mmc/core/sdio_bus.h | 3 +++ drivers/mmc/core/sdio_cis.h | 3 +++ drivers/mmc/core/sdio_ops.h | 4 ++++ drivers/mmc/core/slot-gpio.h | 2 ++ 13 files changed, 54 insertions(+), 4 deletions(-) -- 1.9.1 diff --git a/drivers/mmc/core/block.h b/drivers/mmc/core/block.h index cdabb2e..00b7483 100644 --- a/drivers/mmc/core/block.h +++ b/drivers/mmc/core/block.h @@ -1 +1,9 @@ +#ifndef _MMC_CORE_BLOCK_H +#define _MMC_CORE_BLOCK_H + +struct mmc_queue; +struct request; + int mmc_blk_issue_rq(struct mmc_queue *mq, struct request *req); + +#endif diff --git a/drivers/mmc/core/bus.h b/drivers/mmc/core/bus.h index 00a1971..93b516a 100644 --- a/drivers/mmc/core/bus.h +++ b/drivers/mmc/core/bus.h @@ -11,6 +11,10 @@ #ifndef _MMC_CORE_BUS_H #define _MMC_CORE_BUS_H +struct mmc_host; +struct mmc_card; +struct device_type; + #define MMC_DEV_ATTR(name, fmt, args...) \ static ssize_t mmc_##name##_show (struct device *dev, struct device_attribute *attr, char *buf) \ { \ diff --git a/drivers/mmc/core/core.h b/drivers/mmc/core/core.h index 0fa86a2..29b91ae 100644 --- a/drivers/mmc/core/core.h +++ b/drivers/mmc/core/core.h @@ -12,6 +12,10 @@ #define _MMC_CORE_CORE_H #include +#include + +struct mmc_host; +struct mmc_card; #define MMC_CMD_RETRIES 3 diff --git a/drivers/mmc/core/host.h b/drivers/mmc/core/host.h index 992bf53..366ce79 100644 --- a/drivers/mmc/core/host.h +++ b/drivers/mmc/core/host.h @@ -10,7 +10,8 @@ */ #ifndef _MMC_CORE_HOST_H #define _MMC_CORE_HOST_H -#include + +struct mmc_host; int mmc_register_host_class(void); void mmc_unregister_host_class(void); diff --git a/drivers/mmc/core/mmc_ops.h b/drivers/mmc/core/mmc_ops.h index abd525e..e76365a 100644 --- a/drivers/mmc/core/mmc_ops.h +++ b/drivers/mmc/core/mmc_ops.h @@ -12,6 +12,11 @@ #ifndef _MMC_MMC_OPS_H #define _MMC_MMC_OPS_H +#include + +struct mmc_host; +struct mmc_card; + int mmc_select_card(struct mmc_card *card); int mmc_deselect_cards(struct mmc_host *host); int mmc_set_dsr(struct mmc_host *host); diff --git a/drivers/mmc/core/pwrseq.h b/drivers/mmc/core/pwrseq.h index d69e751..39c911a 100644 --- a/drivers/mmc/core/pwrseq.h +++ b/drivers/mmc/core/pwrseq.h @@ -8,7 +8,11 @@ #ifndef _MMC_CORE_PWRSEQ_H #define _MMC_CORE_PWRSEQ_H -#include +#include + +struct mmc_host; +struct device; +struct module; struct mmc_pwrseq_ops { void (*pre_power_on)(struct mmc_host *host); diff --git a/drivers/mmc/core/queue.h b/drivers/mmc/core/queue.h index dac8c3d..0cea02a 100644 --- a/drivers/mmc/core/queue.h +++ b/drivers/mmc/core/queue.h @@ -1,6 +1,11 @@ #ifndef MMC_QUEUE_H #define MMC_QUEUE_H +#include +#include +#include +#include + static inline bool mmc_req_is_special(struct request *req) { return req && @@ -9,7 +14,6 @@ static inline bool mmc_req_is_special(struct request *req) req_op(req) == REQ_OP_SECURE_ERASE); } -struct request; struct task_struct; struct mmc_blk_data; diff --git a/drivers/mmc/core/sd.h b/drivers/mmc/core/sd.h index aab824a..1ada980 100644 --- a/drivers/mmc/core/sd.h +++ b/drivers/mmc/core/sd.h @@ -1,10 +1,13 @@ #ifndef _MMC_CORE_SD_H #define _MMC_CORE_SD_H -#include +#include extern struct device_type sd_type; +struct mmc_host; +struct mmc_card; + int mmc_sd_get_cid(struct mmc_host *host, u32 ocr, u32 *cid, u32 *rocr); int mmc_sd_get_csd(struct mmc_host *host, struct mmc_card *card); void mmc_decode_cid(struct mmc_card *card); diff --git a/drivers/mmc/core/sd_ops.h b/drivers/mmc/core/sd_ops.h index ffc2305..ac7223c 100644 --- a/drivers/mmc/core/sd_ops.h +++ b/drivers/mmc/core/sd_ops.h @@ -12,6 +12,11 @@ #ifndef _MMC_SD_OPS_H #define _MMC_SD_OPS_H +#include + +struct mmc_card; +struct mmc_host; + int mmc_app_set_bus_width(struct mmc_card *card, int width); int mmc_send_app_op_cond(struct mmc_host *host, u32 ocr, u32 *rocr); int mmc_send_if_cond(struct mmc_host *host, u32 ocr); diff --git a/drivers/mmc/core/sdio_bus.h b/drivers/mmc/core/sdio_bus.h index 567a768..b69a254 100644 --- a/drivers/mmc/core/sdio_bus.h +++ b/drivers/mmc/core/sdio_bus.h @@ -11,6 +11,9 @@ #ifndef _MMC_CORE_SDIO_BUS_H #define _MMC_CORE_SDIO_BUS_H +struct mmc_card; +struct sdio_func; + struct sdio_func *sdio_alloc_func(struct mmc_card *card); int sdio_add_func(struct sdio_func *func); void sdio_remove_func(struct sdio_func *func); diff --git a/drivers/mmc/core/sdio_cis.h b/drivers/mmc/core/sdio_cis.h index 4d903c2..16aa563 100644 --- a/drivers/mmc/core/sdio_cis.h +++ b/drivers/mmc/core/sdio_cis.h @@ -14,6 +14,9 @@ #ifndef _MMC_SDIO_CIS_H #define _MMC_SDIO_CIS_H +struct mmc_card; +struct sdio_func; + int sdio_read_common_cis(struct mmc_card *card); void sdio_free_common_cis(struct mmc_card *card); diff --git a/drivers/mmc/core/sdio_ops.h b/drivers/mmc/core/sdio_ops.h index 5660c7f..e1c36d6 100644 --- a/drivers/mmc/core/sdio_ops.h +++ b/drivers/mmc/core/sdio_ops.h @@ -12,8 +12,12 @@ #ifndef _MMC_SDIO_OPS_H #define _MMC_SDIO_OPS_H +#include #include +struct mmc_host; +struct mmc_card; + int mmc_send_io_op_cond(struct mmc_host *host, u32 ocr, u32 *rocr); int mmc_io_rw_direct(struct mmc_card *card, int write, unsigned fn, unsigned addr, u8 in, u8* out); diff --git a/drivers/mmc/core/slot-gpio.h b/drivers/mmc/core/slot-gpio.h index 8c1854d..a06fd84 100644 --- a/drivers/mmc/core/slot-gpio.h +++ b/drivers/mmc/core/slot-gpio.h @@ -8,6 +8,8 @@ #ifndef _MMC_CORE_SLOTGPIO_H #define _MMC_CORE_SLOTGPIO_H +struct mmc_host; + int mmc_gpio_alloc(struct mmc_host *host); #endif