to define that macro either in the board configuration files.
Signed-off-by: Timur Tabi ti...@freescale.com
---
include/configs/MPC8349ITX.h |7 ---
include/configs/MPC8536DS.h |6 --
include/configs/MPC8544DS.h |6 --
include/configs/MPC8572DS.h |6 --
include
, while the 85xx is all big-endian.
83xx DMA = Elo DMA controller
85xx/86xx DMA = Elo Plus DMA controller
The Elo Plus supports list descriptors in addition to link
descriptors. The Elo only supports link descriptors.
--
Timur Tabi
Linux kernel developer at Freescale
ethernet nodes that match (or aliases) we will not attempt to do
anything.
Signed-off-by: Kumar Gala ga...@kernel.crashing.org
Acked-by: Timur Tabi ti...@freescale.com
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot
address is greater
than %u, force it to %u.\n, MAX_NUM_PORTS, MAX_NUM_PORTS);
But since you defined MAX_NUM_PORTS, you should also do this:
u8 mac[MAX_NUM_PORTS][6]; /* 0x42 - 0x71 MAC addresses */
To indicate where the dependency on 8 comes from.
--
Timur Tabi
Linux
On Wed, May 20, 2009 at 12:31 AM, Wolfgang Denk w...@denx.de wrote:
It's you who has OS X, and who has a problem. Only when Ah itchez, Ah
scratchez.
Just so I'm clear on this, you're saying that you're not really
concerned about how well U-Boot works for other people?
--
Timur Tabi
Linux
understand how it works when it *does*
work.
A functioning build process on OS X will allow me to do more work on U-Boot,
adding more features and fixing more bugs, than I normally would be able to do.
But I can't fix every problem myself.
--
Timur Tabi
Linux kernel developer at Freescale
.
The kernel is orders of magnitude more complex than U-Boot, and yet I
can build it just fine on OS X.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
relocate_code *
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
On Mon, May 4, 2009 at 7:47 PM, Mike Frysinger vap...@gentoo.org wrote:
On Monday 04 May 2009 09:27:11 Timur Tabi wrote:
On Fri, May 1, 2009 at 7:10 PM, Jean-Christophe wrote:
to compile on Mac OS you will have to move to the laster Mainline ,
What is that?
the latest git source tree
On Fri, May 1, 2009 at 7:10 PM, Jean-Christophe PLAGNIOL-VILLARD
plagn...@jcrosoft.com wrote:
to compile on Mac OS you will have to move to the laster Mainline ,
What is that?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
, that would be okay for now. But
according to Jean-Chrisophe's email, it looks like this would be not
be trivial to fix.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman
it doesn't compile. Has anyone tried this before?
What am I missing?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
trying to do.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
the individual commands.
I reviewed the code briefly, and it looks okay. The general idea is
good, so I ACK these patches.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman
*) (CONFIG_SYS_INIT_RAM_ADDR + CONFIG_SYS_GBL_DATA_OFFSET);
That usually means that your LAWBARs are misprogrammed.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Change netdev.h to use CONFIG_ULI526X instead of CONFIG_ULI526. CONFIG_ULI526X
is used everywhere else, so that's the correct macro name. Without this fix,
Ethernet will not work on the Freescale MPC8610 HPCD.
Signed-off-by: Timur Tabi ti...@freescale.com
---
include/netdev.h |2 +-
1
? I don't think accuracy is that
important. The advantage of a table is that it's easier to debug and
the code is simpler.
I'm inclined to NACK this patch.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
are it's not even going to define stack space
for it.
#ifdef CONFIG_COOL_FEATURE
{
u32 myvarrocks = foo * bar * bar;
gd-neato = myvarrocks
}
#endif
Is this an acceptable compromise?
This is what we do today, and I think it's ugly.
--
Timur Tabi
Linux
(for understanding).
Sorry, I don't understand what you're talking about. The #ifdefs are
used to enable feature-specific code on platforms that have that feature.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http
.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
be variables declared at the top of iterators.
That's just my two cents.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
ugly than variable declarations right in the
middle of 200 lines of code.
This is where I disagree.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
to get to work. The person who
has a car is obviously going to tell you that your life will be easier
if you get a car also.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo
hillel.a...@freescale.com
Just for the record:
Acked-by: Timur Tabi ti...@freescale.com
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
represent, according to the table, a divider
of 768. The RM also says that I2C clock is 1/2 CCB, so that means that
your CCB should be 400,000 * 768 * 2 = 614MHz. Is that right?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U
hope I can retrigger it.
I was willing to write a lot of the code to implement that change
(getting rid of i2c_xxx_bus_num), but aer Wolfgang shot it down, I
gave up. If you do manage to turn the tide, let me know.
--
Timur Tabi
Linux kernel developer at Freescale
On Thu, Feb 19, 2009 at 10:24 AM, Heiko Schocher h...@denx.de wrote:
Signed-off-by: Heiko Schocher h...@denx.de
---
Could you add a description that says what I2C mux support is? That
would make it easier to review this patch.
--
Timur Tabi
Linux kernel developer at Freescale
On Tue, Feb 17, 2009 at 3:49 PM, Kumar Gala ga...@kernel.crashing.org wrote:
The default value is 256M so I don't thing the change is need anymore.
Huh? I think his patch is supposed to address the fact that 256MB is
the limit, and he wants to add more.
--
Timur Tabi
Linux kernel developer
On Wed, Feb 11, 2009 at 5:58 PM, Mike Frysinger vap...@gentoo.org wrote:
will you post a patch to fix the issue or shall i ?
I'd rather you do it, because then you can test the code to make sure
it really works.
--
Timur Tabi
Linux kernel developer at Freescale
board specific file.
Ugh.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
the idea
of a current i2c bus is a bad one.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
I2C design. You guys
obviously have everything under control, so good luck.
--
Timur Tabi
Linux Kernel Developer @ Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
low-priority for me, though.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
All implementations of the functions i2c_reg_read() and i2c_reg_write() are
identical. We can save space and simplify the code by converting these
functions into inlines and putting them in i2c.h.
Signed-off-by: Timur Tabi [EMAIL PROTECTED]
---
v3: add 8xx-specific code, and include PRINTDs
Is there any documentation for this global variable? I'm trying to fix a bug in
NetStartAgain(), and I'm trying to understand the purpose of this variable.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
, CONFIG_SYS_I2C_SLAVE);
i2c_read(i2c_addr, reg, 1, buf, 1);
return (buf);
}
Is the call to i2c_init() necessary on 8xx boards? Shouldn't I2C have already
been initialized?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U
Mike Frysinger wrote:
dont wait ... it'll be a bit before i post it, so it'll be easier if i just
rebase against that
If I post the patch that assumes that that i2c_reg_read() passes 1 instead of 0,
it'll break blackfin.
--
Timur Tabi
Linux kernel developer at Freescale
you?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
(bit
28)?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
, and just say that code
comments are unnecessary because the git log contains everything we need to
know?
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
-boot/2008-October/041173.html
On Thu, Oct 2, 2008 at 10:06 AM, Timur Tabi [EMAIL PROTECTED] wrote:
All implementations of the functions i2c_reg_read() and i2c_reg_write() are
identical. We can save space and simplify the code by converting these
functions into inlines and putting them in i2c.h
: alignment.
--
Timur Tabi
Linux kernel developer at Freescale
___
U-Boot mailing list
U-Boot@lists.denx.de
http://lists.denx.de/mailman/listinfo/u-boot
Wolfgang,
Who is the maintainer for I2C? I don't know who needs to ACK this
patch for it to go in, and I don't know who's supposed to pick it up,
either.
On Thu, Oct 2, 2008 at 10:06 AM, Timur Tabi [EMAIL PROTECTED] wrote:
All implementations of the functions i2c_reg_read() and i2c_reg_write
601 - 644 of 644 matches
Mail list logo