linux-ide.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Jeff Garzik <jeff@garzik.org>
To: Yinghai Lu <yinghai@kernel.org>
Cc: Peer Chen <pchen@nvidia.com>, Kuan Luo <kluo@nvidia.com>,
	Christoph Hellwig <hch@infradead.org>,
	linux-kernel@vger.kernel.org, linux-ide@vger.kernel.org,
	Ingo Molnar <mingo@elte.hu>,
	Andrew Morton <akpm@linux-foundation.org>
Subject: Re: [PATCH] sata_nv: sgpio for nvidia mcp55 -v3
Date: Mon, 13 Apr 2009 04:18:39 -0400	[thread overview]
Message-ID: <49E2F55F.5060108@garzik.org> (raw)
In-Reply-To: <4977DAAF.5060006@kernel.org>

Yinghai Lu wrote:
> Impact: new features
> 
> based on patch on
>     http://marc.info/?l=linux-ide&m=116289338705418&w=2
> 
> 1. update the patch for 2.6.19 to latest upstream ( 2.6.28?)
> 2. fix shared sgpio to support several mcp55 + io55, so every mcp55 have
>    seperate spinlock
> 3. use scratch_source as numbering of sgpio instead of address of struct,
>    so could go through kexec/kdump
> 
> v2: revert NV_ON and NV_OFF, so turn on Activity LED all the time when disk is idle.
> v3: use one timer per sgpio instead of per nv_host, so for mcp55+io55 system will use 2 timers
>     instead of 6 timers.
> 
> Signed-off-by: Yinghai Lu <yinghai@kernel.org>
> 
> ---
>  drivers/ata/sata_nv.c |  562 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 562 insertions(+)


Is this really over 500 LOC, just to blink some lights from the data 
xfer hot path?  :)

	Jeff




  parent reply	other threads:[~2009-04-13  8:18 UTC|newest]

Thread overview: 25+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <15F501D1A78BD343BE8F4D8DB854566B059FE0B3@hkemmail01.nvidia.com>
2006-10-31  8:43 ` [PATCH] SCSI: Add the SGPIO support for sata_nv.c Peer Chen
2006-10-31 10:40   ` Christoph Hellwig
2006-10-31 12:37     ` Prakash Punnoor
2006-11-07  9:55     ` Peer Chen
2007-11-07  4:09       ` Yinghai Lu
2009-01-18  9:20         ` Yinghai Lu
2009-01-18  9:22           ` [PATCH] sata_nv: sgpio for nvidia mcp55 Yinghai Lu
2009-01-21  2:07             ` [PATCH] sata_nv: sgpio for nvidia mcp55 -v2 Yinghai Lu
2009-01-21 14:30               ` Yan Seiner
2009-01-21 18:48                 ` Yinghai Lu
2009-01-22  4:46                   ` Yan Seiner
2009-01-22  5:07                     ` Yinghai Lu
2009-01-22 16:22                       ` Yan Seiner
2009-01-22 17:26                         ` Yinghai Lu
2009-01-23  1:44                           ` Yan Seiner
2009-01-23  1:58                             ` Yinghai Lu
2009-01-23  3:18                               ` Yan Seiner
2009-01-23  7:30                                 ` Yinghai Lu
2009-01-22  2:32               ` [PATCH] sata_nv: sgpio for nvidia mcp55 -v3 Yinghai Lu
2009-01-27  4:57                 ` Andrew Morton
2009-04-13  8:18                 ` Jeff Garzik [this message]
2009-04-13 15:50                   ` Yinghai Lu
2009-01-18 13:49           ` [PATCH] SCSI: Add the SGPIO support for sata_nv.c Yan Seiner
2009-01-20  7:36             ` Peer Chen
2006-11-01  1:39   ` Jeff Garzik

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=49E2F55F.5060108@garzik.org \
    --to=jeff@garzik.org \
    --cc=akpm@linux-foundation.org \
    --cc=hch@infradead.org \
    --cc=kluo@nvidia.com \
    --cc=linux-ide@vger.kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=mingo@elte.hu \
    --cc=pchen@nvidia.com \
    --cc=yinghai@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 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).