24 Jul, 2013

1 commit

  • Like many other projects, U-Boot has a tradition of including big
    blocks of License headers in all files. This not only blows up the
    source code with mostly redundant information, but also makes it very
    difficult to generate License Clearing Reports. An additional problem
    is that even the same lincenses are referred to by a number of
    slightly varying text blocks (full, abbreviated, different
    indentation, line wrapping and/or white space, with obsolete address
    information, ...) which makes automatic processing a nightmare.

    To make this easier, such license headers in the source files will be
    replaced with a single line reference to Unique Lincense Identifiers
    as defined by the Linux Foundation's SPDX project [1]. For example,
    in a source file the full "GPL v2.0 or later" header text will be
    replaced by a single line:

    SPDX-License-Identifier: GPL-2.0+

    We use the SPDX Unique Lincense Identifiers here; these are available
    at [2].

    Note: From the legal point of view, this patch is supposed to be only
    a change to the textual representation of the license information,
    but in no way any change to the actual license terms. With this patch
    applied, all files will still be licensed under the same terms they
    were before.

    Note 2: The apparent difference between the old "COPYING" and the new
    "Licenses/gpl-2.0.txt" only results from switching to the upstream
    version of the license which is differently formatted; there are not
    any actual changes to the content.

    Note 3: There are some recurring questions about linense issues, such
    as:
    - Is a "All Rights Reserved" clause a problem in GPL code?
    - Are files without any license header a problem?
    - Do we need license headers at all?

    The following excerpt from an e-mail by Daniel B. Ravicher should help
    with these:

    | Message-ID:
    | Date: Wed, 21 Oct 2009 18:35:22 -0400
    | From: "Daniel B. Ravicher"
    | To: Wolfgang Denk
    | Subject: Re: GPL and license cleanup questions
    |
    | Mr. Denk,
    |
    | Wolfgang Denk wrote:
    | > - There are a number of files which do not include any specific
    | > license information at all. Is it correct to assume that these files
    | > are automatically covered by the "GPL v2 or later" clause as
    | > specified by the COPYING file in the top level directory of the
    | > U-Boot source tree?
    |
    | That is a very fact specific analysis and could be different across the
    | various files. However, if the contributor could reasonably be expected
    | to have known that the project was licensed GPLv2 or later at the time
    | she made her contribution, then a reasonably implication is that she
    | consented to her contributions being distributed under those terms.
    |
    | > - Do such files need any clean up, for example should we add GPL
    | > headers to them, or is this not needed?
    |
    | If the project as a whole is licensed under clear terms, you need not
    | identify those same terms in each file, although there is no harm in
    | doing so.
    |
    | > - There are other files, which include both a GPL license header
    | > _plus_ some copyright note with an "All Rights Reserved" clause. It
    | > has been my understanding that this is a conflict, and me must ask
    | > the copyright holders to remove such "All Rights Reserved" clauses.
    | > But then, some people claim that "All Rights Reserved" is a no-op
    | > nowadays. License checking tools (like OSLC) seem to indicate this is
    | > a problem, but then we see quite a lot of "All rights reserved" in
    | > BSD-licensed files in gcc and glibc. So what is the correct way to
    | > deal with such files?
    |
    | It is not a conflict to grant a license and also reserve all rights, as
    | implicit in that language is that you are reserving all "other" rights
    | not granted in the license. Thus, a file with "Licensed under GPL, All
    | Rights Reserved" would mean that it is licensed under the GPL, but no
    | other rights are given to copy, modify or redistribute it.
    |
    | Warm regards,
    | --Dan
    |
    | Daniel B. Ravicher, Legal Director
    | Software Freedom Law Center (SFLC) and Moglen Ravicher LLC
    | 1995 Broadway, 17th Fl., New York, NY 10023
    | (212) 461-1902 direct (212) 580-0800 main (212) 580-0898 fax
    | ravicher@softwarefreedom.org www.softwarefreedom.org

    [1] http://spdx.org/
    [2] http://spdx.org/licenses/

    Signed-off-by: Wolfgang Denk

    Wolfgang Denk
     

23 Jul, 2013

1 commit


22 Jul, 2013

3 commits

  • The problem here is that uboot can't mount ext4 filesystem with
    commit "50ce4c07df1" applied. We use hard-coded "SECTOR_SIZE"(512)
    before this commit, now we introduce (block_dev_desc_t *)->log2blksz
    to replace this macro. And after we calling do_ls()->fs_set_blk_dev(),
    the variable log2blksz is not initialized, which it's not correct.

    And this patch try to solve the problem by caculating the value of
    log2blksz from variable blksz.

    Lan Yixun (dlan)
     
  • Fix reading ext4_extent_header struture on BE machines. Some 16 bit
    fields where converted to 32 bit fields, due to the byte swap on BE
    machines the containing value was corrupted. Therefore reading ext4
    filesystems on BE machines where broken before.

    Signed-off-by: Rommel Custodio
    [sent via git-send-email; rework commit message]
    Signed-off-by: Andreas Bießmann
    Reviewed-by: Simon Glass
    Tested-by: Simon Glass
    Tested-by: Lukasz Majewski

    Rommel Custodio
     
  • As both the DDR SDRAM and NAND flash connect to EBI on at91sam9n12
    and share the lower 8 bits data line. If use low drive of the data
    line, it will cause DDR data access corrupt in lower address, so
    change the data line to high drive mode

    This will fix the Linux kernel boot issue when use Lower address

    Signed-off-by: Bo Shen
    Acked-by: Andreas Bießmann

    Bo Shen
     

21 Jul, 2013

1 commit

  • The old value of 0x000e0030 will cause ethernet
    timeout issues on the sabrelite and possibly other
    boards using the KSZ9021.
    I have no explanation as to why.

    But this is a correct change, the TRM will be updated
    to show that 00b is the only valid setting for bits
    19-18 of DRAM_RESET.

    My thanks go to Liu Hui(Jason) for this information.

    Acked-by: Fabio Estevam
    Acked-by: Stefano Babic
    Signed-off-by: Troy Kisky

    Troy Kisky
     

20 Jul, 2013

2 commits

  • Commit de1d786e (add support for Xilinx 1000BASE-X phy (GTX)) introduced the
    checking for ESTATUS_1000_XHALF, but it incorrectly sets the
    SUPPORTED_1000baseX_Full flag in this case.

    Set the SUPPORTED_1000baseX_Half flag instead.

    Acked-by: Charles Coldwell
    Reviewed-By: Sascha Silbe
    Reviewed-by: Joe Hershberger
    Signed-off-by: Fabio Estevam

    Fabio Estevam
     
  • de1d786 [add support for Xilinx 1000BASE-X phy (GTX)] introduced a
    check for the extended status register in order to support
    1Gbps-capable PHYs that don't have the 1000BASE-T registers. Since
    Extended Status only indicates what the PHY (i.e. the local side) is
    capable of, this broke communication with non-1Gbps peers.

    Only check the extended status if the 1000BASE-T registers are
    actually missing so we don't end up setting speed to 1Gbps even though
    the previous test (for the combination of local and peer support for
    1Gbps) already indicated we can't do 1Gbps with the current peer.

    Signed-off-by: Sascha Silbe
    Tested-by: Fabio Estevam
    Reviewed-by: Joe Hershberger

    Sascha Silbe
     

19 Jul, 2013

2 commits


17 Jul, 2013

12 commits


16 Jul, 2013

8 commits

  • Before setting: mipi_lcd_device.reverse_panel = 1, the Trats's LCD panel
    was flipped by 180 degrees.

    The flip was caused by following change:
    Exynos: Change get_timer() to work correctly
    SHA1: 3d00c0cb96ff93a929700b80d89cb905e5ab5315

    This commit fixed udelay(), which is necessary (due to HW LCD controller
    oddity) for mipi-dsi correct operation. As a result the display orientation
    has been switched.

    As a follow up, the hwrevision() function has been removed, since it was
    used only in this particular place.

    Test HW: Trats Exynos4210 rev 0.

    Signed-off-by: Lukasz Majewski
    Signed-off-by: Kyungmin Park
    Cc: Minkyu Kang
    Acked-by: Minkyu Kang

    Łukasz Majewski
     
  • Provide proper setting for the APLL fout frequency calculation for
    Exynos4 based targets (especially Exynos4210 - Trats board).

    Signed-off-by: Lukasz Majewski
    Cc: Minkyu Kang
    Acked-by: Minkyu Kang
    Acked-by: Simon Glass
    Tested-by: Simon Glass

    Łukasz Majewski
     
  • Fix:

    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

    Wolfgang Denk
     
  • With CONFIG_SYS_64BIT_LBA, lbaint_t gets defined as a 64-bit type,
    which is required to represent block numbers for storage devices that
    exceed 2TiB (the block size usually is 512B), e.g. recent hard drives

    We now use lbaint_t for partition offset to reflect the lbaint_t change,
    and access partitions beyond or crossing the 2.1TiB limit.
    This required changes to signature of ext4fs_devread(), and type of all
    variables relatives to block sector.

    ext2/ext4 fs uses logical block represented by a 32 bit value. Logical
    block is a multiple of device block sector. To avoid overflow problem
    when calling ext4fs_devread(), we need to cast the sector parameter.

    Signed-off-by: Frédéric Leroy

    Frederic Leroy
     
  • Some ARM compilers may emit code that makes unaligned accesses when
    faced with constructs such as:

    char name[12] = "bootmenu_";

    same fix as commit: 064d55f8bc8d7d205ed0be6abb6717e92eeb7cad

    =========================================================
    data abort

    MAYBE you should read doc/README.arm-unaligned-accesses

    pc : [] lr : []
    sp : 3f346a58 ip : 3ff9c8e6 fp : 02000060
    r10: 00000000 r9 : 3df47fc0 r8 : 3f347f40
    r7 : 00000000 r6 : 00000000 r5 : 00000003 r4 : 3f759140
    r3 : 000003f0 r2 : 00000000 r1 : 000003f1 r0 : 00000000
    Flags: nzCv IRQs on FIQs off Mode SVC_32
    Resetting CPU ...
    ======================================================

    Signed-off-by: Lan Yixun (dlan)

    Lan Yixun (dlan)
     
  • "cramfsload uImage_1" succeeds even though the actual file is named
    "uImage".

    Fix file name comparison when one name is the prefix of the other.

    Signed-off-by: Holger Brunck
    cc: Wolfgang Denk
    cc: Albert ARIBAUD

    Holger Brunck
     
  • adjust instructions for the invocation of Patman's self test: the -t
    flag appears to have a different meaning now, refer to the --test option
    for the builtin unit test; adjust a directory location and make sure to
    run the file which resides in the source directory

    Signed-off-by: Gerhard Sittig
    Acked-by: Simon Glass

    Gerhard Sittig
     
  • The block device expects to see lbaint_t for the blknr parameter. Change
    the SCSI read/write functions to suit.

    This fixes the following build warnings for coreboot:

    cmd_scsi.c: In function ‘scsi_scan’:
    cmd_scsi.c:119:30: error: assignment from incompatible pointer type [-Werror]
    cmd_scsi.c:120:32: error: assignment from incompatible pointer type [-Werror]

    Signed-off-by: Simon Glass

    Simon Glass
     

15 Jul, 2013

1 commit

  • The files board/qi/qi_lb60/qi_lb60.c and include/configs/qi_lb60.h were
    licensed under the GPL v3 or later, and not v2 or later. As this is
    incompatible with the project, revert this board support until the
    responsible parties are available to re-license (if so desired) under
    GPL v2.

    Signed-off-by: Tom Rini

    Tom Rini
     

13 Jul, 2013

5 commits


12 Jul, 2013

4 commits