From mboxrd@z Thu Jan  1 00:00:00 1970
From: Michal Simek <michal.simek@xilinx.com>
Date: Fri, 4 Dec 2015 09:20:57 +0100
Subject: [U-Boot] [PATCH v3 02/14] ARM: zynq: Remove CONFIG_API
In-Reply-To: <5660828D.30903@myspectrum.nl>
References: <3846048879467c3510abb54b930cdc05772d469e.1449154075.git.michal.simek@xilinx.com>
 <8982bb88db1e211d67574a5fd3186d04389e03d5.1449154075.git.michal.simek@xilinx.com>
 <CAD6G_RTUZmMFteGFvuaqixtWCJFe_jYRgDhrBV73P08+8-c37w@mail.gmail.com>
 <5660828D.30903@myspectrum.nl>
Message-ID: <56614CE9.9020002@xilinx.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

On 3.12.2015 18:57, Jeroen Hofstee wrote:
> Hi,
> 
> On 03-12-15 17:09, Jagan Teki wrote:
>> On 3 December 2015 at 20:17, Michal Simek <michal.simek@xilinx.com>
>> wrote:
>>> CONFIG_API is causing compilation error when DM_ETH is enabled because
>>> eth_get_dev() is not available.
>> Then how come, freebsd elf generate w/o CONFIG_API?
>>
>>> Signed-off-by: Michal Simek <michal.simek@xilinx.com>
>>> ---
>>>
>>> Changes in v3: None
>>> Changes in v2: None
>>>
>>>   include/configs/zynq-common.h | 1 -
>>>   1 file changed, 1 deletion(-)
>>>
>>> diff --git a/include/configs/zynq-common.h
>>> b/include/configs/zynq-common.h
>>> index a9e6bf046a97..5db501188b18 100644
>>> --- a/include/configs/zynq-common.h
>>> +++ b/include/configs/zynq-common.h
>>> @@ -297,7 +297,6 @@
>>>
>>>   /* Boot FreeBSD/vxWorks from an ELF image */
>>>   #if defined(CONFIG_ZYNQ_BOOT_FREEBSD)
>>> -# define CONFIG_API
>>>   # define CONFIG_SYS_MMC_MAX_DEVICE     1
>>>   #endif
>>>
>>
> 
> FreeBSD boot relies on CONFIG_API. Removing it will brick FreeBSD boots....

It will. But for me it is better to move driver to DM and break FreeBSD
boot than removing ethernet driver.
Feel free to fix CONFIG_API and we can enable that option again.

Thanks,
Michal