Re: ata_piix on MacBook4,1: fail to startup in a certain situation

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



At Wed, 28 May 2008 23:23:48 +0900,
Tejun Heo wrote:
> 
> > $ sudo /sbin/lspci -nnvvv
> > 00:1f.2 IDE interface [0101]: Intel Corporation 82801HBM/HEM (ICH8M/ICH8M-E) SATA IDE Controller [8086:2828] (rev 03) (prog-if 8f [Master SecP SecO PriP PriO])
> > 	Subsystem: Apple Computer Inc. Unknown device [106b:00a1]
> > 	Control: I/O+ Mem+ BusMaster+ SpecCycle- MemWINV- VGASnoop- ParErr- Stepping- SERR- FastB2B-
> > 	Status: Cap+ 66MHz+ UDF- FastB2B+ ParErr- DEVSEL=medium >TAbort- <TAbort- <MAbort- >SERR- <PERR-
> > 	Latency: 0
> > 	Interrupt: pin B routed to IRQ 18
> > 	Region 0: I/O ports at 60f8 [size=8]
> > 	Region 1: I/O ports at 611c [size=4]
> > 	Region 2: I/O ports at 60f0 [size=8]
> > 	Region 3: I/O ports at 6118 [size=4]
> > 	Region 4: I/O ports at 6020 [size=16]
> > 	Region 5: I/O ports at 4000 [size=16]
> > 	Capabilities: [70] Power Management version 3
> > 		Flags: PMEClk- DSI- D1- D2- AuxCurrent=0mA PME(D0-,D1-,D2-,D3hot+,D3cold-)
> > 		Status: D0 PME-Enable- DSel=0 DScale=0 PME-
> 
> Hmmm... What is apple doing with ich8m?  :-(
> 
> Does the attached patch fix the problem?

Unfortunately no...
With this patch, booting will stop during the execution of
the following lines (in ata_piix.c: piix_init_sidpr()):

        if (pci_resource_start(pdev, PIIX_SIDPR_BAR) == 0 ||
            pci_resource_len(pdev, PIIX_SIDPR_BAR) != PIIX_SIDPR_LEN)
                return;

which might suggest this is not ata_piix related problem I'm afraid...

Again, please note if I keep the AC adapter plugged to MacBook,
there is no such problem happening. It happens only when running with battery.

> -- 
> tejun
> [2 ata_piix-apple.patch <text/x-patch (7bit)>]
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index a9027b8..4e90d94 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -251,6 +251,7 @@ static const struct pci_device_id piix_pci_tbl[] = {
>  	{ 0x8086, 0x2828, PCI_ANY_ID, PCI_ANY_ID, 0, 0, ich8_sata },
>  	/* Mobile SATA Controller IDE (ICH8M), Apple */
>  	{ 0x8086, 0x2828, 0x106b, 0x00a0, 0, 0, ich8m_apple_sata },
> +	{ 0x8086, 0x2828, 0x106b, 0x00a1, 0, 0, ich8m_apple_sata },
>  	/* SATA Controller IDE (ICH9) */
>  	{ 0x8086, 0x2920, PCI_ANY_ID, PCI_ANY_ID, 0, 0, ich8_sata },
>  	/* SATA Controller IDE (ICH9) */

--
Kohji
--
To unsubscribe from this list: send the line "unsubscribe linux-ide" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html

[Index of Archives]     [Linux Filesystems]     [Linux SCSI]     [Linux RAID]     [Git]     [Kernel Newbies]     [Linux Newbie]     [Security]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Samba]     [Device Mapper]

  Powered by Linux