public inbox for u-boot@lists.denx.de
 help / color / mirror / Atom feed
* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
@ 2008-01-09 21:36 Wolfgang Denk
  2008-01-09 21:45 ` Timur Tabi
                   ` (12 more replies)
  0 siblings, 13 replies; 24+ messages in thread
From: Wolfgang Denk @ 2008-01-09 21:36 UTC (permalink / raw)
  To: u-boot

Hello everybody,

this is a heads-up especially for all custodians...

Please review the following list of postings / patches that - to the
best of my knowledge - habve not been processed yet; at least I did
not mark them as rejected, nor did I find them in the current U-Boot
tree.

If possible, please apply and send pull requests ASAP!   Thanks.

[It would be helpful to me if you could include my private message ID
(the number in column 1) when referring to  one  of  these  postings;
thanks.]

 9799  09/18 urwithsughosh at gma  Re: [U-Boot-Users] [PATCH]: Reset issue fix on mpc85xx
 9804  09/18 Jason Jin          [PATCH] fsl_pci_init Fix command register initialization issue.
 9952  09/24 Anatolij Gustschi  [U-Boot-Users] [PATCH] Add Fujitsu CoralP/Lime video driver
 9967  09/24 Troy Kisky         Re: [U-Boot-Users] PATCH - Fix oob data copied into supplied buffer
 9969  09/25 Shinya Kuribayash  Re: [U-Boot-Users] [MIPS] Make mips_io_port_base more generic
