> >
> > Line 374 is the "IDSEL 0x16..." line here:
> > interrupt-map = <
> > /* IRQ mapping for pci slots and ALI M1533
> > ...
> > * management core also isn't used.
> >
> From: Stephen Warren
>
> Previously, the #line parsing regex ended with ({WS}+[0-9]+)?. The {WS}
> could match line-break characters. If the #line directive did not contain
> the optional flags field at the end, this could cause any integer data on
> the next line to be consumed as part of the
> This will allow callers to rebuild .dtb files when any of the /include/d
> .dtsi files are modified, not just the top-level .dts file.
>
> Signed-off-by: Stephen Warren
> ---
> This patch is against the Linux kernel's copy of dtc, but it applies to
> upstream dtc with a couple of trivial confli
> On Tue, Nov 16, 2010 at 12:49:51PM -0800, John Bonesio wrote:
> > Pull in recent changes from the main dtc repository. These changes primarily
> > allow multiple device trees to be declared which are merged by dtc. This
> > feature allows us to include a basic dts file and then provide more
> >
>
> Hmm, is there some documentation for how to use this feature?
> Specifically I have a custom board with multiple discrete computers on
> it which are only very slightly physically different from each other
> and I'd like to be able to avoid maintaining 2 nearly-exact copies of
> the same DTS f
>
> For this to make sense, it really needs to also rename irq_host_ops
> structure, the IRQ_HOST_MAP* #defines, and the irq_*_host functions.
> It also /should/ adjust the users of irq_domain to rename function
> local variables and structure members. Otherwise there will be an
> even worse mism
>
> I always reserve the right to change my mind. }:->
Naturally!
> Sorry about that,
No problem. The thought back then was that we'd hit the "global"
piece and then we'd buy the time to hit the local pieces as we could.
> but I actually applied the patch this morning with the thought that
On Thu, 2009-02-19 at 17:21 +0530, Vijay Nikam wrote:
> Also is it possible to compile device tree on Linux host and genreate
> dtb for powerpc ? ? ? If yes, then how ? ? ? please let me know ...
> thanks ...
Uh, get a copy of the DTC using:
$ git clone git://git.jdl.com/software/dtc.git
> On Fri, Mar 27, 2009 at 8:16 AM, Michal Simek wrote:
> > Hi Ben,
> >
> > could you please move DTC to any generic location. I would like to use it f
> or Microblaze cpu too.
>
> Michal,
>
> David Gibson is the one responsible for the in-tree dtc source, not
> Ben, and you should prepare the pa
> On Tue, Jun 28, 2011 at 09:47:11AM -0400, Josh Boyer wrote:
> > The *p variable is declared and used to save inb->ptr, however p is
> > later never used. This has been the case since commit 6c0f3676 and can
> > lead to build failures with -Werror=unused-but-set-variable:
> >
> > flattree.c:
> On Tue, Jun 28, 2011 at 08:47:09AM -0400, Josh Boyer wrote:
> > Commit 376ab6f2 removed the old style check functionality from DTC,
> > however the check option and variable were not removed. This leads to
> > build failures when -Werror=unused-but-set-variable is specified:
> >
> > dtc.c:
So, like, the other day David Gibson mumbled:
> When I released libfdt, I forgot to add a copyright notice to
> dumptrees.c (probably because the program is so trivial). Apparently
> the lack causes trouble for Debian, so this patch adds one. I've gone
> through the git history and double checked
So, like, the other day David Gibson mumbled:
> This patch makes a bunch of updates to the TODO files for dtc and
> libfdt, some of them rather overdue.
Applied.
jdl
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listi
So, like, the other day Timur Tabi mumbled:
> diff --git a/arch/powerpc/platforms/86xx/mpc8610_hpcd.c
> b/arch/powerpc/platforms/86xx/mpc8610_hpc
> d.c
> index 6390895..6e1bde3 100644
> --- a/arch/powerpc/platforms/86xx/mpc8610_hpcd.c
> +++ b/arch/powerpc/platforms/86xx/mpc8610_hpcd.c
> @@ -34,9 +
Folks,
I'd like to release a DTC 1.1 Real Soon Now!
However, I've been back-logged due to the holidays.
If you have any outstanding DTC or libfdt patches
that I have not yet applied and that you would like
in that release, please remind me by reposting it
and CC:'ing [EMAIL PROTECTED]
Thanks!
jd
On Fri, 2007-12-14 at 10:28, Kumar Gala wrote:
> > Are there any tools to autoconvert (preferably without losing
> > comments)?
>
> Not sure of jon had a perl script or something.
No, I don't. It's hard to tell what should conceptually
remain hex, and what might need to be decimalized.
Sure,
So, like, the other day David Gibson mumbled:
>
> I posted this one a while back, but I guess it dropped through the
> cracks.
Yep, sorry.
> There's also the patch I sent you quite a long time ago that
> adds a clarifying readme about the licensing situation - that might be
> an idea for v1.1, t
So, like, the other day Paul Gortmaker mumbled:
> Here is a small patch to clean up the usage info and the error returns
> for ftdump -- not sure what the future holds for ftdump vs. simply using
> "dtc -I dtb -O dts someblob.dtb" ...
>
> Paul.
Paul,
Any chance of a signed-off-by line?
Thanks,
So, like, the other day Scott Wood mumbled:
>
> I forgot to tag them as such, but these are of course dtc patches and
> not Linux patches.
All three applied (to DTC :-)).
Will you follow up with a patch for /bininc/ or /bin-include/
as well now? (I'm fine with either, and I forget where David's
So, like, the other day David Gibson mumbled:
> Ah, I think I sent it by private mail, so it won't be in patchwork.
> Here you are:
Ah, that explains it.
> dtc/libfdt: Add README clarifying licensing
>
> The fact the dtc are distributed together, but have different licenses
> can be a bit confus
So, like, the other day Scott Wood mumbled:
> Also, free file->dir when freeing file.
>
> Signed-off-by: Scott Wood <[EMAIL PROTECTED]>
Applied.
Thanks.
jdl
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linu
So, like, the other day Scott Wood mumbled:
> The \n is provided by yyerror().
>
> Signed-off-by: Scott Wood <[EMAIL PROTECTED]>
Applied.
Thanks,
jdl
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.org
https://ozlabs.org/mailman/listinfo/linuxppc-dev
So, like, the other day David Gibson mumbled:
>
>
> Hrm. Can we leave this one until after 1.1? Since it adds new
> syntax, I think it would be worth cogitating a bit longer.
Double hrm.
Scott, can you remind us why this feature was requested
off the U-Boot list, please? I'm trying to decid
So, like, the other day Scott Wood mumbled:
> On Sun, Jan 06, 2008 at 02:37:16PM +1100, David Gibson wrote:
> > > out:
> > > + free((void *)file->dir);
> >
> > That cast shouldn't be there.
>
> It generates a constness warning otherwise. I was on the fence over
> whether to just remove the cons
Guys,
I have the bulk of a port for the iomega StorCenter that
Andy and I have been working on. We'd like your feedback!
Specifically, it's not quite working yet, and for lack of
actually getting serial output, we're having some difficulty
still. I post it here in a desperate attempt to let som
Based on the Kurobox DTS files.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Ignore the flash bits. They are from the previous
World View and need to be updated still.
arch/powerpc/boot/dts/storcente
Use cuImage bootwrapper until U-Boot port is completed.
Derived heavily from Linkstation port.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Nope, I have NOT verified that the bd_t file that
is used here byte-identical to U-Boot&
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
This is known to have just a ton of crap in the
config still. Not to worry. It'll be trimmed
down some later...
arch/powerpc/configs/storcenter_defconfig | 1538 +++
From: Jon Loeliger <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Paul,
Continue to fight the Good Fight by removing
old include file references that managed to
creep in recently.
These are the last few in arch/powerpc directly.
If you could pick this up f
So, like, the other day "Grant Likely" mumbled:
> > +
> > + ranges = <8000 8000 7000/* pci mem space */
> > + fdf0 fdf0 0010/* EUMB */
> > + fe00 fe00 00c0/* pci i/o space */
> > +
So, like, the other day "Grant Likely" mumbled:
>
> Curious; why in platforms/embedded6xx vs platforms/82xx?
82xx is nominally the _other_ 82xx family. :-)
This is just more historical precedent, and the fact
that it is similar to the linkstation there.
> > obj-$(CONFIG_MPC7448HPC2) += mpc
So, like, the other day "Grant Likely" mumbled:
> On 1/7/08, Jon Loeliger <[EMAIL PROTECTED]> wrote:
> > Specifically, it's not quite working yet, and for lack of
> > actually getting serial output, we're having some difficulty
> > still. I
So, like, the other day Scott Wood mumbled:
> > --- a/arch/powerpc/platforms/embedded6xx/Kconfig
> > +++ b/arch/powerpc/platforms/embedded6xx/Kconfig
> > @@ -16,6 +16,17 @@ config LINKSTATION
> > Linkstation-I HD-HLAN and HD-HGLAN versions, and PPC-based
> > Terastation systems should
So, like, the other day Stephen Rothwell mumbled:
>
> Hi Jon,
>
> > Continue to fight the Good Fight by removing
> > old include file references that managed to
> > creep in recently.
>
> Thanks for doing this.
By $DIETY, I'll obtain Janitorial Karma one way or another!
The question now is, wh
So, like, the other day Olof Johansson mumbled:
>
> > @@ -0,0 +1,90 @@
> > +#include
> > +#include
> > +#include
> > +#include
> > +#include
> > +#include
> > +#include
>
> Stephen will tell you to include linux/of_platform.h instead. :-)
Yeah, watch me outright NAK it too. :-)
jdl
_
Folks,
Here's the second pass at the patches to add the basic
iomega StorCenter port to arch/powerpc. This version
of the patches addresses the comments from the first
round, cleaning up the Kconfig, DTS and board file.
Thanks,
jdl
arch/powerpc/boot/Makefile |3 +-
arc
So, like, the other day Scott Wood mumbled:
> On Tue, Jan 08, 2008 at 04:06:56PM -0600, Jon Loeliger wrote:
> > +/ {
> > + model = "StorCenter";
> > + compatible = "storcenter";
> > + #address-cells = <1>;
> > + #size-cells = <
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Cleaned up config options by starting with ancient
working config and oldconfig'ing it. Ugh and all.
arch/powerpc/configs/storcenter_defconfig | 1174 +
Based on the Kurobox DTS files.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Pulled PCI node out.
Fixed flash node.
arch/powerpc/boot/dts/storcenter.dts | 155 ++
1 files changed, 155 inse
Use cuImage bootwrapper until U-Boot port is completed.
Derived heavily from Linkstation port.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
Fixed Kconfig to select options rather than "depend on" them.
Fleshed out sto
So, like, the other day David Gibson mumbled:
> > The doubly-linked list is intended to make it easier to construct search
> > path lists one-at-a-time from arguments in the proper order, without
> > needing to reverse the list at the end.
>
> We've already got that problem with a bunch of the lis
So, like, the other day Scott Wood mumbled:
> This allows /include/s to work when in non-default states,
> such as PROPNODECHAR.
>
> We may want to use state stacks to get rid of BEGIN_DEFAULT() altogether...
>
> Signed-off-by: Scott Wood <[EMAIL PROTECTED]>
Hmmm.. With this applied, I get the
Folks,
I have tagged DTC v1.1.0-rc1 and pushed it out.
Please verify that this is going to be a viable 1.1 release!
Consider this the last call for features and first call
for bug fixes for the 1.1 release. :-)
Thanks,
jdl
___
Linuxppc-dev mailing lis
So, like, the other day Josh Boyer mumbled:
> On Fri, 11 Jan 2008 07:44:33 -0600
> Jon Loeliger <[EMAIL PROTECTED]> wrote:
>
> > So, like, the other day David Gibson mumbled:
> > > This patch adds an fdt_set_name() function to libfdt, mirroring
> > > fdt_ge
So, like, the other day David Gibson mumbled:
> This patch adds an fdt_set_name() function to libfdt, mirroring
> fdt_get_name(). This is a r/w function which alters the name of a
> given device tree node.
>
> Signed-off-by: David Gibson <[EMAIL PROTECTED]>
Applied.
jdl
___
So, like, the other day Scott Wood mumbled:
> Signed-off-by: Scott Wood <[EMAIL PROTECTED]>
> ---
> srcpos.c |4 ++--
> srcpos.h |2 +-
> 2 files changed, 3 insertions(+), 3 deletions(-)
Oh yeah.
jdl
___
Linuxppc-dev mailing list
Linuxppc-dev@
Folks,
Anyone out there with an 8241 board such as
the IoMega StorCenter who would be willing to
help me debug my arch/powerpc cuImage problems?
I feel like I am missing something very basic in
my port... like, "Did you map this memory right?"
or "Did you set _this_ magic bit?"
I have posted the
So, like, the other day "Grant Likely" mumbled:
>
> Since you're getting absolutely no output after uncompression it
> probably means that you're failing in the boot wrapper...
Or not getting to the boot wrapper...
> which means mmu is off, and the serial port is already set up by u-boot.
>
> Yo
Wade Farnsworth wrote:
> +
> + ranges = <0 0 ff80 0080
> + 1 0 fe00 0100
> + 2 0 f820 0010
> + 3 0 f810 0010>;
> +
I think you want just:
ranges = <0 0 f800 800>
right? And
Kumar Gala wrote:
> I'm assuming he's listed CS1, CS2, and CS3 even though we don't have a
> children yet. (second flash chip, CF, and pixis).
>
> - k
OK -- I just wasn't sure if you wanted the other
CS entries for nodes that weren't actually present yet.
jdl
_
Kumar,
Here is the StorCenter port with the MTD partitions
re-written to be hard-coded PHYSMAP partitions for now.
Please pick up for 2.6.25!
Thanks,
jdl
arch/powerpc/boot/Makefile |3 +-
arch/powerpc/boot/cuboot-824x.c | 53 +
arch/powerpc/boot/dts/
Based on the Kurobox DTS files.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/boot/dts/storcenter.dts | 139 ++
1 files changed, 139 insertions(+), 0 deletions(-)
create mode 100644
Use cuImage bootwrapper until U-Boot port is completed.
Derived heavily from Linkstation port.
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/boot/Makefile |3 +-
arch/powerpc/boot/c
Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/configs/storcenter_defconfig | 1174 +
1 files changed, 1174 insertions(+), 0 deletions(-)
create mode 100644 arch/powerpc/configs/storcent
Grant Likely wrote:
> On 1/22/08, Jon Loeliger <[EMAIL PROTECTED]> wrote:
>> Based on the Kurobox DTS files.
>>
>> Signed-off-by: Andy Wilcox <[EMAIL PROTECTED]>
>> Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
>
> Comments b
Stephen Rothwell wrote:
> Hi Jon,
>
> On Tue, 22 Jan 2008 16:37:59 -0600 Jon Loeliger <[EMAIL PROTECTED]> wrote:
>> +++ b/arch/powerpc/platforms/embedded6xx/storcenter.c
>
>> +extern void _nmask_and_or_msr(unsigned long nmask, unsigned long or_val);
>
> This
So, like, the other day Jon Smirl mumbled:
> Convert MPC i2c driver from a platform_driver to a
> of_platform_driver. Add the ability to dynamically load i2c drivers
> based on device tree names. Routine names were changed from fsl_ to
> mpc_ to make them match the file name. Common code moved to
>
Sam Ravnborg wrote:
>> What impact do the have on boards in arch/ppc being able to build?
>
> Byt the way what are the plans for powerpc?
> We have:
> arch/ppc
> arch/ppc64
> arch/powerpc
>
> It was my understanding that arch/powerpc was supposed
> to be the future unified powerpc architecture bu
Kumar,
Here is the StorCenter port with the MTD partitions
re-written to be hard-coded PHYSMAP partitions for now.
Addressed list and private commentary as well.
Janitorial Karma has now caussed it to be 4 patches
instead of the original 3.
Please pick up for 2.6.25!
Thanks,
jdl
_
Prevents miscellaneous users from declaring it locally.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/platforms/embedded6xx/mpc7448_hpc2.c |2 --
include/asm-powerpc/system.h |2 ++
2 files changed, 2 insertions(+), 2 deletions(-)
diff -
Based on the Kurobox DTS files.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
Acked-by: Andy Wilcox <[EMAIL PROTECTED]>
---
arch/powerpc/boot/dts/storcenter.dts | 138 ++
1 files changed, 138 insertions(+), 0 deletions(-)
create mode 100644 arch/
Use cuImage bootwrapper until U-Boot port is completed.
Derived heavily from Linkstation port.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
Acked-by: Andy Wilcox <[EMAIL PROTECTED]>
---
arch/powerpc/boot/Makefile |3 +-
arch/powerpc/boot/c
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
Acked-by: Andy Wilcox <[EMAIL PROTECTED]>
---
arch/powerpc/configs/storcenter_defconfig | 1174 +
1 files changed, 1174 insertions(+), 0 deletions(-)
create mode 100644 arch/powerpc/configs/storcenter_def
Stephen Rothwell wrote:
> Hi Jon,
>> +
>> +dnp = of_find_node_by_type(NULL, "open-pic");
>> +if (dnp == NULL)
>> +return;
>> +
>> +prop = of_get_property(dnp, "reg", &size);
>> +if (prop == NULL)
>> +return;
>
> You need an of_node_put(dnp) before you retur
Kumar Gala wrote:
> On Jan 23, 2008, at 6:50 PM, Stephen Rothwell wrote:
>
>> You need an of_node_put(dnp) before you return.
>
> Fixed up by your friendly maintainer.
Kumar,
I meant, "I won't be sending you a patch."
Thank you!
jdl
___
Linuxppc-de
Folks,
I have tagged and released a DTC 1.1.0.
You may find it using git here:
git://www.jdl.com/software/dtc
A tarball snap-shot is also available here:
http://www.jdl.com/software/dtc-1.1.0.tgz
Please let me know if there are problems with it!
Thanks,
jdl
__
> Heh. `git-format-patch --stdout -5 | diffstat` doesn't work correctly.
> Once I've already stumbled against this, but obviously forgot.
Might try:
git diff --stat HEAD~4
or such!
Enjoy,
jdl
___
Linuxppc-dev mailing list
Linuxppc-dev@ozlabs.or
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/boot/dts/storcenter.dts | 73 +
1 files changed, 38 insertions(+), 35 deletions(-)
diff --git a/arch/powerpc/boot/dts/storcenter.dts
b/arch/powerpc/boot/dts/storcenter.dts
index 6aa1d69..2
Also fixed a few minor indent problems as well.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
arch/powerpc/boot/dts/mpc8610_hpcd.dts | 227 +++---
arch/powerpc/boot/dts/mpc8641_hpcn.dts | 333
2 files changed, 281 insertions(+
So, like, the other day Josh Boyer mumbled:
>
> There are lots of powerpc sub-trees. Kumar's, Geoff's, mine, Olof's,
> Grant's, Vitaly's are just the ones I can think of the top of my head.
>
> Shouldn't we just ask Paul to sync up more often rather than have
> Andrew track X number of trees tha
On Mon, 2008-01-28 at 08:42, Jon Smirl wrote:
> PowerPC device trees use a different naming convention than the Linux kernel.
> Provide alias names for i2c drivers in order to allow them to be loaded by
> device tree name. The OF_ID macro ensures that the aliases are only present
> in powerpc b
So, like, the other day Kumar Gala mumbled:
> 86xx uses the flat device tree for all its needs so we dont need explicit
> IRQ info. Its not clear why this code existed since 86xx never existed in
> arch/ppc.
Sorry, my fault.
If you Sign-off-by: that patch, a maintainer can apply it! :-)
jdl
___
Scott Wood wrote:
>> +// Temporary -- will go away once kernel uses ranges for
>> get_immrbase().
>> +reg = <0xf000 0x00053000>;
>
> The patch to use ranges for get_immrbase() just went in, so we can drop this
> now.
Most excellent.
Time for YARDS -- Yet Another Rou
Zhang Wei wrote:
> Signed-off-by: Zhang Wei <[EMAIL PROTECTED]>
> ---
> arch/powerpc/boot/dts/mpc8641_hpcn.dts | 13 +
> 1 files changed, 13 insertions(+), 0 deletions(-)
>
> diff --git a/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> b/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> index 556a
Zhang Wei wrote:
> Signed-off-by: Zhang Wei <[EMAIL PROTECTED]>
> ---
> arch/powerpc/boot/dts/mpc8641_hpcn.dts | 13 +
> 1 files changed, 13 insertions(+), 0 deletions(-)
>
> diff --git a/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> b/arch/powerpc/boot/dts/mpc8641_hpcn.dts
> index 556a
So, like, the other day David Gibson mumbled:
> > +/ {
> > + model = "StorCenter";
> > + compatible = "storcenter";
>
> This really needs a vendor prefix.
Hey Grant!
Here's how dumb I am: You were thinking "Something's wrong here..."
and kept saying "Needs a model name!", and I kept saying
So, like, the other day "Zhang Wei" mumbled:
> >
> > The 8641 DTS file has been converted to /dts-v1/ format now.
> > Please rewrite this patch with explicit hex numbers for
> > addresses, even if 0x0, and decimal for IRQs.
>
> I have seen other nodes in mpc8641_hpcn.dts still use old style.
> Is
Cleaned up IRQ layout and removed unsused ISU allocations.
Fixed RTC address typo from /dts-v1/ conversion.
Incorporated list suggestions to use an "iomega," vendor prefix,
and to use a node reference rather than a hard path.
Signed-off-by: Jon Loeliger
---
Kumar,
I tried to use
Johan Borkhuis wrote:
> Hello,
>
> I was using kernel version 2.6.14 (ppc) on a MVME3100 board (MPC8540
> processor). We are planning to move to 2.6.20 (powerpc), but I have some
> problems with the initialization of a PCI-PCI bridge.
> Connected to the MVME3100 board is a PCI-PCI bridge (HiNT,
Dave Jiang wrote:
> diff --git a/arch/powerpc/platforms/85xx/mpc85xx_ads.c
> b/arch/powerpc/platforms/85xx/mpc85xx_ads.c
> index 4e03050..024393c 100644
> --- a/arch/powerpc/platforms/85xx/mpc85xx_ads.c
> +++ b/arch/powerpc/platforms/85xx/mpc85xx_ads.c
> @@ -26,6 +26,8 @@
> #include
> #include
So, like, the other day David Gibson mumbled:
> This patch adds an fdt_next_node() function which can be used to
> iterate through nodes of the tree while keeping track of depth. This
> function is used to simplify the iteration code in a lot of other
> functions, and is also exported for use by l
So, like, the other day Kumar Gala mumbled:
> If we add to an empty lmb region with a non-zero base we will not coalesce
> the number of regions done to one. This causes problems on ppc32 for the
s/done/down
> memory region as its assumed to only have one region.
>
> We can fix this be easily s
So, like, the other day David Gibson mumbled:
> fdt_add_subnode_namelen() has a bug if asked to add a subnode to a
> node which has NOP tags interspersed with its properties. In this
> case fdt_add_subnode_namelen() will put the new subnode before the
> first NOP tag, even if there are properties
So, like, the other day David Gibson mumbled:
> For ages dtc has included a sample dts, comment-test.dts, for checking
> various lexical corner cases in comment processing. In fact, it
> predates the automated testsuite, and has never been integrated into
> it. This patch addresses this oversight
So, like, the other day David Gibson mumbled:
> In light of the recently discovered bug with NOP handling, this adds
> some more testcases for NOP handling. Specifically, it adds a helper
> program which will add a NOP tag after every existing tag in a dtb,
> and runs the standard battery of tests
So, like, the other day David Gibson mumbled:
> Since fdt_node_offset_by_compatible() was converted to the new
> fdt_next_node() iterator, a chunk of initialization code became
> redundant, but was not removed by oversight. This patch cleans it up.
>
> Signed-off-by: David Gibson <[EMAIL PROTECTE
So, like, the other day David Gibson mumbled:
> Currently the CHECK_HEADER() macro is defined local to fdt_ro.c.
> However, there are a handful of functions (fdt_move, rw_check_header,
> fdt_open_into) from other files which could also use it (currently
> they open-code something more-or-less ident
So, like, the other day Scott Wood mumbled:
> >
> > Can I ask; what is the intended usage of such a thing? How large
> > would a typical binary blob be?
>
> I use it for embedding guest device trees in a hypervisor's device tree.
Why wouldn't we instead, say, extend the source sytax
to allow a
David Gibson wrote:
>> node {
>> prop = /incbin/("path/to/data");
>> };
>>
>> node {
>> prop = /incbin/("path/to/data", 8, 16);
>> };
>
> I still dislike the syntax, but haven't thought of a better one yet.
> There are some issues with the implementation too, but I've been a bit
> too bu
So, like, the other day Linas Vepstas mumbled:
>
> Isn't anyone concerned about the defacto fork-of-source-code that
> this causes?
You betcha. That's why I'm negative on the notion,
but I won't stand in front of _that_ train either.
> Which will be the official version?
The one I am maintain
Guys,
And just for the record, here is the QAD script that
I used to verify that all the existing DTS files in
the arch/powerpc/boot/dts directory are byte-identical
before and after my recent modifications. Naturally,
all the DTS files are still in "legacy dts-version 0"
still. But I don't thin
using "-" for ranges looks
suspiciously like the subtraction of two expressions,
so its syntax has been changed to use ".." as the range
indicator.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
dtc-lexer.l | 180 ++
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
tests/get_name.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/tests/get_name.c b/tests/get_name.c
index 2481741..a76bdf8 100644
--- a/tests/get_name.c
+++ b/tests/get_name.c
@@ -55,7 +55,7 @@ void check_nam
Folks,
This 4 part patch series for the DTC has:
0001-Reformat-grammar-rules-to-not-mix-language-syntax-an.patch
0002-Quiet-a-bogus-May-be-used-uninitialized-warning.patch
0003-Appease-the-printf-format-Gods-with-a-correct-typ.patch
0004-Begin-the-path-to-sane-literals-and-express
Use consistent indenting on all rule actions.
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
No functional changes!
dtc-parser.y | 152 +-
1 files changed, 118 insertions(+), 34 deletions(-)
diff --git a/dtc-parser.y
So, like, the other day Kumar Gala mumbled:
>
> On Oct 18, 2007, at 6:19 PM, Paul Mackerras wrote:
>
> > Your commit message doesn't give any reason why you are doing this, or
> > any explanation of what goes wrong without it. In fact, the commit
> > message is completely empty. :) Please resub
Signed-off-by: Jon Loeliger <[EMAIL PROTECTED]>
---
flattree.c |2 +-
1 files changed, 1 insertions(+), 1 deletions(-)
diff --git a/flattree.c b/flattree.c
index a26e71b..5889900 100644
--- a/flattree.c
+++ b/flattree.c
@@ -136,7 +136,7 @@ static void asm_emit_cell(void *e, cell
So, like, the other day Segher Boessenkool mumbled:
> > Property names have been limited to start with
> > characters from the set [a-zA-Z,._#?]. That is, the
> > digits and the expression symbols have been removed.
>
> This cannot work; many property names start with a digit,
> for example.
Are
So, like, the other day David Gibson mumbled:
> Currently, fdt_string() returns a (non-const) char *, despite taking a
> const void *fdt. This is inconsistent with all the other read-only
> functions which all return const pointers into the blob.
>
> This patch fixes that.
>
> Signed-off-by: Dav
So, like, the other day David Gibson mumbled:
> This patch makes the helper macros in trees.S use separate labels for
> the end of each dt subblock, rather than using only start labels.
> This means that the macros can now be used to create trees with the
> subblocks in non-standard orders.
>
> In
1 - 100 of 355 matches
Mail list logo