| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
The recipe checksummed oe-core's generic MIT text, which never changes
and so verified nothing. Declare MIT with an SPDX identifier in all five
rules files and checksum the line in one of them; SRC_URI fetches all
five unconditionally regardless of which RULE subset a machine installs.
Validated on MACHINE=ls1046ardb, which installs the 73- and 74- rules.
do_install was forced so the candidate package was actually rebuilt.
Intended output change: both installed files grow by the SPDX line, so
PKGSIZE goes 1624 -> 1686. FILELIST is unchanged.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
SRC_URI ships only 10-imx.rules from this layer, with no license header
to checksum, so the recipe checksummed oe-core's generic MIT text, which
never changes and so verified nothing. Declare MIT with an SPDX
identifier in the rules file and checksum that line, the form oe-core
uses for recipes that ship their own files.
Validated on MACHINE=ls1046ardb. Intended output change: the rules file
grows by the SPDX line, so PKGSIZE goes 1612 -> 1643. FILELIST and
runtime dependencies are unchanged.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A bbappend to nativesdk-packagegroup-sdk-host added
nativesdk-imx-usb-loader, nativesdk-mxsldr, nativesdk-u-boot-mkimage and
nativesdk-wayland to RDEPENDS unconditionally, so an SDK built for any
machine in a build with meta-freescale in BBLAYERS picked up i.MX
flashing tools. Measured on qemuarm64 with bitbake -e: RDEPENDS gained
exactly those four.
The obvious fix -- scoping that RDEPENDS to a machine override -- would
be worse than the leak. That packagegroup has a fixed PACKAGE_ARCH of
${SDK_ARCH}-nativesdk, so making its contents depend on MACHINE would
emit different package content under one package architecture, and two
SDKs built from the same TMPDIR would overwrite each other in the feed.
oe-core's own recipe is careful here: it varies only by SDK_ARCH and
DISTRO_FEATURES, never by MACHINE.
Drop the bbappend and add the tools to TOOLCHAIN_HOST_TASK in
imx-base.inc instead. populate_sdk_base consumes that variable in the
SDK recipe, which is already machine-specific, so there is no
package-architecture hazard. imx-base.inc is required only by i.MX
machine configurations -- 27 of them, no QorIQ -- and the append is
additionally scoped to imx-generic-bsp to match the IMAGE_CLASSES line
above it.
Measured with bitbake -e, before vs after:
qemuarm64 TOOLCHAIN_HOST_TASK gains none of the four tools
imx6qdlsabresd TOOLCHAIN_HOST_TASK gains exactly the four tools;
nothing else in the expansion moves
Note for QorIQ: those machines previously received these tools through
the unconditional bbappend and no longer do. Three of the four are
i.MX-specific, so this is intended, but it is a behaviour change for
machines this layer does own.
bitbake -p: 1166 recipes, 0 errors. No SDK was built, so the effect on a
populated SDK is inferred from TOOLCHAIN_HOST_TASK, not measured.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
oelint.vars.noncoreoverride flags every statement in a bbappend that
carries no MACHINE or DISTRO override suffix, because an unconditional
override of core behaviour fails yocto-check-layer. The check is purely
syntactic: it cannot see that a statement's effective value is already
machine-gated, which is exactly how most of this layer is written.
DRM-REMOVE = "" <- flagged
DRM-REMOVE:imxgpu:mx6-nxp-bsp = "drm-gl drm-gles2"
PACKAGECONFIG:remove = "${DRM-REMOVE}" <- flagged
Neither line can be scoped. A fallback default cannot carry an override
for the machines it is the fallback for, and the dispatch line that
consumes it is a no-op wherever the helper expands empty. Scoping them
anyway would damage correct metadata: weston-init reads HAS_G2D as "elif
${HAS_G2D};" in shell, where an empty value is a syntax error.
Suppress the 58 sites proven inert, each with its own rationale. The
proof is a bitbake -e comparison on MACHINE=qemuarm64 -- a machine this
layer does not own -- with meta-freescale in and out of BBLAYERS. At
every suppressed site SRC_URI, DEPENDS, PACKAGECONFIG, OPENGL_PKGCONFIGS
and REQUIRED_DISTRO_FEATURES are unchanged; eight recipes show no
effective difference at all. The helper variables are defined in the
datastore but every consumer is override-scoped. For the 18
FILESEXTRAPATHS sites there is a stronger result: FILESPATH and
FILESEXTRAPATHS are both in BB_BASEHASH_IGNORE_VARS, so they cannot
reach a task signature.
Two anonymous python blocks are deliberately left unsuppressed and
explained in place instead. For those, oelint anchors the finding to the
line before the block, so a directive placed above only moves the anchor
onto itself: the finding survives and inlinesuppress_na is reported for
the directive. oelint.task.noanonpython anchors the same block to the
'python' line and suppresses correctly, so the two rules disagree about
where the construct starts. Upstream oelint bug candidate.
The rule count goes 76 -> 18 and the layer total 548 -> 491. The 16
findings that remain are not noise: they are measured, genuine
unconditional changes to machines this layer does not own, and are left
open for separate fixes.
This change is comment-only; no non-comment line is added or modified.
Validated with bitbake -p (1166 recipes, 0 errors) and an authoritative
driver rescan showing zero introduced findings in any of the 24 edited
files. The residual override/outofcontext churn in the rescan is the
known shared-metadata scan non-determinism and nets to zero.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
oelint.vars.dependsordered flags the three nativesdk-* entries as out
of alphabetical order. Sort them (imx-usb-loader, mxsldr,
u-boot-mkimage); the conditional wayland entry stays last. RDEPENDS is
a set, so this is build-neutral.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
Rename udev-rules-qoriq.bb to udev-rules-qoriq_1.0.bb so the recipe
filename carries an explicit version (file.underscores). PV was already
the default "1.0", so this is no functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
Rename udev-rules-imx.bb to udev-rules-imx_1.0.bb so the recipe
filename carries an explicit version (file.underscores). PV was already
the default "1.0", so this is no functional change.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and HOMEPAGE and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Add the missing mandatory SUMMARY and set SECTION.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Align the continuation lines of multi-line variable assignments, as
suggested by oelint-adv's auto-fix for a single rule
(oelint.vars.multilineident), isolated to one commit.
The auto-fix strips the leading space from a value's first line, which
for a ':append' is a required separator; those five recipes had their
leading space restored by hand, so ':append'/':prepend' concatenation
is unchanged. A few multi-line values (e.g. DESCRIPTION) gain internal
whitespace, which is cosmetic and collapses on display.
Build-tested: parsing all 2948 recipes for an i.MX machine reports
0 errors. A residual set (the restored ':append' recipes, whose
separator space oelint cannot reconcile with its indent preference,
plus opencv/dpdk where oelint's own fix does not satisfy its checker)
is left for per-recipe handling.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
Remove the stray space between an assignment's opening quote and its
line-continuation backslash, and the space before '%' in inline
${@...} expressions. Generated mechanically by oelint-adv's auto-fix
for a single rule (oelint.vars.notneededspace), isolated to one commit.
The affected values are whitespace-separated lists (leading space is
insignificant) and Python format expressions ("%s"% p is equivalent),
so there is no functional change; a re-run of oelint-adv reports the
rule clean.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
Normalize list continuations, append spacing, quote style, and simple
flag values so the metadata follows oelint BitBake syntax expectations.
Tested with: git diff --check.
Signed-off-by: Luciano Dittgen <luciano.dittgen@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
Fixes the warning:
WARNING: udev-rules-qoriq-1.0-r0 do_unpack: udev-rules-qoriq:
the directory pointed to by the S variable doesn't exist - please set S
within the recipe to point to where the source has been unpacked to
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
|
| |
|
|
|
|
| |
Add HOMEPAGE variable to udev-rules-imx recipe.
Signed-off-by: Weisser, Pascal <pascal.weisser.ext@karlstorz.com>
|
| |
|
|
| |
Signed-off-by: Zelan Zou <zelan.zou@nxp.com>
|
| |
|
|
|
|
|
|
|
|
|
| |
imx-pxp-g2d needs root access to
* /dev/pxp_device
* /dev/mali*
* /dev/dma_heap/linux,cma*
These are necessary for weston to start as non-root user.
Signed-off-by: Alexander Stein <alexander.stein@ew.tq-group.com>
|
| |
|
|
| |
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
WARNING: udev-rules-imx-1.0-r0 do_unpack: udev-rules-imx: the directory
${WORKDIR}/${BP} (.../udev-rules-imx/1.0/udev-rules-imx-1.0) pointed to
by the S variable doesn't exist - please set S within the recipe to
point to where the source has been unpacked to
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
Setting S here to something local is redundant moreover master oe-core
flags it now
Using S = ${WORKDIR} is no longer supported
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
| |
|
|
|
|
| |
This is being enforced in master oe-core
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
| |
|
|
| |
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
| |
|
|
|
| |
Fixes: #1498.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
| |
Signed-off-by: Khem Raj <raj.khem@gmail.com>
|
| |
|
|
| |
Signed-off-by: Chunrong Guo <chunrong.guo@nxp.com>
|
| |
|
|
| |
Signed-off-by: Chunrong Guo <chunrong.guo@nxp.com>
|
| |
|
|
|
|
|
| |
*remove rule for t1023
Signed-off-by: Chunrong Guo <chunrong.guo@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Chunrong Guo <chunrong.guo@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
| |
Those are untested and anyone interested in supporting them (mainly
barebox) ought to send the support to OE-Core instead.
This also removes the corresponding code from the
image_types_fsl.bbclass as it is not going to be used from now on.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
The OE-Core now warns when someone uses ${COREBASE}/LICENSE so we
should move to ${COMMON_LICENSE_DIR}/MIT to fix this deprecated use.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Commit d2313434 already exists in eudev to prevent udev from managing
an MMC Replay-protected memory block (RPMB)
Testing of an LS1021A-TWR & eMMC 4.51 with master branches of poky and
meta-freescale minus the patch to 60-persistent-storage.rules
confirmed that udev ignores mmcblk0rpmb on boot.
referenced upstream patch:
https://github.com/gentoo/eudev/commit/d2313434acc1cf5387637e72f76c2ebe36d523e1
Change-Id: Ib91aed741d18a87081e82e7c71e36363bb8b3939
Signed-off-by: Bob Cochran <yocto@mindchasers.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Ting Liu <ting.liu@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Ting Liu <ting.liu@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
Let all qoriq boards share the same interfaces file, no difference
between qoriq-ppc and qoriq-arm.
Signed-off-by: Ting Liu <ting.liu@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Fabio Berton <fabio.berton@ossystems.com.br>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
Patch fixes the resume funcion which does not update teh dmix state
Signed-off-by: Lauren Post <lauren.post@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
| |
* set RULE to empty for ls2080ardb.
* use loop to handle the case when RULE is empty.
* set ALLOW_EMPTY to 1.
Signed-off-by: Ting Liu <ting.liu@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
| |
Signed-off-by: Ting Liu <ting.liu@nxp.com>
|
| |
|
|
|
|
|
| |
eudev_3.1.5.bbappend -> eudev_%.bbappend
Signed-off-by: Zhenhua Luo <zhenhua.luo@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
|
|
|
| |
This includes into the SDK Boundary Devices i.MX/Vybrid recovery
utility, a host tool which allows to boot an image via USB in
serial downloader mode.
It currently supports i.MX5, i.MX6, i.MX7, and Vybrid families.
Signed-off-by: Hector Palacios <hector.palacios@digi.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
| |
Signed-off-by: Zhenhua Luo <zhenhua.luo@nxp.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Signed-off-by: Ting Liu <ting.liu@freescale.com>
Signed-off-by: Zhenhua Luo <zhenhua.luo@nxp.com>
Acked-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
udev-extraconf installs file mount.blacklist too. rpm/ndb bits should
go into udev-extraconf recipes, otherwise there will be file conflict.
Signed-off-by: Ting Liu <ting.liu@freescale.com>
Acked-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
| |
Signed-off-by: Zhenhua Luo <zhenhua.luo@freescale.com>
Acked-by: Daiane Angolini <daiane.angolini@freescale.com>
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
| |
The recipe shouldn't be used for non-QorIQ SoCs so this should have
the COMPATIBLE_MACHINE properly set.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
| |
|
|
|
|
|
|
| |
This commit is just a rename of all contents of meta-fsl-ppc
subdirectory to this layer's root, merging the contents of common
files, subsequent changes are based on top of that.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|
|
|
The meta-fsl-arm is going to be used as the base for this layer. It
contains a clean history and allowing a more granullar set of changes.
This commit is just a rename of all contents of meta-fsl-arm
subdirectory to this layer's root, subsequent changes are based on top
of that.
Signed-off-by: Otavio Salvador <otavio@ossystems.com.br>
|