From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from fllv0016.ext.ti.com (fllv0016.ext.ti.com [198.47.19.142]) by mx.groups.io with SMTP id smtpd.web09.1827.1580500575288476572 for ; Fri, 31 Jan 2020 11:56:15 -0800 Authentication-Results: mx.groups.io; dkim=pass header.i=@ti.com header.s=ti-com-17q1 header.b=gCk9Pe3x; spf=pass (domain: ti.com, ip: 198.47.19.142, mailfrom: denys@ti.com) Received: from fllv0035.itg.ti.com ([10.64.41.0]) by fllv0016.ext.ti.com (8.15.2/8.15.2) with ESMTP id 00VJuEp3083111 for ; Fri, 31 Jan 2020 13:56:14 -0600 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=ti.com; s=ti-com-17Q1; t=1580500574; bh=NK4ogd/VNYJusuBugl5KBjw/7RfLz1n3GtGZ4ylV05Q=; h=Date:From:To:CC:Subject:References:In-Reply-To; b=gCk9Pe3xgPPz5GWqstlLsQPu9yCf8zfFb2q1R1mBNtg/MmDSsph/bHp3QZcScKJCC 2UxrNzjkePaNkAn1yFPN+GcaAfU+rsOTDlvTpQzUzSfprNNNogKFNML9C4LxU0vcRD H1wm4YUQdk0CthiMCgKvTQomoG9OiANMDtw5dEBo= Received: from DFLE108.ent.ti.com (dfle108.ent.ti.com [10.64.6.29]) by fllv0035.itg.ti.com (8.15.2/8.15.2) with ESMTP id 00VJuET1083301 for ; Fri, 31 Jan 2020 13:56:14 -0600 Received: from DFLE109.ent.ti.com (10.64.6.30) by DFLE108.ent.ti.com (10.64.6.29) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1847.3; Fri, 31 Jan 2020 13:56:14 -0600 Received: from fllv0039.itg.ti.com (10.64.41.19) by DFLE109.ent.ti.com (10.64.6.30) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_128_CBC_SHA256_P256) id 15.1.1847.3 via Frontend Transport; Fri, 31 Jan 2020 13:56:14 -0600 Received: from localhost (ileax41-snat.itg.ti.com [10.172.224.153]) by fllv0039.itg.ti.com (8.15.2/8.15.2) with ESMTP id 00VJuDcE043325; Fri, 31 Jan 2020 13:56:13 -0600 Date: Fri, 31 Jan 2020 14:56:12 -0500 From: "Denys Dmytriyenko" To: "Andrew F. Davis" CC: Subject: Re: [meta-ti][master][PATCH 2/3] Move Graphics and Multimedia drivers and apps out of recipes-bsp Message-ID: <20200131195611.GG23242@beryl> References: <20200131192111.29813-1-afd@ti.com> <20200131192111.29813-2-afd@ti.com> <20200131193318.GC23242@beryl> MIME-Version: 1.0 In-Reply-To: User-Agent: Mutt/1.5.24 (2015-08-30) X-EXCLAIMER-MD-CONFIG: e1e8a2fd-e40a-4ac6-ac9b-f7e9cc9ee180 Content-Type: text/plain; charset="us-ascii" Content-Disposition: inline On Fri, Jan 31, 2020 at 02:39:52PM -0500, Andrew F. Davis wrote: > On 1/31/20 2:33 PM, Denys Dmytriyenko wrote: > > On Fri, Jan 31, 2020 at 02:21:10PM -0500, Andrew F. Davis wrote: > >> recipes-bsp is looking a bit random at the moment, lets move graphics > >> recipies into recipes-graphics and some multimedia related ones into > >> recipes-multimedia. > > > > Yeah, let me sit on this one for a bit - I'm slightly torn here, as I like > > some of the new names/locations, but not all them... > > > > > Same, I wasn't sure if I even wanted to post this one, but some later > work I did depended on the new locations and I didn't feel like un-doing it. > > Anyway here is another thing I was thinking: > > recipes-bsp -> firmwares > recipes-kernel -> all kernel stuff including kernel modules > recipes-graphics -> just the UM side drivers and apps > > Right now we have kernel module recipes in several places, usually > bundled either their assosiated firmware or UM drivers. If we split > things out like above we can keep everything in more consistent locations. > > Let me know if you have a prefrence either way. I've been noticing that recipes-bsp has been growing out of proportion. Some old names are historic, but maybe even more correct, e.g.: https://en.wikipedia.org/wiki/PowerVR I like "vivante" rename. Still thinking about "img" vs. powervr due to above. Historically, out-of-tree kernel modules lived in recipes-bsp, while user-space apps and libs in recipes-graphics/recipes-multimedia. But I might agree to moving drivers away from recipes-bsp to leave that only for firmwares and maybe LLDs... Denys > > Andrew > > > > > >> Signed-off-by: Andrew F. Davis > >> --- > >> .../{ti-img-encode-decode => vxd-dec-fw}/vxd-dec-fw_git.bb | 0 > >> .../ti-img-rogue-driver/0001-km-support-general-toolchain.patch | 0 > >> .../ti-img-rogue}/ti-img-rogue-driver_1.10.5371573.bb | 0 > >> .../{libgles => ti-img-rogue}/ti-img-rogue-umlibs_1.10.5371573.bb | 0 > >> ...001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch | 0 > >> .../ti-img-sgx}/ti-sgx-ddk-km_1.17.4948957.bb | 0 > >> .../{libgles => ti-img-sgx}/ti-sgx-ddk-um_1.17.4948957.bb | 0 > >> .../ti-vivante-gc320}/ti-gc320-driver_5.0.11.p7.bb | 0 > >> .../{libgal => ti-vivante-gc320}/ti-gc320-libs_5.0.11.p7.bb | 0 > >> .../ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb | 0 > >> .../ti-img-encode-decode/viddec-test-app_1.0.0.bb | 0 > >> .../ti-img-encode-decode/videnc-test-app_1.0.0.bb | 0 > >> 12 files changed, 0 insertions(+), 0 deletions(-) > >> rename recipes-bsp/{ti-img-encode-decode => vxd-dec-fw}/vxd-dec-fw_git.bb (100%) > >> rename {recipes-bsp/powervr-drivers => recipes-graphics/ti-img-rogue}/ti-img-rogue-driver/0001-km-support-general-toolchain.patch (100%) > >> rename {recipes-bsp/powervr-drivers => recipes-graphics/ti-img-rogue}/ti-img-rogue-driver_1.10.5371573.bb (100%) > >> rename recipes-graphics/{libgles => ti-img-rogue}/ti-img-rogue-umlibs_1.10.5371573.bb (100%) > >> rename {recipes-bsp/powervr-drivers => recipes-graphics/ti-img-sgx}/ti-sgx-ddk-km/0001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch (100%) > >> rename {recipes-bsp/powervr-drivers => recipes-graphics/ti-img-sgx}/ti-sgx-ddk-km_1.17.4948957.bb (100%) > >> rename recipes-graphics/{libgles => ti-img-sgx}/ti-sgx-ddk-um_1.17.4948957.bb (100%) > >> rename {recipes-bsp/gc320-drivers => recipes-graphics/ti-vivante-gc320}/ti-gc320-driver_5.0.11.p7.bb (100%) > >> rename recipes-graphics/{libgal => ti-vivante-gc320}/ti-gc320-libs_5.0.11.p7.bb (100%) > >> rename {recipes-bsp => recipes-multimedia}/ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb (100%) > >> rename {recipes-bsp => recipes-multimedia}/ti-img-encode-decode/viddec-test-app_1.0.0.bb (100%) > >> rename {recipes-bsp => recipes-multimedia}/ti-img-encode-decode/videnc-test-app_1.0.0.bb (100%) > >> > >> diff --git a/recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb b/recipes-bsp/vxd-dec-fw/vxd-dec-fw_git.bb > >> similarity index 100% > >> rename from recipes-bsp/ti-img-encode-decode/vxd-dec-fw_git.bb > >> rename to recipes-bsp/vxd-dec-fw/vxd-dec-fw_git.bb > >> diff --git a/recipes-bsp/powervr-drivers/ti-img-rogue-driver/0001-km-support-general-toolchain.patch b/recipes-graphics/ti-img-rogue/ti-img-rogue-driver/0001-km-support-general-toolchain.patch > >> similarity index 100% > >> rename from recipes-bsp/powervr-drivers/ti-img-rogue-driver/0001-km-support-general-toolchain.patch > >> rename to recipes-graphics/ti-img-rogue/ti-img-rogue-driver/0001-km-support-general-toolchain.patch > >> diff --git a/recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.10.5371573.bb b/recipes-graphics/ti-img-rogue/ti-img-rogue-driver_1.10.5371573.bb > >> similarity index 100% > >> rename from recipes-bsp/powervr-drivers/ti-img-rogue-driver_1.10.5371573.bb > >> rename to recipes-graphics/ti-img-rogue/ti-img-rogue-driver_1.10.5371573.bb > >> diff --git a/recipes-graphics/libgles/ti-img-rogue-umlibs_1.10.5371573.bb b/recipes-graphics/ti-img-rogue/ti-img-rogue-umlibs_1.10.5371573.bb > >> similarity index 100% > >> rename from recipes-graphics/libgles/ti-img-rogue-umlibs_1.10.5371573.bb > >> rename to recipes-graphics/ti-img-rogue/ti-img-rogue-umlibs_1.10.5371573.bb > >> diff --git a/recipes-bsp/powervr-drivers/ti-sgx-ddk-km/0001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch b/recipes-graphics/ti-img-sgx/ti-sgx-ddk-km/0001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch > >> similarity index 100% > >> rename from recipes-bsp/powervr-drivers/ti-sgx-ddk-km/0001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch > >> rename to recipes-graphics/ti-img-sgx/ti-sgx-ddk-km/0001-km-support-OpenEmbedded-hardfp-toolchain-w-o-gnueabi.patch > >> diff --git a/recipes-bsp/powervr-drivers/ti-sgx-ddk-km_1.17.4948957.bb b/recipes-graphics/ti-img-sgx/ti-sgx-ddk-km_1.17.4948957.bb > >> similarity index 100% > >> rename from recipes-bsp/powervr-drivers/ti-sgx-ddk-km_1.17.4948957.bb > >> rename to recipes-graphics/ti-img-sgx/ti-sgx-ddk-km_1.17.4948957.bb > >> diff --git a/recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb b/recipes-graphics/ti-img-sgx/ti-sgx-ddk-um_1.17.4948957.bb > >> similarity index 100% > >> rename from recipes-graphics/libgles/ti-sgx-ddk-um_1.17.4948957.bb > >> rename to recipes-graphics/ti-img-sgx/ti-sgx-ddk-um_1.17.4948957.bb > >> diff --git a/recipes-bsp/gc320-drivers/ti-gc320-driver_5.0.11.p7.bb b/recipes-graphics/ti-vivante-gc320/ti-gc320-driver_5.0.11.p7.bb > >> similarity index 100% > >> rename from recipes-bsp/gc320-drivers/ti-gc320-driver_5.0.11.p7.bb > >> rename to recipes-graphics/ti-vivante-gc320/ti-gc320-driver_5.0.11.p7.bb > >> diff --git a/recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb b/recipes-graphics/ti-vivante-gc320/ti-gc320-libs_5.0.11.p7.bb > >> similarity index 100% > >> rename from recipes-graphics/libgal/ti-gc320-libs_5.0.11.p7.bb > >> rename to recipes-graphics/ti-vivante-gc320/ti-gc320-libs_5.0.11.p7.bb > >> diff --git a/recipes-bsp/ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb b/recipes-multimedia/ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb > >> similarity index 100% > >> rename from recipes-bsp/ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb > >> rename to recipes-multimedia/ti-img-encode-decode/ti-img-encode-decode_1.0.0.bb > >> diff --git a/recipes-bsp/ti-img-encode-decode/viddec-test-app_1.0.0.bb b/recipes-multimedia/ti-img-encode-decode/viddec-test-app_1.0.0.bb > >> similarity index 100% > >> rename from recipes-bsp/ti-img-encode-decode/viddec-test-app_1.0.0.bb > >> rename to recipes-multimedia/ti-img-encode-decode/viddec-test-app_1.0.0.bb > >> diff --git a/recipes-bsp/ti-img-encode-decode/videnc-test-app_1.0.0.bb b/recipes-multimedia/ti-img-encode-decode/videnc-test-app_1.0.0.bb > >> similarity index 100% > >> rename from recipes-bsp/ti-img-encode-decode/videnc-test-app_1.0.0.bb > >> rename to recipes-multimedia/ti-img-encode-decode/videnc-test-app_1.0.0.bb > >> -- > >> 2.17.1 > >>