Re: [Patch] Make suspend/resume work with ich chipset in force AHCI mode

[Sergei Shtylyov <sshtylyov@mvista.com>]

Hello.

On 16-02-2011 12:07, Joerg Dorchain wrote:

> Hello all,

> this patch in needed in addition to the previous one make
> suspend/resume work in the forced ahci mode.

    This remark should probably follow --- tear line (after the signoff).

> During resume from suspend to ram, the kernel pci layer restores
> the registers for the SATA controller once, then says okay, and
> sets dev->state_saved = false. However, since the restore goes
> from highest address (the BARs [base address registers]) to
> lowest register, some of the higher registers are set as RO
> because according to the lower registers controller is in PIIX
> mode.  This patch introduces a workaround for
> this problem, hacking around the PCI API by setting
> pdev->state_saved = true
> before we do the restore, basically leaving the pci config space
> untouched.

> Bye,

> Joerg

    "Hello" and "bye" shouldn't be the part of the patch description.

> Signed-off-by: joerg Dorchain<joerg@dorchain.net>

    Need space before the email address.

WBR, Sergei