[PATCH] x86 64: Add new device Cordoba Edge Platform

[Xiaojun Liu]

Hi,

Thank you for your guide!
I will resubmit it. 

-----Original Message-----
From: Paul Spooren <mail at aparcar.org> 
Sent: Tuesday, October 31, 2023 1:18 AM
To: Xiaojun Liu <xiaojun.liu at silicom.co.il>
Cc: Philip Prindeville <philipp_subx at redfish-solutions.com>; openwrt-devel <openwrt-devel at lists.openwrt.org>
Subject: Re: [PATCH] x86 64: Add new device Cordoba Edge Platform

Hi,

We kind of don’t add image just to contain extra drivers. Extra UCI defaults for eth ordering are fine but please don’t add an extra image if it just contains 1-2 extra packages, installable via the ImageBuilder or OPKG.

> 
>> On Oct 23, 2023, at 8:52 PM, Xiaojun Liu <xiaojun.liu at silicom.co.il> wrote:
>> 
>> Add new device Cordoba Edge Platform
>> Device name:Cordoba Edge Platform
>> hardware specifications: CPU - Intel Atom C3000
>>                         WiFi - mt7915e
>> 
>> Signed-off-by: Xiaojun Liu mailto:xiaojun.liu at silicom.co.il
>> ---
>> 
>> target/linux/x86/base-files/etc/board.d/02_network | 11 +++++++++++
>> target/linux/x86/image/64.mk                       | 10 ++++++++++
>> 2 files changed, 21 insertions(+)
>> 
>> 
>> diff --git a/target/linux/x86/base-files/etc/board.d/02_network
>> b/target/linux/x86/base-files/etc/board.d/02_network
>> index e00e8c04dd..be56153695 100644
>> --- a/target/linux/x86/base-files/etc/board.d/02_network
>> +++ b/target/linux/x86/base-files/etc/board.d/02_network
>> @@ -75,6 +75,17 @@ traverse-technologies-geos)
>>        macaddr="$(cat /sys/class/net/eth0/address)" 2>/dev/null
>>        [ -n "$macaddr" ] && ucidef_set_interface_macaddr "wan" "$macaddr"
>>        ;;
>> +silicom-80500-0214-e53)
>> +        ucidef_set_network_device_path "wan0" "pci0000:00/0000:00:16.0/0000:03:00.0"
>> +        ucidef_set_network_device_path "wan1" "pci0000:00/0000:00:16.0/0000:03:00.1"
>> +        ucidef_set_network_device_path "media0" "pci0000:00/0000:00:17.0/0000:02:00.0"
>> +        ucidef_set_network_device_path "media1" "pci0000:00/0000:00:17.0/0000:02:00.1"
>> +        ucidef_set_network_device_path "eth0" "pci0000:00/0000:00:0c.0/0000:04:00.0"
>> +        ucidef_set_network_device_path "eth1" "pci0000:00/0000:00:0e.0/0000:05:00.0"
>> +        ucidef_set_network_device_path "eth2" "pci0000:00/0000:00:0f.0/0000:06:00.0"
>> +        ucidef_set_network_device_path "eth3" "pci0000:00/0000:00:10.0/0000:07:00.0"
>> +        ucidef_set_interfaces_lan_wan "eth0 eth1 eth2 eth3" "wan0"
>> +        ;;
>> esac
>> board_config_flush
>> 
>> diff --git a/target/linux/x86/image/64.mk 
>> b/target/linux/x86/image/64.mk index 5ec9978b66..4addddeb98 100644
>> --- a/target/linux/x86/image/64.mk
>> +++ b/target/linux/x86/image/64.mk
>> @@ -8,3 +8,13 @@ define Device/generic
>>   GRUB2_VARIANT := generic
>> endef
>> TARGET_DEVICES += generic
>> +
>> +define Device/cordoba
>> +  DEVICE_VENDOR := Cordoba
>> +  DEVICE_MODEL := x86/64
>> +  DEVICE_PACKAGES += \
>> +        kmod-igc kmod-ixgbe \
>> +        kmod-mt7915-firmware
>> +  GRUB2_VARIANT := generic
>> +endef
>> +TARGET_DEVICES += cordoba

Please remove those lines in 64.mk.

Thanks for you work and interest in OpenWrt support for your device.

Best,
Paul