Re: [PATCH 08/35] common: Move relocate_code() to init.h

2019-12-11 Thread Timur Tabi (ti...@kernel.org)
On 12/11/19 6:47 PM, Simon Glass wrote: - * void relocate_code (addr_moni) + * void relocate_code(addr_moni) Is this really necessary? I think you're bloating your patch with these comment changes. Your patch would be half the size without these changes, so I don't think this really qualifi

Re: [U-Boot] [PATCH 1/2] p1022ds: Change Timur's email address

2018-05-02 Thread Timur Tabi
On 05/02/2018 06:12 PM, Fabio Estevam wrote: ti...@freescale.com is not a valid email for quite some time, so change it to Timur's updated email. Signed-off-by: Fabio Estevam Acked-by: Timur Tabi ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH v2] MAINTAINERS: Switch nxp.com domain

2018-05-01 Thread Timur Tabi
On 5/1/18 1:54 PM, Fabio Estevam wrote: Timur, would you like to keep maintaining p1022ds? If so, then please send an email switching your address toti...@tabi.org. Ok. ___ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot

Re: [U-Boot] ACPI in general

2016-10-06 Thread Timur Tabi
Bin Meng wrote: >I believe that ACPI systems generally expect EFI runtime services to >be present as well. I know ours does. This is not true. ACPI does not require any EFI runtime services. Please re-read my sentence. I said "generally expect". If you guys want to spend the man-years neces

Re: [U-Boot] ACPI in general

2016-10-06 Thread Timur Tabi
On Wed, Oct 5, 2016 at 8:58 PM, Tom Rini wrote: > > I think that takes too narrow of a view. If silicon is sold, someone > will put it somewhere. And if there's firmware that works, and the > buyer can modify to suit their design, they'll use it. I believe that ACPI systems generally expect EFI

Re: [U-Boot] ACPI in general

2016-10-05 Thread Timur Tabi
Tom Rini wrote: Well, I wouldn't phrase it quite like that. I would ask, do we want to go down this path? How far down would we want to go, if so? ACPI is pretty complicated, more so than DT. UEFI is also open source. I think you need to find a very compelling reason to reinvent the wheel.

Re: [U-Boot] ACPI in general

2016-10-05 Thread Timur Tabi
On Wed, Oct 5, 2016 at 9:46 AM, Simon Glass wrote: >> Is there any activity to bring ACPI to other than x86 arch? If not, do >> we have a plan to do so? > > Not that I know of, sorry. Note that ACPI for ARM exists on Linux already, and as far as I know, all ARM ACPI systems use UEFI, not U-Boot.

Re: [U-Boot] [PATCH v2 1/3] dm: gpio: Add driver for MPC85XX GPIO controller

2016-05-10 Thread Timur Tabi
Mario Six wrote: On Tue, May 10, 2016 at 2:30 PM, Timur Tabi mailto:ti...@tabi.org>> wrote: Mario Six wrote: + addr = fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, dev->of_offset, + "reg"

Re: [U-Boot] [PATCH v2 1/3] dm: gpio: Add driver for MPC85XX GPIO controller

2016-05-10 Thread Timur Tabi
Mario Six wrote: + addr = fdtdec_get_addr_size_auto_noparent(gd->fdt_blob, dev->of_offset, + "reg", 0, &size); Why can't you use platform_get_resource()? ___ U-Boot mailing list U-Boot@lists.denx.d

Re: [U-Boot] [PATCH 1/3] dm: gpio: Add driver for MPC85XX GPIO controller

2016-05-10 Thread Timur Tabi
Mario Six wrote: The problem is that in 36-bit mode the physical addresses are 64-bit, which means that you get 64-bit integers when you read something from the device tree with fdtdec_get_addr. But the device tree addresses themselves seem to be 32-bit, because if I read a property like 'reg =

Re: [U-Boot] [PATCH 1/3] dm: gpio: Add driver for MPC85XX GPIO controller

2016-05-08 Thread Timur Tabi
On Mon, May 2, 2016 at 6:43 AM, Mario Six wrote: > Regarding the address width discrepancy: The system I'm working on is a > P1022 Qoriq, which has 36 bit width, which implies that phys_addr_t needs > to be 64 bit. But the everything else (including the GPIO controller) uses > 32 bits, thus the de

Re: [U-Boot] drivers/elbc:Memory leak fix

2015-05-23 Thread Timur Tabi
On Fri, May 22, 2015 at 9:55 PM, Kushwaha Prabhakar wrote: > > It is not getting catch during GCC build. But during static analysis using > Klockwork, this is shown as error. Then Klockwork is wrong. ___ U-Boot mailing list U-Boot@lists.denx.de http:/

Re: [U-Boot] [PATCH] powerpc/t1040qds: Remove Video - HDMI support

2014-08-07 Thread Timur Tabi
On Fri, Apr 18, 2014 at 4:32 AM, dongsheng.w...@freescale.com wrote: > > If we have, why kernel doesn't support DIU on T1040QDS? That I must remove it > reason. Because no one added that support yet. You should finish the job by fixing the kernel. ___

