[U-Boot] Please pull u-boot-ubi

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

The following changes since commit a350d0d37d7c113afbd9b0d2cf60ac542007f9e6:
  Wolfgang Denk (1):
        Merge branch 'master' of /home/wd/git/u-boot/custodians

are available in the git repository at:

  git://www.denx.de/git/u-boot-ubi.git master

Simon Kagstrom (1):
      Handle VID header offset in ubi part command

 common/cmd_ubi.c |   21 ++++++++++++++++-----
 1 files changed, 16 insertions(+), 5 deletions(-)

[U-Boot] Please pull u-boot-ubi

[Simon Kagstrom]

On Thu, 9 Jul 2009 09:59:56 +0200
Stefan Roese <sr@denx.de> wrote:

> The following changes since commit a350d0d37d7c113afbd9b0d2cf60ac542007f9e6:
>   Wolfgang Denk (1):
>         Merge branch 'master' of /home/wd/git/u-boot/custodians
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-ubi.git master
> 
> Simon Kagstrom (1):
>       Handle VID header offset in ubi part command

Thanks for applying this! However, I think the "[PATCH 8/8]: Command
improvements for ubifs" patch should also be uncontroversial:

  http://lists.denx.de/pipermail/u-boot/2009-July/055602.html

It's basically just a check for the number of arguments passed and a
description of what the command does.


To use ubifs on ARM, I also need the rest of the patch series

   http://lists.denx.de/pipermail/u-boot/2009-July/055594.html

but I haven't seen any comments on that so far.

// Simon

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

On Thursday 09 July 2009 10:16:11 Simon Kagstrom wrote:
> > The following changes since commit
> > a350d0d37d7c113afbd9b0d2cf60ac542007f9e6: Wolfgang Denk (1):
> >         Merge branch 'master' of /home/wd/git/u-boot/custodians
> >
> > are available in the git repository at:
> >
> >   git://www.denx.de/git/u-boot-ubi.git master
> >
> > Simon Kagstrom (1):
> >       Handle VID header offset in ubi part command
>
> Thanks for applying this! However, I think the "[PATCH 8/8]: Command
> improvements for ubifs" patch should also be uncontroversial:
>
>   http://lists.denx.de/pipermail/u-boot/2009-July/055602.html
>
> It's basically just a check for the number of arguments passed and a
> description of what the command does.

Yes. And I already acked it on the list. Please note that I'm "only" the co-
custodian for the UBI repository. We have no UBIFS custodian. So it needs to 
get into mainline via Wolfgang.

> To use ubifs on ARM, I also need the rest of the patch series
>
>    http://lists.denx.de/pipermail/u-boot/2009-July/055594.html
>
> but I haven't seen any comments on that so far.

No comments doesn't have to be a bad sign. So nobody has something to complain 
about them. In fact I think your patches are good. So I'll probably send a few 
acked-by's soon.

Thanks.

Best regards,
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 at denx.de
=====================================================================

[U-Boot] Please pull u-boot-ubi

[Simon Kagstrom]

On Thu, 9 Jul 2009 10:24:42 +0200
Stefan Roese <sr@denx.de> wrote:

> > Thanks for applying this! However, I think the "[PATCH 8/8]: Command
> > improvements for ubifs" patch should also be uncontroversial:
> 
> Yes. And I already acked it on the list. Please note that I'm "only" the co-
> custodian for the UBI repository. We have no UBIFS custodian. So it needs to 
> get into mainline via Wolfgang.

Ah, OK! I simply assumed UBI/UBIFS maintenance went together and that
it simply got missed.

> > To use ubifs on ARM, I also need the rest of the patch series
> >
> >    http://lists.denx.de/pipermail/u-boot/2009-July/055594.html
> >
> > but I haven't seen any comments on that so far.
> 
> No comments doesn't have to be a bad sign. So nobody has something to complain 
> about them. In fact I think your patches are good. So I'll probably send a few 
> acked-by's soon.

Well my worry with the series is that it touches pretty much all
architectures, which I haven't been able to test. So that is basically
where I'd like some feedback.

// Simon

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

On Thursday 09 July 2009 10:32:02 Simon Kagstrom wrote:
> > > To use ubifs on ARM, I also need the rest of the patch series
> > >
> > >    http://lists.denx.de/pipermail/u-boot/2009-July/055594.html
> > >
> > > but I haven't seen any comments on that so far.
> >
> > No comments doesn't have to be a bad sign. So nobody has something to
> > complain about them. In fact I think your patches are good. So I'll
> > probably send a few acked-by's soon.
>
> Well my worry with the series is that it touches pretty much all
> architectures, which I haven't been able to test. So that is basically
> where I'd like some feedback.

Unfortunately we don't have many boards in mainline U-Boot yet that include 
UBI/UBIFS till now. So problems in this area might get unnoticed right now.

