Skip to contentSkip to conversation

ARM: dts: am43xx-epos-evm: Enable SGX on EPOS boards

PATCH In review Andrew F. Davis 3 participants 4 messages Sep 18, 2017 – Oct 2, 2017 Device tree bindings

lore
Share & tools
lore

Andrew F. Davis

author
#0 lore raw

PATCH 1/1 ARM: dts: am43xx-epos-evm: Enable SGX on EPOS boards

From: Yogesh Siraswar <[email protected]>

The SGX IP is valid and usable on AM438x devices, enable this here.

Signed-off-by Yogesh Siraswar <[email protected]>
Signed-off-by Andrew F. Davis <[email protected]>
---
 arch/arm/boot/dts/am43x-epos-evm.dts | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/boot/dts/am43x-epos-evm.dts +4 -0
index 9d276af7c539..17c081b2cff3 100644
@@ -796,3 +796,7 @@
 	assigned-clocks = <&mux_synctimer32k_ck>;
 	assigned-clock-parents = <&clkdiv32k_ick>;
 };
+
+&sgx {
+	status = "okay";
+};
Signature · 1 line
2.14.1
build bot
#1 lore raw
reproduce: wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross # save the attached .config to linux build tree make.cross…

Hi Yogesh,

[auto build test ERROR on robh/for-next]
[also build test ERROR on v4.14-rc1 next-20170920]
[if your patch is applied to the wrong git tree, please drop us a note to help improve the system]

url:    https://github.com/0day-ci/linux/commits/Andrew-F-Davis/ARM-dts-am43xx-epos-evm-Enable-SGX-on-EPOS-boards/20170920-154002
base:   https://git.kernel.org/pub/scm/linux/kernel/git/robh/linux.git for-next
config: arm-s3c6400_defconfig (attached as .config)
compiler: arm-linux-gnueabi-gcc (Debian 6.1.1-9) 6.1.1 20160705

reproduce:
wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross chmod +x ~/bin/make.cross
# save the attached .config to linux build tree
make.cross ARCH=arm

All errors (new ones prefixed by >>):

>> Error: arch/arm/boot/dts/am43x-epos-evm.dts:800.1-5 Label or path sgx not found
>> FATAL ERROR: Syntax error parsing input tree

---
0-DAY kernel test infrastructure Open Source Technology Center
https://lists.01.org/pipermail/kbuild-all Intel Corporation

author
#2 lore raw
In reply to Andrew F. Davis
Quoted 17 lines · Andrew F. Davis

From: Yogesh Siraswar <[email protected]>

The SGX IP is valid and usable on AM438x devices, enable this here.

Signed-off-by Yogesh Siraswar <[email protected]>
Signed-off-by Andrew F. Davis <[email protected]>
---
 arch/arm/boot/dts/am43x-epos-evm.dts | 4 ++++
 1 file changed, 4 insertions(+)
arch/arm/boot/dts/am43x-epos-evm.dts +0 -0
index 9d276af7c539..17c081b2cff3 100644

arch/arm/boot/dts/am43x-epos-evm.dts line 802

Quoted 7 lines
arch/arm/boot/dts/am43x-epos-evm.dts +4 -0
@@ -796,3 +796,7 @@
 	assigned-clocks = <&mux_synctimer32k_ck>;
 	assigned-clock-parents = <&clkdiv32k_ick>;
 };
+
+&sgx {
+	status = "okay";
+};

Didn't see the parent node didn't make upstream this cycle, ignore for now, sorry for the noise.

#3 lore raw
In reply to Andrew F. Davis
Quoted 4 lines · Andrew F. Davis

From: Yogesh Siraswar <[email protected]>

The SGX IP is valid and usable on AM438x devices, enable this here.

This still produces:

arch/arm/boot/dts/am43x-epos-evm.dts:796.1-5 Label or path sgx not found

The parent node for the interconnect target module can be added the same ways as done in "ARM: dts: Add nodes for missing omap4 interconnect target modules". But the child sgx device binding still needs to be documented before we can add those.

Regards,

Tony