[oe] [PATCH][mtd-utils][how to] Numonyx chip support

[Michal Stawinski]

Can anyone help me on this, please?
I am really into doing some more work, though I need your help to do this.
I've tried some googling and messing around with bb files, but I am more of
a C coder, and scripting/tools are not really my thing. Any help would be
apprecieated.

Maybe it is wrong mailing-list- should I be writing somewhere else?

TIA, Michal Stawinski

2010/2/25 Michał Stawinski <nazgee at gmail.com>

> Hi all,
> I am a newb in working with open projects, so please take it easy if I'll
> do something wrong.
>
> Below (and in the attachment) is "git format-patch" output.
> It introduces support for Numonyx memories, that have oobsize=64 and
> pagesize=4096, that are used e.g. in OMAP3, igep0020b machine.
>
> Solution is very straightforward, though you'll probably reject my patch,
> as diff contains full path to files that i altered (not relative to
> mtd-utils). I hope however, that you can tell me how to generate patches for
> sepcific packages correctly, so it won't contain whole path (e.g.
> a/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c)
>
> That is how I've done it:
> - "git pull"      //to update oe dev branch
> - bitbake mtd-utils      //to get *.c files refreshed
> - git checkout -b mtd_utils      //to create my new development branch
> - git add
> build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c
> //to add files I am going to alter
> - git commit     //to create a starting point for my development
> - :::editing=>compiling=>testing:::
> - git commit     //to check in changes I've made
> - git format-patch HEAD^     //to create patch from changes I've made
>
> What am I doing wrong? Sources of packages like mitd-utils are not added to
> .git repository in OE tree, and when I add it manually to it (as above), and
> then create patch, whole path relative to OR root is generated, which makes
> my patch useles to other builds than the one I did (IGEPv2).
>
> I really'd like to contribute, but I can't find any how-to realted to *.c
> sources editing (*.bb editing is straightforward, as these files are already
> in oe git tree).
>
> //==================================================
> ---
>  .../mtd-utils-1.3.1-r0/git/nanddump.c              |    1 +
>  .../mtd-utils-1.3.1-r0/git/nandwrite.c             |    1 +
>  2 files changed, 2 insertions(+), 0 deletions(-)
>
> diff --git
> a/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c
> b/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c
> index 6926c66..dfc15de 100644
> ---
> a/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c
> +++
> b/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nanddump.c
> @@ -210,6 +210,7 @@ int main(int argc, char * const argv[])
>
>      /* Make sure device page sizes are valid */
>      if (!(meminfo.oobsize == 128 && meminfo.writesize == 4096) &&
> +            !(meminfo.oobsize == 64 && meminfo.writesize == 4096) &&
>              !(meminfo.oobsize == 64 && meminfo.writesize == 2048) &&
>              !(meminfo.oobsize == 32 && meminfo.writesize == 1024) &&
>              !(meminfo.oobsize == 16 && meminfo.writesize == 512) &&
> diff --git
> a/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nandwrite.c
> b/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nandwrite.c
> index b77edd6..bd45879 100644
> ---
> a/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nandwrite.c
> +++
> b/build/tmp-igep0020/work/armv7a-angstrom-linux-gnueabi/mtd-utils-1.3.1-r0/git/nandwrite.c
> @@ -294,6 +294,7 @@ int main(int argc, char * const argv[])
>      if (!(meminfo.oobsize == 16 && meminfo.writesize == 512) &&
>              !(meminfo.oobsize == 8 && meminfo.writesize == 256) &&
>              !(meminfo.oobsize == 64 && meminfo.writesize == 2048) &&
> +            !(meminfo.oobsize == 64 && meminfo.writesize == 4096) &&
>              !(meminfo.oobsize == 128 && meminfo.writesize == 4096)) {
>          fprintf(stderr, "Unknown flash (not normal NAND)\n");
>          close(fd);
> --
> 1.6.3.3
>