Re: [U-Boot] [PATCH 03/11] kconfig: add board Kconfig and defconfig files

2014-05-02 Thread Timur Tabi
g SYS_CONFIG_NAME > + string > + default "P1022DS" > + > +config BOARD_MAINTAINER > + string > + default "Timur Tabi " My new email address is now ti...@tabi.org. I don't work for Freescale any mor

Re: [U-Boot] [PATCH v2 2/2] Powerpc/QE: Add QE support for T1040

2014-02-02 Thread Timur Tabi
On Mon, Jan 27, 2014 at 2:14 AM, Zhao Qiang wrote: > +#ifndef CONFIG_PPC_T1040 Nack. You need to write this code without all this T1040-specific #ifdefs everywhere. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-bo

Re: [U-Boot] [PATCH] Powerpc/QE: Add QE support for T1040

2014-01-26 Thread Timur Tabi
qiang.z...@freescale.com wrote: T1040QDS has microcode for Fman and another microcode for QE. The two microcodes is different. QE microcode is needed. Then why do you have an #else statement: +#ifdef CONFIG_PPC_T1040 + qe_upload_firmware((const void *)CONFIG_SYS_QE_UCODE_FW_ADDR); +#els

Re: [U-Boot] [PATCH] Powerpc/QE: Add QE support for T1040

2014-01-25 Thread Timur Tabi
qiang.z...@freescale.com wrote: Other boards have either Fman or QE, So CONFIG_SYS_QE_FMAN_FW_ADDR can be used either Fman or QE. But T1040QDS have both Fman and QE. Does the T1040QDS have microcode for both Fman and QE? Normally, QE microcode is not needed.

Re: [U-Boot] [PATCH] Powerpc/QE: Add QE support for T1040

2014-01-25 Thread Timur Tabi
qiang.z...@freescale.com wrote: CONFIG_SYS_QE_FMAN_FW_ADDR is used for Fman microcode while CONFIG_SYS_QE_UCODE_FW_ADDR used For QE microcode. No, CONFIG_SYS_QE_FMAN_FW_ADDR is used for both QE and Fman microcode. That's why it says "QE_FMAN" in it. This is documented in the README. ___

Re: [U-Boot] [PATCH] Powerpc/QE: Add QE support for T1040

2014-01-25 Thread Timur Tabi
On Fri, Jan 24, 2014 at 3:05 AM, Zhao Qiang wrote: > +#ifdef CONFIG_PPC_T1040 > + qe_upload_firmware((const void *)CONFIG_SYS_QE_UCODE_FW_ADDR); > +#else > qe_upload_firmware((const void *)CONFIG_SYS_QE_FMAN_FW_ADDR); > +#endif > It's been a while since I looked at the QE code in U

Re: [U-Boot] T4QDS e6500 core and U-Boot 64-bit

2014-01-25 Thread Timur Tabi
On Fri, Jan 24, 2014 at 7:45 AM, Wolfgang Denk wrote: > > For the "test" > part, it is probably much easier to add a customized memory test (or > fix just the existing memory test such that it can be built for a 64 > bit mode) and use this, then trying to run all of U-Boot in 64 bit > mode. There

Re: [U-Boot] [PATCH 7/9][v2] net: tsec: Use portable types and accessors for BDs

2013-10-07 Thread Timur Tabi
Claudiu Manoil wrote: I think you're suggesting that virt_to_phys() should be used to fix that, right? Yes. However, virt_to_phys() is equivalent to that simple cast in most cases as there's no CONFIG_PHYS_64BIT for the platforms with eTSEC. I'm actually not sure if there's a platform with eT

Re: [U-Boot] [PATCH 7/9][v2] net: tsec: Use portable types and accessors for BDs

2013-10-05 Thread Timur Tabi
On Sat, Oct 5, 2013 at 9:31 AM, Timur Tabi wrote: > > + out_be32(®s->tbase, (u32)&txbd[0]); > + out_be32(®s->rbase, (u32)&rxbd[0]); > > &rxbd[0] is a virtual address. > > Doesn't rbase require a physical address? You're assuming that virt ==

Re: [U-Boot] [PATCH 7/9][v2] net: tsec: Use portable types and accessors for BDs

2013-10-05 Thread Timur Tabi
On Fri, Oct 4, 2013 at 11:25 AM, Claudiu Manoil wrote: > [v3] declaring the BDs as __iomem to avoid casting submitted: > http://patchwork.ozlabs.org/patch/280664/ + out_be32(®s->tbase, (u32)&txbd[0]); + out_be32(®s->rbase, (u32)&rxbd[0]); &rxbd[0] is a virtual address. Doesn't rbase require a

Re: [U-Boot] [PATCH 7/9] net: tsec: Use portable types and accessors for BDs

2013-10-03 Thread Timur Tabi
On Mon, Sep 30, 2013 at 4:44 AM, Claudiu Manoil wrote: > +#define GET_BD_STAT(T, i) be16_to_cpu((__force __be16)T##BD(i).status) > +#define SET_BD_STAT(T, i, v) T##BD(i).status = (__force __u16)cpu_to_be16(v) > +#define GET_BD_BLEN(T, i) be16_to_cpu((__force __be16)T##BD(i).length) > +#define SET_

