New issue
Advanced search Search tips
Note: Color blocks (like █ or █) mean that a user may not be available. Tooltip shows the reason.

Issue 779159 link

Starred by 1 user
Status: Fixed
Owner:
█ pbe...@chromium.org
Last visit > 30 days ago
Closed: May 2018
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 2
Type: Feature



Sign in to add a comment

remove model.dtsi files from device image

Project Member Reported by pbe...@chromium.org, Oct 27 2017 
currently the model.dtsi files in unibuild end up being copied to the final device, but all runtime depends on the compiled .dtb instead, so they can be removed.
Project Member

Comment 1 by bugdroid1@chromium.org, Oct 30 2017 

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/0202868575c1e5fbd92a942847e815c6711cf43d

commit 0202868575c1e5fbd92a942847e815c6711cf43d
Author: Patrick Berny <pberny@chromium.org>
Date: Mon Oct 30 04:55:24 2017

chromeos-config: change ...-bsp to DEPEND

Change the RDEPEND of chromeos-config-bsp to DEPEND, since
on the image only the config.dtb is required, not the .dtsi.

BUG= chromium:779159 
TEST=./build_image, check that .dtsi are gone from final image

Change-Id: I4051be1452829d8ad95bc58200fda0ca3a73785c
Reviewed-on: https://chromium-review.googlesource.com/742184
Commit-Ready: YH Lin <yueherngl@chromium.org>
Tested-by: Patrick Berny <pberny@chromium.org>
Reviewed-by: Simon Glass <sjg@chromium.org>
Reviewed-by: Jason Clinton <jclinton@chromium.org>

[rename] https://crrev.com/0202868575c1e5fbd92a942847e815c6711cf43d/chromeos-base/chromeos-config/chromeos-config-0.0.1-r8.ebuild
[modify] https://crrev.com/0202868575c1e5fbd92a942847e815c6711cf43d/chromeos-base/chromeos-config/chromeos-config-0.0.1.ebuild

Comment 2 by zalcorn@chromium.org, Dec 26 2017

Components: Infra

Comment 3 by mmoss@chromium.org, Jan 2 2018

Components: -Infra Infra>Client>ChromeOS
[It appears that a bunch of old cros issues bulk-added the "Infra" component recently, but they should probably be "Infra>Client>ChromeOS".]

Comment 4 by ayatane@chromium.org, May 17 2018

Status: Fixed (was: Untriaged)
Looks like this was fixed
►

Sign in to add a comment