[Buildroot] [PATCH v5 1/1] sunxi-mali-driver-mainline: new package

[Giulio Benetti]

Hello Everybody,

I was thinking about doing a unique patchset for:
sunxi-mali-driver-mainline
and
sunxi-mali-mainline

They're very connected each other.
What do you think?

Il 25/11/2017 21:33, Giulio Benetti ha scritto:
> Hi Arnout,
> 
> Il 25/11/2017 18:59, Arnout Vandecappelle ha scritto:
>>   Hi Giulio,
>>
>> On 21-11-17 23:07, Giulio Benetti wrote:
>>> Added kernel allwinner mali driver package.
>>> Used combined with userspace Allwinner openGL blob,
>>> it gives possibility to use 3D openGL SoC acceleration.
>>
>>   I should have asked before, but: where do those binaries come from? 
>> Are they
>> from the mali-sunxi package? Or not yet packaged in Buildroot?
> 
> It will be combined with sunxi-mali-mainline,
> I've submitted a patch still pending:
> [PATCH v4 1/1] sunxi-mali-mainline: new package
> http://buildroot-busybox.2317881.n4.nabble.com/PATCH-v4-1-1-sunxi-mali-mainline-new-package-td180182.html 
> 
> 
>>
>>
>> [snip]
>>> index fe5ccc4..8de2227 100644
>>> --- a/package/Config.in
>>> +++ b/package/Config.in
>>> @@ -496,6 +496,7 @@ endmenu
>>>       source "package/statserial/Config.in"
>>>       source "package/stm32flash/Config.in"
>>>       source "package/sunxi-cedarx/Config.in"
>>> +    source "package/sunxi-mali-driver-mainline/Config.in"
>>>       source "package/sunxi-mali/Config.in"
>>
>>   foo-bar sorts after foo, so swap these two lines.
> 
> Ok, on next patch I will correct it
> 
>>
>>>       source "package/sysstat/Config.in"
>>>       source "package/targetcli-fb/Config.in"
>>> diff --git a/package/sunxi-mali-driver-mainline/Config.in 
>>> b/package/sunxi-mali-driver-mainline/Config.in
>>> new file mode 100644
>>> index 0000000..5b7b2ba
>>> --- /dev/null
>>> +++ b/package/sunxi-mali-driver-mainline/Config.in
>>> @@ -0,0 +1,22 @@
>>> +comment "sunxi-mali-driver-mainline needs a Linux kernel to be built"
>>> +    depends on !BR2_LINUX_KERNEL
>>> +
>>> +config BR2_PACKAGE_SUNXI_MALI_DRIVER_MAINLINE
>>> +    bool "sunxi-mali-driver-mainline"
>>> +    depends on BR2_LINUX_KERNEL
>>> +    select BR2_LINUX_NEEDS_MODULES # not using kernel-module infra
>>> +    help
>>> +      This package builds and installs the Linux kernel driver for
>>> +      the Mali GPU on Allwinner platforms, compatible with recent
>>> +      Linux kernels (>= 4.4). Note that it should be installed
>>> +      together with the corresponding userspace binary blobs to
>>
>>   I guess with 'binary blobs' you mean the OpenGL libraries, which are 
>> only
>> provided as binaries? 'binary blobs' to me sounds like something like 
>> firmware
>> blobs: is put in your rootfs but doesn't run on the target processor. 
>> So I'd
>> rephrase as: ... together with the corresponding userspace OpenGL 
>> libraries,
>> which are provided as binaries only.
> 
> Ok
> 
>>
>>   And preferably also explain where you can get those.
> 
> Ok, I will add github sunxi-mali by mripard
> 
>>
>>> +      provide complete OpenGL functionality.
>>> +
>>> +      sunxi-mali-driver-mainline requires a Linux kernel >= 4.4 with 
>>> the
>>> +      following options enabled:
>>> +
>>> +      - CONFIG_CMA
>>> +      - CONFIG_DMA_CMA
>>> +      - CONFIG_DRM_FBDEV_OVERALLOC >= 200 for double buffer or more
>>> +
>>> +      https://github.com/mripard/sunxi-mali/blob/master/README.md
>>> diff --git 
>>> a/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk 
>>> b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>> new file mode 100644
>>> index 0000000..1692646
>>> --- /dev/null
>>> +++ b/package/sunxi-mali-driver-mainline/sunxi-mali-driver-mainline.mk
>>> @@ -0,0 +1,26 @@
>>> +################################################################################ 
>>>
>>> +#
>>> +# sunxi-mali-driver-mainline
>>> +#
>>> +################################################################################ 
>>>
>>> +
>>> +SUNXI_MALI_DRIVER_MAINLINE_VERSION = 
>>> 5b7b3c0b6e1736676c90928816d271debbb09d95
>>> +SUNXI_MALI_DRIVER_MAINLINE_SITE = $(call 
>>> github,mripard,sunxi-mali,$(SUNXI_MALI_DRIVER_MAINLINE_VERSION))
>>> +SUNXI_MALI_DRIVER_MAINLINE_DEPENDENCIES = linux
>>> +
>>> +SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS = \
>>> +    KDIR=$(LINUX_DIR) \
>>> +    CROSS_COMPILE=$(TARGET_CROSS) \
>>> +    INSTALL_MOD_PATH=$(TARGET_DIR)
>>> +
>>> +define SUNXI_MALI_DRIVER_MAINLINE_BUILD_CMDS
>>> +    cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>> +        $(SHELL) ./build.sh -r r6p2 -a
>>
>>   Shouldn't the r6p2 be configurable? That can be done in a later 
>> patch, though.
> 
> You're right, but at this time there's only r6p2 available.
> Anyway I can add it parametric, so it will be ready for any next versions.
> 
>>
>>   Regards,
>>   Arnout
>>
>>> +endef
>>> +
>>> +define SUNXI_MALI_DRIVER_MAINLINE_INSTALL_TARGET_CMDS
>>> +    cd $(@D) && $(SUNXI_MALI_DRIVER_MAINLINE_MAKE_OPTS) \
>>> +        $(SHELL) ./build.sh -r r6p2 -i
>>> +endef
>>> +
>>> +$(eval $(generic-package))
>>>
>>
> 
> 


-- 
Giulio Benetti
R&D Manager &
Advanced Research

MICRONOVA SRL
Sede: Via A. Niedda 3 - 35010 Vigonza (PD)
Tel. 049/8931563 - Fax 049/8931346
Cod.Fiscale - P.IVA 02663420285
Capitale Sociale € 26.000 i.v.
Iscritta al Reg. Imprese di Padova N. 02663420285
Numero R.E.A. 258642