Hi Linus,
i915 has a backlight fix for some panels, a pm and a fencing fix,
along with some GVT fixes.
amdgpu has a backlight fix across suspend/resume, an object
destruction ordering issue fix, and a displayport fix
nouveau has two backlight fixes, and a fix for some lockups.
Pretty quiet week, seems like everyone was fixing backlights.
Dave.
The following changes since commit 0c8efd610b58cb23cefdfa12015799079aef94ae:
Linux 4.16-rc5 (2018-03-11 17:25:09 -0700)
are available in the Git …
[View More]repository at:
git://people.freedesktop.org/~airlied/linux tags/drm-fixes-for-v4.16-rc6
for you to fetch changes up to 3a1b5de36fdf403d1b004e537dc13997633d65df:
Merge tag 'drm-intel-fixes-2018-03-15' of
git://anongit.freedesktop.org/drm/drm-intel into drm-fixes (2018-03-16
12:51:35 +1000)
----------------------------------------------------------------
i915, amd and nouveau fixes
----------------------------------------------------------------
Alex Deucher (1):
drm/amdgpu: save/restore backlight level in legacy dce code
Chris Wilson (2):
drm/i915: Only prune fences after wait-for-all
drm/i915: Kick the rps worker when changing the boost frequency
Christian König (2):
drm/amdgpu: fix prime teardown order
drm/radeon: fix prime teardown order
Dave Airlie (4):
Merge branch 'drm-fixes-4.16' of
git://people.freedesktop.org/~agd5f/linux into drm-fixes
Merge tag 'drm-intel-fixes-2018-03-14' of
git://anongit.freedesktop.org/drm/drm-intel into drm-fixes
Merge branch 'linux-4.16' of git://github.com/skeggsb/linux into drm-fixes
Merge tag 'drm-intel-fixes-2018-03-15' of
git://anongit.freedesktop.org/drm/drm-intel into drm-fixes
Karol Herbst (1):
drm/nouveau/bl: fix backlight regression
Lukas Wunner (1):
drm/nouveau/bl: Fix oops on driver unbind
Michel Dänzer (1):
drm/amdgpu/dce: Don't turn off DP sink when disconnected
Min He (1):
drm/i915/gvt: keep oa config in shadow ctx
Mustamin B Mustaffa (1):
drm/i915: Enable VBT based BL control for DP
Māris Nartišs (1):
drm/nouveau/mmu: ALIGN_DOWN correct variable
Rodrigo Vivi (1):
Merge tag 'gvt-fixes-2018-03-15' of
https://github.com/intel/gvt-linux into drm-intel-fixes
Xiong Zhang (1):
drm/i915/gvt: Add runtime_pm_get/put into gvt_switch_mmio
Zhenyu Wang (1):
drm/i915/gvt: fix user copy warning by whitelist workload rb_tail field
fred gao (1):
drm/i915/gvt: Correct the privilege shadow batch buffer address
drivers/gpu/drm/amd/amdgpu/amdgpu_connectors.c | 31 +++++------
drivers/gpu/drm/amd/amdgpu/amdgpu_gem.c | 2 -
drivers/gpu/drm/amd/amdgpu/amdgpu_mode.h | 1 +
drivers/gpu/drm/amd/amdgpu/amdgpu_object.c | 2 +
drivers/gpu/drm/amd/amdgpu/atombios_encoders.c | 4 +-
drivers/gpu/drm/amd/amdgpu/atombios_encoders.h | 5 ++
drivers/gpu/drm/amd/amdgpu/dce_v10_0.c | 8 +++
drivers/gpu/drm/amd/amdgpu/dce_v11_0.c | 8 +++
drivers/gpu/drm/amd/amdgpu/dce_v6_0.c | 8 +++
drivers/gpu/drm/amd/amdgpu/dce_v8_0.c | 8 +++
drivers/gpu/drm/i915/gvt/cmd_parser.c | 8 +++
drivers/gpu/drm/i915/gvt/mmio_context.c | 2 +
drivers/gpu/drm/i915/gvt/scheduler.c | 71 ++++++++++++++++++++++++--
drivers/gpu/drm/i915/gvt/scheduler.h | 5 ++
drivers/gpu/drm/i915/i915_gem.c | 16 ++++--
drivers/gpu/drm/i915/i915_sysfs.c | 10 +++-
drivers/gpu/drm/i915/intel_dp.c | 10 ++--
drivers/gpu/drm/nouveau/nouveau_backlight.c | 14 ++---
drivers/gpu/drm/nouveau/nvkm/subdev/mmu/vmm.c | 2 +-
drivers/gpu/drm/radeon/radeon_gem.c | 2 -
drivers/gpu/drm/radeon/radeon_object.c | 2 +
21 files changed, 169 insertions(+), 50 deletions(-)
[View Less]
https://bugs.freedesktop.org/show_bug.cgi?id=105401
Bug ID: 105401
Summary: Unable to start X on Vega AMDGPU(0):
amdgpu_setup_kernel_mem failed
Product: DRI
Version: XOrg git
Hardware: Other
OS: All
Status: NEW
Severity: normal
Priority: medium
Component: DRM/AMDgpu
Assignee: dri-devel(a)lists.freedesktop.org
Reporter: tpearson(a)…
[View More]raptorengineering.com
A similar issue to the one described in Bug #105172 is affecting Vega GPUs on
POWER9 machines. Log files attached.
System configuration:
Software versions:
Linux 4.16.0-rc4 ppc64el
GPU hardware:
Radeon RX Vega 56
CPU hardware:
POWER9 PowerNV
--
You are receiving this mail because:
You are the assignee for the bug.
[View Less]
Hi Dave,
Sorry for the last minute and for sending 2 pull requests
in a short time, but we just got a pull request from GVT.
It passes our CI-fast-feedback and the full run is still
running.
Please if we still have time please consider pulling it,
otherwise this will be part of next regular one.
Here goes drm-intel-fixes-2018-03-15:
Only GVT fixes:
- Two warnings fix for runtime pm and usr copy (Xiong, Zhenyu)
- OA context fix for vGPU profiling (Min)
- privilege batch buffer reloc fix (…
[View More]Fred)
Thanks,
Rodrigo.
The following changes since commit 0c8efd610b58cb23cefdfa12015799079aef94ae:
Linux 4.16-rc5 (2018-03-11 17:25:09 -0700)
are available in the git repository at:
git://anongit.freedesktop.org/drm/drm-intel tags/drm-intel-fixes-2018-03-15
for you to fetch changes up to 05b429a8eed83084a8a7fc04c97120a5ba07b5be:
Merge tag 'gvt-fixes-2018-03-15' of https://github.com/intel/gvt-linux into drm-intel-fixes (2018-03-15 15:37:57 -0700)
----------------------------------------------------------------
Only GVT fixes:
- Two warnings fix for runtime pm and usr copy (Xiong, Zhenyu)
- OA context fix for vGPU profiling (Min)
- privilege batch buffer reloc fix (Fred)
----------------------------------------------------------------
Chris Wilson (2):
drm/i915: Only prune fences after wait-for-all
drm/i915: Kick the rps worker when changing the boost frequency
Min He (1):
drm/i915/gvt: keep oa config in shadow ctx
Mustamin B Mustaffa (1):
drm/i915: Enable VBT based BL control for DP
Rodrigo Vivi (1):
Merge tag 'gvt-fixes-2018-03-15' of https://github.com/intel/gvt-linux into drm-intel-fixes
Xiong Zhang (1):
drm/i915/gvt: Add runtime_pm_get/put into gvt_switch_mmio
Zhenyu Wang (1):
drm/i915/gvt: fix user copy warning by whitelist workload rb_tail field
fred gao (1):
drm/i915/gvt: Correct the privilege shadow batch buffer address
drivers/gpu/drm/i915/gvt/cmd_parser.c | 8 ++++
drivers/gpu/drm/i915/gvt/mmio_context.c | 2 +
drivers/gpu/drm/i915/gvt/scheduler.c | 71 +++++++++++++++++++++++++++++++--
drivers/gpu/drm/i915/gvt/scheduler.h | 5 +++
drivers/gpu/drm/i915/i915_gem.c | 16 ++++++--
drivers/gpu/drm/i915/i915_sysfs.c | 10 ++++-
drivers/gpu/drm/i915/intel_dp.c | 10 ++---
7 files changed, 105 insertions(+), 17 deletions(-)
[View Less]
Hi all,
the main goal of this patchset is to simplify clk management code in
exynos5433 drivers by using clk bulk API. In order to achieve that,
patch #1 adds a new function to clk core, which dynamically allocates
clk_bulk_data array and fills its id fields.
Best regards,
Maciej Purski
Maciej Purski (8):
clk: Add clk_bulk_alloc functions
media: s5p-jpeg: Use bulk clk API
drm/exynos/decon: Use clk bulk API
drm/exynos/dsi: Use clk bulk API
drm/exynos: mic: Use clk bulk API
drm/…
[View More]exynos/hdmi: Use clk bulk API
[media] exynos-gsc: Use clk bulk API
[media] s5p-mfc: Use clk bulk API
drivers/clk/clk-bulk.c | 16 +++++
drivers/clk/clk-devres.c | 37 +++++++++--
drivers/gpu/drm/exynos/exynos5433_drm_decon.c | 50 +++++----------
drivers/gpu/drm/exynos/exynos_drm_dsi.c | 68 +++++++++-----------
drivers/gpu/drm/exynos/exynos_drm_mic.c | 44 +++++--------
drivers/gpu/drm/exynos/exynos_hdmi.c | 85 ++++++++-----------------
drivers/media/platform/exynos-gsc/gsc-core.c | 55 ++++++----------
drivers/media/platform/exynos-gsc/gsc-core.h | 2 +-
drivers/media/platform/s5p-jpeg/jpeg-core.c | 45 ++++++-------
drivers/media/platform/s5p-jpeg/jpeg-core.h | 2 +-
drivers/media/platform/s5p-mfc/s5p_mfc_common.h | 6 +-
drivers/media/platform/s5p-mfc/s5p_mfc_pm.c | 41 +++++-------
include/linux/clk.h | 64 +++++++++++++++++++
13 files changed, 263 insertions(+), 252 deletions(-)
--
2.7.4
[View Less]
There was a patch submitted by Keith Packard which prevents fbcon from
using non-desktop displays, but this breaks vive, and other HMD
development/use on embedded and other fbdev systems (
https://patchwork.kernel.org/patch/10053989/ ).
Even if the vive is the only device connected, it will still not permit it
to be operated. See https://github.com/linux-sunxi/linux-sunxi/issues/291
for dri with a lot of debugging turned on.
I can understand that most users would probably prefer that the …
[View More]vive isn't
automatically used if no other displays are available, however, the current
behavior prevents use of the vive on all devices that use fbdev for their
primary output.
This patch allows enabling of non-desktop devices both as a kernel command
line as well as by setting /sys/module/drm_kms_helper/par
ameters/drm_fbdev_permit_non_desktop.
I've never sent an email to the kernel devel list, so I'm still a little
nervous. Especially because this is against a different branch, and I'm
starting to think that I should be messaging there, but this is something
that really needs to go upstream.
Signed-off-by:
diff --git a/drivers/gpu/drm/drm_fb_helper.c
b/drivers/gpu/drm/drm_fb_helper.c
index 035784ddd..8bfaf79ff 100644
--- a/drivers/gpu/drm/drm_fb_helper.c
+++ b/drivers/gpu/drm/drm_fb_helper.c
@@ -55,6 +55,11 @@ MODULE_PARM_DESC(drm_fbdev_overalloc,
"Overallocation of the fbdev buffer (%) [default="
__MODULE_STRING(CONFIG_DRM_FBDEV_OVERALLOC) "]");
+static bool drm_fbdev_permit_non_desktop;
+module_param(drm_fbdev_permit_non_desktop, bool, 0644);
+MODULE_PARM_DESC(drm_fbdev_permit_non_desktop,
+ "Allow the framebuffer to use non-desktop displays
[default=off]");
+
static LIST_HEAD(kernel_fb_helper_list);
static DEFINE_MUTEX(kernel_fb_helper_lock);
@@ -2109,7 +2114,7 @@ static bool drm_connector_enabled(struct
drm_connector *connector, bool strict)
{
bool enable;
- if (connector->display_info.non_desktop)
+ if (connector->display_info.non_desktop &&
!drm_fbdev_permit_non_desktop)
return false;
if (strict)
[View Less]
Until now, the drm-intel commit access have been handed out ad hoc,
without transparency, consistency, or fairness. With pressure to add
more committers, this is no longer tenable, if it ever was. Document the
requirements and expectations around becoming a drm-intel committer.
The Linux kernel operates in a model where, by and large, only
maintainers commit patches. Maintainer teams are no longer rare, but the
drm-intel and drm-misc maintainer/committer model is definitely an
outlier.
The …
[View More]drm-intel maintainers believe that a reasonable level of experience
and track record of working on the driver, as well as actively engaging
in the community upstream, are necessary before becoming a
committer. While the requirements outlined here may seem strict in
contrast with many projects, they are extremely liberal by kernel
standards.
Finally, no rules are carved in stone. We fully expect the requirements
to be adjusted later. However, it will be much easier to start strict
and relax the requirements later than the other way round.
Cc: Gustavo Padovan <gustavo(a)padovan.org>
Cc: Maarten Lankhorst <maarten.lankhorst(a)linux.intel.com>
Cc: Sean Paul <seanpaul(a)chromium.org>
Cc: Dave Airlie <airlied(a)gmail.com>
Cc: dim-tools(a)lists.freedesktop.org
Cc: dri-devel(a)lists.freedesktop.org
Cc: intel-gfx(a)lists.freedesktop.org
Signed-off-by: Jani Nikula <jani.nikula(a)intel.com>
Signed-off-by: Joonas Lahtinen <joonas.lahtinen(a)linux.intel.com>
Signed-off-by: Rodrigo Vivi <rodrigo.vivi(a)intel.com>
---
We encourage the drm-misc maintainers to consider and document your
requirements for committers. While there's certain appeal to aligning on
the rules between drm-misc and drm-intel, we don't think they
necessarily have to be the same.
---
commit-access.rst | 143 ++++++++++++++++++++++++++++++++++++++++++++++++++++++
index.rst | 1 +
2 files changed, 144 insertions(+)
create mode 100644 commit-access.rst
diff --git a/commit-access.rst b/commit-access.rst
new file mode 100644
index 000000000000..dbc50f2b5bd3
--- /dev/null
+++ b/commit-access.rst
@@ -0,0 +1,143 @@
+===============
+ Commit Access
+===============
+
+The drm-misc and drm-intel repositories operate in a maintainer/committer model
+with a large pool committers who can push patches in accordance with the stated
+merge criteria, and maintainers handling pull requests, topic branches, merges,
+and so on.
+
+This document outlines the requirements for becoming a committer.
+
+drm-misc
+--------
+
+TBD.
+
+drm-intel
+---------
+
+Requirements
+~~~~~~~~~~~~
+
+The baseline requirements for becoming a drm-intel committer are:
+
+- Comfortable with the open collaboration model we have.
+
+- Enough experience to gauge reasonably well how much review a patch needs, and
+ when pushing a patch, whether it needs acks from domain experts or
+ maintainers.
+
+- Strong presence in the project communication channels (intel-gfx mailing list
+ and #intel-gfx IRC channel) for topics in their area of expertise.
+
+Since everyone is different and has different focus in their work, there are no
+hard and fast rules, but just indicators and some judgement.
+
+Positive indicators are:
+
+- Decent amount of non-trivial patches merged in the past year (25+ patches,
+ excluding simple code movement, typo fixes and similar patches).
+
+- Decent amount of in-depth review, again 25+ in the past year as the threshold.
+
+- Lots of experience and commit rights in related open-source projects, like
+ Mesa, or kernel trees like drm-misc, since the processes are very similar. 2+
+ years as the threshold here, and this is also fulfilled after 2+ years of
+ working in the virtual upstream team (product tree hacking doesn't count).
+
+- Already merged a complex feature, e.g. spanning multiple subsystems or even
+ involving userspace.
+
+- Active member on freedesktop.org bugzilla. A developer that besides submitting
+ patches to fix bugs is also engaged on bugs discussion giving constructive
+ comments helping to drive the bug entry to a solution. Hence keeping bug list
+ active, clean, and under control.
+
+Contra-indicators are:
+
+- Not around on IRC (taking timezones into account of course).
+
+- Mostly simple patches and rubber-stamping reviews not resulting in in-depth
+ discussions.
+
+- No complete feature patch set merged yet.
+
+As a rule of thumb, commit rights will be granted when most positive indicators
+are fulfilled and no negative indicators present. The current project
+maintainers assess whether a candidate meets the requirements, and make the
+decisions about commit rights.
+
+Nomination
+~~~~~~~~~~
+
+Any developer, who already have demonstrated some of positive requirements, can
+be nominated at any time by anyone: maintainers, committers, managers, peers,
+and even self nomination are accepted.
+
+Nomination process is simply sending an email to the drm-intel
+maintainers. Please nominate one person per email, and Cc: the
+nominee. Nominations are processed by the maintainers on a first come, first
+served basis, as nominations are received.
+
+Nomination is not a guarantee of the commit rights. In case a nomination is
+rejected, the maintainers will give constructive feedback on what areas the
+nominee should try to work on.
+
+When there are several nominees at the same time, the ramp-up time on the tools
+and processes, and the support bandwidth limit the amount of rights that can be
+distributed at once. Maintainers are responsible for collecting all the
+candidates in a list that will be sorted out following some criteria:
+
+- More recent activity. It is easier to absorb and ramp-up a committer that is
+ currently active on i915 with pending patches to get merged.
+
+- More positive indicators.
+
+- Fewer contra-indicators.
+
+After the nomination has been processed and a decision has been made about
+giving or postponing giving out the commit rights to the nominee, there will be
+a reply from the maintainers to the the nominated person only.
+
+In case the nominee is accepted as a committer and accepts the new role, the
+nominee will need to request a freedesktop.org account according to
+https://www.freedesktop.org/wiki/AccountRequests/, and the drm-intel maintainers
+will ack adding the account with drm-intel access on the account request bug. If
+the nominee already has a freedesktop.org account, the drm-intel maintainers
+will file the access request bug.
+
+Revocation
+~~~~~~~~~~
+
+- Self abdicate. Anyone at any time can explicitly request to be removed from
+ the list of drm-intel committers.
+
+- Long period of inactivity on DRM projects and community. It is very common to
+ have people changing jobs and not contributing anymore to DRM related
+ projects. So, in order to keep the list clean and under control, after one
+ year of inactivity, commit rights can be withdrawn with or without
+ notification.
+
+- Repeatedly breaking the rules on pushing patches. Everyone makes mistakes,
+ especially when ramping up and getting used to the maintainer tools. However,
+ commit rights could be revoked when a committer has been repeatedly observed,
+ and notified, of:
+
+ - Pushing patches without review (or acks from maintainers or domain experts
+ when required).
+
+ - Pushing patches that had a clear rejection from maintainers or other
+ committers or unresolved review comments.
+
+ - Pushing patches without proper tags or links.
+
+ - Pushing patches to other branches than drm-intel-next-queued.
+
+ - Pushing patches without using dim tools.
+
+ - Pushing patches that didn’t pass CI.
+
+- Freedesktop.org Code of Conduct violations may lead to temporary or permanent
+ account or commit rights suspension according to freedesktop.org umbrella
+ rules.
diff --git a/index.rst b/index.rst
index d2142a7898f8..2db033161196 100644
--- a/index.rst
+++ b/index.rst
@@ -23,6 +23,7 @@ Contents:
:maxdepth: 2
repositories
+ commit-access
drm-misc
drm-intel
dim
--
2.11.0
[View Less]
Hi,
Thanks for reviews of previous iterations.
This patchset introduces USB physical connector bindings, together with
working example.
I have removed RFC prefix - the patchset seems to be heading
to a happy end :)
v5: fixed extra parenthesis in dts, renamed extcon function.
v4: improved binding descriptions, added missing reg in dts.
v3: Separate binding for Samsung 11-pin connector, added full-blown USB-C
example.
v2: I have addressed comments by Rob and Laurent, thanks
Changes in …
[View More]datail are described in the patches.
Regards
Andrzej
Andrzej Hajda (5):
dt-bindings: add bindings for USB physical connector
dt-bindings: add bindings for Samsung micro-USB 11-pin connector
arm64: dts: exynos: add micro-USB connector node to TM2 platforms
arm64: dts: exynos: add OF graph between MHL and USB connector
extcon: add possibility to get extcon device by OF node
Maciej Purski (1):
drm/bridge/sii8620: use micro-USB cable detection logic to detect MHL
.../connector/samsung,usb-connector-11pin.txt | 49 +++++++++++
.../bindings/connector/usb-connector.txt | 75 +++++++++++++++++
.../boot/dts/exynos/exynos5433-tm2-common.dtsi | 38 ++++++++-
drivers/extcon/extcon.c | 44 +++++++---
drivers/gpu/drm/bridge/sil-sii8620.c | 97 +++++++++++++++++++++-
include/linux/extcon.h | 6 ++
6 files changed, 293 insertions(+), 16 deletions(-)
create mode 100644 Documentation/devicetree/bindings/connector/samsung,usb-connector-11pin.txt
create mode 100644 Documentation/devicetree/bindings/connector/usb-connector.txt
--
2.16.2
[View Less]
gcc-8 reports that we access an array with a negative index
in an error case:
drivers/gpu/ipu-v3/ipu-prg.c: In function 'ipu_prg_channel_disable':
drivers/gpu/ipu-v3/ipu-prg.c:252:43: error: array subscript -22 is below array bounds of 'struct ipu_prg_channel[3]' [-Werror=array-bounds]
This moves the range check in front of the first time that
variable gets used.
Signed-off-by: Arnd Bergmann <arnd(a)arndb.de>
----
v1: Originally sent on Dec 4, 2017.
v2: Sent again on Jan 16, after no …
[View More]reply, Phillipp said it was merged
v3: Ran into a related problem with CONFIG_UBSAN, sent again fixing both
---
drivers/gpu/ipu-v3/ipu-prg.c | 12 +++++++++---
1 file changed, 9 insertions(+), 3 deletions(-)
diff --git a/drivers/gpu/ipu-v3/ipu-prg.c b/drivers/gpu/ipu-v3/ipu-prg.c
index 97b99500153d..83f9dd934a5d 100644
--- a/drivers/gpu/ipu-v3/ipu-prg.c
+++ b/drivers/gpu/ipu-v3/ipu-prg.c
@@ -250,10 +250,14 @@ void ipu_prg_channel_disable(struct ipuv3_channel *ipu_chan)
{
int prg_chan = ipu_prg_ipu_to_prg_chan(ipu_chan->num);
struct ipu_prg *prg = ipu_chan->ipu->prg_priv;
- struct ipu_prg_channel *chan = &prg->chan[prg_chan];
+ struct ipu_prg_channel *chan;
u32 val;
- if (!chan->enabled || prg_chan < 0)
+ if (prg_chan < 0)
+ return;
+
+ chan = &prg->chan[prg_chan];
+ if (!chan->enabled)
return;
pm_runtime_get_sync(prg->dev);
@@ -280,13 +284,15 @@ int ipu_prg_channel_configure(struct ipuv3_channel *ipu_chan,
{
int prg_chan = ipu_prg_ipu_to_prg_chan(ipu_chan->num);
struct ipu_prg *prg = ipu_chan->ipu->prg_priv;
- struct ipu_prg_channel *chan = &prg->chan[prg_chan];
+ struct ipu_prg_channel *chan;
u32 val;
int ret;
if (prg_chan < 0)
return prg_chan;
+ chan = &prg->chan[prg_chan];
+
if (chan->enabled) {
ipu_pre_update(prg->pres[chan->used_pre], *eba);
return 0;
--
2.9.0
[View Less]