linux-arm-kernel.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: arnd@arndb.de (Arnd Bergmann)
To: linux-arm-kernel@lists.infradead.org
Subject: [PATCH] mmc: dw_mmc: fix dw_mci_get_cd
Date: Thu, 16 Jan 2014 12:12:10 +0100	[thread overview]
Message-ID: <201401161212.11246.arnd@arndb.de> (raw)
In-Reply-To: <20140115162210.GP15937@n2100.arm.linux.org.uk>

On Wednesday 15 January 2014, Russell King - ARM Linux wrote:
> The issue here is that we'll potentially now end up with three
> inversions.  One in the GPIO layers.  One in mmc_gpio_get_cd(), and
> another in every driver which uses the GPIO layer inversion.  This
> is hardly the right approach as it leads to multiple points where
> confusion about the inverted status can occur.

I did not mean to suggest adding a third inversion. We already have
to deal with the one in the GPIO layer and the one in mmc_gpio_get_cd(),
which I absolutely agree is unfortunate. As was pointed out already,
the gpio driver used in this system does not support the inversion
in the DT binding, so the only proper solution is to use the
inversion from mmc_gpio_get_cd if MMC_CAP2_CD_ACTIVE_HIGH is set
in host->caps2. This flag gets set from mmc_of_parse based
on the presence of the "cd-inverted" flag, in the absence of the
gpio flags. The dw_mmc driver does not use mmc_of_parse() at the
moment, and while it probably should use that in the future,
it should definitely read the same DT properties with the
same semantics already and not introduce new properties.

	Arnd

  parent reply	other threads:[~2014-01-16 11:12 UTC|newest]

Thread overview: 47+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-01-09 14:35 [PATCH v7 0/3] mmc: dw_mmc: add dw_mmc-k3 Zhangfei Gao
2014-01-09 14:35 ` [PATCH 1/3] mmc: dw_mmc: use slot-gpio to handle cd pin Zhangfei Gao
2014-01-09 14:38   ` Arnd Bergmann
2014-01-14 15:58   ` Kevin Hilman
2014-01-14 17:23     ` Olof Johansson
2014-01-15  1:09       ` zhangfei
2014-01-15  5:16         ` zhangfei
2014-01-15  7:15     ` [PATCH] mmc: dw_mmc: fix dw_mci_get_cd Zhangfei Gao
2014-01-15  7:31       ` zhangfei
2014-01-15 10:07       ` Zhangfei Gao
2014-01-15 11:58         ` Seungwon Jeon
2014-01-15 12:10           ` zhangfei
2014-01-15 12:26             ` Seungwon Jeon
2014-01-15 13:56               ` zhangfei
2014-01-15 13:59                 ` Arnd Bergmann
2014-01-15 14:20                   ` Arnd Bergmann
2014-01-15 15:36                     ` zhangfei
2014-01-15 16:01                   ` Russell King - ARM Linux
2014-01-15 16:07                     ` Arnd Bergmann
2014-01-15 16:22                       ` Russell King - ARM Linux
2014-01-16  2:10                         ` zhangfei
2014-01-16 11:12                         ` Arnd Bergmann [this message]
2014-01-16 11:25                           ` Russell King - ARM Linux
2014-01-15 14:38                 ` Seungwon Jeon
2014-01-15 15:32                   ` zhangfei
2014-01-15 15:51                     ` Zhangfei Gao
2014-01-15 15:59                       ` Russell King - ARM Linux
2014-01-15 15:54                     ` Arnd Bergmann
2014-01-09 14:35 ` [PATCH 2/3] mmc: dw_mmc: add dw_mmc-k3 for k3 platform Zhangfei Gao
2014-01-09 14:45   ` Arnd Bergmann
2014-01-10 13:39   ` Seungwon Jeon
2014-01-10 14:12     ` zhangfei
2014-01-13  2:09       ` Seungwon Jeon
2014-01-13  2:37         ` zhangfei
2014-01-13  5:32           ` Seungwon Jeon
2014-01-13  8:30             ` zhangfei
2014-01-14  9:38               ` Seungwon Jeon
2014-01-14  9:47                 ` zhangfei
2014-01-09 14:35 ` [PATCH 3/3] clk: hisilicon: add hi3620_mmc_clks Zhangfei Gao
2014-01-09 14:38   ` Arnd Bergmann
2014-01-10  3:41 ` [PATCH v7 0/3] mmc: dw_mmc: add dw_mmc-k3 Jaehoon Chung
2014-01-12 16:35 ` Chris Ball
2014-01-13  1:00   ` Zhangfei Gao
  -- strict thread matches above, loose matches on Subject: below --
2014-01-16  2:31 [PATCH v2] mmc: dw_mmc: fix dw_mci_get_cd Zhangfei Gao
2014-01-16 12:48 ` [PATCH] " Zhangfei Gao
2014-01-16 18:29   ` Kevin Hilman
2014-01-17 12:35     ` Seungwon Jeon
2014-01-17 14:25       ` 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=201401161212.11246.arnd@arndb.de \
    --to=arnd@arndb.de \
    --cc=linux-arm-kernel@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).