diff mbox

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

Message ID 20170918173130.17657-1-afd@ti.com (mailing list archive)
State New, archived
Headers show

Commit Message

Andrew Davis Sept. 18, 2017, 5:31 p.m. UTC
From: Yogesh Siraswar <yogeshs@ti.com>

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

Signed-off-by: Yogesh Siraswar <yogeshs@ti.com>
Signed-off-by: Andrew F. Davis <afd@ti.com>
---
 arch/arm/boot/dts/am43x-epos-evm.dts | 4 ++++
 1 file changed, 4 insertions(+)

Comments

kernel test robot Sept. 20, 2017, 10:15 a.m. UTC | #1
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
Andrew Davis Sept. 20, 2017, 3:15 p.m. UTC | #2
On 09/18/2017 12:31 PM, Andrew F. Davis wrote:
> From: Yogesh Siraswar <yogeshs@ti.com>
> 
> The SGX IP is valid and usable on AM438x devices, enable this
> here.
> 
> Signed-off-by: Yogesh Siraswar <yogeshs@ti.com>
> Signed-off-by: Andrew F. Davis <afd@ti.com>
> ---
>  arch/arm/boot/dts/am43x-epos-evm.dts | 4 ++++
>  1 file changed, 4 insertions(+)
> 
> diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
> index 9d276af7c539..17c081b2cff3 100644
> --- a/arch/arm/boot/dts/am43x-epos-evm.dts
> +++ b/arch/arm/boot/dts/am43x-epos-evm.dts
> @@ -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.
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Tony Lindgren Oct. 2, 2017, 2:52 p.m. UTC | #3
* Andrew F. Davis <afd@ti.com> [170918 10:32]:
> From: Yogesh Siraswar <yogeshs@ti.com>
> 
> 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
--
To unsubscribe from this list: send the line "unsubscribe linux-omap" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox

Patch

diff --git a/arch/arm/boot/dts/am43x-epos-evm.dts b/arch/arm/boot/dts/am43x-epos-evm.dts
index 9d276af7c539..17c081b2cff3 100644
--- a/arch/arm/boot/dts/am43x-epos-evm.dts
+++ b/arch/arm/boot/dts/am43x-epos-evm.dts
@@ -796,3 +796,7 @@ 
 	assigned-clocks = <&mux_synctimer32k_ck>;
 	assigned-clock-parents = <&clkdiv32k_ick>;
 };
+
+&sgx {
+	status = "okay";
+};