Did you at least compile-test your patchset on some of the other architectures 
(PPC, MIPS...)?

Thanks.

Best regards,
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 at denx.de
=====================================================================

[U-Boot] Please pull u-boot-ubi

[Simon Kagstrom]

On Thu, 9 Jul 2009 10:36:58 +0200
Stefan Roese <sr@denx.de> wrote:

> Unfortunately we don't have many boards in mainline U-Boot yet that include 
> UBI/UBIFS till now. So problems in this area might get unnoticed right now.
> 
> Did you at least compile-test your patchset on some of the other architectures 
> (PPC, MIPS...)?

Well, the only other cross-compiler I had around is for powerpc. I
built for the MPC8349ITX board, and that compiled just fine. But then
again PPC already had ffs/fls.

// Simon

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

On Thursday 09 July 2009 10:49:47 Simon Kagstrom wrote:
> > Did you at least compile-test your patchset on some of the other
> > architectures (PPC, MIPS...)?
>
> Well, the only other cross-compiler I had around is for powerpc. I
> built for the MPC8349ITX board, and that compiled just fine. But then
> again PPC already had ffs/fls.

Well, that's a good sign then. You didn't break the PPC implementation with 
your patch. ;)

Best regards,
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 at denx.de
=====================================================================

[U-Boot] Please pull u-boot-ubi

[Wolfgang Denk]

Dear Stefan Roese,

In message <200907091024.42759.sr@denx.de> you wrote:
>
> Yes. And I already acked it on the list. Please note that I'm "only" the co-
> custodian for the UBI repository. We have no UBIFS custodian. So it needs to 
> get into mainline via Wolfgang.

I see UBIFS and UBI as one topic. Please run it through your UBI
repo. Thanks.


Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Experience is what causes a person to make new  mistakes  instead  of
old ones.

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

On Thursday 09 July 2009 12:05:31 Wolfgang Denk wrote:
> > Yes. And I already acked it on the list. Please note that I'm "only" the
> > co- custodian for the UBI repository. We have no UBIFS custodian. So it
> > needs to get into mainline via Wolfgang.
>
> I see UBIFS and UBI as one topic. Please run it through your UBI
> repo. Thanks.

So I'm promoted from UBI co-custodian to UBIFS custodian? ;)

OK, I'll try to take care of UBIFS related patches for now. Will send an 
updated ubi/ubifs pull request soon.

Thanks.

Best regards,
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 at denx.de
=====================================================================

[U-Boot] Please pull u-boot-ubi

[Wolfgang Denk]

Dear Stefan Roese,

In message <200909231605.39337.sr@denx.de> you wrote:
> The following changes since commit 3b6a9267f0de7b85d387fa4123d0b58379363447:
>   Wolfgang Denk (1):
>         board/flagadm/flash.c: fix compile warning
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-ubi.git master
> 
> Simon Kagstrom (1):
>       ubifs: Add support for looking up directory and relative symlinks
> 
>  fs/ubifs/ubifs.c |   68 ++++++++++++++++++++++++++++++++++-------------------
>  1 files changed, 43 insertions(+), 25 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
You're dead, Jim.
	-- McCoy, "The Tholian Web", stardate unknown

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

The following changes since commit 3b6a9267f0de7b85d387fa4123d0b58379363447:
  Wolfgang Denk (1):
        board/flagadm/flash.c: fix compile warning

are available in the git repository at:

  git://www.denx.de/git/u-boot-ubi.git master

Simon Kagstrom (1):
      ubifs: Add support for looking up directory and relative symlinks

 fs/ubifs/ubifs.c |   68 ++++++++++++++++++++++++++++++++++-------------------
 1 files changed, 43 insertions(+), 25 deletions(-)

[U-Boot] Please pull u-boot-ubi

[Wolfgang Denk]

Dear Stefan Roese,

In message <200907200927.39207.sr@denx.de> you wrote:
> The following changes since commit b86b85e2611d57d834795a92453431a1a340c3c9:
>   Ilya Yanok (1):
>         mmc: set bus width to 1 and clock to minimum early during initialization
> 
> are available in the git repository at:
> 
>   git://www.denx.de/git/u-boot-ubi.git master
> 
> Andrzej Wolski (1):
>       ubi: help message correction
> 
>  common/cmd_ubi.c |    4 ++--
>  1 files changed, 2 insertions(+), 2 deletions(-)

Applied, thanks.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
He is truly wise who gains wisdom from another's mishap.

[U-Boot] Please pull u-boot-ubi

[Stefan Roese]

The following changes since commit b86b85e2611d57d834795a92453431a1a340c3c9:
  Ilya Yanok (1):
        mmc: set bus width to 1 and clock to minimum early during initialization

are available in the git repository at:

  git://www.denx.de/git/u-boot-ubi.git master

Andrzej Wolski (1):
      ubi: help message correction

 common/cmd_ubi.c |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)