From: "Gustavo A. R. Silva" <gustavo@embeddedor.com>
To: Maxime Desroches <desroches.maxime@gmail.com>,
gregkhl@linuxfoundation.org
Cc: devel@driverdev.osuosl.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] Staging: rts5208: rtsx_card: Fixed multiple coding style issues
Date: Sat, 6 Oct 2018 22:40:18 +0200 [thread overview]
Message-ID: <98626de4-3914-7c8d-047d-84cd4a339073@embeddedor.com> (raw)
In-Reply-To: <20180929020300.23321-1-desroches.maxime@gmail.com>
Hi,
On 9/29/18 4:03 AM, Maxime Desroches wrote:
> Fixed multiple coding style issues
>
What kind of style issues are you trying to fix?
Please, always be specific about what you are trying to fix. This makes
it easier for the maintainers to review your patch and give you feedback.
Thanks
--
Gustavo
> Signed-off-by: Maxime Desroches <desroches.maxime@gmail.com>
> ---
> drivers/staging/rts5208/rtsx_card.c | 96 +++++++++++------------------
> 1 file changed, 37 insertions(+), 59 deletions(-)
>
> diff --git a/drivers/staging/rts5208/rtsx_card.c b/drivers/staging/rts5208/rtsx_card.c
> index d26a8e372fce..b45abbe29bc2 100644
> --- a/drivers/staging/rts5208/rtsx_card.c
> +++ b/drivers/staging/rts5208/rtsx_card.c
> @@ -647,9 +647,8 @@ int switch_ssc_clock(struct rtsx_chip *chip, int clk)
> dev_dbg(rtsx_dev(chip), "Switch SSC clock to %dMHz (cur_clk = %d)\n",
> clk, chip->cur_clk);
>
> - if ((clk <= 2) || (n > max_n)) {
> + if ((clk <= 2) || (n > max_n))
> return STATUS_FAIL;
> - }
>
> mcu_cnt = (u8)(125 / clk + 3);
> if (mcu_cnt > 7)
> @@ -688,15 +687,13 @@ int switch_ssc_clock(struct rtsx_chip *chip, int clk)
> }
>
> retval = rtsx_send_cmd(chip, 0, WAIT_TIME);
> - if (retval < 0) {
> + if (retval < 0)
> return STATUS_ERROR;
> - }
>
> udelay(10);
> retval = rtsx_write_register(chip, CLK_CTL, CLK_LOW_FREQ, 0);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> chip->cur_clk = clk;
>
> @@ -790,49 +787,44 @@ int switch_normal_clock(struct rtsx_chip *chip, int clk)
> }
>
> retval = rtsx_write_register(chip, CLK_CTL, 0xFF, CLK_LOW_FREQ);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> if (sd_vpclk_phase_reset) {
> retval = rtsx_write_register(chip, SD_VPCLK0_CTL,
> PHASE_NOT_RESET, 0);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> +
> retval = rtsx_write_register(chip, SD_VPCLK1_CTL,
> PHASE_NOT_RESET, 0);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> }
> retval = rtsx_write_register(chip, CLK_DIV, 0xFF,
> (div << 4) | mcu_cnt);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> retval = rtsx_write_register(chip, CLK_SEL, 0xFF, sel);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> if (sd_vpclk_phase_reset) {
> udelay(200);
> retval = rtsx_write_register(chip, SD_VPCLK0_CTL,
> PHASE_NOT_RESET, PHASE_NOT_RESET);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> +
> retval = rtsx_write_register(chip, SD_VPCLK1_CTL,
> PHASE_NOT_RESET, PHASE_NOT_RESET);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> +
> udelay(200);
> }
> retval = rtsx_write_register(chip, CLK_CTL, 0xFF, 0);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> chip->cur_clk = clk;
>
> @@ -878,9 +870,8 @@ int enable_card_clock(struct rtsx_chip *chip, u8 card)
> clk_en |= MS_CLK_EN;
>
> retval = rtsx_write_register(chip, CARD_CLK_EN, clk_en, clk_en);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> return STATUS_SUCCESS;
> }
> @@ -898,9 +889,8 @@ int disable_card_clock(struct rtsx_chip *chip, u8 card)
> clk_en |= MS_CLK_EN;
>
> retval = rtsx_write_register(chip, CARD_CLK_EN, clk_en, 0);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> return STATUS_SUCCESS;
> }
> @@ -924,9 +914,8 @@ int card_power_on(struct rtsx_chip *chip, u8 card)
> rtsx_add_cmd(chip, WRITE_REG_CMD, CARD_PWR_CTL, mask, val1);
>
> retval = rtsx_send_cmd(chip, 0, 100);
> - if (retval != STATUS_SUCCESS) {
> + if (retval != STATUS_SUCCESS)
> return STATUS_FAIL;
> - }
>
> udelay(chip->pmos_pwr_on_interval);
>
> @@ -934,9 +923,8 @@ int card_power_on(struct rtsx_chip *chip, u8 card)
> rtsx_add_cmd(chip, WRITE_REG_CMD, CARD_PWR_CTL, mask, val2);
>
> retval = rtsx_send_cmd(chip, 0, 100);
> - if (retval != STATUS_SUCCESS) {
> + if (retval != STATUS_SUCCESS)
> return STATUS_FAIL;
> - }
>
> return STATUS_SUCCESS;
> }
> @@ -955,9 +943,8 @@ int card_power_off(struct rtsx_chip *chip, u8 card)
> }
>
> retval = rtsx_write_register(chip, CARD_PWR_CTL, mask, val);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> return STATUS_SUCCESS;
> }
> @@ -969,9 +956,8 @@ int card_rw(struct scsi_cmnd *srb, struct rtsx_chip *chip,
> unsigned int lun = SCSI_LUN(srb);
> int i;
>
> - if (!chip->rw_card[lun]) {
> + if (!chip->rw_card[lun])
> return STATUS_FAIL;
> - }
>
> for (i = 0; i < 3; i++) {
> chip->rw_need_retry = 0;
> @@ -1009,36 +995,33 @@ int card_share_mode(struct rtsx_chip *chip, int card)
>
> if (CHECK_PID(chip, 0x5208)) {
> mask = CARD_SHARE_MASK;
> - if (card == SD_CARD) {
> + if (card == SD_CARD)
> value = CARD_SHARE_48_SD;
> - } else if (card == MS_CARD) {
> + else if (card == MS_CARD)
> value = CARD_SHARE_48_MS;
> - } else if (card == XD_CARD) {
> + else if (card == XD_CARD)
> value = CARD_SHARE_48_XD;
> - } else {
> + else
> return STATUS_FAIL;
> - }
>
> } else if (CHECK_PID(chip, 0x5288)) {
> mask = 0x03;
> - if (card == SD_CARD) {
> + if (card == SD_CARD)
> value = CARD_SHARE_BAROSSA_SD;
> - } else if (card == MS_CARD) {
> + else if (card == MS_CARD)
> value = CARD_SHARE_BAROSSA_MS;
> - } else if (card == XD_CARD) {
> + else if (card == XD_CARD)
> value = CARD_SHARE_BAROSSA_XD;
> - } else {
> + else
> return STATUS_FAIL;
> - }
>
> } else {
> return STATUS_FAIL;
> }
>
> retval = rtsx_write_register(chip, CARD_SHARE_MODE, mask, value);
> - if (retval) {
> + if (retval)
> return retval;
> - }
>
> return STATUS_SUCCESS;
> }
> @@ -1050,30 +1033,26 @@ int select_card(struct rtsx_chip *chip, int card)
> if (chip->cur_card != card) {
> u8 mod;
>
> - if (card == SD_CARD) {
> + if (card == SD_CARD)
> mod = SD_MOD_SEL;
> - } else if (card == MS_CARD) {
> + else if (card == MS_CARD)
> mod = MS_MOD_SEL;
> - } else if (card == XD_CARD) {
> + else if (card == XD_CARD)
> mod = XD_MOD_SEL;
> - } else if (card == SPI_CARD) {
> + else if (card == SPI_CARD)
> mod = SPI_MOD_SEL;
> - } else {
> + else
> return STATUS_FAIL;
> - }
>
> retval = rtsx_write_register(chip, CARD_SELECT, 0x07, mod);
> - if (retval) {
> + if (retval)
> return retval;
> - }
> chip->cur_card = card;
>
> retval = card_share_mode(chip, card);
> - if (retval != STATUS_SUCCESS) {
> + if (retval != STATUS_SUCCESS)
> return STATUS_FAIL;
> - }
> }
> -
> return STATUS_SUCCESS;
> }
>
> @@ -1120,9 +1099,8 @@ int detect_card_cd(struct rtsx_chip *chip, int card)
> }
>
> status = rtsx_readl(chip, RTSX_BIPR);
> - if (!(status & card_cd)) {
> + if (!(status & card_cd))
> return STATUS_FAIL;
> - }
>
> return STATUS_SUCCESS;
> }
>
prev parent reply other threads:[~2018-10-06 20:40 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-09-29 2:03 Maxime Desroches
2018-09-29 9:36 ` Greg KH
2018-10-06 20:40 ` Gustavo A. R. Silva [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=98626de4-3914-7c8d-047d-84cd4a339073@embeddedor.com \
--to=gustavo@embeddedor.com \
--cc=desroches.maxime@gmail.com \
--cc=devel@driverdev.osuosl.org \
--cc=gregkhl@linuxfoundation.org \
--cc=linux-kernel@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox
all inboxes | Powered by JetHome®