[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <8A42379416420646B9BFAC9682273B6D01FB8036@limkexm3.ad.analog.com>
Date: Wed, 4 Jun 2008 18:20:52 +0100
From: "Hennerich, Michael" <Michael.Hennerich@...log.com>
To: "David Woodhouse" <dwmw2@...radead.org>,
"Bryan Wu" <cooloney@...nel.org>
Cc: "Hennerich, Michael" <Michael.Hennerich@...log.com>,
<linux-mtd@...ts.infradead.org>, <linux-kernel@...r.kernel.org>,
<mike@...roidmicros.com>
Subject: RE: [PATCH 1/1] mtd m25p80: fix bug - ATmel spi flash fails to becopied to
David,
Bryan sent an incremental update patch, some time ago.
See below ...
Regards,
Michael
From: Michael Hennerich <michael.hennerich@...log.com>
remove duplicate code.
Signed-off-by: Michael Hennerich <michael.hennerich@...log.com>
Signed-off-by: Bryan Wu <cooloney@...nel.org>
---
drivers/mtd/devices/m25p80.c | 2 --
1 files changed, 0 insertions(+), 2 deletions(-)
diff --git a/drivers/mtd/devices/m25p80.c b/drivers/mtd/devices/m25p80.c
index b10649b..7f89b54 100644
--- a/drivers/mtd/devices/m25p80.c
+++ b/drivers/mtd/devices/m25p80.c
@@ -122,8 +122,6 @@ static int write_sr(struct m25p *flash, u8 val)
flash->command[0] = OPCODE_WRSR;
flash->command[1] = val;
- spi_write(flash->spi, flash->command, 2);
-
return spi_write(flash->spi, flash->command, 2); }
--
1.5.5
>-----Original Message-----
>From: David Woodhouse [mailto:dwmw2@...radead.org]
>Sent: Mittwoch, 4. Juni 2008 18:59
>To: Bryan Wu
>Cc: Hennerich, Michael; linux-mtd@...ts.infradead.org; linux-
>kernel@...r.kernel.org; mike@...roidmicros.com
>Subject: Re: [PATCH 1/1] mtd m25p80: fix bug - ATmel spi flash fails to
>becopied to
>
>On Mon, 2008-05-19 at 17:53 +0800, Bryan Wu wrote:
>>
>> >
>> > I just noticed that write_sr contains a duplicated spi_write.
>> > Bryan sends out a new patch.
>> >
>>
>>
>> No problem, it is my fault. I should review the patch carefully.
>
>Was there a new patch coming?
>
>--
>dwmw2
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists