All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeremie Samuel <jeremie.samuel.ext@parrot.com>
To: "linux-mmc@vger.kernel.org" <linux-mmc@vger.kernel.org>
Subject: Control drive strength in software
Date: Mon, 4 Feb 2013 14:08:28 +0100	[thread overview]
Message-ID: <510FB2CC.20201@parrot.com> (raw)

Hi,

I'm working on a SDHCI driver for linux kernel 3.4.

  The IP that I have to control does not control the Driver Strength. 
Driver Type B is always selected. On the other hand, I can control the 
drive strength of the pads in software.

I would like to use the interface select_drive_strength of the 
mmc_host_ops. Is it a good idea?

If so, I have an other issue.

This function doesn't have an mmc_host as parameter. It is an issue for 
me since the drive strength parameters can be board specific.
A solution would be to add this parameter:

diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
index 5674504..cb6d117 100644
--- a/include/linux/mmc/host.h
+++ b/include/linux/mmc/host.h
@@ -134,7 +134,7 @@ struct mmc_host_ops {
         /* The tuning command opcode value is different for SD and eMMC 
cards */
         int     (*execute_tuning)(struct mmc_host *host, u32 opcode);
         void    (*enable_preset_value)(struct mmc_host *host, bool enable);
-       int     (*select_drive_strength)(unsigned int max_dtr, int 
host_drv, int card_drv);
+       int     (*select_drive_strength)(struct mmc_host *host, unsigned 
int max_dtr, int host_drv, int card_drv);
         void    (*hw_reset)(struct mmc_host *host);
  };

Is this solution a good idea?

Also, the drive strength configuration might be different according to 
the mode (not only UHS). So, I would like to control the drive strength 
for the default speed and the high speed. Would it be relevant to add 
this a call to this function in mmc_sd_switch_hs or in mmc_sd_init_card?

Thanks,

Jeremie Samuel

             reply	other threads:[~2013-02-04 13:10 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2013-02-04 13:08 Jeremie Samuel [this message]
2013-02-11 17:55 ` Control drive strength in software Chris Ball

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=510FB2CC.20201@parrot.com \
    --to=jeremie.samuel.ext@parrot.com \
    --cc=linux-mmc@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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.