Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755699AbYGEQwc (ORCPT ); Sat, 5 Jul 2008 12:52:32 -0400 Received: (majordomo@vger.kernel.org) by vger.kernel.org id S1752018AbYGEQwN (ORCPT ); Sat, 5 Jul 2008 12:52:13 -0400 Received: from mga01.intel.com ([192.55.52.88]:25039 "EHLO mga01.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751877AbYGEQwM (ORCPT ); Sat, 5 Jul 2008 12:52:12 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="4.30,307,1212390000"; d="scan'208";a="349123509" From: Tomas Winkler To: drzeus-list@drzeus.cx Cc: marcel@holtmann.org, linux-kernel@vger.kernel.org, Tomas Winkler Subject: [PATCH 2/2 V2] MMC: cleanup endianity conversions and style problems Date: Sat, 5 Jul 2008 19:52:05 +0300 Message-Id: <1215276725-15588-2-git-send-email-tomas.winkler@intel.com> X-Mailer: git-send-email 1.5.4.1 In-Reply-To: <1215276725-15588-1-git-send-email-tomas.winkler@intel.com> References: <1215276725-15588-1-git-send-email-tomas.winkler@intel.com> Content-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii" Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 3851 Lines: 133 This patch cleans up endianity conversions im mmc core and style errors in the 'for' statements Signed-off-by: Tomas Winkler --- drivers/mmc/card/block.c | 5 ++--- drivers/mmc/core/mmc_ops.c | 15 ++++++++------- drivers/mmc/core/sd_ops.c | 7 ++++--- 3 files changed, 14 insertions(+), 13 deletions(-) diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c index 4b0f822..fbee07f 100644 --- a/drivers/mmc/card/block.c +++ b/drivers/mmc/card/block.c @@ -144,7 +144,7 @@ struct mmc_blk_request { static u32 mmc_sd_num_wr_blocks(struct mmc_card *card) { int err; - u32 blocks; + __be32 blocks; struct mmc_request mrq; struct mmc_command cmd; @@ -203,9 +203,8 @@ static u32 mmc_sd_num_wr_blocks(struct mmc_card *card) if (cmd.error || data.error) return (u32)-1; - blocks = ntohl(blocks); - return blocks; + return be32_to_cpu(blocks); } static int mmc_blk_issue_rq(struct mmc_queue *mq, struct request *req) diff --git a/drivers/mmc/core/mmc_ops.c b/drivers/mmc/core/mmc_ops.c index 64b05c6..8187aea 100644 --- a/drivers/mmc/core/mmc_ops.c +++ b/drivers/mmc/core/mmc_ops.c @@ -267,17 +267,18 @@ mmc_send_cxd_data(struct mmc_card *card, struct mmc_host *host, int mmc_send_csd(struct mmc_card *card, u32 *csd) { int ret, i; + __be32 csd_buf[4]; if (!mmc_host_is_spi(card->host)) return mmc_send_cxd_native(card->host, card->rca << 16, csd, MMC_SEND_CSD); - ret = mmc_send_cxd_data(card, card->host, MMC_SEND_CSD, csd, 16); + ret = mmc_send_cxd_data(card, card->host, MMC_SEND_CSD, csd_buf, 16); if (ret) return ret; - for (i = 0;i < 4;i++) - csd[i] = be32_to_cpu(csd[i]); + for (i = 0; i < 4; i++) + csd[i] = be32_to_cpu(csd_buf[i]); return 0; } @@ -285,7 +286,7 @@ int mmc_send_csd(struct mmc_card *card, u32 *csd) int mmc_send_cid(struct mmc_host *host, u32 *cid) { int ret, i; - + __be32 cid_buf[4]; if (!mmc_host_is_spi(host)) { if (!host->card) return -EINVAL; @@ -293,12 +294,12 @@ int mmc_send_cid(struct mmc_host *host, u32 *cid) cid, MMC_SEND_CID); } - ret = mmc_send_cxd_data(NULL, host, MMC_SEND_CID, cid, 16); + ret = mmc_send_cxd_data(NULL, host, MMC_SEND_CID, cid_buf, 16); if (ret) return ret; - for (i = 0;i < 4;i++) - cid[i] = be32_to_cpu(cid[i]); + for (i = 0; i < 4; i++) + cid[i] = be32_to_cpu(cid_buf[i]); return 0; } diff --git a/drivers/mmc/core/sd_ops.c b/drivers/mmc/core/sd_ops.c index 0d96080..7ef7b6c 100644 --- a/drivers/mmc/core/sd_ops.c +++ b/drivers/mmc/core/sd_ops.c @@ -254,6 +254,7 @@ int mmc_app_send_scr(struct mmc_card *card, u32 *scr) struct mmc_command cmd; struct mmc_data data; struct scatterlist sg; + __be32 scr_buf[2]; BUG_ON(!card); BUG_ON(!card->host); @@ -282,7 +283,7 @@ int mmc_app_send_scr(struct mmc_card *card, u32 *scr) data.sg = &sg; data.sg_len = 1; - sg_init_one(&sg, scr, 8); + sg_init_one(&sg, scr_buf, 8); mmc_set_data_timeout(&data, card); @@ -293,8 +294,8 @@ int mmc_app_send_scr(struct mmc_card *card, u32 *scr) if (data.error) return data.error; - scr[0] = be32_to_cpu(scr[0]); - scr[1] = be32_to_cpu(scr[1]); + scr[0] = be32_to_cpu(scr_buf[0]); + scr[1] = be32_to_cpu(scr_buf[1]); return 0; } -- 1.5.4.1 --------------------------------------------------------------------- Intel Israel (74) Limited This e-mail and any attachments may contain confidential material for the sole use of the intended recipient(s). Any review or distribution by others is strictly prohibited. If you are not the intended recipient, please contact the sender and delete all copies. -- 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/