public inbox for linux-mmc@vger.kernel.org
 help / color / mirror / Atom feed
From: Kukjin Kim <kgene.kim@samsung.com>
To: linux-arm-kernel@lists.infradead.org,
	linux-samsung-soc@vger.kernel.org, linux-mmc@vger.kernel.org
Cc: ben-linux@fluff.org,
	'Marek Szyprowski' <m.szyprowski@samsung.com>,
	'Andrew Morton' <akpm@linux-foundation.org>
Subject: 'error: 'SDHCI_QUIRK_NO_HISPD_BIT' undeclared' and 'undefined reference to `sdhci_card_detect''
Date: Fri, 13 Aug 2010 16:32:07 +0900	[thread overview]
Message-ID: <008d01cb3ab9$ae73da60$0b5b8f20$%kim@samsung.com> (raw)

Hi, 

This is just information about Samsung sdmmc stuff building error now.

I found Marek's 'sdhci-s3c: enable SDHCI_QUIRK_NO_HISPD_BIT quirk' in the
Linus' tree.
(commit ID: a1d5646005af1247d6ae78434bb4db15b07a07b2)

But not defined the quirk yet...so following build error happened with
s3c6400_defconfig in Linus' latest.

drivers/mmc/host/sdhci-s3c.c: In function 'sdhci_s3c_probe':
drivers/mmc/host/sdhci-s3c.c:400: error: 'SDHCI_QUIRK_NO_HISPD_BIT'
undeclared (first use in this function)
drivers/mmc/host/sdhci-s3c.c:400: error: (Each undeclared identifier is
reported only once
drivers/mmc/host/sdhci-s3c.c:400: error: for each function it appears in.)
make[4]: *** [drivers/mmc/host/sdhci-s3c.o] Error 1
make[3]: *** [drivers/mmc/host] Error 2
make[2]: *** [drivers/mmc] Error 2
make[1]: *** [drivers] Error 2

Kyungmin Park's below patch can solve this and it is in mmotm now.
(commit ID: 2935b9e7fcc4bea3751b8d039b383b2036a7d36d)

But I think, to update quirk definition should being in Marek's patch for
avoiding build error.
Of course, I'm not sure whether the commit order changed.
Anyway, in this case, will be solved after merging mm tree.


And second case is same.

Marek's 'sdhci-s3c: add support for new card detection methods' cause
following build error.
(commit ID:17866e14f3a4f219e94f1374ece7226479418ff8)

drivers/built-in.o: In function `sdhci_s3c_notify_change':
/home/kgene/linux/linux-2.6-mainline-dev/drivers/mmc/host/sdhci-s3c.c:255:
undefined reference to `sdhci_card_detect'
make[1]: *** [.tmp_vmlinux1] Error 1
make: *** [sub-make] Error 2

And Andrew's patch(b567e5dd5a34c184e5642100e752cb87e064bb97) can solve this.
(of course this needs another patches...)

Anyway...
Marek, in future please make sure your patch has no building problem before
submitting.
(or it can help to add some kind of dependency note in your patch)

Thanks.

Best regards,
Kgene.
--
Kukjin Kim <kgene.kim@samsung.com>, Senior Engineer,
SW Solution Development Team, Samsung Electronics Co., Ltd.

             reply	other threads:[~2010-08-13  7:32 UTC|newest]

Thread overview: 6+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-08-13  7:32 Kukjin Kim [this message]
2010-08-13  8:13 ` 'error: 'SDHCI_QUIRK_NO_HISPD_BIT' undeclared' and 'undefined reference to `sdhci_card_detect'' Marek Szyprowski
2010-08-17  4:26   ` Kukjin Kim
2010-08-18 18:33   ` Andrew Morton
2010-08-19  1:10     ` Kyungmin Park
2010-08-19  1:59       ` Kyungmin Park

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='008d01cb3ab9$ae73da60$0b5b8f20$%kim@samsung.com' \
    --to=kgene.kim@samsung.com \
    --cc=akpm@linux-foundation.org \
    --cc=ben-linux@fluff.org \
    --cc=linux-arm-kernel@lists.infradead.org \
    --cc=linux-mmc@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=m.szyprowski@samsung.com \
    /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