All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jani Nikula <jani.nikula@linux.intel.com>
To: Andrzej Pietrasiewicz <andrzej.p@collabora.com>,
	Maling list - DRI developers <dri-devel@lists.freedesktop.org>,
	Thierry Reding <thierry.reding@gmail.com>,
	Jonathan Hunter <jonathanh@nvidia.com>,
	Eric Anholt <eric@anholt.net>, Shawn Guo <shawnguo@kernel.org>,
	Joonas Lahtinen <joonas.lahtinen@linux.intel.com>,
	Rodrigo Vivi <rodrigo.vivi@intel.com>
Subject: Re: Next round of associating ddc adapters with connectors
Date: Thu, 28 Nov 2019 12:26:06 +0200	[thread overview]
Message-ID: <874kyonvpd.fsf@intel.com> (raw)
In-Reply-To: <00af93c8-4e59-3ff0-6bd0-69289b34846c@collabora.com>

On Fri, 22 Nov 2019, Andrzej Pietrasiewicz <andrzej.p@collabora.com> wrote:
> Dear Maintainers,
>
> Can you please express your opinion on these patches:
>
> [tegra] https://patchwork.freedesktop.org/patch/327099/?series=65831&rev=1
> [vc4] https://patchwork.freedesktop.org/patch/327102/?series=65831&rev=1
> [zte] https://patchwork.freedesktop.org/patch/327106/?series=65831&rev=1
> [zte] https://patchwork.freedesktop.org/patch/327112/?series=65831&rev=1
> [i915] https://patchwork.freedesktop.org/patch/327120/?series=65831&rev=1

I've expressed my opinion in [1] and that hasn't changed. I don't like
the interface, and a few years down the line I expect there's going to
be a patch series renaming drm_connector_init_with_ddc() back to
drm_connector_init() with an extra parameter, and NULL will be passed
where ddc is not relevant.

Anyway, that ship has sailed, you didn't even care to reply, and nobody
else seems to mind, so meh, and ack on the patch. Indeed I would've
applied it already, but it no longer applies, so please send the rebased
i915 patch individually to intel-gfx@lists.freedesktop.org so we'll also
get CI coverage against the current drm-tip tree.

BR,
Jani.


[1] http://lore.kernel.org/r/875znlp6yk.fsf@intel.com


>
> ?
>
> Of the originally posted patches:
>
> https://patchwork.freedesktop.org/series/62764/
>
> only the above are still outstanding, the others have been applied
> to at least drm-misc-next or are already upstream.
>
> Regards,
>
> Andrzej

-- 
Jani Nikula, Intel Open Source Graphics Center
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

  reply	other threads:[~2019-11-28 10:26 UTC|newest]

Thread overview: 19+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-11-22 11:57 Next round of associating ddc adapters with connectors Andrzej Pietrasiewicz
2019-11-28 10:26 ` Jani Nikula [this message]
2019-11-28 13:51   ` [PATCH] drm/i915: Provide ddc symlink in hdmi connector sysfs directory Andrzej Pietrasiewicz
2019-11-28 13:51     ` [Intel-gfx] " Andrzej Pietrasiewicz
2019-11-28 13:57     ` Jani Nikula
2019-11-28 13:57       ` [Intel-gfx] " Jani Nikula
2019-11-28 15:01       ` Andrzej Pietrasiewicz
2019-11-28 15:01         ` [Intel-gfx] " Andrzej Pietrasiewicz
2019-11-28 15:01         ` Andrzej Pietrasiewicz
2019-12-13 16:10         ` Andrzej Pietrasiewicz
2019-12-13 16:10           ` [Intel-gfx] " Andrzej Pietrasiewicz
2019-12-19 13:20           ` Jani Nikula
2019-12-19 13:20             ` [Intel-gfx] " Jani Nikula
2019-11-28 14:42   ` ✗ Fi.CI.BUILD: failure for " Patchwork
2019-11-28 14:42     ` [Intel-gfx] " Patchwork
2019-11-28 17:15   ` ✓ Fi.CI.BAT: success for drm/i915: Provide ddc symlink in hdmi connector sysfs directory (rev2) Patchwork
2019-11-28 17:15     ` [Intel-gfx] " Patchwork
2019-11-29 22:57   ` ✗ Fi.CI.IGT: failure " Patchwork
2019-11-29 22:57     ` [Intel-gfx] " Patchwork

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=874kyonvpd.fsf@intel.com \
    --to=jani.nikula@linux.intel.com \
    --cc=andrzej.p@collabora.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=eric@anholt.net \
    --cc=jonathanh@nvidia.com \
    --cc=joonas.lahtinen@linux.intel.com \
    --cc=rodrigo.vivi@intel.com \
    --cc=shawnguo@kernel.org \
    --cc=thierry.reding@gmail.com \
    /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.