10068  09/28 Vasiliy Leoenenko  [PATCH] M18 flash (Sibley) support (attempt 2)
10078  09/28 Dan Wilson         [U-Boot-Users] [PATCH] tsec.c patch for receiver reset
10081  09/29 Shinya Kuribayash  [MIPS] Make mips_io_port_base more generic
10159  10/04 Vlad Lungu         [U-Boot-Users] [PATCH] Target support for qemu -M mips
10281  10/10 Vlad Lungu         [PATCH] Fixed pcnet io_base
10282  10/10 Vlad Lungu         [PATCH] Added Am79C970A chip id to pcnet
10397  10/12 Tony Li            [U-Boot-Users] [PATCH] mpc83xx: Add ATM makefile entry for MPC8360EM
10676  10/22 Anton Vorontsov    [U-Boot-Users] [PATCH] MPC8568E-MDS: reset UCCs to use them reliably
10686  10/23 Shinya Kuribayash  Re: [U-Boot-Users] [PATCH 1/1] Fix mips_io_port_base multiple declar
10690  10/22 Anton Vorontsov    [U-Boot-Users] [PATCH] MPC8568E-MDS: set up QE pario for UART1
10810  10/25 Johannes Stezenba  [U-Boot-Users] [patch 0/3] KB9202 CodeSourcery ARM EABI toolchain bu
11193  11/07 llandre            [U-Boot-Users] [PATCH] ARM optimized memcpy/memmove routines
11230  11/09 "Martin Krause"    RE: [U-Boot-Users] [PATCH] TQM5200B: update MTD partition layout
11473  11/19 Matthias Fuchs     Re: [U-Boot-Users] flash hardware protection with CFI driver does no
11522  11/20 stefano babic      [U-Boot-Users] [PATCH] Fix gcc issues in pxa-regs.h
11523  11/20 stefano babic      [U-Boot-Users] [PATCH] MMC for PXA 27X (resubmit)
11635  11/23 sivaji             [U-Boot-Users] MPC8548 problem in Boot Sequencer mode
11911  12/01 Daniel Mack        Re: [U-Boot-Users] [PATCH] add support for Toradex Colibri module
11928  12/03 Yoshihiro Shimoda  [PATCH] serial_sh: Add support for SH7720
11929  12/03 Yoshihiro Shimoda  [PATCH] sh: Add support SH3 and SH7720
11930  12/03 Yoshihiro Shimoda  [PATCH] Add support for MS7720RP02 board
12002  12/06 Ladislav Michl     [U-Boot-Users] [PATCH] ARM: Fix at91rm9200dk base address
12003  12/06 Ladislav Michl     [U-Boot-Users] [PATCH] ARM: AT91RM9200 based boards config cleanup
12007  12/07 Ladislav Michl     [U-Boot-Users] [PATCH] ARM: Board voiceblue update
12044  12/08 Jean-Christophe P  Re: [U-Boot-Users] PATCH: add support for MII-connected ethernet swi
12046  12/08 Jean-Christophe P  Re: [U-Boot-Users] PATCH: ixp42x UART: support 230400bps, enable RTS
12057  12/09 Jean-Christophe P  Re: [U-Boot-Users] PATCH: add support for MII-connected ethernet swi
12096  12/10 Jean-Christophe P  [U-Boot-Users] [PATCH] bf537-stamp: remove already defined is_zero_e
12257  12/13 Jean-Christophe P  [U-Boot-Users] [PATCH] Nios2: remove common/cmd_bdinfo.c unused vari
12382  12/18 Hebbar             Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function on
12383  12/18 Hebbar             Re: [U-Boot-Users] [PATCH] ARM926: Remove compiler warning: target C
12384  12/18 Hebbar             Re: [U-Boot-Users] [PATCH] ARM: I2C Init function call in lib_arm/bo
12385  12/18 Hebbar             Re: [U-Boot-Users] [PATCH] ARM926: Display Ethernet info in do_bdinf
12390  12/19 Kumar Gala         [U-Boot-Users] [PATCH] Reworked FSL Book-E TLB macros to be more rea
12392  12/19 Joakim Tjernlund   Re: [U-Boot-Users] 83xx, FSL_UEC reducing boot latency, printf causi
12393  12/19 Stefan Roese       [U-Boot-Users] fdt chosen node handling
12394  12/19 Joakim Tjernlund   Re: [U-Boot-Users] 83xx, FSL_UEC reducing boot latency, printf causi
12395  12/19 "Gururaja Hebbar   Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function on
12396  12/19 Dirk Behme         Re: [U-Boot-Users] [PATCH] ARM926: compile cpu_init_crit function on
12404  12/19 Harald Welte       [U-Boot-Users] [PATCH] ARM: arm920/s3c24xx: IRQ demulitplexer callba
12405  12/19 Harald Welte       [U-Boot-Users] [PATCH] ARM: arm920t: Allow use of 'gd' pointer from 
12408  12/19 Harald Welte       [U-Boot-Users] [PATCH] ARM: s3c24xx: Multiple serial port support
12412  12/19 Harald Welte       [U-Boot-Users] [PATCH] ARM: S3C24x0 SoC NAND controller support
12510  12/22 Jean-Christophe P  [U-Boot-Users] [PATCH 1/1] qemu-mips: active HUSH PARSER, AUTO_COMPL
12548  12/25 Anton Vorontsov    [U-Boot-Users] [PATCH] mpc83xx: UEC: add support for Broadcom BCM548
12549  12/25 Anton Vorontsov    [U-Boot-Users] [PATCH] mpc83xx: add support for the MPC8360E-RDK
12602  12/27 Wolfgang Denk      [U-Boot-Users] [PATCH] OMAP5912: fix FIFO handling in UART driver
12818  01/04 Jon Loeliger       [U-Boot-Users] [PATCH] Convert MPC8610HPCD to use libfdt.
12820  01/04 Timur Tabi         Re: [U-Boot-Users] [PATCH] 8610: Fix lingering compile warnings.
12859  01/07 Timur Tabi         [U-Boot-Users] [PATCH v2] 85xx: add ability to upload QE firmware
12866  01/07 Kumar Gala         Re: [U-Boot-Users] [PATCH] MPC85xx KGDB support
12870  01/08 Kumar Gala         [U-Boot-Users] [PATCH] 85xx: Remove cache config from configs.h
12934  01/08 Ben Warren         [U-Boot-Users] [PATCH] Fix Ethernet init() return codes
12938  01/08 Ben Warren         [U-Boot-Users] [PATCH V2] Fix Ethernet init() return codes.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Applying computer technology is simply finding the  right  wrench  to
pound in the correct screw.

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
@ 2008-01-09 21:45 ` Timur Tabi
  2008-01-09 21:47 ` Jerry Van Baren
                   ` (11 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Timur Tabi @ 2008-01-09 21:45 UTC (permalink / raw)
  To: u-boot

Wolfgang Denk wrote:
> 12820  01/04 Timur Tabi         Re: [U-Boot-Users] [PATCH] 8610: Fix lingering compile warnings.

This is just a comment to a patch from Jon.

> 12859  01/07 Timur Tabi         [U-Boot-Users] [PATCH v2] 85xx: add ability to upload QE firmware

Andy sent a pull request earlier today to pick up this patch.

-- 
Timur Tabi
Linux kernel developer at Freescale

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
  2008-01-09 21:45 ` Timur Tabi
@ 2008-01-09 21:47 ` Jerry Van Baren
  2008-01-09 22:47 ` Andy Fleming
                   ` (10 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Jerry Van Baren @ 2008-01-09 21:47 UTC (permalink / raw)
  To: u-boot

Wolfgang Denk wrote:
> Hello everybody,
> 
> this is a heads-up especially for all custodians...
> 
> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.
> 
> If possible, please apply and send pull requests ASAP!   Thanks.
> 
> [It would be helpful to me if you could include my private message ID
> (the number in column 1) when referring to  one  of  these  postings;
> thanks.]

> 12393  12/19 Stefan Roese       [U-Boot-Users] fdt chosen node handling

This was discussion email, culminating in my patch "[PATCH] Fine grained 
per property /chosen updating." which has been applied.
   <http://article.gmane.org/gmane.comp.boot-loaders.u-boot/34563/>

Thanks,
gvb

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
  2008-01-09 21:45 ` Timur Tabi
  2008-01-09 21:47 ` Jerry Van Baren
@ 2008-01-09 22:47 ` Andy Fleming
  2008-01-09 23:17 ` Matthias Fuchs
                   ` (9 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Andy Fleming @ 2008-01-09 22:47 UTC (permalink / raw)
  To: u-boot

On Jan 9, 2008 3:36 PM, Wolfgang Denk <wd@denx.de> wrote:

>  9799  09/18 urwithsughosh at gma  Re: [U-Boot-Users] [PATCH]: Reset issue fix on mpc85xx

A version of this patch is already committed:
df90968b48fb34fa9072fab150db2ac89678f537

>  9804  09/18 Jason Jin          [PATCH] fsl_pci_init Fix command register initialization issue.

> 10078  09/28 Dan Wilson         [U-Boot-Users] [PATCH] tsec.c patch for receiver reset

This one was commited: 5c7ea64bb74a850a2b2303f853a8270695ad8602


> 10676  10/22 Anton Vorontsov    [U-Boot-Users] [PATCH] MPC8568E-MDS: reset UCCs to use them reliably
> 10690  10/22 Anton Vorontsov    [U-Boot-Users] [PATCH] MPC8568E-MDS: set up QE pario for UART1

These two are in the current pull request


> 11635  11/23 sivaji             [U-Boot-Users] MPC8548 problem in Boot Sequencer mode

This isn't a patch, but I am tracking the issue.  :)


> 12390  12/19 Kumar Gala         [U-Boot-Users] [PATCH] Reworked FSL Book-E TLB macros to be more rea

This is in the current pull request


> 12859  01/07 Timur Tabi         [U-Boot-Users] [PATCH v2] 85xx: add ability to upload QE firmware

In the current pull request

> 12866  01/07 Kumar Gala         Re: [U-Boot-Users] [PATCH] MPC85xx KGDB support

This one has some outstanding issues, so unless I hear something from
the author (not Kumar, but I don't have the name in front of me at the
moment), it won't go into the next release.

> 12870  01/08 Kumar Gala         [U-Boot-Users] [PATCH] 85xx: Remove cache config from configs.h

In the current pull request


Oh, and sorry about top-posting and over-quoting.  I can only guess it
was because I was tired.  :)

I have rebased my tree, and will send another pull request soon.

Andy

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (2 preceding siblings ...)
  2008-01-09 22:47 ` Andy Fleming
@ 2008-01-09 23:17 ` Matthias Fuchs
  2008-01-10  8:48 ` Haavard Skinnemoen
                   ` (8 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Matthias Fuchs @ 2008-01-09 23:17 UTC (permalink / raw)
  To: u-boot

On Wednesday 09 January 2008 22:36, Wolfgang Denk wrote:
> 11473  11/19 Matthias Fuchs     Re: [U-Boot-Users] flash hardware protection with CFI driver does no

This issue did not made it into any patch. We discussed it and ended up
with an idea how to add a config option to allow board specific protected flash ranges.

Because I have an open issue on this I will try to provide a patch 'til the end of this week.

Wolfgang, thanks for the reminder.

Matthias

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (3 preceding siblings ...)
  2008-01-09 23:17 ` Matthias Fuchs
@ 2008-01-10  8:48 ` Haavard Skinnemoen
  2008-01-10 10:50 ` Jean-Christophe PLAGNIOL-VILLARD
                   ` (7 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Haavard Skinnemoen @ 2008-01-10  8:48 UTC (permalink / raw)
  To: u-boot

On Wed, 09 Jan 2008 22:36:14 +0100
Wolfgang Denk <wd@denx.de> wrote:

> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.

I can't see this one anywhere:

[PATCH] Conditionally add -fno-stack-protector to CFLAGS
http://groups.google.com.mx/group/osdeve_mirror_boot-loaders_u-boot/browse_thread/thread/c7a0045307cde22

It doesn't matter all that much to me, but I get lots of build errors
on powerpc without it. Other arches may be affected too.

Haavard

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (4 preceding siblings ...)
  2008-01-10  8:48 ` Haavard Skinnemoen
@ 2008-01-10 10:50 ` Jean-Christophe PLAGNIOL-VILLARD
  2008-01-10 15:34 ` Scott McNutt
                   ` (6 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Jean-Christophe PLAGNIOL-VILLARD @ 2008-01-10 10:50 UTC (permalink / raw)
  To: u-boot

On 22:36 Wed 09 Jan     , Wolfgang Denk wrote:
> Hello everybody,
> 
> this is a heads-up especially for all custodians...
> 
> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.
> 
> If possible, please apply and send pull requests ASAP!   Thanks.
> 
> [It would be helpful to me if you could include my private message ID
> (the number in column 1) when referring to  one  of  these  postings;
> thanks.]

> 12046  12/08 Jean-Christophe P  Re: [U-Boot-Users] PATCH: ixp42x UART: support 230400bps, enable RTS
Replace by IXP: Add full baud-rate support for ixp42x, ixp45x and ixp46x
	and already in the tree as commit 7b74ebe723e576baedf5a8b6240589b19b845a1b

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (5 preceding siblings ...)
  2008-01-10 10:50 ` Jean-Christophe PLAGNIOL-VILLARD
@ 2008-01-10 15:34 ` Scott McNutt
  2008-01-10 20:43 ` Ben Warren
                   ` (5 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Scott McNutt @ 2008-01-10 15:34 UTC (permalink / raw)
  To: u-boot


> If possible, please apply and send pull requests ASAP!   Thanks.

> 12257  12/13 Jean-Christophe P  [U-Boot-Users] [PATCH] Nios2: remove common/cmd_bdinfo.c unused vari

Done.

Regards,
--Scott

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (6 preceding siblings ...)
  2008-01-10 15:34 ` Scott McNutt
@ 2008-01-10 20:43 ` Ben Warren
  2008-01-11  4:04 ` Kim Phillips
                   ` (4 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Ben Warren @ 2008-01-10 20:43 UTC (permalink / raw)
  To: u-boot

Wolfgang Denk wrote:
> Hello everybody,
>
> this is a heads-up especially for all custodians...
>
> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.
>
> If possible, please apply and send pull requests ASAP!   Thanks.
>
> [It would be helpful to me if you could include my private message ID
> (the number in column 1) when referring to  one  of  these  postings;
> thanks.]
>
>   
<snip>
> 10078  09/28 Dan Wilson         [U-Boot-Users] [PATCH] tsec.c patch for receiver reset
>   
Looks like Andy already applied this one:
http://article.gmane.org/gmane.comp.boot-loaders.u-boot/32643
> 12044  12/08 Jean-Christophe P  Re: [U-Boot-Users] PATCH: add support for MII-connected ethernet swi
>
> 12057  12/09 Jean-Christophe P  Re: [U-Boot-Users] PATCH: add support for MII-connected ethernet swi
>   
These are both responses to a patch submitted by Michael Schwingen. A 
revised patch was never submitted that I know of.
> 12096  12/10 Jean-Christophe P  [U-Boot-Users] [PATCH] bf537-stamp: remove already defined is_zero_e
>   
I'll pull this one in.
> 12548  12/25 Anton Vorontsov    [U-Boot-Users] [PATCH] mpc83xx: UEC: add support for Broadcom BCM548
>   
I've asked Anton for changes to this one
> 12934  01/08 Ben Warren         [U-Boot-Users] [PATCH] Fix Ethernet init() return codes
> 12938  01/08 Ben Warren         [U-Boot-Users] [PATCH V2] Fix Ethernet init() return codes.
>   
You've pulled this in. Sorry for being Captain Obvious.

regards,
Ben

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (7 preceding siblings ...)
  2008-01-10 20:43 ` Ben Warren
@ 2008-01-11  4:04 ` Kim Phillips
  2008-01-11 10:06 ` Martin Krause
                   ` (3 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Kim Phillips @ 2008-01-11  4:04 UTC (permalink / raw)
  To: u-boot

On Wed, 09 Jan 2008 22:36:14 +0100
Wolfgang Denk <wd@denx.de> wrote:

> 10397  10/12 Tony Li            [U-Boot-Users] [PATCH] mpc83xx: Add ATM makefile entry for MPC8360EM

this is already in your mainline tree (commit
281df457c1aa50d2752165d0c5c3282d4027b974).

> 12392  12/19 Joakim Tjernlund   Re: [U-Boot-Users] 83xx, FSL_UEC reducing boot latency, printf causi

this was never intended to be a final patch; we're still working out the
issues.

> 12394  12/19 Joakim Tjernlund   Re: [U-Boot-Users] 83xx, FSL_UEC reducing boot latency, printf causi

same as above.  I'll try and get back to it before the Jan. 17 deadline.

> 12548  12/25 Anton Vorontsov    [U-Boot-Users] [PATCH] mpc83xx: UEC: add support for Broadcom BCM548

Ben Warren is handling the newer version of this one (and it looks like
Anton will resubmit it again).

> 12549  12/25 Anton Vorontsov    [U-Boot-Users] [PATCH] mpc83xx: add support for the MPC8360E-RDK

I'm about to ask you to pull the newer version of this.

so, assuming you pull on my next request, the above can be removed from
your list.

Kim

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (8 preceding siblings ...)
  2008-01-11  4:04 ` Kim Phillips
@ 2008-01-11 10:06 ` Martin Krause
  2008-01-11 21:06 ` Michael Schwingen
                   ` (2 subsequent siblings)
  12 siblings, 0 replies; 24+ messages in thread
From: Martin Krause @ 2008-01-11 10:06 UTC (permalink / raw)
  To: u-boot

u-boot-users-bounces at lists.sourceforge.net wrote on :
> Hello everybody,
> 
> this is a heads-up especially for all custodians...
> 
> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.
> 
> If possible, please apply and send pull requests ASAP!   Thanks.
> 
> [It would be helpful to me if you could include my private message ID
> (the number in column 1) when referring to  one  of  these  postings;
> thanks.]
> 
> 11230 11/09 "Martin Krause"    RE: [U-Boot-Users] [PATCH] TQM5200B: update
> MTD partition layout

After the first version of the patch was NAKed, I reveived no (positiv) 
answer to my then proposed alternative flash layout and the offer to submit
a new patch. So I assumed the patch will also not be accepted and did no
further work on this.

Best regards,
Martin Krause

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (9 preceding siblings ...)
  2008-01-11 10:06 ` Martin Krause
@ 2008-01-11 21:06 ` Michael Schwingen
  2008-01-12  6:27   ` Stefan Roese
  2008-01-14  8:01   ` [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Haavard Skinnemoen
  2008-01-15 14:48 ` Shinya Kuribayashi
  2008-01-15 15:39 ` Nobuhiro Iwamatsu
  12 siblings, 2 replies; 24+ messages in thread
From: Michael Schwingen @ 2008-01-11 21:06 UTC (permalink / raw)
  To: u-boot

(resend due to mailserver problems)

On Wed, Jan 09, 2008 at 10:36:14PM +0100, Wolfgang Denk wrote:
> Hello everybody,
> 
> this is a heads-up especially for all custodians...
> 
> Please review the following list of postings / patches that - to the
> best of my knowledge - habve not been processed yet; at least I did
> not mark them as rejected, nor did I find them in the current U-Boot
> tree.

Not sure about the source of the problem, but the CONFIG_CFI_FLASH_LEGACY
support seems to be broken - the code for
flash_read_jedec_ids_intel and flash_read_jedec_ids_amd which are called in
cfi_flash.c seems to be missing. I do not see any missing/outstanding
patches that seem relevant to this problem.

Am I looking at the wrong branch?

cu
Michael

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-11 21:06 ` Michael Schwingen
@ 2008-01-12  6:27   ` Stefan Roese
  2008-01-12 13:43     ` [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation Michael Schwingen
  2008-01-14  8:01   ` [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Haavard Skinnemoen
  1 sibling, 1 reply; 24+ messages in thread
From: Stefan Roese @ 2008-01-12  6:27 UTC (permalink / raw)
  To: u-boot

Hi Michael,

On Friday 11 January 2008, Michael Schwingen wrote:
> > this is a heads-up especially for all custodians...
> >
> > Please review the following list of postings / patches that - to the
> > best of my knowledge - habve not been processed yet; at least I did
> > not mark them as rejected, nor did I find them in the current U-Boot
> > tree.
>
> Not sure about the source of the problem, but the CONFIG_CFI_FLASH_LEGACY
> support seems to be broken - the code for
> flash_read_jedec_ids_intel and flash_read_jedec_ids_amd which are called in
> cfi_flash.c seems to be missing. I do not see any missing/outstanding
> patches that seem relevant to this problem.
>
> Am I looking at the wrong branch?

No. You CFI/JEDEC rework should be available in the official repository right 
now. Could be that something slipped though. Best would be if you would 
provide a patch that adds the missing JEDEC stuff.

Thanks.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation
  2008-01-12  6:27   ` Stefan Roese
@ 2008-01-12 13:43     ` Michael Schwingen
  2008-01-12 18:48       ` Wolfgang Denk
  0 siblings, 1 reply; 24+ messages in thread
From: Michael Schwingen @ 2008-01-12 13:43 UTC (permalink / raw)
  To: u-boot

On Sat, Jan 12, 2008 at 07:27:49AM +0100, Stefan Roese wrote:
> 
> No. You CFI/JEDEC rework should be available in the official repository right 
> now. Could be that something slipped though. Best would be if you would 
> provide a patch that adds the missing JEDEC stuff.

Okay - after a closer look, just a small change is required.

Signed-off-by: Michael Schwingen <michael@schwingen.org>

diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
index 4f61e36..3a3439c 100644
--- a/drivers/mtd/cfi_flash.c
+++ b/drivers/mtd/cfi_flash.c
@@ -1446,11 +1446,11 @@ static void flash_read_jedec_ids (flash_info_t * info)
 	switch (info->vendor) {
 	case CFI_CMDSET_INTEL_STANDARD:
 	case CFI_CMDSET_INTEL_EXTENDED:
-		flash_read_jedec_ids_intel(info);
+		cmdset_amd_read_jedec_ids(info);
 		break;
 	case CFI_CMDSET_AMD_STANDARD:
 	case CFI_CMDSET_AMD_EXTENDED:
-		flash_read_jedec_ids_amd(info);
+		cmdset_intel_read_jedec_ids(info);
 		break;
 	default:
 		break;

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation
  2008-01-12 13:43     ` [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation Michael Schwingen
@ 2008-01-12 18:48       ` Wolfgang Denk
  2008-01-12 19:29         ` Michael Schwingen
  0 siblings, 1 reply; 24+ messages in thread
From: Wolfgang Denk @ 2008-01-12 18:48 UTC (permalink / raw)
  To: u-boot

In message <20080112134301.GA6978@discworld.dascon.de> you wrote:
> On Sat, Jan 12, 2008 at 07:27:49AM +0100, Stefan Roese wrote:
> > 
> > No. You CFI/JEDEC rework should be available in the official repository right 
> > now. Could be that something slipped though. Best would be if you would 
> > provide a patch that adds the missing JEDEC stuff.
> 
> Okay - after a closer look, just a small change is required.

Hm... either you got the patch inverse, or something is seriously
broken:

>  	case CFI_CMDSET_INTEL_STANDARD:
>  	case CFI_CMDSET_INTEL_EXTENDED:
> -		flash_read_jedec_ids_intel(info);
> +		cmdset_amd_read_jedec_ids(info);
>  		break;

For INTEL flashes use the AMD command set, and ...

>  	case CFI_CMDSET_AMD_STANDARD:
>  	case CFI_CMDSET_AMD_EXTENDED:
> -		flash_read_jedec_ids_amd(info);
> +		cmdset_intel_read_jedec_ids(info);

...for AMD flashes use the INTEL command set?

This makes no sense to me.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
"If anything can go wrong, it will."                   - Edsel Murphy

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation
  2008-01-12 18:48       ` Wolfgang Denk
@ 2008-01-12 19:29         ` Michael Schwingen
  2008-01-13 14:09           ` Stefan Roese
  0 siblings, 1 reply; 24+ messages in thread
From: Michael Schwingen @ 2008-01-12 19:29 UTC (permalink / raw)
  To: u-boot

On Sat, Jan 12, 2008 at 07:48:26PM +0100, Wolfgang Denk wrote:
> Hm... either you got the patch inverse, or something is seriously
> broken:

I vote for the second option - don't know what got into me. The strange
thing is that it worked, nevertheless ...

cu
Michael

Signed-off-by: Michael Schwingen <michael@schwingen.org>

diff --git a/drivers/mtd/cfi_flash.c b/drivers/mtd/cfi_flash.c
index 4f61e36..eb509f5 100644
--- a/drivers/mtd/cfi_flash.c
+++ b/drivers/mtd/cfi_flash.c
@@ -1446,11 +1446,11 @@ static void flash_read_jedec_ids (flash_info_t * info)
 	switch (info->vendor) {
 	case CFI_CMDSET_INTEL_STANDARD:
 	case CFI_CMDSET_INTEL_EXTENDED:
-		flash_read_jedec_ids_intel(info);
+		cmdset_intel_read_jedec_ids(info);
 		break;
 	case CFI_CMDSET_AMD_STANDARD:
 	case CFI_CMDSET_AMD_EXTENDED:
-		flash_read_jedec_ids_amd(info);
+		cmdset_amd_read_jedec_ids(info);
 		break;
 	default:
 		break;

^ permalink raw reply related	[flat|nested] 24+ messages in thread

* [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation
  2008-01-12 19:29         ` Michael Schwingen
@ 2008-01-13 14:09           ` Stefan Roese
  0 siblings, 0 replies; 24+ messages in thread
From: Stefan Roese @ 2008-01-13 14:09 UTC (permalink / raw)
  To: u-boot

On Saturday 12 January 2008, Michael Schwingen wrote:
> On Sat, Jan 12, 2008 at 07:48:26PM +0100, Wolfgang Denk wrote:
> > Hm... either you got the patch inverse, or something is seriously
> > broken:
>
> I vote for the second option - don't know what got into me. The strange
> thing is that it worked, nevertheless ...
>
> cu
> Michael
>
> Signed-off-by: Michael Schwingen <michael@schwingen.org>

Thanks. Applied to cfi-flash repository.

Pull request will follow soon.

Best regards,
Stefan

=====================================================================
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: +49-8142-66989-0 Fax: +49-8142-66989-80  Email: office at denx.de
=====================================================================

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-11 21:06 ` Michael Schwingen
  2008-01-12  6:27   ` Stefan Roese
@ 2008-01-14  8:01   ` Haavard Skinnemoen
  2008-01-14  8:07     ` Michael Schwingen
  1 sibling, 1 reply; 24+ messages in thread
From: Haavard Skinnemoen @ 2008-01-14  8:01 UTC (permalink / raw)
  To: u-boot

On Fri, 11 Jan 2008 22:06:30 +0100
rincewind at discworld.dascon.de (Michael Schwingen) wrote:

> Not sure about the source of the problem, but the
> CONFIG_CFI_FLASH_LEGACY support seems to be broken - the code for
> flash_read_jedec_ids_intel and flash_read_jedec_ids_amd which are
> called in cfi_flash.c seems to be missing. I do not see any
> missing/outstanding patches that seem relevant to this problem.

Hmm...I guess my compile tests didn't hit the CONFIG_CFI_FLASH_LEGACY
case. Are there any boards that are actually using this in mainline?

Thanks for fixing it.

Haavard

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-14  8:01   ` [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Haavard Skinnemoen
@ 2008-01-14  8:07     ` Michael Schwingen
  0 siblings, 0 replies; 24+ messages in thread
From: Michael Schwingen @ 2008-01-14  8:07 UTC (permalink / raw)
  To: u-boot

Haavard Skinnemoen wrote:
> Hmm...I guess my compile tests didn't hit the CONFIG_CFI_FLASH_LEGACY
> case. Are there any boards that are actually using this in mainline?
>   
Not yet. AcTux-4 will be, as soon as it is fit for inclusion.

cu
Michael

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (10 preceding siblings ...)
  2008-01-11 21:06 ` Michael Schwingen
@ 2008-01-15 14:48 ` Shinya Kuribayashi
  2008-01-16 10:16   ` Vlad Lungu
  2008-01-15 15:39 ` Nobuhiro Iwamatsu
  12 siblings, 1 reply; 24+ messages in thread
From: Shinya Kuribayashi @ 2008-01-15 14:48 UTC (permalink / raw)
  To: u-boot

Hi,

Wolfgang Denk wrote:
>  9969  09/25 Shinya Kuribayash  Re: [U-Boot-Users] [MIPS] Make mips_io_port_base more generic
> 10081  09/29 Shinya Kuribayash  [MIPS] Make mips_io_port_base more generic
> 10686  10/23 Shinya Kuribayash  Re: [U-Boot-Users] [PATCH 1/1] Fix mips_io_port_base multiple declar

these are already in v1.3.0. Please drop them from your list. 

> 10159  10/04 Vlad Lungu         [U-Boot-Users] [PATCH] Target support for qemu -M mips
> 12510  12/22 Jean-Christophe P  [U-Boot-Users] [PATCH 1/1] qemu-mips: active HUSH PARSER, AUTO_COMPL

I don't feel no need to be in a rush to these stuffs. And they needs some
possible cleanups, a MAINTAINER file entry, and so on. With these changes,
I'll pull request in the future.

  Shinya

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
                   ` (11 preceding siblings ...)
  2008-01-15 14:48 ` Shinya Kuribayashi
@ 2008-01-15 15:39 ` Nobuhiro Iwamatsu
  12 siblings, 0 replies; 24+ messages in thread
From: Nobuhiro Iwamatsu @ 2008-01-15 15:39 UTC (permalink / raw)
  To: u-boot

On Wed, 09 Jan 2008 22:36:14 +0100
Wolfgang Denk <wd@denx.de> wrote:

> 11928  12/03 Yoshihiro Shimoda  [PATCH] serial_sh: Add support for SH7720
> 11929  12/03 Yoshihiro Shimoda  [PATCH] sh: Add support SH3 and SH7720
> 11930  12/03 Yoshihiro Shimoda  [PATCH] Add support for MS7720RP02 board

I picked up and apply these patches. 
These patches are included in next request-pull. 

regards,
 Nobuhiro
-- 
Nobuhiro Iwamatsu
	GPG ID : 3170EBE9

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-15 14:48 ` Shinya Kuribayashi
@ 2008-01-16 10:16   ` Vlad Lungu
  2008-01-16 12:16     ` Shinya Kuribayashi
  0 siblings, 1 reply; 24+ messages in thread
From: Vlad Lungu @ 2008-01-16 10:16 UTC (permalink / raw)
  To: u-boot

Shinya Kuribayashi wrote:
>   
>> 10159  10/04 Vlad Lungu         [U-Boot-Users] [PATCH] Target support for qemu -M mips
>> 12510  12/22 Jean-Christophe P  [U-Boot-Users] [PATCH 1/1] qemu-mips: active HUSH PARSER, AUTO_COMPL
>>     
>
> I don't feel no need to be in a rush to these stuffs. And they needs some
> possible cleanups, a MAINTAINER file entry, and so on. With these changes,
> I'll pull request in the future.
>   
Just saw your comments on JC's patch. Last message in:

http://www.nabble.com/Target-support-for-qemu--M-mips-td14375017.html

Can you please be more specific about the issues you have with my patch? 
I haven't seen any negative feedback (or much positive feedback :-] )
on the list or in private e-mail. I'm open to suggestions, as long as I 
get any ;-)

As for pushing into mainline, I don't want to rush this, but I have sent 
and re-sent this since end of  September. If the next merge window opens 
in the summer,
I'll just give up.
 
Vlad

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-16 10:16   ` Vlad Lungu
@ 2008-01-16 12:16     ` Shinya Kuribayashi
  2008-01-16 12:18       ` Vlad Lungu
  0 siblings, 1 reply; 24+ messages in thread
From: Shinya Kuribayashi @ 2008-01-16 12:16 UTC (permalink / raw)
  To: u-boot

Vlad Lungu wrote:
> Just saw your comments on JC's patch. Last message in:
> 
> http://www.nabble.com/Target-support-for-qemu--M-mips-td14375017.html
> 
> Can you please be more specific about the issues you have with my patch? 
> I haven't seen any negative feedback (or much positive feedback :-] )
> on the list or in private e-mail. I'm open to suggestions, as long as I 
> get any ;-)

Ok, I'll review the patch. Wait a while.

> As for pushing into mainline, I don't want to rush this, but I have sent 
> and re-sent this since end of  September. If the next merge window opens 
> in the summer,
> I'll just give up.

So I have to push this now :-)

  Shinya

^ permalink raw reply	[flat|nested] 24+ messages in thread

* [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches
  2008-01-16 12:16     ` Shinya Kuribayashi
@ 2008-01-16 12:18       ` Vlad Lungu
  0 siblings, 0 replies; 24+ messages in thread
From: Vlad Lungu @ 2008-01-16 12:18 UTC (permalink / raw)
  To: u-boot

Shinya Kuribayashi wrote:
> Vlad Lungu wrote:
>> Just saw your comments on JC's patch. Last message in:
>>
>> http://www.nabble.com/Target-support-for-qemu--M-mips-td14375017.html
>>
>> Can you please be more specific about the issues you have with my 
>> patch? I haven't seen any negative feedback (or much positive 
>> feedback :-] )
>> on the list or in private e-mail. I'm open to suggestions, as long as 
>> I get any ;-)
>
> Ok, I'll review the patch. Wait a while.
>
OK.
>> As for pushing into mainline, I don't want to rush this, but I have 
>> sent and re-sent this since end of  September. If the next merge 
>> window opens in the summer,
>> I'll just give up.
>
> So I have to push this now :-)
You don't HAVE to, but it would be nice to consider it.

Vlad

^ permalink raw reply	[flat|nested] 24+ messages in thread

end of thread, other threads:[~2008-01-16 12:18 UTC | newest]

Thread overview: 24+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2008-01-09 21:36 [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Wolfgang Denk
2008-01-09 21:45 ` Timur Tabi
2008-01-09 21:47 ` Jerry Van Baren
2008-01-09 22:47 ` Andy Fleming
2008-01-09 23:17 ` Matthias Fuchs
2008-01-10  8:48 ` Haavard Skinnemoen
2008-01-10 10:50 ` Jean-Christophe PLAGNIOL-VILLARD
2008-01-10 15:34 ` Scott McNutt
2008-01-10 20:43 ` Ben Warren
2008-01-11  4:04 ` Kim Phillips
2008-01-11 10:06 ` Martin Krause
2008-01-11 21:06 ` Michael Schwingen
2008-01-12  6:27   ` Stefan Roese
2008-01-12 13:43     ` [U-Boot-Users] [PATCH] fix CONFIG_FLASH_CFI_LEGACY compilation Michael Schwingen
2008-01-12 18:48       ` Wolfgang Denk
2008-01-12 19:29         ` Michael Schwingen
2008-01-13 14:09           ` Stefan Roese
2008-01-14  8:01   ` [U-Boot-Users] CUSTODIANS: List of open issues / not applied patches Haavard Skinnemoen
2008-01-14  8:07     ` Michael Schwingen
2008-01-15 14:48 ` Shinya Kuribayashi
2008-01-16 10:16   ` Vlad Lungu
2008-01-16 12:16     ` Shinya Kuribayashi
2008-01-16 12:18       ` Vlad Lungu
2008-01-15 15:39 ` Nobuhiro Iwamatsu

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox