* [meta-fsl-arm PATCH v2 1/3] imx-base.inc: Ensure MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT is expanded
@ 2015-10-07 18:50 Otavio Salvador
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community Otavio Salvador
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 3/3] linux-fslc-mx6: Update revision to 964e5a3 revision Otavio Salvador
0 siblings, 2 replies; 5+ messages in thread
From: Otavio Salvador @ 2015-10-07 18:50 UTC (permalink / raw)
To: meta-freescale Mailing List; +Cc: Otavio Salvador
The MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT needs to be expanded and
cannot have a default overriden later, otherwise it does not respect
the default set in the SoC.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2:
- New patch
conf/machine/include/imx-base.inc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/conf/machine/include/imx-base.inc b/conf/machine/include/imx-base.inc
index d2fe875..a398164 100644
--- a/conf/machine/include/imx-base.inc
+++ b/conf/machine/include/imx-base.inc
@@ -105,7 +105,7 @@ SOC_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT_mx6sl = "1"
# Handle Vivante kernel driver setting:
# 0 - machine does not have Vivante GPU driver support
# 1 - machine has Vivante GPU driver support
-MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT ??= "${SOC_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT}"
+MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT ?= "${SOC_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT}"
# mx6 GPU libraries
PREFERRED_PROVIDER_virtual/egl_mx6q ?= "imx-gpu-viv"
--
2.6.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community
2015-10-07 18:50 [meta-fsl-arm PATCH v2 1/3] imx-base.inc: Ensure MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT is expanded Otavio Salvador
@ 2015-10-07 18:50 ` Otavio Salvador
2015-10-07 19:08 ` Daiane Angolini
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 3/3] linux-fslc-mx6: Update revision to 964e5a3 revision Otavio Salvador
1 sibling, 1 reply; 5+ messages in thread
From: Otavio Salvador @ 2015-10-07 18:50 UTC (permalink / raw)
To: meta-freescale Mailing List; +Cc: Otavio Salvador
The driver source code provided by Freescale need fixes to properly
work. This forks the code to a more community driven development,
allowing for a faster development of the driver and easier integration
of fixes.
To allow visual identification, when checking for bug reports, the
'+fslc' suffix is added to the module.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2:
- New patch
.../kernel-module-imx-gpu-viv/updatemakefile.patch | 158 ---------------------
.../work-around-include-file-rename.patch | 30 ----
.../kernel-module-imx-gpu-viv_5.0.11.p7.1+fslc.bb | 17 +++
.../kernel-module-imx-gpu-viv_5.0.11.p7.1.bb | 23 ---
4 files changed, 17 insertions(+), 211 deletions(-)
delete mode 100644 recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/updatemakefile.patch
delete mode 100644 recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/work-around-include-file-rename.patch
create mode 100644 recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1+fslc.bb
delete mode 100644 recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1.bb
diff --git a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/updatemakefile.patch b/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/updatemakefile.patch
deleted file mode 100644
index 2ca10d4..0000000
--- a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/updatemakefile.patch
+++ /dev/null
@@ -1,158 +0,0 @@
-Upstream-Status: Pending
-
-Signed-off-by: Neena Busireddy <neena.busireddy@freescale.com>
-
-diff -Naur kernel-module-imx-gpu-viv-5.0.11.p7.1_org/kernel-module-imx-gpu-viv-src/Makefile kernel-module-imx-gpu-viv-5.0.11.p7.1/kernel-module-imx-gpu-viv-src/Makefile
---- kernel-module-imx-gpu-viv-5.0.11.p7.1_org/kernel-module-imx-gpu-viv-src/Makefile 2015-09-22 10:46:18.280385659 -0500
-+++ kernel-module-imx-gpu-viv-5.0.11.p7.1/kernel-module-imx-gpu-viv-src/Makefile 2015-09-22 10:46:51.000000000 -0500
-@@ -1,20 +1,54 @@
- ##############################################################################
- #
--# Copyright (C) 2005 - 2014 by Vivante Corp.
-+# The MIT License (MIT)
- #
--# This program is free software; you can redistribute it and/or modify
--# it under the terms of the GNU General Public License as published by
--# the Free Software Foundation; either version 2 of the license, or
--# (at your option) any later version.
-+# Copyright (c) 2014 Vivante Corporation
-+#
-+# Permission is hereby granted, free of charge, to any person obtaining a
-+# copy of this software and associated documentation files (the "Software"),
-+# to deal in the Software without restriction, including without limitation
-+# the rights to use, copy, modify, merge, publish, distribute, sublicense,
-+# and/or sell copies of the Software, and to permit persons to whom the
-+# Software is furnished to do so, subject to the following conditions:
-+#
-+# The above copyright notice and this permission notice shall be included in
-+# all copies or substantial portions of the Software.
-+#
-+# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
-+# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
-+# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
-+# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
-+# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING
-+# FROM, OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER
-+# DEALINGS IN THE SOFTWARE.
-+#
-+##############################################################################
-+#
-+# The GPL License (GPL)
-+#
-+# Copyright (C) 2014 Vivante Corporation
-+#
-+# This program is free software; you can redistribute it and/or
-+# modify it under the terms of the GNU General Public License
-+# as published by the Free Software Foundation; either version 2
-+# of the License, or (at your option) any later version.
- #
- # This program is distributed in the hope that it will be useful,
- # but WITHOUT ANY WARRANTY; without even the implied warranty of
--# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
-+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- # GNU General Public License for more details.
- #
- # You should have received a copy of the GNU General Public License
--# along with this program; if not write to the Free Software
--# Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
-+# along with this program; if not, write to the Free Software Foundation,
-+# Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
-+#
-+##############################################################################
-+#
-+# Note: This software is released under dual MIT and GPL licenses. A
-+# recipient may use this file under the terms of either the MIT license or
-+# GPL License. If you wish to use only one license not the other, you can
-+# indicate your decision by deleting one of the above license notices in your
-+# version of this file.
- #
- ##############################################################################
-
-@@ -23,16 +57,15 @@
- # Linux build file for kernel HAL driver.
- #
-
--AQROOT ?= .
--
- include $(AQROOT)/config
-
--KERNEL_DIR ?= $(KERNEL_PATH)
-+KERNEL_DIR ?= $(TOOL_DIR)/kernel
-
- OS_KERNEL_DIR := hal/os/linux/kernel
- ARCH_KERNEL_DIR := hal/kernel/arch
- ARCH_VG_KERNEL_DIR := hal/kernel/archvg
- HAL_KERNEL_DIR := hal/kernel
-+HOST := $(shell hostname)
-
- # Check and include platform config.
- ifneq ($(PLATFORM),)
-@@ -63,6 +96,14 @@
- $(OS_KERNEL_DIR)/gc_hal_kernel_debugfs.o \
- $(OS_KERNEL_DIR)/gc_hal_kernel_allocator.o \
-
-+ifneq ($(CONFIG_DMA_SHARED_BUFFER),)
-+OBJS += $(OS_KERNEL_DIR)/allocator/default/gc_hal_kernel_allocator_dmabuf.o
-+endif
-+
-+ifneq ($(CONFIG_IOMMU_SUPPORT),)
-+OBJS += $(OS_KERNEL_DIR)/gc_hal_kernel_iommu.o
-+endif
-+
- ifneq ($(PLATFORM),)
- OBJS += $(OS_KERNEL_DIR)/gc_hal_kernel_probe.o
- OBJS += $(OS_KERNEL_DIR)/platform/$(PLATFORM).o
-@@ -83,6 +124,10 @@
- OBJS += $(ARCH_KERNEL_DIR)/gc_hal_kernel_context.o \
- $(ARCH_KERNEL_DIR)/gc_hal_kernel_hardware.o
-
-+ifeq ($(VIVANTE_ENABLE_3D), 1)
-+OBJS += $(ARCH_KERNEL_DIR)/gc_hal_kernel_recorder.o
-+endif
-+
- ifeq ($(VIVANTE_ENABLE_VG), 1)
- OBJS +=\
- $(HAL_KERNEL_DIR)/gc_hal_kernel_vg.o\
-@@ -99,10 +144,6 @@
- OBJS += $(OS_KERNEL_DIR)/gc_hal_kernel_sync.o
- endif
-
--ifneq ($(CONFIG_ANDROID),)
--EXTRA_CFLAGS += -DANDROID
--endif
--
- ifeq ($(SECURITY), 1)
- OBJS += $(OS_KERNEL_DIR)/gc_hal_kernel_security_channel.o \
- $(HAL_KERNEL_DIR)/gc_hal_kernel_security.o
-@@ -118,15 +159,16 @@
-
- # Define targets.
- all:
-- @make V=$(V) ARCH=$(ARCH_TYPE) -C $(KERNEL_DIR) SUBDIRS=`pwd` modules
-+ @$(MAKE) V=$(V) ARCH=$(ARCH_TYPE) -C $(KERNEL_DIR) SUBDIRS=`pwd` modules
-
- clean:
- @rm -rf $(OBJS)
-- @rm -rf modules.order Module.symvers
-+ @rm -rf modules.order Module.symvers .tmp_versions
- @find $(AQROOT) -name ".gc_*.cmd" | xargs rm -f
-
- install: all
- @mkdir -p $(SDK_DIR)/drivers
-+ @cp $(MODULE_NAME).ko $(SDK_DIR)/drivers
-
- else
-
-@@ -261,8 +303,10 @@
- EXTRA_CFLAGS += -I$(AQROOT)/hal/kernel/archvg
- endif
-
--obj-m += galcore.o
-+EXTRA_CFLAGS += -DHOST=\"$(HOST)\"
-+
-+obj-m = $(MODULE_NAME).o
-
--galcore-objs := $(OBJS)
-+$(MODULE_NAME)-objs = $(OBJS)
-
- endif
diff --git a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/work-around-include-file-rename.patch b/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/work-around-include-file-rename.patch
deleted file mode 100644
index 5b92392..0000000
--- a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv/work-around-include-file-rename.patch
+++ /dev/null
@@ -1,30 +0,0 @@
-From 3ec514cf260b82f4701b6fed521ce470d9faf8c9 Mon Sep 17 00:00:00 2001
-From: Gary Thomas <gary@mlbassoc.com>
-Date: Tue, 6 Oct 2015 09:32:22 -0600
-Subject: [PATCH] kernel-module-imx-gpu-viv: Better work-around for change in name of busfreq-imx.h
-
-The renaming of include/linux/busfreq-imx6.h to include/linux/busfreq-imx.h
-is not consistent over the many kernel versions currently being built. This
-changeset works around this inconsistency by creating a local symbolic link
-to whatever file is actually present in the kernel sources.
-
-Signed-off-by: Gary Thomas <gary@mlbassoc.com>
-Upstream-status: Innapropriate [requires OE recipe support]
-----
-Index: kernel-module-imx-gpu-viv-5.0.11.p7.1/kernel-module-imx-gpu-viv-src/hal/os/linux/kernel/platform/freescale/gc_hal_kernel_platform_imx6q14.c
-===================================================================
---- kernel-module-imx-gpu-viv-5.0.11.p7.1.orig/kernel-module-imx-gpu-viv-src/hal/os/linux/kernel/platform/freescale/gc_hal_kernel_platform_imx6q14.c
-+++ kernel-module-imx-gpu-viv-5.0.11.p7.1/kernel-module-imx-gpu-viv-src/hal/os/linux/kernel/platform/freescale/gc_hal_kernel_platform_imx6q14.c
-@@ -74,11 +74,8 @@
- #include <linux/pm_runtime.h>
- #if LINUX_VERSION_CODE < KERNEL_VERSION(3, 10, 0)
- #include <mach/busfreq.h>
--#elif LINUX_VERSION_CODE < KERNEL_VERSION(3, 14, 0)
--#include <linux/busfreq-imx6.h>
--#include <linux/reset.h>
- #else
--#include <linux/busfreq-imx.h>
-+#include "busfreq-imx.h"
- #include <linux/reset.h>
- #endif
- #endif
diff --git a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1+fslc.bb b/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1+fslc.bb
new file mode 100644
index 0000000..eaaa383
--- /dev/null
+++ b/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1+fslc.bb
@@ -0,0 +1,17 @@
+# Copyright (C) 2015 O.S. Systems Software LTDA.
+# Copyright (C) 2015 Freescale Semiconductor
+
+SUMMARY = "Kernel loadable module for Vivante GPU"
+DESCRIPTION = "Provides flexibility to switch graphics between different kernels in future \
+releases. This package uses same source code as GPU kernel driver source."
+LICENSE = "GPLv2"
+LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e"
+
+PV .= "+git${SRCPV}"
+
+inherit module
+
+SRCREV = "eeeb23c0fb1cee01318088d417025263479c44ac"
+SRC_URI = "git://github.com/Freescale/kernel-module-imx-gpu-viv.git;protocol=https"
+
+S = "${WORKDIR}/git"
diff --git a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1.bb b/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1.bb
deleted file mode 100644
index 7e8414b..0000000
--- a/recipes-kernel/kernel-modules/kernel-module-imx-gpu-viv_5.0.11.p7.1.bb
+++ /dev/null
@@ -1,23 +0,0 @@
-# Copyright (C) 2015 Freescale Semiconductor
-
-SUMMARY = "Kernel loadable module for Vivante GPU"
-DESCRIPTION = "Provides flexibility to switch graphics between different kernels in future \
-releases. This package uses same source code as GPU kernel driver source."
-LICENSE = "GPLv2"
-LIC_FILES_CHKSUM = "file://COPYING;md5=12f884d2ae1ff87c09e5b7ccc2c4ca7e"
-
-inherit module
-
-SRC_URI = "${FSL_MIRROR}/${PN}-${PV}.tar.gz \
- file://updatemakefile.patch \
- file://work-around-include-file-rename.patch \
-"
-
-SRC_URI[md5sum] = "a251a94390986371f75b338ad938e46f"
-SRC_URI[sha256sum] = "9aaef0a62bc2be69dc568228192b060c54970b5c700fee602d83a4d13e04a9b3"
-
-# Work around inconsistent naming of <linux/busfreq-imx.h>
-do_compile_prepend () {
- ln -sf ${STAGING_KERNEL_DIR}/include/linux/busfreq-imx*.h kernel-module-imx-gpu-viv-src/hal/os/linux/kernel/platform/freescale/busfreq-imx.h
-}
-
--
2.6.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [meta-fsl-arm PATCH v2 3/3] linux-fslc-mx6: Update revision to 964e5a3 revision
2015-10-07 18:50 [meta-fsl-arm PATCH v2 1/3] imx-base.inc: Ensure MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT is expanded Otavio Salvador
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community Otavio Salvador
@ 2015-10-07 18:50 ` Otavio Salvador
1 sibling, 0 replies; 5+ messages in thread
From: Otavio Salvador @ 2015-10-07 18:50 UTC (permalink / raw)
To: meta-freescale Mailing List; +Cc: Otavio Salvador
The 964e5a3 revision includes the 3.14.54 stable release fixes plus
following changes contributed by Gateworks:
7292dc7 ASoC: add tda1997x codec support
aa4ee3d ipu3: keep buffer offset aligned
05608b5 ipu3: fix stripe calculation
30c6d90 thermal: imx: allow crit_delta to be passed in via kernel param
334fc35 arm: dts: add handles to anatop regulators
c7a4d85 fec: Russell Kings corruption fix
0c2c11f net: fec: fix the warning found by dma debug
b5b53b5 pci: imx6: Add delay after LTSSM transition
6dba6b0 fec_main: add mb() before setting status
05ab2a6 net: fec: fix NULL pointer dereference in fec_enet_timeout_work
07465db drivers: video: mxc: Add support for interlace on hdmi out
3eca3ca imx: thermal: use CPU temperature grade info for thresholds
172ad49 media: pci: tw6869: Implement ENUM_FRAMESIZES ioctl
a7d429e media: pci: tw6869: add device registered dev info
957f3fa media: pci: tw6869: initial add
cb61124 ARM: dts: imx: fix invalid #address-cells value
ea53bf7f imx:busfreq: disable busfreq driver by default
e3e114d video: mxc: ldb: use native mode for display timings
fa6b13a net: igb: register mii_bus for SerDes w/ external phy
09359b6 net: igb: add phy read/write functions that accept phy addr
d6daeac net: igb: add i210/i211 support for phy read/write
00b2ceb video: mxc: add adv739x TV encoder
3117a93 mxc_capture: add tda1997x HDMI receiver video support
9348099 imx: add imx tda1997x i2s DAI support
b81d263 mfd: add tda1997x HDMI receiver support
dba225f mxc_capture: check for init_camera_struct failure
09f7c1f mxc_capture: add device registration info
1db12fc mxc_hdmi: present sysfs edid as binary data instead of ascii
bd0601d mxc_hdmi: enable overflow interrupt after initialization complete
d6cc9e6 mxc_ipuv3: show device name for overlay fb
a7f44be mxc_ipuv3: show ipu_id/disp_id of registered device
e9ea090 regulator: Add LTC3676 support
3e12a168 regulator: core: Fix the init of DT defined fixed regulators
154b8be hwmon: mma8451: add fxos8700 magnetometer support
e19d671 hwmon: mma8451: add fxos8700 accel support
760c31c cpufreq: imx: add a 25mV tollerance to the operating point voltages
ef2eda1 cpufreq: imx: show soc voltage when debugging
b205762d cpufreq: imx: init mutex before it can be used
04bc9ed input: edt-ft5x06 - add device-tree config for screen res and inversion
1ca89b6 Input: edt-ft5x06 - fix an i2c write for M09 support
dc5f6c9 Input: edt-ft5x06 - add a missing condition
f2854dd Input: edt-ft5x06 - add support for M09 firmware version
390394c Input: edt-ft5x06 - ignore touchdown events
6b60fb3 Input: edt-ft5x06 - adjust delays to conform datasheet
74bceb0 Input: edt-ft5x06 - add DT support
481da81 input: goodix: add invert and rotate support via device-tree
6297f18 input: goodix: add support for performing init sqequence
4746ef3 input: goodix: add device-tree support
For compatibility with the Vivante kernel module build, the following
commit has be also applied:
964e5a3 FSLC: Add busfreq-imx.h header for forward compatibility
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
---
Changes in v2:
- Add busfreq-imx.h workaround fix
recipes-kernel/linux/linux-fslc-mx6_3.14-1.0.x.bb | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/recipes-kernel/linux/linux-fslc-mx6_3.14-1.0.x.bb b/recipes-kernel/linux/linux-fslc-mx6_3.14-1.0.x.bb
index 6cbe9d3..f46c0b3 100644
--- a/recipes-kernel/linux/linux-fslc-mx6_3.14-1.0.x.bb
+++ b/recipes-kernel/linux/linux-fslc-mx6_3.14-1.0.x.bb
@@ -11,6 +11,6 @@ include linux-fslc.inc
PV .= "+git${SRCPV}"
SRCBRANCH = "3.14-1.0.x-mx6"
-SRCREV = "4bae14aef7b8f340f30598d2b076e9ed7b7cba56"
+SRCREV = "964e5a3e65936e07f5d5189d233b8f8843687776"
COMPATIBLE_MACHINE = "(mx6)"
--
2.6.1
^ permalink raw reply related [flat|nested] 5+ messages in thread
* Re: [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community Otavio Salvador
@ 2015-10-07 19:08 ` Daiane Angolini
2015-10-07 19:44 ` Ann Thornton
0 siblings, 1 reply; 5+ messages in thread
From: Daiane Angolini @ 2015-10-07 19:08 UTC (permalink / raw)
To: Otavio Salvador; +Cc: meta-freescale Mailing List
On Wed, Oct 7, 2015 at 3:50 PM, Otavio Salvador <otavio@ossystems.com.br> wrote:
> The driver source code provided by Freescale need fixes to properly
s/need/needs
I would drop the "properly work" completely.
> work. This forks the code to a more community driven development,
I would drop "a more" and would keep only "This fork is a community
driven source code"
> allowing for a faster development of the driver and easier integration
and allow a faster development and easier integration of fixes.
> of fixes.
>
> To allow visual identification, when checking for bug reports, the
> '+fslc' suffix is added to the module.
Maybe:
"The "+fslc" suffix is added to the module during < puts here where I
can see +fslc > to make a clear identification."
I would not say "visual" as a script is not "visual".
>
> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
> ---
>
> Changes in v2:
> - New patch
>
(...)
> +
> +SUMMARY = "Kernel loadable module for Vivante GPU"
> +DESCRIPTION = "Provides flexibility to switch graphics between different kernels in future \
> +releases. This package uses same source code as GPU kernel driver source."
Replace by something like:
"This package uses an exact copy of GPU kernel driver source code of
same version"
Daiane
^ permalink raw reply [flat|nested] 5+ messages in thread
* Re: [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community
2015-10-07 19:08 ` Daiane Angolini
@ 2015-10-07 19:44 ` Ann Thornton
0 siblings, 0 replies; 5+ messages in thread
From: Ann Thornton @ 2015-10-07 19:44 UTC (permalink / raw)
To: meta-freescale
[-- Attachment #1: Type: text/plain, Size: 1741 bytes --]
Just a little more detail...
On 10/7/2015 2:08 PM, Daiane Angolini wrote:
> On Wed, Oct 7, 2015 at 3:50 PM, Otavio Salvador <otavio@ossystems.com.br> wrote:
>> The driver source code provided by Freescale need fixes to properly
> s/need/needs
>
> I would drop the "properly work" completely.
>
>> work. This forks the code to a more community driven development,
> I would drop "a more" and would keep only "This fork is a community
> driven source code"
s/source code/development/
>
>> allowing for a faster development of the driver and easier integration
> and allow a faster development and easier integration of fixes.
"and allow faster development and ..."
>> of fixes.
>>
>> To allow visual identification, when checking for bug reports, the
>> '+fslc' suffix is added to the module.
> Maybe:
> "The "+fslc" suffix is added to the module during < puts here where I
> can see +fslc > to make a clear identification."
>
> I would not say "visual" as a script is not "visual".
>
>
>> Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
>> ---
>>
>> Changes in v2:
>> - New patch
>>
> (...)
>
>> +
>> +SUMMARY = "Kernel loadable module for Vivante GPU"
>> +DESCRIPTION = "Provides flexibility to switch graphics between different kernels in future \
>> +releases. This package uses same source code as GPU kernel driver source."
> Replace by something like:
>
> "This package uses an exact copy of GPU kernel driver source code of
> same version"
"This package uses an exact copy of the GPU kernel driver source code of
the same version"
> Daiane
--
Ann Thornton
/Microcontrollers Software and Applications
Freescale Semiconductors
email: Ann.Thornton@freescale.com/
[-- Attachment #2: Type: text/html, Size: 3651 bytes --]
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-10-07 19:44 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-07 18:50 [meta-fsl-arm PATCH v2 1/3] imx-base.inc: Ensure MACHINE_HAS_VIVANTE_KERNEL_DRIVER_SUPPORT is expanded Otavio Salvador
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 2/3] kernel-module-imx-gpu-viv: Fork driver for FSL Community Otavio Salvador
2015-10-07 19:08 ` Daiane Angolini
2015-10-07 19:44 ` Ann Thornton
2015-10-07 18:50 ` [meta-fsl-arm PATCH v2 3/3] linux-fslc-mx6: Update revision to 964e5a3 revision Otavio Salvador
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.