* [DistroKit] [PATCH] patches: move TF-A patches to the platforms @ 2023-01-06 11:54 Michael Olbrich 2023-06-12 12:23 ` Robert Schwebel 0 siblings, 1 reply; 5+ messages in thread From: Michael Olbrich @ 2023-01-06 11:54 UTC (permalink / raw) To: distrokit On all other platforms "ptxdist lint" complains because they are unused. These patches are platform specific so they should be in the platform. Move the patches into the correct platform to fix this. Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de> --- ...001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch | 0 .../0002-build-makefile-add-helper-to-detect-linker-options.patch | 0 .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 .../tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch | 0 {patches => configs/platform-v7a/patches}/tf-a-v2.7/series | 0 .../0001-build-makefile-add-helper-to-detect-linker-options.patch | 0 ...002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch | 0 .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 .../0004-makefile-link-with-z-noexecstack.patch | 0 .../platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series | 0 10 files changed, 0 insertions(+), 0 deletions(-) rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch (100%) rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch (100%) rename {patches/imx-atf-imx_5.4.24_2.1.0 => configs/platform-v7a/patches/tf-a-v2.7}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch (100%) rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/series (100%) rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch (100%) rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch (100%) rename {patches/tf-a-v2.7 => configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch (100%) rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series (100%) diff --git a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch b/configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch similarity index 100% rename from patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch rename to configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch diff --git a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch similarity index 100% rename from patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch rename to configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch similarity index 100% rename from patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch rename to configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch diff --git a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch similarity index 100% rename from patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch rename to configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch diff --git a/patches/tf-a-v2.7/series b/configs/platform-v7a/patches/tf-a-v2.7/series similarity index 100% rename from patches/tf-a-v2.7/series rename to configs/platform-v7a/patches/tf-a-v2.7/series diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch similarity index 100% rename from patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch similarity index 100% rename from patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch diff --git a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch similarity index 100% rename from patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch similarity index 100% rename from patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/series b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series similarity index 100% rename from patches/imx-atf-imx_5.4.24_2.1.0/series rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series -- 2.30.2 ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] patches: move TF-A patches to the platforms 2023-01-06 11:54 [DistroKit] [PATCH] patches: move TF-A patches to the platforms Michael Olbrich @ 2023-06-12 12:23 ` Robert Schwebel 2023-06-12 12:25 ` Ahmad Fatoum 0 siblings, 1 reply; 5+ messages in thread From: Robert Schwebel @ 2023-06-12 12:23 UTC (permalink / raw) To: Michael Olbrich; +Cc: distrokit Hi, could you rebase the series? Upstream has 2.8 meanwhile. rsc On Fri, Jan 06, 2023 at 12:54:24PM +0100, Michael Olbrich wrote: > On all other platforms "ptxdist lint" complains because they are unused. > > These patches are platform specific so they should be in the platform. > Move the patches into the correct platform to fix this. > > Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de> > --- > ...001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch | 0 > .../0002-build-makefile-add-helper-to-detect-linker-options.patch | 0 > .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 > .../tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch | 0 > {patches => configs/platform-v7a/patches}/tf-a-v2.7/series | 0 > .../0001-build-makefile-add-helper-to-detect-linker-options.patch | 0 > ...002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch | 0 > .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 > .../0004-makefile-link-with-z-noexecstack.patch | 0 > .../platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series | 0 > 10 files changed, 0 insertions(+), 0 deletions(-) > rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch (100%) > rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch (100%) > rename {patches/imx-atf-imx_5.4.24_2.1.0 => configs/platform-v7a/patches/tf-a-v2.7}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) > rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch (100%) > rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/series (100%) > rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch (100%) > rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch (100%) > rename {patches/tf-a-v2.7 => configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) > rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch (100%) > rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series (100%) > > diff --git a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch b/configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > similarity index 100% > rename from patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > rename to configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > diff --git a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > similarity index 100% > rename from patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > rename to configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > similarity index 100% > rename from patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > rename to configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > diff --git a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > similarity index 100% > rename from patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > rename to configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > diff --git a/patches/tf-a-v2.7/series b/configs/platform-v7a/patches/tf-a-v2.7/series > similarity index 100% > rename from patches/tf-a-v2.7/series > rename to configs/platform-v7a/patches/tf-a-v2.7/series > diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > similarity index 100% > rename from patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > similarity index 100% > rename from patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > diff --git a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > similarity index 100% > rename from patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > similarity index 100% > rename from patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/series b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series > similarity index 100% > rename from patches/imx-atf-imx_5.4.24_2.1.0/series > rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series > -- > 2.30.2 > > > -- Pengutronix e.K. | Dipl.-Ing. Robert Schwebel | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] patches: move TF-A patches to the platforms 2023-06-12 12:23 ` Robert Schwebel @ 2023-06-12 12:25 ` Ahmad Fatoum 2023-06-13 7:20 ` Michael Olbrich 0 siblings, 1 reply; 5+ messages in thread From: Ahmad Fatoum @ 2023-06-12 12:25 UTC (permalink / raw) To: Robert Schwebel, Michael Olbrich; +Cc: distrokit On 12.06.23 14:23, Robert Schwebel wrote: > Hi, > > could you rebase the series? Upstream has 2.8 meanwhile. We should be using the same package for both platforms, so this patch shouldn't be necessary anymore. > > rsc > > On Fri, Jan 06, 2023 at 12:54:24PM +0100, Michael Olbrich wrote: >> On all other platforms "ptxdist lint" complains because they are unused. >> >> These patches are platform specific so they should be in the platform. >> Move the patches into the correct platform to fix this. >> >> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de> >> --- >> ...001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch | 0 >> .../0002-build-makefile-add-helper-to-detect-linker-options.patch | 0 >> .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 >> .../tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch | 0 >> {patches => configs/platform-v7a/patches}/tf-a-v2.7/series | 0 >> .../0001-build-makefile-add-helper-to-detect-linker-options.patch | 0 >> ...002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch | 0 >> .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 >> .../0004-makefile-link-with-z-noexecstack.patch | 0 >> .../platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series | 0 >> 10 files changed, 0 insertions(+), 0 deletions(-) >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch (100%) >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch (100%) >> rename {patches/imx-atf-imx_5.4.24_2.1.0 => configs/platform-v7a/patches/tf-a-v2.7}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch (100%) >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/series (100%) >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch (100%) >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch (100%) >> rename {patches/tf-a-v2.7 => configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch (100%) >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series (100%) >> >> diff --git a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch b/configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch >> similarity index 100% >> rename from patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch >> rename to configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch >> diff --git a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch >> similarity index 100% >> rename from patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch >> rename to configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch >> similarity index 100% >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch >> rename to configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch >> diff --git a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch >> similarity index 100% >> rename from patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch >> rename to configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch >> diff --git a/patches/tf-a-v2.7/series b/configs/platform-v7a/patches/tf-a-v2.7/series >> similarity index 100% >> rename from patches/tf-a-v2.7/series >> rename to configs/platform-v7a/patches/tf-a-v2.7/series >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch >> similarity index 100% >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch >> similarity index 100% >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch >> diff --git a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch >> similarity index 100% >> rename from patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch >> similarity index 100% >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/series b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series >> similarity index 100% >> rename from patches/imx-atf-imx_5.4.24_2.1.0/series >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series >> -- >> 2.30.2 >> >> >> > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] patches: move TF-A patches to the platforms 2023-06-12 12:25 ` Ahmad Fatoum @ 2023-06-13 7:20 ` Michael Olbrich 2023-06-20 12:52 ` Robert Schwebel 0 siblings, 1 reply; 5+ messages in thread From: Michael Olbrich @ 2023-06-13 7:20 UTC (permalink / raw) To: Ahmad Fatoum; +Cc: distrokit On Mon, Jun 12, 2023 at 02:25:08PM +0200, Ahmad Fatoum wrote: > On 12.06.23 14:23, Robert Schwebel wrote: > > could you rebase the series? Upstream has 2.8 meanwhile. > > We should be using the same package for both platforms, so this patch shouldn't > be necessary anymore. There are other platforms that don't build tf-a at all, so ptxdist lint will fail for those. And you need to update tf-a for both platform in lock-step. If that's ok, then we can leave it as it is. I only use v7a and v8a to test ptxdist itself, so I don't mind lint failures for other platforms. Michael > > On Fri, Jan 06, 2023 at 12:54:24PM +0100, Michael Olbrich wrote: > >> On all other platforms "ptxdist lint" complains because they are unused. > >> > >> These patches are platform specific so they should be in the platform. > >> Move the patches into the correct platform to fix this. > >> > >> Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de> > >> --- > >> ...001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch | 0 > >> .../0002-build-makefile-add-helper-to-detect-linker-options.patch | 0 > >> .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 > >> .../tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch | 0 > >> {patches => configs/platform-v7a/patches}/tf-a-v2.7/series | 0 > >> .../0001-build-makefile-add-helper-to-detect-linker-options.patch | 0 > >> ...002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch | 0 > >> .../0003-feat-build-add-support-for-new-binutils-versions.patch | 0 > >> .../0004-makefile-link-with-z-noexecstack.patch | 0 > >> .../platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series | 0 > >> 10 files changed, 0 insertions(+), 0 deletions(-) > >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch (100%) > >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch (100%) > >> rename {patches/imx-atf-imx_5.4.24_2.1.0 => configs/platform-v7a/patches/tf-a-v2.7}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) > >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch (100%) > >> rename {patches => configs/platform-v7a/patches}/tf-a-v2.7/series (100%) > >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch (100%) > >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch (100%) > >> rename {patches/tf-a-v2.7 => configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0}/0003-feat-build-add-support-for-new-binutils-versions.patch (100%) > >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch (100%) > >> rename {patches => configs/platform-v8a/patches}/imx-atf-imx_5.4.24_2.1.0/series (100%) > >> > >> diff --git a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch b/configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > >> similarity index 100% > >> rename from patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > >> rename to configs/platform-v7a/patches/tf-a-v2.7/0001-fix-stm32mp1-fdts-stm32mp1-align-DDR-regulators-with.patch > >> diff --git a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > >> similarity index 100% > >> rename from patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > >> rename to configs/platform-v7a/patches/tf-a-v2.7/0002-build-makefile-add-helper-to-detect-linker-options.patch > >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > >> similarity index 100% > >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > >> rename to configs/platform-v7a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > >> diff --git a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > >> similarity index 100% > >> rename from patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > >> rename to configs/platform-v7a/patches/tf-a-v2.7/0004-makefile-link-with-z-noexecstack.patch > >> diff --git a/patches/tf-a-v2.7/series b/configs/platform-v7a/patches/tf-a-v2.7/series > >> similarity index 100% > >> rename from patches/tf-a-v2.7/series > >> rename to configs/platform-v7a/patches/tf-a-v2.7/series > >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > >> similarity index 100% > >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0001-build-makefile-add-helper-to-detect-linker-options.patch > >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > >> similarity index 100% > >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0002-build-makefile-avoid-gcc-warning-that-triggers-with-.patch > >> diff --git a/patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > >> similarity index 100% > >> rename from patches/tf-a-v2.7/0003-feat-build-add-support-for-new-binutils-versions.patch > >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0003-feat-build-add-support-for-new-binutils-versions.patch > >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > >> similarity index 100% > >> rename from patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/0004-makefile-link-with-z-noexecstack.patch > >> diff --git a/patches/imx-atf-imx_5.4.24_2.1.0/series b/configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series > >> similarity index 100% > >> rename from patches/imx-atf-imx_5.4.24_2.1.0/series > >> rename to configs/platform-v8a/patches/imx-atf-imx_5.4.24_2.1.0/series > >> -- > >> 2.30.2 > >> > >> > >> > > > > -- > Pengutronix e.K. | | > Steuerwalder Str. 21 | http://www.pengutronix.de/ | > 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | > Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | > > > -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-5555 | ^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [DistroKit] [PATCH] patches: move TF-A patches to the platforms 2023-06-13 7:20 ` Michael Olbrich @ 2023-06-20 12:52 ` Robert Schwebel 0 siblings, 0 replies; 5+ messages in thread From: Robert Schwebel @ 2023-06-20 12:52 UTC (permalink / raw) To: Ahmad Fatoum, distrokit On Tue, Jun 13, 2023 at 09:20:50AM +0200, Michael Olbrich wrote: > On Mon, Jun 12, 2023 at 02:25:08PM +0200, Ahmad Fatoum wrote: > > On 12.06.23 14:23, Robert Schwebel wrote: > > > could you rebase the series? Upstream has 2.8 meanwhile. > > > > We should be using the same package for both platforms, so this patch shouldn't > > be necessary anymore. > > There are other platforms that don't build tf-a at all, so ptxdist lint > will fail for those. And you need to update tf-a for both platform in > lock-step. If that's ok, then we can leave it as it is. > > I only use v7a and v8a to test ptxdist itself, so I don't mind lint > failures for other platforms. For the moment, I'll leave it as it is. rsc -- Pengutronix e.K. | Dipl.-Ing. Robert Schwebel | Steuerwalder Str. 21 | https://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 | Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917-9 | ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2023-06-20 12:52 UTC | newest] Thread overview: 5+ messages (download: mbox.gz / follow: Atom feed) -- links below jump to the message on this page -- 2023-01-06 11:54 [DistroKit] [PATCH] patches: move TF-A patches to the platforms Michael Olbrich 2023-06-12 12:23 ` Robert Schwebel 2023-06-12 12:25 ` Ahmad Fatoum 2023-06-13 7:20 ` Michael Olbrich 2023-06-20 12:52 ` Robert Schwebel
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox