* [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