Re: [U-Boot] declaring and initializing variables

2013-09-30 Thread Timur Tabi
On Mon, Sep 30, 2013 at 7:04 PM, York Sun wrote: > > struct ccsr_ddr __iomem *ddr = (void *) CONFIG_FOO_ADDR; > struct ccsr_ddr __iomem *ddr = > (struct ccsr_ddr __iomem *) CONFIG_FOO_ADDR; > > You have told me the second format is preferred. I have been using this > format since. But in p

Re: [U-Boot] How do ARM platform initialize DDR?

2013-09-22 Thread Timur Tabi
On Thu, Sep 19, 2013 at 3:57 PM, Tom Rini wrote: > > Thinking back, as a rule of thumb, PowerPC has SPD I2C data > available, usually. That's not the rule for ARM. One of a few choices > happen: Even if you do have SPD on your ARM chip, that's still no guarantee that you can program the DDR con

Re: [U-Boot] How do ARM platform initialize DDR?

2013-09-22 Thread Timur Tabi
On Tue, Sep 17, 2013 at 10:59 AM, York Sun wrote: > I am wondering if there is generic > DDR driver used by many ARM platforms, like the one we have for > powerpc/mpc85xx SoCs. Sadly, no such thing exists. There is no generic DDR controller for ARM. I'm working on DDR controller support for our

Re: [U-Boot] [PATCH v2] powerpc/c29xpcie: Getting DDR SPD image from 16-bit sub-address EEPROM

2013-09-22 Thread Timur Tabi
On Sat, Sep 21, 2013 at 8:31 PM, Po Liu wrote: > +void get_spd(generic_spd_eeprom_t *spd, u8 i2c_address) > +{ > + int ret = i2c_read(i2c_address, 0, 2, (uchar *)spd, (uchar) is wrong. int i2c_read(uint8_t chip, unsigned int addr, int alen, uint8_t *buffer

Re: [U-Boot] [PATCH] Enable the DIU for P1022DS board.

2013-09-06 Thread Timur Tabi
On Fri, Sep 6, 2013 at 4:21 AM, Jason Jin wrote: > > #ifndef CONFIG_FSL_DIU_FB > +#define CONFIG_ATI > #endif Is this really necessary? If the DIU is disabled, why would someone use a PCI video controller? ___ U-Boot mailing list U-Boot@lists.denx.de

Re: [U-Boot] [PATCH] powerpc/eeprom: update MAX_NUM_PORTS to adapt non-256-bytes EEPROM

2013-08-30 Thread Timur Tabi
On Fri, Aug 30, 2013 at 5:07 AM, Shengzhou Liu wrote: > #ifdef CONFIG_SYS_I2C_EEPROM_NXID > +/* some boards with non-256-bytes EEPROM have special define */ > +/* for MAX_NUM_PORTS in board-specific file */ > +#ifndef MAX_NUM_PORTS > #define MAX_NUM_PORTS 23 > +#endif > #define NXID_VERSION

Re: [U-Boot] [PATCH] fsl: Fix compiler warnings from gcc-4.4 in sys_eeprom code

2013-08-30 Thread Timur Tabi
On Mon, Jul 6, 2009 at 10:11 AM, Kumar Gala wrote: > > #ifdef CONFIG_SYS_I2C_EEPROM_NXID > /* Is this a valid NXID EEPROM? */ > -#define is_valid (*((u32 *)e.id) == (('N' << 24) | ('X' << 16) | ('I' << 8) > | 'D')) > +#define is_valid ((e.id[0] == 'N') || (e.id[1] == 'X') || \ > +

Re: [U-Boot] [PATCH] powerpc/eeprom: update MAX_NUM_PORTS to fix program failure

2013-08-30 Thread Timur Tabi
aders need only consider the MACSIZE field to determine if this space is used for additional MAC addresses." Which means that you can fill the EEPROM with MAC addresses. If you do the math, that means 31 addresses. -- Timur Tabi ___ U-Boot maili

Re: [U-Boot] [PATCH] powerpc/eeprom: update MAX_NUM_PORTS to fix program failure

2013-08-29 Thread Timur Tabi
mber needs to be changed to v2, and the code needs to dynamically handle v1 and v2. BTW, your patch breaks EVERY OTHER BOARD. You can't just change the 23 to a 9 for every board. Did you test your patch on other boards? -- Timur Tabi ___ U-Boot mail

Re: [U-Boot] [PATCH] powerpc/eeprom: update MAX_NUM_PORTS to fix program failure

2013-08-11 Thread Timur Tabi
On Thu, Aug 8, 2013 at 5:14 AM, Shengzhou Liu wrote: > On some boards, the size of EEPROM is 128 Bytes instead of 256. > so we set default MAX_NUM_PORTS to 9 rather than previous 23 to > avoid the programming failure, we can define MAX_NUM_PORTS in > board-specific header file to overwrite the def

Re: [U-Boot] [PATCH 2/2 V3] PPC MPC83xx: Fix MPC8323ERDB build warning

2013-07-14 Thread Timur Tabi
Wolfgang Denk wrote: mpc8323erdb.c: In function 'mac_read_from_eeprom': mpc8323erdb.c:198:3: warning: dereferencing type-punned pointer will break strict-aliasing rules [-Wstrict-aliasing] Signed-off-by: Wolfgang Denk cc: Timur Tabi cc: Kim Phillips Ack. --

Re: [U-Boot] [PATCH 2/2 V2] PPC MPC83xx: Fix MPC8323ERDB build warning

2013-07-08 Thread Timur Tabi
On 07/08/2013 01:19 PM, Wolfgang Denk wrote: > + uint32_t crc_buf; > + > + memcpy(&crc_buf, &buf[24], sizeof(unsigned int)); Well, this is wrong, because it should be sizeof(uint32_t). Or maybe sizeof(crc_buf)

Re: [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning

2013-07-04 Thread Timur Tabi
4]) { > + unsigned int crc_buf; This should be a uint32_t, because you specifically want only four bytes. If you fix that, Acked-by: Timur Tabi ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH 2/2] PPC MPC83xx: Fix MPC8323ERDB build warning

2013-06-28 Thread Timur Tabi
On Fri, Jun 28, 2013 at 4:47 PM, Andy Fleming wrote: > Well, he hasn't been at Freescale for years, now, so that's not surprising. > It's not entirely clear to me whether that board is one we actively > support, but I suspect we can get away with maintenance moving over to Kim. > That said, I gue

Re: [U-Boot] [PATCH 10/10] powerpc: p1022ds: support TPL on the P1022DS

2013-06-18 Thread Timur Tabi
On Tue, Jun 18, 2013 at 5:39 AM, wrote: > From: Ying Zhang > > Support TPL on the P1022DS. Please define "TPL". I have no idea what it is. ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] powerpc/85xx: Add P1023RDB board support

2013-04-16 Thread Timur Tabi
On Tue, Apr 16, 2013 at 4:00 AM, Chunhe Lan wrote: > > +#define CONFIG_SYS_BAUDRATE_TABLE \ > + {300, 600, 1200, 2400, 4800, 9600, 19200, 38400, 115200} It's amazing that Freescale is still producing board files that are missing 57,600 baud. ___

Re: [U-Boot] [u-boot-release] [PATCH 1/3] powerpc/p5040: enable NAND boot support

2013-01-16 Thread Timur Tabi
; freescale - BSC9131RDB:BSC9131RDB,SPIFLASH > stxgp3 powerpc mpc85xx stxgp3 stx > stxssa powerpc mpc85xx stxssa stx >- stxssa > -- Timur Tabi Linux kernel developer at Free

Re: [U-Boot] [PATCH] Introduce a global bool type

2013-01-08 Thread Timur Tabi
f the C language that were invented in the past 20 years. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] Introduce a global bool type

2013-01-08 Thread Timur Tabi
= 0; > k = 2; > > if ((i | j | k) == true) ... Ok, but this is just wrong. i, j, and k are not boolean types, so they should not be compared with 'true' or 'false'. I don't think you'll find any disagreement with that. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [u-boot-release] [PATCH] Introduce a global bool type

2013-01-07 Thread Timur Tabi
ol was introduced in C99, so it should be safe to use this new definition instead of a hand-coded enum. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [u-boot-release] [PATCH 6/7] getenv_f() env variable exist w/o needing a buffer

2013-01-07 Thread Timur Tabi
is found, -1 if not), other than being able to copy the > variable's value to the address 0. > > Signed-off-by: James Yang Acked-by: Timur Tabi -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [u-boot-release] [PATCH] boards/T4240qds:Fix IFC AMASK init as per FPGA register space

2012-12-18 Thread Timur Tabi
Prabhakar Kushwaha wrote: > +#define CONFIG_SYS_AMASK3IFC_AMASK(4*1024) IFC_AMASK(4 * 1024) Spaces around binary operators, please. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de h

[U-Boot] [PATCH] powerpc/t4qds: move VSC3316 config data from t4qds.h to t4qds.c

2012-12-12 Thread Timur Tabi
Static variables should be defined in C files, not header files, because otherwise every C file that #includes the header file will generate a duplicate of the variables. Since the vsc3316_xxx[] arrays are only used by t4qds.c anyway, just put the variables there. Signed-off-by: Timur Tabi

[U-Boot] [PATCH 1/2] powerpc/85xx: update the work-around for P4080 erratum SERDES-9

2012-11-01 Thread Timur Tabi
The documented work-around for P4080 erratum SERDES-9 has been updated. It is now compatible with the work-around for erratum A-4580. This requires adding a few bitfield macros for the BnTTLCRy0 register. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c | 20

[U-Boot] [PATCH 2/2] powerpc/85xx: implement check for erratum A-004580 work-around

2012-11-01 Thread Timur Tabi
ane sub-structure in serdes_corenet_t, so we make it a named struct. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/cmd_errata.c | 48 + arch/powerpc/include/asm/config_mpc85xx.h |1 + arch/powerpc/include/asm/immap_85xx.h |2 +- 3 files changed,

[U-Boot] [PATCH] powerpc/85xx: implement check for erratum A-004849 work-around

2012-10-25 Thread Timur Tabi
plement in PBI than in U-Boot itself. It is still useful, however, for the 'errata' command to tell us whether the work-around has been applied. For A-004849, we can do this by verifying that the values in the specific registers that the work-around says to update. Signed-off-by: T

[U-Boot] [PATCH] powerpc/85xx/p5040: add CONFIG_SYS_PPC64, del CONFIG_SYS_FSL_ELBC_MULTIBIT_ECC

2012-10-23 Thread Timur Tabi
-Boot. It's a holdover from the SDK. Signed-off-by: Timur Tabi --- This patch should be applied before "[v4] powerpc/85xx: add support for the Freescale P5040DS Superhydra reference board" arch/powerpc/include/asm/config_mpc85xx.h |2 +- 1 files changed, 1 insertions(+), 1 dele

[U-Boot] [PATCH] [v4] powerpc/85xx: add support for the Freescale P5040DS Superhydra reference board

2012-10-23 Thread Timur Tabi
The P5040DS reference board (a.k.a "Superhydra") is an enhanced version of P3041DS/P5020DS ("Hydra") reference board. Signed-off-by: Timur Tabi Signed-off-by: Shaohui Xie --- board/freescale/common/Makefile |2 + board/freescale/common/ngpixis.h

Re: [U-Boot] Please pull u-boot-mpc85xx.git

2012-10-23 Thread Timur Tabi
format, we just implement it. So I don't think there is any reasonable objection to my patch. Please apply it. Without it, we have a broken implementation of the EEPROM parser, and *that* is not acceptable. -- Timur Tabi Linux kernel developer at Freescale _

Re: [U-Boot] [PATCH] powerpc/fm: fix TBI PHY address settings

2012-10-16 Thread Timur Tabi
Xie Shaohui-B21989 wrote: >> Is this still true with Fman v3? > [S.H] No. Fman V3 does not need to set TBIPA on FM1@DTSEC1. Does this patch break Fman V3? -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@list

[U-Boot] [PATCH 6/7] [v5] powerpc/85xx: Add P5040 processor support

2012-10-05 Thread Timur Tabi
enforcement Extra privileged level for hypervisor support QorIQ Trust Architecture 1.1 Secure boot, secure debug, tamper detection, volatile key storage Signed-off-by: Timur Tabi --- v5: fixed all the LIODNs, for real this time. arch/powerpc/cpu/mpc85xx/Makefile |3 + arch/powerpc

Re: [U-Boot] [PATCH 6/7] [v4] powerpc/85xx: Add P5040 processor support

2012-10-05 Thread Timur Tabi
Kim Phillips wrote: > need another two DECO entries. Doh, I merged the change into the wrong commit. Ugh. Thanks for catching that. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx

[U-Boot] [PATCH 6/7] [v4] powerpc/85xx: Add P5040 processor support

2012-10-05 Thread Timur Tabi
enforcement Extra privileged level for hypervisor support QorIQ Trust Architecture 1.1 Secure boot, secure debug, tamper detection, volatile key storage Signed-off-by: Timur Tabi --- v4: forgot a couple LIODN updates arch/powerpc/cpu/mpc85xx/Makefile |3 + arch/powerpc/cpu

[U-Boot] [PATCH 7/7] [v3] powerpc/85xx: add support for the Freescale P5040DS Superhydra reference board

2012-10-05 Thread Timur Tabi
The P5040DS reference board (a.k.a "Superhydra") is an enhanced version of P3041DS/P5020DS ("Hydra") reference board. Signed-off-by: Timur Tabi Signed-off-by: Shaohui Xie --- arch/powerpc/cpu/mpc85xx/p5040_ids.c| 66 ++-- board/freescale/common/Makefile

[U-Boot] [PATCH 6/7] [v3] powerpc/85xx: Add P5040 processor support

2012-10-05 Thread Timur Tabi
enforcement Extra privileged level for hypervisor support QorIQ Trust Architecture 1.1 Secure boot, secure debug, tamper detection, volatile key storage Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/Makefile |3 + arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c | 70

[U-Boot] [PATCH 3/7] powerpc/85xx: define SRIO LIODN functions only if SRIO is defined

2012-10-05 Thread Timur Tabi
The P5040 does not have SRIO support, so there are no SRIO LIODNs. Therefore, the functions that set the SRIO LIODNs should not be compiled. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/liodn.c |8 arch/powerpc/include/asm/fsl_liodn.h |4 +++- 2 files changed, 11

[U-Boot] [PATCH 2/7] powerpc/85xx: introduce SET_PCI_LIODN_BASE, for setting PCI LIODNs

2012-10-05 Thread Timur Tabi
macro that puts the liodn in the correct register. Signed-off-by: Laurentiu Tudor Signed-off-by: Timur Tabi --- arch/powerpc/include/asm/fsl_liodn.h |5 + arch/powerpc/include/asm/immap_85xx.h |4 +++- 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/arch/powerpc

[U-Boot] [PATCH 5/7] powerpc/p5040ds: add per pci endpoint liodn offset list

