linux-amlogic.lists.infradead.org archive mirror
 help / color / mirror / Atom feed
From: linux@armlinux.org.uk (Russell King - ARM Linux)
To: linus-amlogic@lists.infradead.org
Subject: [PATCH] drm/bridge: dw_hdmi: add cec notifier support
Date: Thu, 6 Jul 2017 13:50:25 +0100	[thread overview]
Message-ID: <20170706125025.GL4902@n2100.armlinux.org.uk> (raw)
In-Reply-To: <dc03b967-9d59-48c2-3365-0dc54ba4a891@baylibre.com>

On Thu, Jul 06, 2017 at 01:55:58PM +0200, Neil Armstrong wrote:
> On 07/06/2017 01:45 PM, Russell King - ARM Linux wrote:
> > Well, from what I can see in 4.12, the cec-notifier stuff is rather
> > broken (tda998x has stopped working as its stuck with a physical
> > address of f.f.f.f) so I think the whole thing is rather moot right
> > now.  I don't yet know what's going on with that, other than the
> > notifier stuff seems to not be working, despite being enabled in
> > the .config.
> > 
> 
> Indeed, I missed some parts of the thread, sorry for the confusion.
> 
> Anyway, is it a showstopper to have this patch merged separately ?

It shouldn't be provided it makes _this_ merge window or the -rc
following, but it doesn't really comprise a "fix" in that the feature
never worked in the previous kernel (and we are supposed to have the
rule that features are merged prior to the merge window.)

It's up to the maintainers to decide what to do at this point.

> If you prefer re-posting a serie with this patch inside, no problem,
> but the Amlogic platform still needs this patch to have CEC working.

I suspect (because I've not seen what has been queued to be merged yet)
that even the dw-hdmi "native" side will be similarly broken - unless
Hans merged some other changes to make it work.

So, I suggest we all sit tight and wait until what was merged appears
in Linus' tree.

-- 
RMK's Patch system: http://www.armlinux.org.uk/developer/patches/
FTTC broadband for 0.8mile line: currently at 9.6Mbps down 400kbps up
according to speedtest.net.

  reply	other threads:[~2017-07-06 12:50 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2017-07-06 10:33 [PATCH] drm/bridge: dw_hdmi: add cec notifier support Neil Armstrong
2017-07-06 10:43 ` Jose Abreu
2017-07-06 11:27   ` Neil Armstrong
2017-07-06 11:05 ` Russell King - ARM Linux
2017-07-06 11:29   ` Neil Armstrong
2017-07-06 11:45     ` Russell King - ARM Linux
2017-07-06 11:55       ` Neil Armstrong
2017-07-06 12:50         ` Russell King - ARM Linux [this message]
2017-07-06 11:56       ` Russell King - ARM Linux
2017-07-06 13:38         ` Russell King - ARM Linux
2017-07-06 14:19           ` Russell King - ARM Linux
  -- strict thread matches above, loose matches on Subject: below --
2017-07-31 12:53 Neil Armstrong

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=20170706125025.GL4902@n2100.armlinux.org.uk \
    --to=linux@armlinux.org.uk \
    --cc=linus-amlogic@lists.infradead.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;
as well as URLs for NNTP newsgroup(s).