public inbox for linux-mtd@lists.infradead.org
 help / color / mirror / Atom feed
From: Vitaly Wool <vwool@ru.mvista.com>
To: Charles Manning <manningc2@actrix.gen.nz>
Cc: yuri.golovach@mindspeed.com, linux-mtd@lists.infradead.org,
	yaffs@stoneboat.aleph1.co.uk
Subject: Re: [PATCH] treat OOB as a single chunk of oobavail bytes
Date: Sat, 28 Jan 2006 12:19:57 +0300	[thread overview]
Message-ID: <43DB373D.2000906@ru.mvista.com> (raw)
In-Reply-To: <200601271126.23808.manningc2@actrix.gen.nz>

Hi Charles,

no it's not yet in CVS. I think I'll be able to commit it if we get any 
positive feedback on this patch (i. e. if it's actually working for 
anyone else than me :))
As for write_oobfree, it looks to be  useful for JFFSx to stop messing 
with oobinfos in JFFSx code.

Best regards,
   Vitaly

Charles Manning wrote:

>On Friday 27 January 2006 01:51, Vitaly Wool wrote:
>  
>
>>Hi Yuri,
>>
>>well, lemme just summarize what you hafta do in order to make YAFFS2
>>work with my patches.
>>
>>1. Apply the following patch to the mtd code:
>>http://lists.infradead.org/pipermail/linux-mtd/2005-December/014648.html
>>(yes, it's a single patch!)
>>2. Modify the fs/yaffs2/yaffs_mtdif2.c to use read_oobfree/write_oobfree
>>where appropriate.
>>
>>Hope that helps,
>>
>>   Vitaly
>>    
>>
>
>Hi Vitaly
>
>Is this in CVS yet? If not, I hope it will be there soon.
>
>From what I see, this is going to be just what we need.
>
>YAFFS2 does not need to write_oobfree as a seperate function (it is always 
>written as a single page write). YAFFS2 only needs read_oobfree.
>
>However, YAFFS1 could use the write_oobfree function at some future date if 
>YAFFS1 gets moved to this interface to remove the current reliance on hanging 
>the ugly ECC parameter stuff through the interface.
>
>I am pretty busy right now, but will try get the YAFFS2 side sorted ASAP. If 
>the YAFFS2 stuff is CVSed before the mtd then I'll just point people to this 
>patch.
>
>Thanx!
>
>Charles
>
>
>  
>

  reply	other threads:[~2006-01-30  1:55 UTC|newest]

Thread overview: 10+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <OFD870B79D.90A804FE-ON88257101.0037ED08-C2257101.0037DF6B@mindspeed.com>
2006-01-26 12:51 ` [PATCH] treat OOB as a single chunk of oobavail bytes Vitaly Wool
2006-01-26 22:26   ` Charles Manning
2006-01-28  9:19     ` Vitaly Wool [this message]
     [not found] <OF1F9E1072.64E03318-ON88257102.0060A9BE-C2257102.0060ED3A@mindspeed.com>
2006-01-28  9:15 ` Vitaly Wool
2006-01-29 15:19   ` Vitaly Wool
     [not found] <OFFED20C5C.C2FC40DE-ON882570FF.005D6B36-C22570FF.005ED903@mindspeed.com>
2006-01-24 13:02 ` Vitaly Wool
2005-11-29 15:02 Vitaly Wool
2005-11-29 22:12 ` Charles Manning
2005-11-30  8:54   ` Vitaly Wool
2005-12-01 22:03     ` Charles Manning

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=43DB373D.2000906@ru.mvista.com \
    --to=vwool@ru.mvista.com \
    --cc=linux-mtd@lists.infradead.org \
    --cc=manningc2@actrix.gen.nz \
    --cc=yaffs@stoneboat.aleph1.co.uk \
    --cc=yuri.golovach@mindspeed.com \
    /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