2012-10-05 Thread Timur Tabi
From: Laurentiu Tudor Add a new device tree property named "fsl,liodn-offset-list" holding a list of per pci endpoint permitted liodn offsets. This property is useful in virtualization scenarios that implement per pci endpoint partitioning. The final liodn of a partitioned pci endpoint is calcula

[U-Boot] [PATCH 1/7] powerpc/mpc85xx: fix Unicode characters in release.S

2012-10-05 Thread Timur Tabi
Commit 709389b6 unintentionally used the Unicode version of the apostrophy. Replace it with the normal ASCII version. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/release.S | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/powerpc

[U-Boot] [PATCH 4/7] powerpc/85xx: move SRIO configuration out of corenet_ds.h

2012-10-05 Thread Timur Tabi
The P5040 does not have SRIO, so don't put the SRIO definitions in corenet_ds.h. They belong in the board-specific header files. Signed-off-by: Timur Tabi --- include/configs/P3041DS.h|4 include/configs/P4080DS.h|4 include/configs/P5020DS.h|4 in

Re: [U-Boot] [PATCH 4/6] [v2] powerpc/85xx: Add P5040 processor support

2012-10-03 Thread Timur Tabi
97, 98), >> +SET_SEC_DECO_LIODN_ENTRY(1, 99, 100), >> +}; > NACK - the SEC in the p5040 has four DECOs, not two. I should have the proper LIODN information later today. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [u-boot-release] [RFC PATCH 17/17] ns16550: wait for TENT before initializing

2012-10-01 Thread Timur Tabi
MT)) > + ; > + > serial_out(CONFIG_SYS_NS16550_IER, &com_port->ier); > #if (defined(CONFIG_OMAP) && !defined(CONFIG_OMAP3_ZOOM2)) || \ > defined(CONFIG_AM33XX) -- Timur Tabi Linux kernel developer at Freescale ___

[U-Boot] [PATCH 3/6] powerpc/85xx: move SRIO configuration out of corenet_ds.h

2012-10-01 Thread Timur Tabi
The P5040 does not have SRIO, so don't put the SRIO definitions in corenet_ds.h. They belong in the board-specific header files. Signed-off-by: Timur Tabi --- include/configs/P3041DS.h|4 include/configs/P4080DS.h|4 include/configs/P5020DS.h|4 in

[U-Boot] [PATCH 6/6] [v2] powerpc/85xx: add support for the Freescale P5040DS Superhydra reference board

2012-10-01 Thread Timur Tabi
The P5040DS reference board (a.k.a "Superhydra") is an enhanced version of P3041DS/P5020DS ("Hydra") reference board. Signed-off-by: Timur Tabi Signed-off-by: Shaohui Xie --- board/freescale/common/Makefile |2 + board/freescale/common/ngpixis.h

[U-Boot] [PATCH 2/6] powerpc/85xx: introduce SET_PCI_LIODN_BASE, for setting PCI LIODNs

2012-10-01 Thread Timur Tabi
macro that puts the liodn in the correct register. Signed-off-by: Laurentiu Tudor Signed-off-by: Timur Tabi --- arch/powerpc/include/asm/fsl_liodn.h |5 + arch/powerpc/include/asm/immap_85xx.h |4 +++- 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/arch/powerpc

[U-Boot] [PATCH 4/6] [v2] powerpc/85xx: Add P5040 processor support

2012-10-01 Thread Timur Tabi
enforcement Extra privileged level for hypervisor support QorIQ Trust Architecture 1.1 Secure boot, secure debug, tamper detection, volatile key storage Signed-off-by: Timur Tabi --- The LIODNs for the SEC's DECOs are apparently wrong, but the correct values are not available. They will be

[U-Boot] [PATCH 5/6] powerpc/p5040ds: add per pci endpoint liodn offset list

2012-10-01 Thread Timur Tabi
From: Laurentiu Tudor Add a new device tree property named "fsl,liodn-offset-list" holding a list of per pci endpoint permitted liodn offsets. This property is useful in virtualization scenarios that implement per pci endpoint partitioning. The final liodn of a partitioned pci endpoint is calcula

[U-Boot] [PATCH 1/6] powerpc/mpc85xx: fix Unicode characters in release.S

2012-10-01 Thread Timur Tabi
Commit 709389b6 unintentionally used the Unicode version of the apostrophy. Replace it with the normal ASCII version. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/release.S | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch/powerpc

Re: [U-Boot] [PATCH] SBx81CFC960: embed Fman microcode in the bootloader

2012-09-24 Thread Timur Tabi
posed to be #included into customer code. Unfortunately, not all Freescale developers have gotten the message. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

Re: [U-Boot] [PATCH] SBx81CFC960: embed Fman microcode in the bootloader

2012-09-21 Thread Timur Tabi
image as C code array of bytes. > > What exactly is the licensing of this code? It's not GPL-compatible, so linking the code with U-boot and shipping the binary as one blob is probably a GPL violation. That's why I created the "QE firmware binary format" and wrote all that c

Re: [U-Boot] arch/powerpc/lib/board.c:get_effective_memsize() for 4GB+ systems

