From: Rupjyoti Sarmah on
Hi Stephen,

Thanks for your suggestions.
I used git-send-email earlier and emails never appeared in the archive.
This time I sent through outlook & the message appeared in the archive.
Looks like outlook did that word wrap.

I am not sure why the emails from the git-send-email did not get
delivered. At the time of sending I got message delivered logs.

Regards,
Rupjyoti Sarmah




-----Original Message-----
From: Stephen Rothwell [mailto:sfr(a)canb.auug.org.au]
Sent: Wednesday, March 17, 2010 10:30 AM
To: Rupjyoti Sarmah
Cc: linux-ide(a)vger.kernel.org; Jeff Garzik; Rupjyoti Sarmah;
linuxppc-dev(a)ozlabs.org; Rupjyoti Sarmah; linux-kernel(a)vger.kernel.org
Subject: Re: [PATCH] 460EX on-chip SATA driver<kernel 2.6.33> <
resubmission : 01>

Hi,

Just a couple of quick points ...

On Tue, 16 Mar 2010 21:23:16 -0700 "Rupjyoti Sarmah" <rsarmah(a)amcc.com>
wrote:
>
> This patch enables the on-chip DWC SATA controller of the AppliedMicro

> processor 460EX.
>
> Signed-off-by: Rupjyoti Sarmah <rsarmah(a)appliedmicro.com> ,Mark
> Miesfeld <mmiesfeld(a)appliedmicro.com>, Prodyut Hazarika
> <phazarika(a)appliedmicro.com>

Please put one Signed-off-by line per person.

> ---
> arch/powerpc/boot/dts/canyonlands.dts | 8 +
> drivers/ata/Kconfig | 9 +
> drivers/ata/Makefile | 1 +
> drivers/ata/sata_dwc.c | 1965
> +++++++++++++++++++++++++++++++++
> 4 files changed, 1983 insertions(+), 0 deletions(-) create mode
> 100644 drivers/ata/sata_dwc.c

This patch has been word wrapped by your mailer (or something along the
way).
--
Cheers,
Stephen Rothwell sfr(a)canb.auug.org.au
http://www.canb.auug.org.au/~sfr/
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
From: Stefan Roese on
Hi Rup,

On Wednesday 17 March 2010 05:23:16 Rupjyoti Sarmah wrote:
> This patch enables the on-chip DWC SATA controller of the AppliedMicro
> processor 460EX.

Apart from the other comments, did you take a look at the latest version of
this driver in our linux-2.6-denx repository? We did some work on this driver
for a new Canyonlands release last year. It's hard to check this for me now,
with this line wrapped patch (the drivers diverged somewhat). So either check
for yourself, or send a new, not line-wrapped, patch version so that I can
better check myself.

BTW: It would be nice if you would add me to Cc on DWC-SATA related patches,
since I've been involved in the development at some time.

Thanks.

Cheers,
Stefan

--
DENX Software Engineering GmbH, MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-0 Fax: (+49)-8142-66989-80 Email: office(a)denx.de
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo(a)vger.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/