From: augulis.darius@gmail.com (Darius Augulis)
To: linux-arm-kernel@lists.infradead.org
Subject: S3C2416 fixes (gpiolib, SDHCI, compilation without FB device)
Date: Thu, 08 Jul 2010 20:08:35 +0300 [thread overview]
Message-ID: <4C360613.1000506@gmail.com> (raw)
In-Reply-To: <000201cb1e6e$8b3f7a20$a1be6e60$%szyprowski@samsung.com>
On 07/08/2010 10:23 AM, Marek Szyprowski wrote:
> Hello,
>
> On Tuesday, July 06, 2010 7:03 PM Darius Augulis wrote:
>
>> On 06/28/2010 09:01 AM, Marek Szyprowski wrote:
>>> Hello,
>>>
>>> On Friday, June 25, 2010 12:56 PM Darius Augulis wrote:
>>>
>>>> On 06/22/2010 04:16 PM, Yauhen Kharuzhy wrote:
>>>>> This patch series fixes SDHCI on S3C2416. I got SD card working with
>> this
>>>> patches
>>>>> and with patches for GPIO card detection support by Marek Szyprowski.
>>>>>
>>>>
>>>> Unfortunately I did not get SD card working with these patches. I'm
>>>> using custom board, not SMDK2416, but SD slot connection and schematics
>>>> are the same like in SMDK2416. I've tried to debug a bit this problem
>>>> and this is what I got: kernel do detection of card insert/remove,
>>>> driver prints messages:
>>>>
>>>> s3c-sdhci s3c-sdhci.1: card inserted.
>>>> s3c-sdhci s3c-sdhci.1: card removed.
>>>>
>>>> Then I put some printk in s3c-sdhci driver and MMC core and found that
>>>> mmc_rescan() is called only once at startup and no more when
>>>> inserting/removing SD card. Also mmc_rescan() stops at that point where
>>>> mmc_power_up() is called and does not return. And mmc_power_up() stops
>>>> where mmc_delay(10) is called and does not return too.
>>>> Every time when inserting/removing SD card only functions
>>>> sdhci_tasklet_card() and mmc_detect_change() are called, and nothing
>>>> happens. Kernel doesn't detect the card inserted.
>>>> Maybe this info could help to debug where the problem is - in my custom
>>>> board or in your patches?
>>>
>>> Have you tried with polled card detection method (SDHCI_CD_NONE)? If it
>>> doesn't work I suspect something with the hardware. Checking with mmc
>> debug
>>> enabled might also help figuring this issue out.
>>
>> yes, I've tried that, but didn't get sd card working. I got schematics
>> from China manufacturer, SD card connection is the same like in
>> smdk2416. with mmc debug enabled I found that SD card is detected when
>> inserted or removed, but after detection card initialisation is not
>> performed.
>
> Does it work without my patches or with other Linux kernel? You can try to
> check the custom Linux 2.6.29 from
> git://git.kernel.org/pub/scm/linux/kernel/git/kki_ap/linux-2.6-samsung.git
I did try with and without your patches. First, I applied only s3c2416
realated patches from Yauhen, then I've added your patches, but nothing
changed.
>
> I would like to know whether my patches cause the problem or there is some
> kind of hardware issue with your board.
>
> Best regards
> --
> Marek Szyprowski
> Samsung Poland R&D Center
>
next prev parent reply other threads:[~2010-07-08 17:08 UTC|newest]
Thread overview: 19+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-06-22 13:16 S3C2416 fixes (gpiolib, SDHCI, compilation without FB device) Yauhen Kharuzhy
2010-06-22 13:16 ` [PATCH 1/4] ARM: S3C24XX: Fix gpiolib support for ports K..M Yauhen Kharuzhy
2010-06-22 13:16 ` [PATCH 2/4] ARM: S3C2416: Set name of the framebuffer device only if it present Yauhen Kharuzhy
2010-06-22 13:16 ` [PATCH 3/4] ARM: S3C2416: Add clock definition for HSMMC1 Yauhen Kharuzhy
2010-06-22 13:16 ` [PATCH 4/4] ARM: S3C2416: Add platform helpers for setup SDHCI Yauhen Kharuzhy
2010-06-22 13:19 ` Maurus Cuelenaere
2010-06-22 14:05 ` Yauhen Kharuzhy
2010-06-22 13:45 ` Darius Augulis
2010-06-22 14:19 ` Yauhen Kharuzhy
2010-06-22 13:40 ` [PATCH 1/4] ARM: S3C24XX: Fix gpiolib support for ports K..M Vasily Khoruzhick
2010-06-25 10:55 ` S3C2416 fixes (gpiolib, SDHCI, compilation without FB device) Darius Augulis
2010-06-28 6:01 ` Marek Szyprowski
2010-07-06 17:02 ` Darius Augulis
2010-07-08 7:23 ` Marek Szyprowski
2010-07-08 17:08 ` Darius Augulis [this message]
2010-10-18 12:30 ` Kukjin Kim
2010-10-18 13:00 ` Darius Augulis
2010-10-18 13:04 ` Kukjin Kim
2010-10-18 13:11 ` Darius Augulis
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=4C360613.1000506@gmail.com \
--to=augulis.darius@gmail.com \
--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).