From mboxrd@z Thu Jan 1 00:00:00 1970 From: Paulraj, Sandeep Date: Mon, 14 Sep 2009 17:29:10 -0500 Subject: [U-Boot] [PATCH] OMAP3 MMC: Fix warning dereferencing type-punned pointer In-Reply-To: <1252831222-32222-1-git-send-email-dirk.behme@googlemail.com> References: <1252831222-32222-1-git-send-email-dirk.behme@googlemail.com> Message-ID: <0554BEF07D437848AF01B9C9B5F0BC5D927DAB12@dlee01.ent.ti.com> List-Id: MIME-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit To: u-boot@lists.denx.de > -----Original Message----- > From: u-boot-bounces at lists.denx.de [mailto:u-boot-bounces at lists.denx.de] > On Behalf Of Dirk Behme > Sent: Sunday, September 13, 2009 4:40 AM > To: u-boot at lists.denx.de > Subject: [U-Boot] [PATCH] OMAP3 MMC: Fix warning dereferencing type-punned > pointer > > Fix warning > > dereferencing type-punned pointer will break strict-aliasing rules > > Signed-off-by: Dirk Behme > > --- If nobody has any issues with this patch I would like to push this to u-boot-ti as the u-boot-mmc fork seems not synced up with the latest U-Boot GIT tree. > > Patch against recent u-boot-ti git > f4f92c81835ccc43f74925737dfd478edb0182f2 > "TI: OMAP3: Overo Tobi ethernet support" > > Compile tested only ./MAKEALL ARM_CORTEX_A8 > > drivers/mmc/omap3_mmc.c | 56 ++++++++++++++++++++++++------------------ > ------ > 1 file changed, 29 insertions(+), 27 deletions(-) > > Index: u-boot-ti/drivers/mmc/omap3_mmc.c > =================================================================== > --- u-boot-ti.orig/drivers/mmc/omap3_mmc.c > +++ u-boot-ti/drivers/mmc/omap3_mmc.c > @@ -235,8 +235,13 @@ unsigned char mmc_detect_card(mmc_card_d > unsigned char err; > unsigned int argument = 0; > unsigned int ocr_value, ocr_recvd, ret_cmd41, hcs_val; > - unsigned int resp[4]; > unsigned short retry_cnt = 2000; > + union { > + unsigned int resp[4]; > + mmc_resp_r3 *r3; > + mmc_resp_r6 *r6; > + } mmc_resp; > + > > /* Set to Initialization Clock */ > err = mmc_clock_config(CLK_400KHZ, 0); > @@ -247,18 +252,18 @@ unsigned char mmc_detect_card(mmc_card_d > argument = 0x00000000; > > ocr_value = (0x1FF << 15); > - err = mmc_send_cmd(MMC_CMD0, argument, resp); > + err = mmc_send_cmd(MMC_CMD0, argument, mmc_resp.resp); > if (err != 1) > return err; > > argument = SD_CMD8_CHECK_PATTERN | SD_CMD8_2_7_3_6_V_RANGE; > - err = mmc_send_cmd(MMC_SDCMD8, argument, resp); > + err = mmc_send_cmd(MMC_SDCMD8, argument, mmc_resp.resp); > hcs_val = (err == 1) ? > MMC_OCR_REG_HOST_CAPACITY_SUPPORT_SECTOR : > MMC_OCR_REG_HOST_CAPACITY_SUPPORT_BYTE; > > argument = 0x0000 << 16; > - err = mmc_send_cmd(MMC_CMD55, argument, resp); > + err = mmc_send_cmd(MMC_CMD55, argument, mmc_resp.resp); > if (err == 1) { > mmc_card_cur->card_type = SD_CARD; > ocr_value |= hcs_val; > @@ -272,24 +277,24 @@ unsigned char mmc_detect_card(mmc_card_d > } > > argument = ocr_value; > - err = mmc_send_cmd(ret_cmd41, argument, resp); > + err = mmc_send_cmd(ret_cmd41, argument, mmc_resp.resp); > if (err != 1) > return err; > > - ocr_recvd = ((mmc_resp_r3 *) resp)->ocr; > + ocr_recvd = mmc_resp.r3->ocr; > > while (!(ocr_recvd & (0x1 << 31)) && (retry_cnt > 0)) { > retry_cnt--; > if (mmc_card_cur->card_type == SD_CARD) { > argument = 0x0000 << 16; > - err = mmc_send_cmd(MMC_CMD55, argument, resp); > + err = mmc_send_cmd(MMC_CMD55, argument, mmc_resp.resp); > } > > argument = ocr_value; > - err = mmc_send_cmd(ret_cmd41, argument, resp); > + err = mmc_send_cmd(ret_cmd41, argument, mmc_resp.resp); > if (err != 1) > return err; > - ocr_recvd = ((mmc_resp_r3 *) resp)->ocr; > + ocr_recvd = mmc_resp.r3->ocr; > } > > if (!(ocr_recvd & (0x1 << 31))) > @@ -318,22 +323,22 @@ unsigned char mmc_detect_card(mmc_card_d > if (!(ocr_recvd & ocr_value)) > return 0; > > - err = mmc_send_cmd(MMC_CMD2, argument, resp); > + err = mmc_send_cmd(MMC_CMD2, argument, mmc_resp.resp); > if (err != 1) > return err; > > if (mmc_card_cur->card_type == MMC_CARD) { > argument = mmc_card_cur->RCA << 16; > - err = mmc_send_cmd(MMC_CMD3, argument, resp); > + err = mmc_send_cmd(MMC_CMD3, argument, mmc_resp.resp); > if (err != 1) > return err; > } else { > argument = 0x00000000; > - err = mmc_send_cmd(MMC_SDCMD3, argument, resp); > + err = mmc_send_cmd(MMC_SDCMD3, argument, mmc_resp.resp); > if (err != 1) > return err; > > - mmc_card_cur->RCA = ((mmc_resp_r6 *) resp)->newpublishedrca; > + mmc_card_cur->RCA = mmc_resp.r6->newpublishedrca; > } > > writel(readl(&mmc_base->con) & ~OD, &mmc_base->con); > @@ -437,10 +442,12 @@ unsigned char configure_mmc(mmc_card_dat > { > unsigned char ret_val; > unsigned int argument; > - unsigned int resp[4]; > unsigned int trans_clk, trans_fact, trans_unit, retries = 2; > - mmc_csd_reg_t Card_CSD; > unsigned char trans_speed; > + union { > + unsigned int resp[4]; > + mmc_csd_reg_t Card_CSD; > + } mmc_resp; > > ret_val = mmc_init_setup(); > > @@ -453,21 +460,16 @@ unsigned char configure_mmc(mmc_card_dat > } while ((retries > 0) && (ret_val != 1)); > > argument = mmc_card_cur->RCA << 16; > - ret_val = mmc_send_cmd(MMC_CMD9, argument, resp); > + ret_val = mmc_send_cmd(MMC_CMD9, argument, mmc_resp.resp); > if (ret_val != 1) > return ret_val; > > - ((unsigned int *) &Card_CSD)[3] = resp[3]; > - ((unsigned int *) &Card_CSD)[2] = resp[2]; > - ((unsigned int *) &Card_CSD)[1] = resp[1]; > - ((unsigned int *) &Card_CSD)[0] = resp[0]; > - > if (mmc_card_cur->card_type == MMC_CARD) > - mmc_card_cur->version = Card_CSD.spec_vers; > + mmc_card_cur->version = mmc_resp.Card_CSD.spec_vers; > > - trans_speed = Card_CSD.tran_speed; > + trans_speed = mmc_resp.Card_CSD.tran_speed; > > - ret_val = mmc_send_cmd(MMC_CMD4, MMC_DSR_DEFAULT << 16, resp); > + ret_val = mmc_send_cmd(MMC_CMD4, MMC_DSR_DEFAULT << 16, > mmc_resp.resp); > if (ret_val != 1) > return ret_val; > > @@ -491,18 +493,18 @@ unsigned char configure_mmc(mmc_card_dat > return ret_val; > > argument = mmc_card_cur->RCA << 16; > - ret_val = mmc_send_cmd(MMC_CMD7_SELECT, argument, resp); > + ret_val = mmc_send_cmd(MMC_CMD7_SELECT, argument, mmc_resp.resp); > if (ret_val != 1) > return ret_val; > > /* Configure the block length to 512 bytes */ > argument = MMCSD_SECTOR_SIZE; > - ret_val = mmc_send_cmd(MMC_CMD16, argument, resp); > + ret_val = mmc_send_cmd(MMC_CMD16, argument, mmc_resp.resp); > if (ret_val != 1) > return ret_val; > > /* get the card size in sectors */ > - ret_val = mmc_read_cardsize(mmc_card_cur, &Card_CSD); > + ret_val = mmc_read_cardsize(mmc_card_cur, &mmc_resp.Card_CSD); > if (ret_val != 1) > return ret_val; > > _______________________________________________ > U-Boot mailing list > U-Boot at lists.denx.de > http://lists.denx.de/mailman/listinfo/u-boot