Commit ce0c1bc13556fbf1bdfa2a4a27ca6744e7beb32a

Authored by Łukasz Majewski
Committed by Minkyu Kang
1 parent d4ea072ca6

mmc:sdhci:fix: Change default interrupts enabled at SDHCI initialization

This patch changes sdhci_init()'s behavior to NOT enable all interrupt
sources by default. Moreover interrupt signaling has been disabled.

This patch do not enable interrupts which aren't served in u-boot
(they are defined at sdhci.h but NOT used elsewhere):
- SDHCI_INT_CARD_INSERT, SDHCI_INT_CARD_REMOVE, SDHCI_BUS_POWER,
  SDHCI_INT_CARD_REMOVE, SDHCI_INT_CARD_INT

Special care shall be put on SDHCI_INT_CARD_INT, which indicates
interrupt generated by SD card.
According to "SD Host Controller Simplified Spec. ver 3.00" when bit 8
(Card Interrupt Status Enable) at "Normal Interrupt Status Enable
Register" (offset 0x34) is set, the card interrupt detection is started.
Then eMMC card may cause the SD controller to set this bit and then this
interrupt is passed to booted OS and might cause kernel crash.

To sum up:
- Only enable interrupts, which are served at u-boot
- This cleanup as a side effect fixes SDHCI's CARD INTERRUPT problem at
  Linux kernel (versions 3.6+, sdhci controller)
- Keep masked bits at "Normal Interrupt Signal Enable Register" (0x38h)

Signed-off-by: Lukasz Majewski <l.majewski@samsung.com>
Signed-off-by: Kyungmin Park <kyungmin.park@samsung.com>
Cc: Lei Wen <leiwen@marvell.com>
Cc: Andy Fleming <afleming@freescale.com>
Acked-by: Jaehoon Chung <jh80.chung@samsung.com>
Signed-off-by: Minkyu Kang <mk7.kang@samsung.com>

Showing 1 changed file with 5 additions and 3 deletions Side-by-side Diff

... ... @@ -412,9 +412,11 @@
412 412 status = sdhci_readl(host, SDHCI_PRESENT_STATE);
413 413 }
414 414  
415   - /* Eable all state */
416   - sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_INT_ENABLE);
417   - sdhci_writel(host, SDHCI_INT_ALL_MASK, SDHCI_SIGNAL_ENABLE);
  415 + /* Enable only interrupts served by the SD controller */
  416 + sdhci_writel(host, SDHCI_INT_DATA_MASK | SDHCI_INT_CMD_MASK
  417 + , SDHCI_INT_ENABLE);
  418 + /* Mask all sdhci interrupt sources */
  419 + sdhci_writel(host, 0x0, SDHCI_SIGNAL_ENABLE);
418 420  
419 421 return 0;
420 422 }