Linux SCSI subsystem development
 help / color / mirror / Atom feed
From: "jameshsu" <jameshsu@mail.acard.com>
To: Alan Cox <alan@lxorguk.ukuu.org.uk>
Cc: linux-scsi@vger.kernel.org, James.Bottomley@SteelEye.com,
	EdwardLian <edwardlian@mail.acard.com>,
	Jason Wu <jason_wu@acard.com>
Subject: Re: Fw: ACARD SCSI driver update for Linux kernel v2.6
Date: Fri, 7 Jan 2005 18:05:29 +0800	[thread overview]
Message-ID: <035001c4f4a0$6b3b4e50$6200a8c0@jameshsu> (raw)
In-Reply-To: 1105052325.17166.284.camel@localhost.localdomain

Hi Alan,

We are in the process to modify the driver in order to meet your suggests.
Following change has been made.
1) Put it back for originator's copyrights to the file.
2) Go back to build up/modify driver from kernel 2.6.10. according to James
Bottomley's suggestions
3) use new header
     <scsi/scsi.h>
    <scsi/scsi_host.h>
    <scsi/scsi_device.h>
    <scsi/scsi_cmnd.h>)
4) enable .max_sectors = 128
5) Either  modify or remove all following declare:
+#if LINUX_VERSION_CODE > KERNEL_VERSION(2,5,0)
+static irqreturn_t atp885_intr_handle(int irq, void *dev_id, struct
pt_regs *regs)
+#else
+static void atp885_intr_handle(int irq, void *dev_id, struct pt_regs
*regs)
+#endif

+static struct Scsi_Host *atp_host[MAX_ADAPTER];
+static int host_index = 0;

6) Reversals of some of improvements like hotplug pci made outside -
Addition of support for the AEC67160Acard
7) more.....
New driver will be submit around 1-2 week later.
During these period, if you have more suggestions, please let us know ASAP
while testing.
Again, Thanks for your help!

James Hsu,
Manager,Engineering Testing Dept.
ACARD TECHNOLOGY CORP.
Taipei Hsien,Taiwan
6F No78,Sec 1,Kwang Fu Road,Sang Chung,
(02)8512-2290 x 2311(O),(02)8512-1524(FAX)


----- Original Message -----
From: "Alan Cox" <alan@lxorguk.ukuu.org.uk>
To: "jameshsu" <jameshsu@mail.acard.com>
Cc: <linux-scsi@vger.kernel.org>
Sent: Friday, January 07, 2005 7:10 AM
Subject: Re: Fw: ACARD SCSI driver update for Linux kernel v2.6


> The trouble with the patch as is, is that it mixes up
>
> - Reversals of improvements like hotplug pci made outside Acard
> - Addition of support for the AEC67160
> - Other unknown improvements Acard may have made
>
> In order for this to be useful we need to be able to see just the
> changes that are important.
>
> I can easily see for example the new PCI identifiers but not the other
> needed changes. In order to do that I really need either a description
> of the relevant changes or those changes versus the existing 2.6 driver
> tree.
>
> Any help welcomed.
>
> Alan
>


  reply	other threads:[~2005-01-07 10:02 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2004-12-30  9:58 Fw: ACARD SCSI driver update for Linux kernel v2.6 jameshsu
2004-12-30 15:20 ` James Bottomley
2005-01-06 23:10 ` Alan Cox
2005-01-07 10:05   ` jameshsu [this message]
2005-01-07 10:11     ` Christoph Hellwig
  -- strict thread matches above, loose matches on Subject: below --
2005-01-14 10:03 jameshsu
2005-01-14 17:44 ` Matthew Wilcox
2005-01-19  2:05   ` jameshsu
2005-03-30 11:48 jameshsu
2005-03-30 12:45 ` 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='035001c4f4a0$6b3b4e50$6200a8c0@jameshsu' \
    --to=jameshsu@mail.acard.com \
    --cc=James.Bottomley@SteelEye.com \
    --cc=alan@lxorguk.ukuu.org.uk \
    --cc=edwardlian@mail.acard.com \
    --cc=jason_wu@acard.com \
    --cc=linux-scsi@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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox