mbox series

[v2,0/5] Rename sunxi-mali-mainline* to sunxi-mali-utgard* and fix Mali loading

Message ID 20211105221800.1451167-1-giulio.benetti@benettiengineering.com
Headers show
Series Rename sunxi-mali-mainline* to sunxi-mali-utgard* and fix Mali loading | expand

Message

Giulio Benetti Nov. 5, 2021, 10:17 p.m. UTC
With this patchset I respin the first 2 patches. While testing on hardware
the patches they work, but I've left back an error while bumping linux
version to 5.14. The error is that Linux's sunxi_defconfig(used by both
a20_olinuxino_lime/2) enables Lima driver, but in this 2 boards we want
to use Mali driver, so when Mali driver tries to load(being it a module),
it finds that Lima already has taken gpu resources and fail. So here I add
3 patches, 1 to add a Linux fragment to disable Lima in Linux and other 2
to use that fragment in a20_olinuxino_lime/2.

Giulio Benetti (3):
  board/olimex/a20_olinuxino: add linux-disable-lima.fragment
  configs/olimex_a20_olinuxino_lime2_defconfig: use
    linux-disable-lima.fragment
  configs/olimex_a20_olinuxino_lime_defconfig: use
    linux-disable-lima.fragment

Miquel Raynal (2):
  package/sunxi-mali-utgard: rename from sunxi-mali-mainline
  package/sunxi-mali-utgard-driver: rename from
    sunxi-mali-mainline-driver

 Config.in.legacy                              | 42 ++++++++++++++++
 DEVELOPERS                                    |  4 +-
 .../a20_olinuxino/linux-disable-lima.fragment |  2 +
 configs/olimex_a20_olinuxino_lime2_defconfig  |  7 +--
 configs/olimex_a20_olinuxino_lime_defconfig   |  7 +--
 docs/manual/adding-packages-virtual.txt       |  2 +-
 package/Config.in                             |  4 +-
 package/glslsandbox-player/Config.in          |  4 +-
 .../glslsandbox-player/glslsandbox-player.mk  |  2 +-
 package/qt5/qt5base/qt5base.mk                |  2 +-
 .../sunxi-mali-mainline-driver.mk             | 41 ----------------
 package/sunxi-mali-mainline/Config.in         | 42 ----------------
 .../sunxi-mali-mainline.mk                    | 49 -------------------
 .../Config.in                                 | 20 ++++----
 .../sunxi-mali-utgard-driver.hash}            |  2 +-
 .../sunxi-mali-utgard-driver.mk               | 41 ++++++++++++++++
 package/sunxi-mali-utgard/Config.in           | 43 ++++++++++++++++
 .../egl.pc                                    |  0
 .../glesv2.pc                                 |  0
 .../sunxi-mali-utgard.hash}                   |  2 +-
 .../sunxi-mali-utgard/sunxi-mali-utgard.mk    | 49 +++++++++++++++++++
 21 files changed, 206 insertions(+), 159 deletions(-)
 create mode 100644 board/olimex/a20_olinuxino/linux-disable-lima.fragment
 delete mode 100644 package/sunxi-mali-mainline-driver/sunxi-mali-mainline-driver.mk
 delete mode 100644 package/sunxi-mali-mainline/Config.in
 delete mode 100644 package/sunxi-mali-mainline/sunxi-mali-mainline.mk
 rename package/{sunxi-mali-mainline-driver => sunxi-mali-utgard-driver}/Config.in (60%)
 rename package/{sunxi-mali-mainline-driver/sunxi-mali-mainline-driver.hash => sunxi-mali-utgard-driver/sunxi-mali-utgard-driver.hash} (66%)
 create mode 100644 package/sunxi-mali-utgard-driver/sunxi-mali-utgard-driver.mk
 create mode 100644 package/sunxi-mali-utgard/Config.in
 rename package/{sunxi-mali-mainline => sunxi-mali-utgard}/egl.pc (100%)
 rename package/{sunxi-mali-mainline => sunxi-mali-utgard}/glesv2.pc (100%)
 rename package/{sunxi-mali-mainline/sunxi-mali-mainline.hash => sunxi-mali-utgard/sunxi-mali-utgard.hash} (70%)
 create mode 100644 package/sunxi-mali-utgard/sunxi-mali-utgard.mk

Comments

Giulio Benetti Nov. 5, 2021, 10:20 p.m. UTC | #1
I've realized only now that this patchset had to be v3, sorry for the noise!

Do I keep v2 as it is or do I resend?

Best regards
Arnout Vandecappelle Jan. 6, 2022, 6:03 p.m. UTC | #2
On 05/11/2021 23:17, Giulio Benetti wrote:
> With this patchset I respin the first 2 patches. While testing on hardware
> the patches they work, but I've left back an error while bumping linux
> version to 5.14. The error is that Linux's sunxi_defconfig(used by both
> a20_olinuxino_lime/2) enables Lima driver, but in this 2 boards we want
> to use Mali driver, so when Mali driver tries to load(being it a module),
> it finds that Lima already has taken gpu resources and fail. So here I add
> 3 patches, 1 to add a Linux fragment to disable Lima in Linux and other 2
> to use that fragment in a20_olinuxino_lime/2.
> 
> Giulio Benetti (3):
>    board/olimex/a20_olinuxino: add linux-disable-lima.fragment
>    configs/olimex_a20_olinuxino_lime2_defconfig: use
>      linux-disable-lima.fragment
>    configs/olimex_a20_olinuxino_lime_defconfig: use
>      linux-disable-lima.fragment
> 
> Miquel Raynal (2):
>    package/sunxi-mali-utgard: rename from sunxi-mali-mainline
>    package/sunxi-mali-utgard-driver: rename from
>      sunxi-mali-mainline-driver

  I applied these two patches to master.

  Luca, your review was very much appreciated so if you could maybe review the 
rest of this series as well? :-)

  Regards,
  Arnout

> 
>   Config.in.legacy                              | 42 ++++++++++++++++
>   DEVELOPERS                                    |  4 +-
>   .../a20_olinuxino/linux-disable-lima.fragment |  2 +
>   configs/olimex_a20_olinuxino_lime2_defconfig  |  7 +--
>   configs/olimex_a20_olinuxino_lime_defconfig   |  7 +--
>   docs/manual/adding-packages-virtual.txt       |  2 +-
>   package/Config.in                             |  4 +-
>   package/glslsandbox-player/Config.in          |  4 +-
>   .../glslsandbox-player/glslsandbox-player.mk  |  2 +-
>   package/qt5/qt5base/qt5base.mk                |  2 +-
>   .../sunxi-mali-mainline-driver.mk             | 41 ----------------
>   package/sunxi-mali-mainline/Config.in         | 42 ----------------
>   .../sunxi-mali-mainline.mk                    | 49 -------------------
>   .../Config.in                                 | 20 ++++----
>   .../sunxi-mali-utgard-driver.hash}            |  2 +-
>   .../sunxi-mali-utgard-driver.mk               | 41 ++++++++++++++++
>   package/sunxi-mali-utgard/Config.in           | 43 ++++++++++++++++
>   .../egl.pc                                    |  0
>   .../glesv2.pc                                 |  0
>   .../sunxi-mali-utgard.hash}                   |  2 +-
>   .../sunxi-mali-utgard/sunxi-mali-utgard.mk    | 49 +++++++++++++++++++
>   21 files changed, 206 insertions(+), 159 deletions(-)
>   create mode 100644 board/olimex/a20_olinuxino/linux-disable-lima.fragment
>   delete mode 100644 package/sunxi-mali-mainline-driver/sunxi-mali-mainline-driver.mk
>   delete mode 100644 package/sunxi-mali-mainline/Config.in
>   delete mode 100644 package/sunxi-mali-mainline/sunxi-mali-mainline.mk
>   rename package/{sunxi-mali-mainline-driver => sunxi-mali-utgard-driver}/Config.in (60%)
>   rename package/{sunxi-mali-mainline-driver/sunxi-mali-mainline-driver.hash => sunxi-mali-utgard-driver/sunxi-mali-utgard-driver.hash} (66%)
>   create mode 100644 package/sunxi-mali-utgard-driver/sunxi-mali-utgard-driver.mk
>   create mode 100644 package/sunxi-mali-utgard/Config.in
>   rename package/{sunxi-mali-mainline => sunxi-mali-utgard}/egl.pc (100%)
>   rename package/{sunxi-mali-mainline => sunxi-mali-utgard}/glesv2.pc (100%)
>   rename package/{sunxi-mali-mainline/sunxi-mali-mainline.hash => sunxi-mali-utgard/sunxi-mali-utgard.hash} (70%)
>   create mode 100644 package/sunxi-mali-utgard/sunxi-mali-utgard.mk
>
Luca Ceresoli Jan. 6, 2022, 10:24 p.m. UTC | #3
Hi Arnout,

On 06/01/22 19:03, Arnout Vandecappelle wrote:
> 
> 
> On 05/11/2021 23:17, Giulio Benetti wrote:
>> With this patchset I respin the first 2 patches. While testing on
>> hardware
>> the patches they work, but I've left back an error while bumping linux
>> version to 5.14. The error is that Linux's sunxi_defconfig(used by both
>> a20_olinuxino_lime/2) enables Lima driver, but in this 2 boards we want
>> to use Mali driver, so when Mali driver tries to load(being it a module),
>> it finds that Lima already has taken gpu resources and fail. So here I
>> add
>> 3 patches, 1 to add a Linux fragment to disable Lima in Linux and other 2
>> to use that fragment in a20_olinuxino_lime/2.
>>
>> Giulio Benetti (3):
>>    board/olimex/a20_olinuxino: add linux-disable-lima.fragment
>>    configs/olimex_a20_olinuxino_lime2_defconfig: use
>>      linux-disable-lima.fragment
>>    configs/olimex_a20_olinuxino_lime_defconfig: use
>>      linux-disable-lima.fragment
>>
>> Miquel Raynal (2):
>>    package/sunxi-mali-utgard: rename from sunxi-mali-mainline
>>    package/sunxi-mali-utgard-driver: rename from
>>      sunxi-mali-mainline-driver
> 
>  I applied these two patches to master.
> 
>  Luca, your review was very much appreciated so if you could maybe
> review the rest of this series as well? :-)

I'm afraid I'm unable to do so these days. Hopefully next week.