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
next 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.