From: Harry Wentland <harry.wentland-5C7GfCeVMHo@public.gmane.org>
To: Tom St Denis <tom.stdenis-5C7GfCeVMHo@public.gmane.org>,
amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org
Subject: Re: Various cleanups for amdgpu_dm
Date: Tue, 17 Oct 2017 19:23:49 -0400 [thread overview]
Message-ID: <d410d7b6-6d6b-a58f-0e9c-4eaf91497a97@amd.com> (raw)
In-Reply-To: <6dadea86-5a2c-e193-d02f-27515408b24c-5C7GfCeVMHo@public.gmane.org>
On 2017-10-17 05:43 PM, Tom St Denis wrote:
> On 17/10/17 04:30 PM, Harry Wentland wrote:
>> Thanks for these cleanups.
>>
>> Some of these could possibly squashed but then, I generally
>> prefer smaller changes than one big lump.
>>
>> Reviewed-by: Harry Wentland <harry.wentland@amd.com>
>
> Hi Harry,
>
> You're welcome. Normally I too agree that squashing is better but since that part of the tree is less familiar I decided to go for a easily bisectable patchset in case I did fubar something. Mostly I just compile test as I go to make sure I didn't typo anything.
>
Yeah, definitely better to keep it small, especially on unfamiliar territory.
> I've pushed these to drm-next. Is there any other place you'd like me to merge them?
>
amd-staging-drm-next is the right place. I'll backmerge them to our internal tree from there.
Harry
> Cheers,
> Tom
_______________________________________________
amd-gfx mailing list
amd-gfx@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/amd-gfx
prev parent reply other threads:[~2017-10-17 23:23 UTC|newest]
Thread overview: 18+ messages / expand[flat|nested] mbox.gz Atom feed top
2017-10-17 16:18 Various cleanups for amdgpu_dm Tom St Denis
[not found] ` <20171017161912.11357-1-tom.stdenis-5C7GfCeVMHo@public.gmane.org>
2017-10-17 16:19 ` [PATCH 01/13] drm/amd/display: Simplify dm_late_init() Tom St Denis
2017-10-17 16:19 ` [PATCH 02/13] drm/amd/display: Fix indentation in dm_suspend() Tom St Denis
2017-10-17 16:19 ` [PATCH 03/13] drm/amd/display: Fix indentation in dm_resume() Tom St Denis
2017-10-17 16:19 ` [PATCH 04/13] drm/amd/display: Fix brace style in amdgpu_dm_update_connector_after_detect() Tom St Denis
2017-10-17 16:19 ` [PATCH 05/13] drm/amd/display: Fix brace style in dm_handle_hpd_rx_irq() Tom St Denis
2017-10-17 16:19 ` [PATCH 06/13] drm/amd/display: Simplify handle_hpd_rx_irq() Tom St Denis
2017-10-17 16:19 ` [PATCH 07/13] drm/amd/display: Fix brace style in amdgpu_dm_initialize_drm_device() Tom St Denis
2017-10-17 16:19 ` [PATCH 08/13] drm/amd/display: Replace block with strncpy() in fill_audio_info() Tom St Denis
2017-10-17 16:19 ` [PATCH 09/13] drm/amd/display: Fix indentation in create_eml_sink() Tom St Denis
2017-10-17 16:19 ` [PATCH 10/13] drm/amd/display: Tidy up dm_drm_plane_reset() Tom St Denis
2017-10-17 16:19 ` [PATCH 11/13] drm/amd/display: Fix brace style in amdgpu_dm_connector_ddc_get_modes() Tom St Denis
2017-10-17 16:19 ` [PATCH 12/13] drm/amd/display: Remove needless cast in amdgpu_dm_connector_init() Tom St Denis
2017-10-17 16:19 ` [PATCH 13/13] drm/amd/display: Fix brace style Tom St Denis
2017-10-17 17:16 ` Various cleanups for amdgpu_dm Andrey Grodzovsky
2017-10-17 20:30 ` Harry Wentland
[not found] ` <5eaea6a7-e3d9-e19d-4d75-3d26e4db9aaf-5C7GfCeVMHo@public.gmane.org>
2017-10-17 21:43 ` Tom St Denis
[not found] ` <6dadea86-5a2c-e193-d02f-27515408b24c-5C7GfCeVMHo@public.gmane.org>
2017-10-17 23:23 ` Harry Wentland [this message]
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=d410d7b6-6d6b-a58f-0e9c-4eaf91497a97@amd.com \
--to=harry.wentland-5c7gfcevmho@public.gmane.org \
--cc=amd-gfx-PD4FTy7X32lNgt0PjOBp9y5qC8QIuHrW@public.gmane.org \
--cc=tom.stdenis-5C7GfCeVMHo@public.gmane.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 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.