2012-09-21 Thread Timur Tabi
d what test platform would you suggest? And what if your extended > testing needs to be a viable option at every boot and that you have > boot time requirements? You can create U-boot "applications" that test your hardware, if you want to test from within U-Boo

Re: [U-Boot] arch/powerpc/lib/board.c:get_effective_memsize() for 4GB+ systems

2012-09-21 Thread Timur Tabi
Robert Thorhuus wrote: > Still I wonder why the choice was made to have U-Boot relocate in high > memory rather than low memory and also not making it easy to configure > the relocation. U-Boot relocates in high memory so that you can load your operating system at address 0. -- T

Re: [U-Boot] arch/powerpc/lib/board.c:get_effective_memsize() for 4GB+ systems

2012-09-21 Thread Timur Tabi
the first software place for test and debug. Maybe I want > to read out RAM contents? It's a boot loader, not a testing platform. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/m

Re: [U-Boot] QE firmware/microcode handling

2012-09-20 Thread Timur Tabi
LWAYS disabled features or rewritten code to make it smaller. If making U-boot larger were so simple, it would have been done by now, I think. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.d

Re: [U-Boot] QE firmware/microcode handling

2012-09-20 Thread Timur Tabi
they're afraid they won't be able to update the firmware. If you can do it, go ahead. I would approve a patch that adds the ability to inject a firmware binary into u-boot.bin, provided it's controlled via a build or CONFIG_SYS_xxx option. -- Timur Tabi Linux kernel developer at Fre

[U-Boot] [PATCH] video/powerpc: don't touch DIU registers that we don't need

2012-09-17 Thread Timur Tabi
Ds in U-Boot. 3) Don't configure any write-back buffer registers, since we don't use that mode. 4) The default values for the THRESHOLDS, SYN_POL, and PLUT registers should be used, so don't touch those registers either. Signed-off-by: Timur Tabi --- dri

Re: [U-Boot] [u-boot-release] [PATCH 4/5] powerpc/85xx: Add P5040 processor support

2012-09-11 Thread Timur Tabi
o those that were > identified as wrong). Plus, if I'd had reviewed this, it would have > four DECOs by now. Ok, I'll just wait until someone tells me what changes to make to this code. -- Timur Tabi Linux kernel developer at Freescale _

[U-Boot] [PATCH 1/5] powerpc/mpc85xx: fix Unicode characters in release.S

2012-08-31 Thread Timur Tabi
The previous commit unintentionally used the Unicode version of the apostrophy. Replace it with the normal ASCII version. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/release.S | 24 1 files changed, 12 insertions(+), 12 deletions(-) diff --git a/arch

[U-Boot] [PATCH 3/5] powerpc/85xx: move SRIO configuration out of corenet_ds.h

2012-08-31 Thread Timur Tabi
The P5040 does not have SRIO, so don't put the SRIO definitions in corenet_ds.h. They belong in the board-specific header files. Signed-off-by: Timur Tabi --- include/configs/P3041DS.h|4 include/configs/P4080DS.h|4 include/configs/P5020DS.h|4 in

[U-Boot] [PATCH 5/5] powerpc/85xx: add support for the Freescale P5040DS Superhydra reference board

2012-08-31 Thread Timur Tabi
The P5040DS reference board (a.k.a "Superhydra") is an enhanced version of P3041DS/P5020DS ("Hydra") reference board. Signed-off-by: Timur Tabi --- board/freescale/common/Makefile |2 + board/freescale/corenet_ds/Makefile |2 + board/

[U-Boot] [PATCH 4/5] powerpc/85xx: Add P5040 processor support

2012-08-31 Thread Timur Tabi
enforcement Extra privileged level for hypervisor support QorIQ Trust Architecture 1.1 Secure boot, secure debug, tamper detection, volatile key storage Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/Makefile |3 + arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c | 43

[U-Boot] [PATCH 2/5] powerpc/85xx: introduce SET_PCI_LIODN_BASE, for setting PCI LIODNs

2012-08-31 Thread Timur Tabi
macro that puts the liodn in the correct register. Signed-off-by: Laurentiu Tudor Signed-off-by: Timur Tabi --- arch/powerpc/include/asm/fsl_liodn.h |5 + arch/powerpc/include/asm/immap_85xx.h |4 +++- 2 files changed, 8 insertions(+), 1 deletions(-) diff --git a/arch/powerpc

Re: [U-Boot] [PATCH 6/7] powerpc/85xx: update P4080DS MDIO bus multiplexer support

2012-08-16 Thread Timur Tabi
x27;t back-port the kernel code, which I've already done for my testing. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx.de/mailman/listinfo/u-boot

[U-Boot] [PATCH 1/7] powerpc/85xx: add support for FM2 DTSEC5

2012-08-14 Thread Timur Tabi
Unlike previous SOCs, the Freescale P5040 has a fifth DTSEC on the second Fman, so add the Fman and SerDes macros for that DTSEC. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c |6 ++ arch/powerpc/include/asm/fsl_serdes.h |1 + arch/powerpc

[U-Boot] [PATCH 6/7] powerpc/85xx: update P4080DS MDIO bus multiplexer support

