Skip to contentSkip to conversation

ubi-utils: Moving mkpfi, pfiflash, ... to old-tools

DISCUSSION Frank Haverkamp 4 participants 12 messages Feb 18, 2008 – Feb 20, 2008 Memory technology devices (MTD)

lore
Share & tools
lore
#0 lore raw

Hi Artem,

I saw that you moved the old ubi-toolchain to the old-tools directory with the comment "sort-me-out" in your git-commits.

We need the tool-chain to generate PFI files for the code-update process. Those pfi-files are suited to be used in streaming mode together with the pfiflash utility. We use the for NOR and NAND flash system the same pfi. Therefor the pfi format does not contain flash geometry information e.g. vid-hdr-offset, erase-block size.

I, and maybe some others on the CC list, would like to see those tools moved from the "sort-me-out" section again into the ubi-tools directory. Please let us know, what you think, needs to be done to get this done.

Thanks,

Frank

Signature · 4 lines
IBM Deutschland Entwicklung GmbH, Schoenaicher Str. 220, 71032
Boeblingen, Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher, Sitz der Gesellschaft: Böblingen,
Registergericht: Amtsgericht Stuttgart, HRB 243294
#1 lore raw
Quoted 5 lines · Frank Haverkamp

We need the tool-chain to generate PFI files for the code-update process. Those pfi-files are suited to be used in streaming mode together with the pfiflash utility. We use the for NOR and NAND flash system the same pfi. Therefor the pfi format does not contain flash geometry information e.g. vid-hdr-offset, erase-block size.

Yeah. I tried to use the tools but they did not generate correct UBI images. I tried to fix this and found out that the internals are too messy to be maintainable, at least by me.

I cleaned-up what I could, made the utilities consistent, small, and easy to maintain.

Vs. the PFI stuff, I also tried to clean-up this, but gave up after few days, and just implemented another utility - ubinize.

I know what is PFI and I decided not to do this, because pfi = configuration ini file + ubi images. I figured that it does not make much sense to inroduce this additional magic format.

Signature · 2 lines
Best regards,
Artem Bityutskiy (Битюцкий Артём)
#2 lore raw
In reply to Frank Haverkamp
Quoted 11 lines · Frank Haverkamp

Hi Artem,

I saw that you moved the old ubi-toolchain to the old-tools directory with the comment "sort-me-out" in your git-commits.

We need the tool-chain to generate PFI files for the code-update process. Those pfi-files are suited to be used in streaming mode together with the pfiflash utility. We use the for NOR and NAND flash system the same pfi. Therefor the pfi format does not contain flash geometry information e.g. vid-hdr-offset, erase-block size.

Quoted 3 lines

I, and maybe some others on the CC list, would like to see those tools moved from the "sort-me-out" section again into the ubi-tools directory. Please let us know, what you think, needs to be done to get this done.

I really do not understand why this move was done anyway. We can move stuff into retirement when we have a full replacement there.

Just shuffling functional tools around is useless and makes the life of developers, users and package maintainers harder for no good reason.

I did not update for some time, but did now and noticed that my packaging scripts need a real overhaul for no benefit.

Yours grumpy,

tglx

#3 lore raw

Hi Thomas,

Quoted 3 lines · Thomas Gleixner

I did not update for some time, but did now and noticed that my packaging scripts need a real overhaul for no benefit.

I talked to Artem. He told me that the tools had deficiencies that prevented him using them. E.g. that he could not put the vid-hdr the offset he wanted it to put. He promissed me to write up the deficiencies so that we can work on them. I am sure that after fixing, we can move the tools back to their old location.

Of course it would be nice, if we could move the tools back even before the fixup, to avoid changing our build scripts. Let us discuss that with Artem tomorrow.

Regards,

Frank

Signature · 4 lines
IBM Deutschland Entwicklung GmbH, Schoenaicher Str. 220, 71032
Boeblingen, Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher, Sitz der Gesellschaft: Böblingen,
Registergericht: Amtsgericht Stuttgart, HRB 243294
#4 lore raw
Quoted 7 lines · On Mon, 18 Feb 2008 17:49:03 +0100 Frank Haverkamp <[email protected]> wrote:

Hi Thomas,

Quoted 3 lines · Thomas Gleixner

I did not update for some time, but did now and noticed that my packaging scripts need a real overhaul for no benefit.

Quoted 5 lines

I talked to Artem. He told me that the tools had deficiencies that prevented him using them. E.g. that he could not put the vid-hdr the offset he wanted it to put. He promissed me to write up the deficiencies so that we can work on them. I am sure that after fixing, we can move the tools back to their old location.

I have a patch for the vid header thing. I also have a patch for bin2nand/nand2bin since they are hard coded to use the SmartMedia ECC byte ordering.

Quoted 3 lines

Of course it would be nice, if we could move the tools back even before the fixup, to avoid changing our build scripts. Let us discuss that with Artem tomorrow.

I'm a bit grumpy about the move as well, not because they were moved, but because there's a dozen or so completely useless commits in the repo. But, oh well.

josh

#5 lore raw

Hi Josh,

Quoted 3 lines · Josh Boyer

I also have a patch for
bin2nand/nand2bin since they are hard coded to use the SmartMedia ECC byte ordering.

I added fixes to be more flexible in the fashion the tools place the OOB-ECC bytes already some time ago. Please have a look if those changes already do what you had in mind here.

