From mboxrd@z Thu Jan 1 00:00:00 1970 Delivery-date: Thu, 09 Dec 2021 07:27:25 +0100 Received: from metis.ext.pengutronix.de ([2001:67c:670:201:290:27ff:fe1d:cc33]) by lore.white.stw.pengutronix.de with esmtps (TLS1.3) tls TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384 (Exim 4.94.2) (envelope-from ) id 1mvCtp-000mv1-Hy for lore@lore.pengutronix.de; Thu, 09 Dec 2021 07:27:25 +0100 Received: from localhost ([127.0.0.1] helo=metis.ext.pengutronix.de) by metis.ext.pengutronix.de with esmtp (Exim 4.92) (envelope-from ) id 1mvCto-00024j-AO; Thu, 09 Dec 2021 07:27:24 +0100 Received: from mail-eopbgr60073.outbound.protection.outlook.com ([40.107.6.73] helo=EUR04-DB3-obe.outbound.protection.outlook.com) by metis.ext.pengutronix.de with esmtps (TLS1.2:ECDHE_RSA_AES_256_GCM_SHA384:256) (Exim 4.92) (envelope-from ) id 1mvCtO-00024T-Mg; Thu, 09 Dec 2021 07:27:00 +0100 ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=S1fyxUTAsgSx0fkjuyjQ8drp7k9BDSTme5mJ/pMp5oNZkSPR9dzKc1oqCVt+EVCg86o5ghdmg5DW+rswGxAwLnhMpzU3etG7nzl52v6HDpb3ak6AIiXVGgISj7U5XYJHSYPg3iLACwwKh0ywAi0oIOdXmiB4P7kuFQcBVNSb/rtYhS+K03FiRrpW11JeiuLuGRvZiK7xINlyytWtLWpbaXKP615UO7WBG37vEUpCzJpR/5B66w0ou3sAzYtdYIeNmSmSzR6fK+TbiRkmmiOi39WjNVihnZdAOE7AZer8i4JzV5E5bpL/HlxfWmvrS8rbHRPVrT4Q6OJLQEtJTVQOsQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=wP4s4XdgI5HTIFB5RjgTFfrHsal4kYUQlvB70jxle7c=; b=ScKwgdZGb+tB9PLackSr6qkkP7kHxa6CVuGQSXA/7kacWZjAALnG0q13IcM0XPxOLG9HjDV1vGjFIomDKR2JxbpIBq2pHea/of+oyHOJtNj2N8hwK7nnPXIAnXWlII8Z1FjbKcrVNfGGSRLePZOtiuha0cl+sbutb/+YbzbQ3luGLr41wxM4/Sw739Z7ensvHTNELWhRW9ta59dkyTfvy2V5CT115qG7Wm/vlXrF9yymz2R2GEIgKxPs9mz6wsPJxJxJTgBPTOXgtmCsJER/mHokt5J+YejZ+8gyKvv/YQAtKNIpN6sXdRAKvGa21iNhA8uJk7JigEANqiGXmZXt8w== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=pass smtp.mailfrom=wolfvision.net; dmarc=pass action=none header.from=wolfvision.net; dkim=pass header.d=wolfvision.net; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=wolfvision.net; s=selector2; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=wP4s4XdgI5HTIFB5RjgTFfrHsal4kYUQlvB70jxle7c=; b=NFviOiwUllwWO41qC2JZsiXOkKZQ+9sUMWHtN2LzKLXv2JY2zrh4KPUQNiCE8gm1ZS/1jQzcL75mpm0eOqD8hGBfTt6rd6lr0T6cAUOBlFnWukDGUXjqyQg772CJdaHKIiGD70f2sAzrdzbTj2VTJcjSJgwlIkQeVDjMiV9WFuo= Authentication-Results: dkim=none (message not signed) header.d=none;dmarc=none action=none header.from=wolfvision.net; Received: from DBBPR08MB4523.eurprd08.prod.outlook.com (2603:10a6:10:c8::19) by DB6PR0801MB1941.eurprd08.prod.outlook.com (2603:10a6:4:76::16) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4755.20; Thu, 9 Dec 2021 06:26:55 +0000 Received: from DBBPR08MB4523.eurprd08.prod.outlook.com ([fe80::81d1:d448:4016:1956]) by DBBPR08MB4523.eurprd08.prod.outlook.com ([fe80::81d1:d448:4016:1956%4]) with mapi id 15.20.4755.022; Thu, 9 Dec 2021 06:26:55 +0000 To: christian.melki@t2data.com, ptxdist@pengutronix.de References: <20211207153905.2386379-1-michael.riesch@wolfvision.net> <20211207153905.2386379-2-michael.riesch@wolfvision.net> <5d9dbc5e-8330-34cd-3a4b-859ebfae277e@t2data.com> From: Michael Riesch Message-ID: Date: Thu, 9 Dec 2021 07:26:53 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.14.0 In-Reply-To: <5d9dbc5e-8330-34cd-3a4b-859ebfae277e@t2data.com> Content-Language: en-US X-ClientProxiedBy: ZR0P278CA0121.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:20::18) To DBBPR08MB4523.eurprd08.prod.outlook.com (2603:10a6:10:c8::19) MIME-Version: 1.0 Received: from [IPv6:2a05:1141:1c9:7c00:2971:328c:3026:3149] (2a05:1141:1c9:7c00:2971:328c:3026:3149) by ZR0P278CA0121.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:20::18) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.4755.21 via Frontend Transport; Thu, 9 Dec 2021 06:26:54 +0000 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 80d75f3c-3c63-4d1a-8ce1-08d9badce507 X-MS-TrafficTypeDiagnostic: DB6PR0801MB1941:EE_ X-Microsoft-Antispam-PRVS: X-MS-Oob-TLC-OOBClassifiers: OLM:8882; X-MS-Exchange-SenderADCheck: 1 X-MS-Exchange-AntiSpam-Relay: 0 X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: b0VWwgbccGWwUvjKdrSPjbd5qYk0riO7+q1xA5L35G04VrsBK4CrcIpLprlZQpLTn3LlbT80JgNJjLosaCkDGWZqqAQ9veI+lN1p9vkPlqzAqUgglF0INoIYwNFt95ULlt1frhGb1sjMbCb89wkU5Rk+OGzeXYSQoXoLmoiv5mFMvcpQer7y9aoV+ccHJNXT/07a9a3I8HoHA94MWLqzOYjtXLOcf3dZ3Tw3i0mV0nPJsDMpLK3Xu91dNmMssggYabHWRdWsmtZJRgUBRJ9h+O0+WPyTTVPblSuXP+7g/jqEAS6gQAizDHYv3UrNfvkA3yWOnPROZTuXpyQoviaVDYTEB9+sfPP+E1fOwkhu8CHkVrYB+KJXwDP6K4xWwRN8bzpSpmIXGr3M8yjstCZnjd0Swst9IN+aCTLbUPOaZEKXWbmy6BfJ+377zberf94uGeXBC/+Y+m83i9ek6/efLCP9t3GR5n+wo1O0NrhFLytdKkhukN2nkME3NqNPPoMWchNvyuf5PxCr+et3ZuJlzgX23WGTOTkCZhY+l1RddqRjzB8TVC17AB8LoSKxe88xtYsmOaxdGYK9XVd3JCBjYaBCRxdlsYx1i5PYEisd/w5MygUQqNXE8coSsKklsO9W55y3OeKCxzjRJ2MDhxsqGtRUgS7wd6gEMorWRRuqwhfOSp/Ik89EkJk0jRlCGhC6L5/HnNRIRPpr8fWm3gAaEmUPrGWqZc+t7Ts/jvUw7b7CqKaD25k4wJbritjCMLWN X-Forefront-Antispam-Report: CIP:255.255.255.255; CTRY:; LANG:en; SCL:1; SRV:; IPV:NLI; SFV:NSPM; H:DBBPR08MB4523.eurprd08.prod.outlook.com; PTR:; CAT:NONE; SFS:(4636009)(366004)(316002)(44832011)(508600001)(8936002)(83380400001)(186003)(6486002)(86362001)(38100700002)(52116002)(2906002)(5660300002)(66946007)(66476007)(2616005)(66556008)(8676002)(36756003)(4326008)(31686004)(53546011)(31696002)(45980500001)(43740500002); DIR:OUT; SFP:1101; X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?dkdIdW1CUUs1VlJxc0hJTUV5c0x0eHJvMTBoSldvT1RHK0VJdDhxejJSekdQ?= =?utf-8?B?QUZpTnpCc0gvN0NRZGZoR1FSVG5OdEgxVmw5alBRSnUwQ1BSL055NWc1aUgr?= =?utf-8?B?L3A3eFlVZkZUSXFMMmdEWTNld0hyY21pS2lBNDV4NnN3ZWpxZWJoQlZyU003?= =?utf-8?B?Q3JmRjRPVE84NDhDSGxmdTY0QXlPSFdNT0ZEOUJ2OHVka0QzUElWcU5NbVNF?= =?utf-8?B?VEF1V2tORHdtLzRMYzUxcXJGQUZ6M0NsQkpkSnFtY1JXbHJ3WHc0RmFQbVZO?= =?utf-8?B?d21nQjJLd3NpNThQZFhML2NDMW5kRm1BcDJrWHlIVU9mb2ZPZTFwNlJWMWd6?= =?utf-8?B?aHdhK2RTNlllczNxTGNLZnhmRld0Q0NtaTk2eC9TTzI5NHNCU2RYeDFTTVE5?= =?utf-8?B?dy91a1ZHZ3BiTW5UemMxdHFMaEpKdEI0Qk4vNVE0OS9IWXZyOXdFS0I0VDRT?= =?utf-8?B?OGgvZnhXZ0N6MVdqUVNyWjRDR1dFcjJEOThTelRoQkdsbWo4MUhPR0VtV2xv?= =?utf-8?B?MXFidDRVUVhjaUV2OWtEN01ER0JEM1ovbGhvQ2hKdjJzQzBUdlh6dW5JaWtB?= =?utf-8?B?VjlKY05SZkFJbFJaVnJDZVdoMEZhNDlPcUVQZG1VK2hUbEEwbkZpNzZqU1pR?= =?utf-8?B?RVdoeS9OSEU3MXpZd2JhN2VTYVBreHNaTXlSbWN0bUR4bzl2RnY3OElQZXcv?= =?utf-8?B?Y0FQdEMva3kxNTZhMEJ3dU8weHNFUXJ4V0xDTnBsZGZBNUhGT0VKZ05OVUhK?= =?utf-8?B?RUlJdk4vZnpsL3R6ck41QzZYcmZCT0I2cGcvL3hHUGVyTEdTNC9SQVBwR1g1?= =?utf-8?B?Z0F1OXBhYU1uc2QveWlYU0MvakVoQWk1ZFlvYklEenk2NWhYcE9RdTJoYjBN?= =?utf-8?B?R1NtNm56b3A2bU14SEh4dVBSWUthNmY3aG42WDZxZEFNUnFsb0FDbkJZZWp6?= =?utf-8?B?by9mQ3JLeTFnOStycFZsaGhZUjYvVlhCSzN3TnRma2FkSERKSHBkeFA1aUlR?= =?utf-8?B?aTgrQTdpN25BVlE4UXpvKzFpdEFLMTNpZ2J0ZUx6Z0Q1dkxiTkV0TFF3VXhT?= =?utf-8?B?Yk1xdzVVcEo0S3pSSUFZTmZQRFcvMS9iV1EvL01lU05zV2tBL1haemNvenhh?= =?utf-8?B?Z01aOTdWSWJJdllOVE53UEtLQUZPckFLbytHMDAwMWJLSEhtUXR0TGNzQnQ3?= =?utf-8?B?ekhQQVo5NVp3UUpkV0lTYzZJQnhkcVBPRFBrWU5GZVpDREgxM1pucWQ0SDVx?= =?utf-8?B?bmkvclRoMGU0LzBzWTNiNmsrTXArS1RuM2ZGQWM0cWk5SU5aQ0RkZE11SGVG?= =?utf-8?B?b2FmZGpiOGtuaGk0dGliU0NDYytpU3BkdFFBc2FaZFN2aU96MWtBcE5TcFQ5?= =?utf-8?B?RlhRMk9UYnBXeTZEeGxzS3BFRUozejBQZ0VSNngzcjQ1QnBpZVBrTzloeXJX?= =?utf-8?B?U0xxcWwvczVnbUM3QThzOXZuZnJGSmNEbDhwY0hkbm9MSWNBbzRxb3NnSmd1?= =?utf-8?B?QUFLbHlhM3R2ZEsvZDBCOVhVbU1xTXloVXc2c3VPYUpNOFVNbStFcktOc0FO?= =?utf-8?B?eStBSmJodDl6UjA1QU5KRkM1M0krWkZZSnNHYXFpOUlYWUtrTlBBaTBvNTFW?= =?utf-8?B?c0s0a2orYXA0Qjk4VWtVVS9KeWFCbGdTQk9PbXNhcVlTWTJzLzRBR2RyN24y?= =?utf-8?B?dXJENTcrdEpvZ3piMW55UHlWV1pNMmY3MTQ1VEhLSXVYOVBkeE1EekNmY3du?= =?utf-8?B?N1RnTEt4MktrQ0EvVW9wU2hLbzV5eTlyZkFYVVg3TmI3WTZONVhtbjVrbEpu?= =?utf-8?B?UTh6RXY3T1dBa1BIN1lGMjRXQWV1N3I5NE4zN1FnT1krZWxYYkFHN3A4S2Vi?= =?utf-8?B?VmZFOElvSVl6T0lRNVZ5V1VMQitOZkhsYmR0aDk5SG1lQlNsVWJtd0FLVGxk?= =?utf-8?B?bjdBSCtFQ1FqZWtyVlJRMzBSUlUxb01UMU9JREU2WHlLdThGK2JnMkRwMzF1?= =?utf-8?B?V3ZZbGtZcnRaSGFBLzdmd3AwSWxzZzFqSUp0VnphUHkrekFxRGdSNFJmcEhY?= =?utf-8?B?ME1ZVHJkNERDc2w0VnBtVmNyUHoyUm5VRWRKUk5Ick9WdTB1WlZZd3BrcSth?= =?utf-8?B?bVdQQjI4VFNsNDZVZlhtMGpUSGRETTE1NkZZV2RRR1JVWWlkL3lVeUw1ei84?= =?utf-8?B?SkU1VlRCNFF3THo3b3VWellSNDBqWklEbE1sZlZhV0xWdWNWV2p0ZElsODNM?= =?utf-8?B?akxNbWFka3h3bk9ickIzN1MwK3BYenhWNlJMUnYyalZZaDdobnExSmRiYzdQ?= =?utf-8?B?ODhzODVmTEJRZWt0aWc1d0Q2WG1TQ1djdnhENzl2QnRVVEFLM0tOSDlsN0hi?= =?utf-8?Q?44TGZxR2oP0+iT7o=3D?= X-OriginatorOrg: wolfvision.net X-MS-Exchange-CrossTenant-Network-Message-Id: 80d75f3c-3c63-4d1a-8ce1-08d9badce507 X-MS-Exchange-CrossTenant-AuthSource: DBBPR08MB4523.eurprd08.prod.outlook.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 09 Dec 2021 06:26:54.8753 (UTC) X-MS-Exchange-CrossTenant-FromEntityHeader: Hosted X-MS-Exchange-CrossTenant-Id: e94ec9da-9183-471e-83b3-51baa8eb804f X-MS-Exchange-CrossTenant-MailboxType: HOSTED X-MS-Exchange-CrossTenant-UserPrincipalName: 6xqZqqY0Cp/EJageszBd9+K/qVCh0K1vz7Fmd+1whUWB8pVnja0qVtXIwu11MKZ1bMHEChB5F2lb3fV4qf8wj3eqt7fEiiP2AYM6SR22wtk= X-MS-Exchange-Transport-CrossTenantHeadersStamped: DB6PR0801MB1941 X-Spam-Checker-Version: SpamAssassin 3.4.2 (2018-09-13) on metis.ext.pengutronix.de X-Spam-Level: X-Spam-Status: No, score=-4.4 required=4.0 tests=AWL,BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,MSGID_FROM_MTA_HEADER, NICE_REPLY_A,RCVD_IN_DNSWL_NONE,RCVD_IN_MSPIKE_H2,SPF_HELO_PASS, SPF_PASS autolearn=ham autolearn_force=no version=3.4.2 Subject: Re: [ptxdist] [PATCH 1/2] ptxd_make_world_{dtb, dtbo}: add support for device tree overlays X-BeenThere: ptxdist@pengutronix.de X-Mailman-Version: 2.1.29 Precedence: list List-Id: PTXdist Development Mailing List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: ptxdist@pengutronix.de Cc: m.tretter@pengutronix.de Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "ptxdist" X-SA-Exim-Connect-IP: 127.0.0.1 X-SA-Exim-Mail-From: ptxdist-bounces@pengutronix.de X-SA-Exim-Scanned: No (on metis.ext.pengutronix.de); SAEximRunCond expanded to false Hi Christian, On 12/7/21 6:30 PM, Christian Melki wrote: > Hi! > > I like this take on dtbos! Thanks for your review! > On 12/7/21 16:39, Michael Riesch wrote: >> Make ptxd_make_dtb more general and provide suitable wrappers to >> support the generation of device tree overlays. >> >> Signed-off-by: Michael Riesch >> --- >> rules/post/ptxd_make_world_dtbo.make | 21 +++++++ >> scripts/lib/ptxd_make_world_dtb.sh | 82 ++++++++++++++++++++-------- >> 2 files changed, 80 insertions(+), 23 deletions(-) >> create mode 100644 rules/post/ptxd_make_world_dtbo.make >> >> diff --git a/rules/post/ptxd_make_world_dtbo.make b/rules/post/ptxd_make_world_dtbo.make >> new file mode 100644 >> index 000000000..61babc653 >> --- /dev/null >> +++ b/rules/post/ptxd_make_world_dtbo.make >> @@ -0,0 +1,21 @@ >> +# -*-makefile-*- >> +# >> +# Copyright (C) 2020 by Michael Tretter >> +# >> +# For further information about the PTXdist project and license conditions >> +# see the README file. >> +# >> + >> +world/dtbo/env = \ >> + $(call world/env, $(1)) \ >> + pkg_dtso_path="$($(1)_DTSO_PATH)" \ >> + pkg_dtso="$($(1)_DTSO)" \ >> + pkg_dtbo_dir="$($(1)_DTBO_DIR)" \ >> + pkg_kernel_src="$($(1)_KERNEL_DIR)" \ >> + pkg_arch="$(GENERIC_KERNEL_ARCH)" >> + >> +world/dtbo = \ >> + $(call world/dtbo/env,$(strip $(1))) \ >> + ptxd_make_world_dtbo >> + >> +# vim: syntax=make >> diff --git a/scripts/lib/ptxd_make_world_dtb.sh b/scripts/lib/ptxd_make_world_dtb.sh >> index f5e796b9d..c1ded5ffb 100644 >> --- a/scripts/lib/ptxd_make_world_dtb.sh >> +++ b/scripts/lib/ptxd_make_world_dtb.sh >> @@ -9,18 +9,10 @@ >> ptxd_make_dtb() { >> local dtc dts tmp_dts dtb deps tmp_deps >> >> - if [[ "${dts_dts}" =~ ^/.* ]]; then >> - ptxd_bailout "'${dts_dts}' must not be an absolute path!" \ >> - "Use _DTS_PATH to specify the search path." >> - fi >> - >> - if ! ptxd_in_path pkg_dts_path "${dts_dts}"; then >> - ptxd_bailout "Device-tree '${dts_dts}' not found in '${pkg_dts_path}'." >> - fi >> - dts="${ptxd_reply}" >> - dtb="${dtb_dir}/$(basename ${dts/%.dts/.dtb})" >> + dts="${dtb_source}" >> + dtb="${dtb_target}" >> >> - dtc="${pkg_build_dir}/scripts/dtc/dtc" >> + dtc="${dtb_kernel_dir}/scripts/dtc/dtc" >> if [ ! -x "${dtc}" ]; then >> dtc=dtc >> fi >> @@ -39,16 +31,16 @@ ptxd_make_dtb() { >> -Wp,-MT,${dtb_deps_target} \ >> -nostdinc \ >> -I"$(dirname "${dts}")" \ >> - -I${pkg_dir}/arch/${pkg_arch}/boot/dts \ >> - -I${pkg_dir}/arch/${pkg_arch}/boot/dts/include \ >> - -I${pkg_dir}/scripts/dtc/include-prefixes \ >> - -I${pkg_dir}/drivers/of/testcase-data \ >> - -I${pkg_dir}/include \ >> + -I${dtb_kernel_dir}/arch/${pkg_arch}/boot/dts \ >> + -I${dtb_kernel_dir}/arch/${pkg_arch}/boot/dts/include \ >> + -I${dtb_kernel_dir}/scripts/dtc/include-prefixes \ >> + -I${dtb_kernel_dir}/drivers/of/testcase-data \ >> + -I${dtb_kernel_dir}/include \ >> -undef -D__DTS__ -x assembler-with-cpp \ >> -o ${tmp_dts} \ >> ${dts} && >> >> - sed -e "\;^ ${pkg_dir}[^ ]*;d" \ >> + sed -e "\;^ ${dtb_kernel_dir}[^ ]*;d" \ >> -e 's;^ \([^ \]*\); $(wildcard \1);' "${tmp_deps}" > "${deps}" && >> # empty line in case all dependencies were removed >> echo >> "${deps}" && >> @@ -57,9 +49,9 @@ ptxd_make_dtb() { >> echo "DTC $(ptxd_print_path "${dtb}")" && >> ptxd_eval \ >> "${dtc}" \ >> - $(ptxd_get_ptxconf PTXCONF_DTC_EXTRA_ARGS) \ >> + "${dtb_extra_args}" \ >> -i "$(dirname "${dts}")" \ >> - -i "${pkg_dir}/arch/${pkg_arch}/boot/dts" \ >> + -i "${dtb_kernel_dir}/arch/${pkg_arch}/boot/dts" \ >> -d "${tmp_deps}" \ >> -I dts -O dtb -b 0 \ >> -o "${dtb}" "${tmp_dts}" && >> @@ -81,17 +73,61 @@ ptxd_make_dtb() { >> export -f ptxd_make_dtb >> >> >> +ptxd_make_world_dtbo() { >> + local dtb_deps_target dtb_extra_args dtb_kernel_dir dtb_source dtb_target >> + >> + ptxd_make_world_init || break >> + >> + dtb_deps_target="${ptx_state_dir}/${pkg_stamp}" >> + dtb_extra_args="-@" > > Looks ok. Do we need to do a dtc version check? > 1.4.4+ I think. Maybe a bit excessive? As pointed out by Michael Olbrich this won't help. > >> + dtb_kernel_dir="${pkg_kernel_src}" >> + if [ -z $dtb_kernel_dir ]; then >> + dtb_kernel_dir="${pkg_dir}" >> + fi >> + >> + echo -e "\nBuilding device tree overlays..." >> + >> + for overlay in ${pkg_dtso}; do >> + if [[ "${overlay}" =~ ^/.* ]]; then >> + ptxd_bailout "'${overlay}' must not be an absolute path!" \ >> + "Use _DTSO_PATH to specify the search path." >> + fi >> + >> + if ! ptxd_in_path pkg_dtso_path "${overlay}"; then >> + ptxd_bailout "Overlay '${overlay}' not found in '${pkg_dtso_path}'." >> + fi >> + dtb_source="${ptxd_reply}" >> + dtb_target="${pkg_pkg_dir}/${pkg_dtbo_dir}/$(basename ${overlay/%.dts*/.dtbo})" >> + >> + ptxd_make_dtb || break >> + done >> +} >> +export -f ptxd_make_world_dtbo >> + >> + >> ptxd_make_world_dtb() { >> - local dtb_deps_target dtb_dir >> + local dtb_deps_target dtb_extra_args dtb_kernel_dir dtb_source dtb_target >> >> ptxd_make_world_init || break >> >> dtb_deps_target="${ptx_state_dir}/${pkg_stamp}" >> - dtb_dir="${pkg_pkg_dir}/boot" >> + dtb_extra_args="$(ptxd_get_ptxconf PTXCONF_DTC_EXTRA_ARGS)" > > Not sure I follow here. Must not the dtb itself be built with -@ for > symbols if dtbos are used? Does EXTRA_ARGS default to -@ or something? You are right, thanks for pointing that out. Will fix this in v2. > >> + dtb_kernel_dir="${pkg_dir}" >> + >> + echo -e "\nBuilding device trees..." >> + >> + for tree in ${pkg_dts}; do >> + if [[ "${tree}" =~ ^/.* ]]; then >> + ptxd_bailout "'${tree}' must not be an absolute path!" \ >> + "Use _DTS_PATH to specify the search path." >> + fi >> >> - echo -e "\nBuilding Device trees..." >> + if ! ptxd_in_path pkg_dts_path "${tree}"; then >> + ptxd_bailout "Overlay '${tree}' not found in '${pkg_dts_path}'." >> + fi >> + dtb_source="${ptxd_reply}" >> + dtb_target="${pkg_pkg_dir}/boot/$(basename ${tree/%.dts/.dtb})" > > Static output dir path? Not sure whether I understand your comment correctly, but the path has been copied from the existing script. It can be discussed whether it makes sense to install the compiled device trees to an alternative location, but I think this is a different topic and beyond the scope of this series. Best regards, Michael > >> >> - for dts_dts in ${pkg_dts}; do >> ptxd_make_dtb || break >> done >> } >> > _______________________________________________ ptxdist mailing list ptxdist@pengutronix.de To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de