2012-08-14 Thread Timur Tabi
-boot code depends on several device tree aliases to help it find the nodes that need to be updated. Signed-off-by: Timur Tabi --- board/freescale/corenet_ds/eth_p4080.c | 172 1 files changed, 129 insertions(+), 43 deletions(-) diff --git a/board/freescale/coren

[U-Boot] [PATCH 3/7] [v2] powerpc/85xx: introduce function serdes_device_from_fm_port()

2012-08-14 Thread Timur Tabi
In order to figure out which SerDes lane a given Fman port is connected to, we need a function that maps the fm_port namespace to the srds_prtcl namespace. Signed-off-by: Timur Tabi --- board/freescale/common/fman.c | 31 +++ board/freescale/common/fman.h |2

[U-Boot] [PATCH 4/7] fm-eth: use fdt_status_disabled() function in ft_fixup_port()

2012-08-14 Thread Timur Tabi
We have a dedicated function for setting the node status now, so use it. Also improve a comment and fix the type of the phandle variable. Signed-off-by: Timur Tabi --- drivers/net/fm/init.c | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/net/fm/init.c

[U-Boot] [PATCH 2/7] fm-eth: add function fm_info_get_phy_address()

2012-08-14 Thread Timur Tabi
Function fm_info_get_phy_address() returns the PHY address for a given Fman port. This is handy when the MDIO code needs to fixup the Ethernet nodes in the device tree to point to PHY nodes for a specific PHY address. Signed-off-by: Timur Tabi --- drivers/net/fm/init.c | 16

[U-Boot] [PATCH 5/7] powerpc/85xx: get rid of enum board_slots in P4080 MDIO driver

2012-08-14 Thread Timur Tabi
enum board_slots contained six values, where SLOT1 == 1, SLOT2 == 2, and so on. This is pointless, so remove it. Also move the lane_to_slot[] array to the top of the file so that it can be used by other functions. Signed-off-by: Timur Tabi --- board/freescale/corenet_ds/eth_p4080.c | 59

Re: [U-Boot] [u-boot-release] [PATCH 3/6] powerpc/85xx: introduce function serdes_device_from_fm_port()

2012-08-13 Thread Timur Tabi
Kim Phillips wrote: > it's ~30% less lines of code, and the maps lie on the same line, > which helps when grepping. OK. -- Timur Tabi Linux kernel developer at Freescale ___ U-Boot mailing list U-Boot@lists.denx.de http://lists.denx

Re: [U-Boot] [u-boot-release] [PATCH 3/6] powerpc/85xx: introduce function serdes_device_from_fm_port()

2012-08-13 Thread Timur Tabi
, [FM2_DTSEC5] = SGMII_FM2_DTSEC5, [FM2_10GEC1] = XAUI_FM2, }; if ((port < FM1_DTSEC1) || (port > FM2_10GEC1)) return NONE; else return srds_table[port]; I'm not sure that's an

[U-Boot] [PATCH 3/6] powerpc/85xx: introduce function serdes_device_from_fm_port()

2012-08-10 Thread Timur Tabi
In order to figure out which SerDes lane a given Fman port is connected to, we need a function that maps the fm_port namespace to the srds_prtcl namespace. Signed-off-by: Timur Tabi --- board/freescale/common/fman.c | 40 board/freescale/common/fman.h

[U-Boot] [PATCH 5/6] powerpc/85xx: get rid of enum board_slots in P4080 MDIO driver

2012-08-10 Thread Timur Tabi
enum board_slots contained six values, where SLOT1 == 1, SLOT2 == 2, and so on. This is pointless, so remove it. Also move the lane_to_slot[] array to the top of the file so that it can be used by other functions. Signed-off-by: Timur Tabi --- board/freescale/corenet_ds/eth_p4080.c | 59

[U-Boot] [PATCH 4/6] fm-eth: use fdt_status_disabled() function in ft_fixup_port()

2012-08-10 Thread Timur Tabi
We have a dedicated function for setting the node status now, so use it. Also improve a comment and fix the type of the phandle variable. Signed-off-by: Timur Tabi --- drivers/net/fm/init.c | 11 +-- 1 files changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/net/fm/init.c

[U-Boot] [PATCH 6/6] powerpc/85xx: update P4080DS MDIO bus multiplexer support

2012-08-10 Thread Timur Tabi
-boot code depends on several device tree aliases to help it find the nodes that need to be updated. Signed-off-by: Timur Tabi --- board/freescale/corenet_ds/eth_p4080.c | 172 1 files changed, 129 insertions(+), 43 deletions(-) diff --git a/board/freescale/coren

[U-Boot] [PATCH 1/6] powerpc/85xx: add support for FM2 DTSEC5

2012-08-10 Thread Timur Tabi
Unlike previous SOCs, the Freescale P5040 has a fifth DTSEC on the second Fman, so add the Fman and SerDes macros for that DTSEC. Signed-off-by: Timur Tabi --- arch/powerpc/cpu/mpc85xx/fsl_corenet_serdes.c |6 ++ arch/powerpc/include/asm/fsl_serdes.h |1 + arch/powerpc

  1   2   3   4   5   6   7   >