From: Jerome Glisse <j.glisse@gmail.com>
To: "Christian König" <deathsimple@vodafone.de>
Cc: dri-devel@lists.freedesktop.org
Subject: Re: [PATCH 00/11] basic UVD support v3
Date: Mon, 8 Apr 2013 10:44:14 -0400 [thread overview]
Message-ID: <20130408144413.GA2239@gmail.com> (raw)
In-Reply-To: <1365417697-2316-1-git-send-email-deathsimple@vodafone.de>
On Mon, Apr 08, 2013 at 12:41:26PM +0200, Christian König wrote:
> Hello everyone,
>
> Third round of this patchset, and only some stupid typos fixed. I've uploaded this patchset as branch uvd-3.9 to git://people.freedesktop.org/~deathsimple/linux.
>
> Please review,
> Christian.
Looks good
Reviewed-by: Jerome Glisse <jglisse@redhat.com>
However i am wondering if it would not have been easier to incorporate
the msg as a new chunk rather as a bo so that you would not have to call
kmap/kunmap on them. But i dunno if GPU writeback stuff to this message
thing (i am guessing so).
Cheers,
Jerome
next prev parent reply other threads:[~2013-04-08 14:47 UTC|newest]
Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top
2013-04-08 10:41 [PATCH 00/11] basic UVD support v3 Christian König
2013-04-08 10:41 ` [PATCH 01/11] drm/radeon: UVD doesn't needs VM on SI v2 Christian König
2013-04-08 10:41 ` [PATCH 02/11] drm/radeon: rework fallback handling v2 Christian König
2013-04-08 10:41 ` [PATCH 03/11] drm/radeon: UVD bringup v8 Christian König
2013-04-08 10:41 ` [PATCH 04/11] drm/radeon: add pm callback for setting uvd clocks Christian König
2013-04-08 10:41 ` [PATCH 05/11] drm/radeon: add radeon_atom_get_clock_dividers helper Christian König
2013-04-08 10:41 ` [PATCH 06/11] drm/radeon: add set_uvd_clocks callback for ON/LN/TN (v4) Christian König
2013-04-08 10:41 ` [PATCH 07/11] drm/radeon: add set_uvd_clocks callback for evergreen Christian König
2013-04-08 10:41 ` [PATCH 08/11] drm/radeon: add set_uvd_clocks callback for SI Christian König
2013-04-08 10:41 ` [PATCH 09/11] drm/radeon: add set_uvd_clocks callback for r7xx v3 Christian König
2013-04-08 10:41 ` [PATCH 10/11] drm/radeon: init UVD clocks to sane defaults Christian König
2013-04-08 10:41 ` [PATCH 11/11] drm/radeon: add UVD tiling addr config v2 Christian König
2013-04-08 14:44 ` Jerome Glisse [this message]
[not found] ` <2c8aa0c195d39b1a150431e895a34619@mail.ud03.udmedia.de>
[not found] ` <5162D947.7040509@vodafone.de>
2013-04-11 1:11 ` [PATCH 00/11] basic UVD support v3 Dieter Nützel
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20130408144413.GA2239@gmail.com \
--to=j.glisse@gmail.com \
--cc=deathsimple@vodafone.de \
--cc=dri-devel@lists.freedesktop.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox