[U-Boot] [PATCH v3 0/4] arm: socfpga: clean up socfpga_common.h

Marek Vasut marex at denx.de
Mon Apr 22 18:42:40 UTC 2019


On 4/22/19 8:23 PM, Simon Goldschmidt wrote:
> Am 22.04.2019 um 20:13 schrieb Marek Vasut:
>> On 4/22/19 8:09 PM, Simon Goldschmidt wrote:
>>> This series cleans up the include/configs/socfpga_common.h file a bit.
>>>
>>> It removes some defines that are used nowhere and cleans up some
>>> leftovers after various subsystems have been converted to use DM.
>>>
>>> Changes in v3:
>>> - changed commit message of 2/4: s/defines/macros and comments/
>>>
>>> Changes in v2:
>>> - added (this) patch to move socfpga_vining to DM_I2C
>>> - remove even more outdated things
>>> - added (this) patch with further cleanups to the socfpga board config
>>>    files
>>>
>>> Simon Goldschmidt (4):
>>>    arm: socfpga: move vining_fpga to DM_I2C
>>>    arm: socfpga: clean up socfpga_common.h
>>>    arm: socfpga: remove CONFIG_SYS_BOOTMAPSZ
>>>    arm: socfpga: clean up board config files
>>>
>>>   configs/socfpga_vining_fpga_defconfig    |  8 ++-
>>>   include/configs/socfpga_arria10_socdk.h  |  6 ---
>>>   include/configs/socfpga_arria5_socdk.h   |  2 -
>>>   include/configs/socfpga_common.h         | 68 ------------------------
>>>   include/configs/socfpga_cyclone5_socdk.h |  2 -
>>>   include/configs/socfpga_de0_nano_soc.h   |  2 -
>>>   include/configs/socfpga_de10_nano.h      |  2 -
>>>   include/configs/socfpga_de1_soc.h        |  2 -
>>>   include/configs/socfpga_is1.h            |  2 -
>>>   include/configs/socfpga_sockit.h         |  2 -
>>>   include/configs/socfpga_socrates.h       |  2 -
>>>   include/configs/socfpga_sr1500.h         | 11 ----
>>>   include/configs/socfpga_vining_fpga.h    | 18 -------
>>>   13 files changed, 7 insertions(+), 120 deletions(-)
>>
>> I wanted to pick the whole thing, but it doesn't apply to
>> u-boot-socfpga/master or u-boot/master . Can you please rebase it ? I
>> pushed updated u-boot-socfpga/master , which matches u-boot/master .
>>
> 
> I'm preparing a pull request. This was just for the patch being on the
> list. Sorry, I should have mentioned that.
> 
> The reason it doesn't apply is that it depens on other patches.

Fine, thanks.

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list