All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-ide@vger.kernel.org
Subject: Re: [PATCH 3/4] ata_piix: Turn on hotplugging support for older chips
Date: Wed, 20 May 2009 16:12:19 -0400	[thread overview]
Message-ID: <4A146423.4090801@garzik.org> (raw)
In-Reply-To: <20090506160947.24191.32143.stgit@t61.ukuu.org.uk>

Alan Cox wrote:
> From: Alan Cox <alan@linux.intel.com>
> 
> We can't do this for the later ones as they have all sorts of magic boot
> time stuff that needs reviewing and the like. However we can do it for the
> older ones and it turns out we need to as some IBM docking stations have a
> second PIIX series device in them and without this change you can't use it
> very well
> 
> Signed-off-by: Alan Cox <alan@linux.intel.com>
> ---
> 
>  drivers/ata/ata_piix.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)
> 
> 
> diff --git a/drivers/ata/ata_piix.c b/drivers/ata/ata_piix.c
> index d51a17c..3434468 100644
> --- a/drivers/ata/ata_piix.c
> +++ b/drivers/ata/ata_piix.c
> @@ -1500,8 +1500,8 @@ static int __devinit piix_init_one(struct pci_dev *pdev,
>  		dev_printk(KERN_DEBUG, &pdev->dev,
>  			   "version " DRV_VERSION "\n");
>  
> -	/* no hotplugging support (FIXME) */
> -	if (!in_module_init)
> +	/* no hotplugging support for later devices (FIXME) */
> +	if (!in_module_init && ent->driver_data >= ich5_sata)
>  		return -ENODEV;

applied



  reply	other threads:[~2009-05-20 20:12 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-05-06 16:08 [PATCH 0/4] Series short description Alan Cox
2009-05-06 16:08 ` [PATCH 1/4] ata_piix: ICH7 does not support correct MWDMA timings Alan Cox
2009-05-06 16:09 ` [PATCH 2/4] ata_piix: The Sony TZ90 needs the cable type hardcoding Alan Cox
2009-05-11 18:33   ` Jeff Garzik
2009-05-06 16:10 ` [PATCH 3/4] ata_piix: Turn on hotplugging support for older chips Alan Cox
2009-05-20 20:12   ` Jeff Garzik [this message]
2009-05-06 16:10 ` [PATCH 4/4] ata_piix: Remove stale comment Alan Cox
2009-05-20 20:12   ` Jeff Garzik
2009-05-11 17:21 ` [PATCH 0/4] Series short description Jeff Garzik
2009-05-11 17:25   ` Alan Cox
2009-05-11 17:33     ` Jeff Garzik
2009-05-11 17:58       ` Alan Cox

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=4A146423.4090801@garzik.org \
    --to=jeff@garzik.org \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=linux-ide@vger.kernel.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.