Yesterday I sent a patch to move the location of bin2nand and nand2bin since those tools are not UBI related. If David accepts the change, you will find them at a different spot (but anyhow, you were on CC list for that patch).

Regards,

Frank

Signature · 4 lines
IBM Deutschland Entwicklung GmbH, Schoenaicher Str. 220, 71032
Boeblingen, Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher, Sitz der Gesellschaft: Böblingen,
Registergericht: Amtsgericht Stuttgart, HRB 243294
#6 lore raw
Quoted 10 lines · On Tue, 19 Feb 2008 09:47:36 +0100 Frank Haverkamp <[email protected]> wrote:

Hi Josh,

Quoted 3 lines · Josh Boyer

I also have a patch for
bin2nand/nand2bin since they are hard coded to use the SmartMedia ECC byte ordering.

I added fixes to be more flexible in the fashion the tools place the OOB-ECC bytes already some time ago. Please have a look if those changes already do what you had in mind here.

No, they don't. That was a major part of what I needed (so thank you!), but ecc layout isn't the same as ecc byte ordering. I'll post a patch soonish.

Quoted 4 lines

Yesterday I sent a patch to move the location of bin2nand and nand2bin since those tools are not UBI related. If David accepts the change, you will find them at a different spot (but anyhow, you were on CC list for that patch).

Yes I saw that. We don't need to wait for David. I'll look it over today.

josh

#7 lore raw

Hi Josh,

Quoted 4 lines · Josh Boyer

No, they don't. That was a major part of what I needed (so thank you!),
but ecc layout isn't the same as ecc byte ordering. I'll post a patch soonish.

I already thought that ;-). Fine nice that we will get a new feature into the tool.

Frank

Signature · 4 lines
IBM Deutschland Entwicklung GmbH, Schoenaicher Str. 220, 71032
Boeblingen, Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher, Sitz der Gesellschaft: Böblingen,
Registergericht: Amtsgericht Stuttgart, HRB 243294
#8 lore raw
In reply to Frank Haverkamp
Quoted 9 lines · Frank Haverkamp

Hi Thomas,

Quoted 3 lines · Thomas Gleixner

I did not update for some time, but did now and noticed that my packaging scripts need a real overhaul for no benefit.

I talked to Artem. He told me that the tools had deficiencies that prevented him using them.

I really give a sh*t whether Artem can use them or not. He is not the center of the universe. If he needs them changed, then he should send a patch or a suitable replacement for the tool, which does not force others to fixup their scripts and build tools for no benefit. This is not a sandbox, this are tools used in production environments.

Artem, please stop this nonsense right now. You can create your own world if you have the urge to do so, but please keep stuff others use intact. This "I don't like that old crap" attitude is an arrogant and at the same time ignorant childish behaviour, which has no place in a serious project.

Quoted 4 lines

E.g. that he could not put the vid-hdr the
offset he wanted it to put. He promissed me to write up the deficiencies so that we can work on them. I am sure that after fixing, we can move the tools back to their old location.

No, they go back to their place immediately, period.

Yours even more grumpy,

tglx

#9 lore raw
Quoted 12 lines · Thomas Gleixner

I really give a sh*t whether Artem can use them or not. He is not the center of the universe. If he needs them changed, then he should send a patch or a suitable replacement for the tool, which does not force others to fixup their scripts and build tools for no benefit. This is not a sandbox, this are tools used in production environments.

Artem, please stop this nonsense right now. You can create your own world if you have the urge to do so, but please keep stuff others use intact. This "I don't like that old crap" attitude is an arrogant and at the same time ignorant childish behaviour, which has no place in a serious project.

Quoted 8 lines
Quoted 4 lines

E.g. that he could not put the vid-hdr the
offset he wanted it to put. He promissed me to write up the deficiencies so that we can work on them. I am sure that after fixing, we can move the tools back to their old location.

No, they go back to their place immediately, period.

Yours even more grumpy,

I apologize for this. Indeed that was not nice and arrogant. The tools are returned to their previous place.

Signature · 2 lines
Best regards,
Artem Bityutskiy (Битюцкий Артём)
#10 lore raw
In reply to Frank Haverkamp
Quoted 9 lines · Frank Haverkamp

I talked to Artem. He told me that the tools had deficiencies that prevented him using them. E.g. that he could not put the vid-hdr the offset he wanted it to put. He promissed me to write up the deficiencies so that we can work on them. I am sure that after fixing, we can move the tools back to their old location.

Of course it would be nice, if we could move the tools back even before the fixup, to avoid changing our build scripts. Let us discuss that with Artem tomorrow.

I can return the tools to the state they were before I touched them, is it OK for you?

Signature · 2 lines
Best regards,
Artem Bityutskiy (Битюцкий Артём)
#11 lore raw

Hi Artem,

Quoted 3 lines · Artem Bityutskiy

I can return the tools to the state they were before I touched them, is
it OK for you?

that would be very nice of you. And if you could still create the list of deficiencies e.g. that the modification of the vid-hdr-offset is missing it would be also nice.

Thanks,

Frank

Signature · 4 lines
IBM Deutschland Entwicklung GmbH, Schoenaicher Str. 220, 71032
Boeblingen, Vorsitzender des Aufsichtsrats: Martin Jetter
Geschäftsführung: Herbert Kircher, Sitz der Gesellschaft: Böblingen,
Registergericht: Amtsgericht Stuttgart, HRB 243294