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

Marek Vasut marex at denx.de
Tue Apr 9 20:24:29 UTC 2019


On 4/9/19 9:45 PM, Simon Goldschmidt wrote:
> Am 09.04.2019 um 21:28 schrieb Marek Vasut:
>> On 4/9/19 8:52 PM, Simon Goldschmidt wrote:
>>> Remove outdated defines (not used any more, outdated due to DM
>>> conversion) from socfpga_common.h.
>>
>> s/defines/macros and comments/ >
>>> Signed-off-by: Simon Goldschmidt <simon.k.r.goldschmidt at gmail.com>
>>> ---
>>>
>>> Changes in v2:
>>> - remove even more outdated things
>>>
>>>   include/configs/socfpga_common.h | 40 --------------------------------
>>>   1 file changed, 40 deletions(-)
>>>
>>> diff --git a/include/configs/socfpga_common.h
>>> b/include/configs/socfpga_common.h
>>> index a65fc804e3..5b5e5f5d43 100644
>>> --- a/include/configs/socfpga_common.h
>>> +++ b/include/configs/socfpga_common.h
>>> @@ -72,29 +72,12 @@
>>>   #define CONFIG_SYS_BARGSIZE    CONFIG_SYS_CBSIZE
>>>                           /* Boot argument buffer size */
>>>   -#ifndef CONFIG_SYS_HOSTNAME
>>> -#define CONFIG_SYS_HOSTNAME    CONFIG_SYS_BOARD
>>> -#endif
>>> -
>>>   /*
>>>    * Cache
>>>    */
>>>   #define CONFIG_SYS_L2_PL310
>>>   #define CONFIG_SYS_PL310_BASE        SOCFPGA_MPUL2_ADDRESS
>>>   -/*
>>> - * EPCS/EPCQx1 Serial Flash Controller
>>> - */
>>> -#ifdef CONFIG_ALTERA_SPI
>>> -/*
>>> - * The base address is configurable in QSys, each board must specify
>>> the
>>> - * base address based on it's particular FPGA configuration. Please
>>> note
>>> - * that the address here is incremented by  0x400  from the Base
>>> address
>>> - * selected in QSys, since the SPI registers are at offset +0x400.
>>> - * #define CONFIG_SYS_SPI_BASE        0xff240400
>>> - */
>>> -#endif
>>> -
>>>   /*
>>>    * Ethernet on SoC (EMAC)
>>>    */
>>> @@ -162,15 +145,6 @@ unsigned int cm_get_qspi_controller_clk_hz(void);
>>>   #define CONFIG_CQSPI_REF_CLK        cm_get_qspi_controller_clk_hz()
>>>   #endif
>>>   -/*
>>> - * Designware SPI support
>>> - */
>>> -
>>> -/*
>>> - * Serial Driver
>>> - */
>>> -#define CONFIG_SYS_NS16550_SERIAL
>>
>> Are you sure about this one ?
> 
> Yes. This is only used in 'drivers/serial/Makefile' if CONFIG_DM_SERIAL
> is not defined. No need to keep it.
> 
> Should I still send a v3 for the commit message?

Would be nice. Can you prep me a PR with the patches for current release?

-- 
Best regards,
Marek Vasut


More information about the U-Boot mailing list