[U-Boot] [PATCH 1/5] Make RBTREE selectable by Kconfig
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de --- lib/Kconfig | 7 +++++++ 1 file changed, 7 insertions(+)
diff --git a/lib/Kconfig b/lib/Kconfig index 3c8de86..fc2be56 100644 --- a/lib/Kconfig +++ b/lib/Kconfig @@ -47,6 +47,13 @@ config REGEX config LIB_RAND bool
+config RBTREE + bool "Red-black Trees (rbtree)" + help + Red-black trees are a type of self-balancing binary search + tree, used for storing sortable key/value data pairs. + This is currently used in u-boot for ubi and ubifs. + source lib/rsa/Kconfig
menu "Hashing Support"
Users who want to use MTD_PARTITIONS can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de --- drivers/mtd/Kconfig | 11 ++++++++++- 1 file changed, 10 insertions(+), 1 deletion(-)
diff --git a/drivers/mtd/Kconfig b/drivers/mtd/Kconfig index 59278d1..787b1cd 100644 --- a/drivers/mtd/Kconfig +++ b/drivers/mtd/Kconfig @@ -1,3 +1,12 @@ -source "drivers/mtd/nand/Kconfig" +menu "Memory Technology Device (MTD) support" + +config MTD_PARTITIONS + bool "mtd partitions support" + help + Adds the MTD partitioning infrastructure from the Linux + kernel. Needed for UBI support.
+endmenu + +source "drivers/mtd/nand/Kconfig" source "drivers/mtd/spi/Kconfig"
On Wed, Jun 10, 2015 at 10:41:00AM +0200, Lars Poeschel wrote:
Users who want to use MTD_PARTITIONS can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
On Wed, Jun 10, 2015 at 10:41:00AM +0200, Lars Poeschel wrote:
Users who want to use MTD_PARTITIONS can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
For the record, not applied to u-boot/master, thanks!
Users who want to use ubi can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de --- common/Kconfig | 10 ++++++++++ 1 file changed, 10 insertions(+)
diff --git a/common/Kconfig b/common/Kconfig index f6478fa..bc2836a 100644 --- a/common/Kconfig +++ b/common/Kconfig @@ -310,6 +310,16 @@ config CMD_FPGA help FPGA support.
+config CMD_UBI + bool "ubi" + select RBTREE + depends on MTD_PARTITIONS + help + UBI is a software layer above MTD layer which admits of LVM-like + logical volumes on top of MTD devices, hides some complexities of + flash chips like wear and bad blocks and provides some other useful + capabilities. Please, consult the MTD web site for more details + (www.linux-mtd.infradead.org). endmenu
On Wed, Jun 10, 2015 at 10:41:01AM +0200, Lars Poeschel wrote:
Users who want to use ubi can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
On Wed, Jun 10, 2015 at 10:41:01AM +0200, Lars Poeschel wrote:
Users who want to use ubi can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
For the record, not applied to u-boot/master, thanks!
Users who want to use lzo decompression can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de --- lib/Kconfig | 6 ++++++ 1 file changed, 6 insertions(+)
diff --git a/lib/Kconfig b/lib/Kconfig index fc2be56..0fe4932 100644 --- a/lib/Kconfig +++ b/lib/Kconfig @@ -54,6 +54,12 @@ config RBTREE tree, used for storing sortable key/value data pairs. This is currently used in u-boot for ubi and ubifs.
+config LZO + bool "lzo decompression support" + help + This adds a library for lzo decrompression. + This is currently used in u-boot for ubifs. + source lib/rsa/Kconfig
menu "Hashing Support"
On Wed, Jun 10, 2015 at 10:41:02AM +0200, Lars Poeschel wrote:
Users who want to use lzo decompression can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
On Wed, Jun 10, 2015 at 10:41:02AM +0200, Lars Poeschel wrote:
Users who want to use lzo decompression can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
For the record, not applied to u-boot/master, thanks!
Users who want to use ubifs can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de --- fs/ubifs/Kconfig | 8 ++++++++ 1 file changed, 8 insertions(+)
diff --git a/fs/ubifs/Kconfig b/fs/ubifs/Kconfig index e69de29..c5f9714 100644 --- a/fs/ubifs/Kconfig +++ b/fs/ubifs/Kconfig @@ -0,0 +1,8 @@ +config CMD_UBIFS + bool "ubifs" + depends on CMD_UBI + select LZO + help + UBIFS is a file system for flash devices which works on top of UBI. + Please, consult the MTD web site for more details + (www.linux-mtd.infradead.org)
On Wed, Jun 10, 2015 at 10:41:03AM +0200, Lars Poeschel wrote:
Users who want to use ubifs can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
On Wed, Jun 10, 2015 at 10:41:03AM +0200, Lars Poeschel wrote:
Users who want to use ubifs can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
For the record, not applied to u-boot/master, thanks!
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Also, you need to use the tools/moveconfig.py tool to update the headers and defconfigs. So...
NAK
Thanks, -Joe
On Wed, Jun 10, 2015 at 08:49:28AM -0500, Joe Hershberger wrote:
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month. This was RFC will you resend this as a "real" PATCH ? Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS has to depend on CMD_UBI.
Thanks, Lars
Hi Lars,
On Wed, Jun 10, 2015 at 10:08 AM, Lars Poeschel poeschel@lemonage.de wrote:
On Wed, Jun 10, 2015 at 08:49:28AM -0500, Joe Hershberger wrote:
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS has to depend on CMD_UBI.
Will do.
Cheers, -Joe
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
Regards, Lars
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
Cheers, -Joe
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
-Joe
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
-Joe
On Tue, Aug 18, 2015 at 01:47:20PM -0500, Joe Hershberger wrote:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
> I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
Note that since it's part of the E1000 series I gave it to you in patchwork :)
On Tue, Aug 18, 2015 at 2:27 PM, Tom Rini trini@konsulko.com wrote:
On Tue, Aug 18, 2015 at 01:47:20PM -0500, Joe Hershberger wrote:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
>> I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html > > Well, ... ok. You won ;-) Your patchset is by far more comprehensive than > mine. > If I see this right in the archives, you did not receive any comments > since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
> This was RFC will you resend this as a "real" PATCH ?
Yes.
> Could you then include two of the dependencies from my patches: > I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
> has to depend on CMD_UBI.
Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
Note that since it's part of the E1000 series I gave it to you in patchwork :)
Sure, I expect to take it in once all feedback is addressed on that series. :)
-Joe
Am Dienstag, 18. August 2015, 14:32:13 schrieb Joe Hershberger:
On Tue, Aug 18, 2015 at 2:27 PM, Tom Rini trini@konsulko.com wrote:
On Tue, Aug 18, 2015 at 01:47:20PM -0500, Joe Hershberger wrote:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de
wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote: > >> I beat you to it: > >> http://lists.denx.de/pipermail/u-boot/2015-May/214261.html%3E%3E >>>>
> > Well, ... ok. You won ;-) Your patchset is by far more > > comprehensive than > > mine. > > If I see this right in the archives, you did not receive any > > comments > > since nearly a month. > > That is true. I really would like a little feedback on it... or > maybe > I can just drop the controversial one and only include these > things... > they seem to be mostly straightforward. > > > This was RFC will you resend this as a "real" PATCH ? > > Yes. > > > Could you then include two of the dependencies from my patches: > > I think CMD_UBI has to depend on MTD_PARTITIONS and obivously > > CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
> > has to depend on CMD_UBI. > > Will do.
I saw no action from you on this one. Can I help out - at least for the decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
I think the setexpr / env location parts from your patchset can then go in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
Note that since it's part of the E1000 series I gave it to you in patchwork :)
Sure, I expect to take it in once all feedback is addressed on that series. :)
Joe, a gentle ping on this. I don't see RBTREE, MTD_PARTIONS and UBIFS in Kconfig yet.
Regards, Lars
Hi Lars,
On Tue, Dec 15, 2015 at 8:44 AM, Lars Poeschel poeschel@lemonage.de wrote:
Am Dienstag, 18. August 2015, 14:32:13 schrieb Joe Hershberger:
On Tue, Aug 18, 2015 at 2:27 PM, Tom Rini trini@konsulko.com wrote:
On Tue, Aug 18, 2015 at 01:47:20PM -0500, Joe Hershberger wrote:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de
wrote:
> Hi Joe, > > On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote: >> >> I beat you to it: >> >> http://lists.denx.de/pipermail/u-boot/2015-May/214261.html%3E%3E >>>>
>> > Well, ... ok. You won ;-) Your patchset is by far more >> > comprehensive than >> > mine. >> > If I see this right in the archives, you did not receive any >> > comments >> > since nearly a month. >> >> That is true. I really would like a little feedback on it... or >> maybe >> I can just drop the controversial one and only include these >> things... >> they seem to be mostly straightforward. >> >> > This was RFC will you resend this as a "real" PATCH ? >> >> Yes. >> >> > Could you then include two of the dependencies from my patches: >> > I think CMD_UBI has to depend on MTD_PARTITIONS and obivously >> > CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
>> > has to depend on CMD_UBI. >> >> Will do. > > I saw no action from you on this one. Can I help out - at least for > the > decompression / ubi / ubifs part ?
Sorry about that... the end of last release got a little crazy.
> I think the setexpr / env location parts from your patchset can then > go > in as seperate patches.
I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
Note that since it's part of the E1000 series I gave it to you in patchwork :)
Sure, I expect to take it in once all feedback is addressed on that series. :)
Joe, a gentle ping on this. I don't see RBTREE, MTD_PARTIONS and UBIFS in Kconfig yet.
Argh! I forgot that I had this sitting in a branch that still had a few targets not building cleanly. This moving configs stuff is a pain when so many boards use them and in odd / slightly different ways.
Now all the work has to be regenerated and reanalyzed for build failures. Maybe I'll just start with one dependency... get that in, then focus on the next, etc. There are so many boards that conflict readily if you have to spend any amount of time resolving issues.
Sorry for the delay. -Joe
Am Dienstag, 15. Dezember 2015, 16:54:45 schrieb Joe Hershberger:
Hi Lars,
On Tue, Dec 15, 2015 at 8:44 AM, Lars Poeschel poeschel@lemonage.de wrote:
Am Dienstag, 18. August 2015, 14:32:13 schrieb Joe Hershberger:
On Tue, Aug 18, 2015 at 2:27 PM, Tom Rini trini@konsulko.com wrote:
On Tue, Aug 18, 2015 at 01:47:20PM -0500, Joe Hershberger wrote:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger
joe.hershberger@gmail.com wrote: > Hi Lars, > > On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel > poeschel@lemonage.de
wrote:
>> Hi Joe, >> >> On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote: >>> >> I beat you to it: >>> >> http://lists.denx.de/pipermail/u-boot/2015-May/214261.html%3E%3E >>> >> >>>> >>> > >>> > Well, ... ok. You won ;-) Your patchset is by far more >>> > comprehensive than >>> > mine. >>> > If I see this right in the archives, you did not receive any >>> > comments >>> > since nearly a month. >>> >>> That is true. I really would like a little feedback on it... or >>> maybe >>> I can just drop the controversial one and only include these >>> things... >>> they seem to be mostly straightforward. >>> >>> > This was RFC will you resend this as a "real" PATCH ? >>> >>> Yes. >>> >>> > Could you then include two of the dependencies from my patches: >>> > I think CMD_UBI has to depend on MTD_PARTITIONS and obivously >>> > CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
>>> > has to depend on CMD_UBI. >>> >>> Will do. >> >> I saw no action from you on this one. Can I help out - at least >> for >> the >> decompression / ubi / ubifs part ? > > Sorry about that... the end of last release got a little crazy. > >> I think the setexpr / env location parts from your patchset can >> then >> go >> in as seperate patches. > > I agree. I'll split them apart and send these out.
I haven't forgotten about this... I plan to get this sent out this week.
I had this ready to go, but conflicts with https://patchwork.ozlabs.org/patch/508134/, so I'm gonna have to regenerate the moves after that goes in.
Note that since it's part of the E1000 series I gave it to you in patchwork :)
Sure, I expect to take it in once all feedback is addressed on that series.
:)
Joe, a gentle ping on this. I don't see RBTREE, MTD_PARTIONS and UBIFS in Kconfig yet.
Argh! I forgot that I had this sitting in a branch that still had a few targets not building cleanly. This moving configs stuff is a pain when so many boards use them and in odd / slightly different ways.
Now all the work has to be regenerated and reanalyzed for build failures. Maybe I'll just start with one dependency... get that in, then focus on the next, etc. There are so many boards that conflict readily if you have to spend any amount of time resolving issues.
I had another idea: Is it reasonable to put the parameters into Kconfig and only change the non-problematic board configs and leave the rest as they are ? That shouldn't harm ? But I think you are right. It is propably the best if you do it step by step and bring one patch after the other in.
Regards, Lars
Am Dienstag, 18. August 2015, 13:47:20 schrieb Joe Hershberger:
Hi Lars,
On Tue, Aug 11, 2015 at 2:29 PM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 11:01 AM, Joe Hershberger
joe.hershberger@gmail.com wrote:
Hi Lars,
On Tue, Jul 28, 2015 at 3:25 AM, Lars Poeschel poeschel@lemonage.de
wrote:
Hi Joe,
On Wed, Jun 10, 2015 at 11:03:59AM -0500, Joe Hershberger wrote:
> I beat you to it: > http://lists.denx.de/pipermail/u-boot/2015-May/214261.html%3E%3E%3E%3E > Well, ... ok. You won ;-) Your patchset is by far more comprehensive than mine. If I see this right in the archives, you did not receive any comments since nearly a month.
That is true. I really would like a little feedback on it... or maybe I can just drop the controversial one and only include these things... they seem to be mostly straightforward.
This was RFC will you resend this as a "real" PATCH ?
Yes.
Could you then include two of the dependencies from my patches: I think CMD_UBI has to depend on MTD_PARTITIONS and obivously CMD_UBIFS
I noticed when adding MTD_PARTITIONS that MTD_DEVICE is not moved over yet. Do you know what the dependency relationship is there? If not I'll dig into it a bit.
Sorry, I am really not sure about this. I think in my configuration MTD_PARTITIONS worked even without MTD_DEVICE. But maybe looking into drivers/mtd/Makefile:8 helps here. It seems that MTD_PARTITIONS needs either MTD_DEVICE, CMD_NAND, CMD_ONENAND or CMD_SF.
Lars
On Wed, Jun 10, 2015 at 08:49:28AM -0500, Joe Hershberger wrote:
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Also, you need to use the tools/moveconfig.py tool to update the headers and defconfigs. So...
NAK
Oh blarg, I missed this part somehow, dang it.. So now what?
On Fri, Jun 19, 2015 at 3:33 PM, Tom Rini trini@konsulko.com wrote:
On Wed, Jun 10, 2015 at 08:49:28AM -0500, Joe Hershberger wrote:
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Also, you need to use the tools/moveconfig.py tool to update the headers and defconfigs. So...
NAK
Oh blarg, I missed this part somehow, dang it.. So now what?
If you haven't pushed it yet you can back it out, right?
-Joe
On Fri, Jun 19, 2015 at 03:39:44PM -0500, Joe Hershberger wrote:
On Fri, Jun 19, 2015 at 3:33 PM, Tom Rini trini@konsulko.com wrote:
On Wed, Jun 10, 2015 at 08:49:28AM -0500, Joe Hershberger wrote:
Hi Lars,
On Wed, Jun 10, 2015 at 3:40 AM, Lars Poeschel poeschel@lemonage.de wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
I beat you to it: http://lists.denx.de/pipermail/u-boot/2015-May/214261.html
Also, you need to use the tools/moveconfig.py tool to update the headers and defconfigs. So...
NAK
Oh blarg, I missed this part somehow, dang it.. So now what?
If you haven't pushed it yet you can back it out, right?
Yay for getting side-tracked. Dropping this series on the floor.
On Wed, Jun 10, 2015 at 10:40:59AM +0200, Lars Poeschel wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
Hey Tom,
On Fri, Jun 19, 2015 at 3:24 PM, Tom Rini trini@konsulko.com wrote:
On Wed, Jun 10, 2015 at 10:40:59AM +0200, Lars Poeschel wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
Applied to u-boot/master, thanks!
Please note the discussion on this. Did you mean to apply it?
Thanks, -Joe
On Wed, Jun 10, 2015 at 10:40:59AM +0200, Lars Poeschel wrote:
Users who want to use RBTREE can now select it by Kconfig. Selecting it by board config include is still possible.
Signed-off-by: Lars Poeschel poeschel@lemonage.de
For the record, not applied to u-boot/master, thanks!
participants (3)
-
Joe Hershberger -
Lars Poeschel -
Tom Rini