All of lore.kernel.org
 help / color / mirror / Atom feed
From: Laurent Pinchart <laurent.pinchart@ideasonboard.com>
To: alsa-devel@alsa-project.org
Cc: Mark Brown <broonie@kernel.org>,
	Lars-Peter Clausen <lars@metafoo.de>,
	Magnus <magnus.damm@gmail.com>,
	alsa-devel@vger.kernel.org, Mark Brown <broonie@linaro.org>
Subject: Re: Abouut adv7511 driver
Date: Sat, 01 Aug 2015 12:44:06 +0300	[thread overview]
Message-ID: <2068811.mrRDGz9M57@avalon> (raw)
In-Reply-To: <87y4hwviyu.wl%kuninori.morimoto.gx@renesas.com>

Hello,

On Friday 31 July 2015 08:21:16 Kuninori Morimoto wrote:
> Hi Lars
> Cc Magnus, Laurent
> 
> > > I noticed Analog Devices has adv7511 support on
> > > http://wiki.analog.com/resources/tools-software/linux-drivers/drm/adv751
> > > 1
> > > 
> > > And I tried this driver on my board, and it works.
> > > 
> > > if my understanding was correct, latest linux kernel doesn't have
> > > adv7511 "sound" support yet.
> > > Thus, I would like to have it on linux kernel.
> > > According to this driver/file, it is under GPL-2, so it is no problem
> > > from license point of view.
> > > 
> > > According to git log Signd-off-by, Lars/Mark are listed on this adv7511
> > > sound driver. but I wonder why it is not on upstream ? Are there some
> > > reason ??
> > > If you don't mind, I would like to send it to ML.
> > > This is OK ?
> > 
> > The reason why the driver is not upstream at the moment is that the way it
> > implements things is definitely not the right way. But there is still a
> > lot of discussion on what actually is the right way to implement audio
> > support for HDMI devices. Once that has been sorted out the driver will
> > hopefully go upstream.

Beside the in-kernel API, have you given a thought to DT bindings for HDMI 
encoders ? Some platforms use DT bindings based on OF graph for the video 
path, would it make sense to model the audio path similarly ? The 
implementation could become a bit of a challenge though.

> Thank you for explaining this background.
> I understand. feel free to contact me if you want me test.

-- 
Regards,

Laurent Pinchart

--
To unsubscribe from this list: send the line "unsubscribe alsa-devel" in

  reply	other threads:[~2015-08-01  9:44 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <87zj2cvqi7.wl%kuninori.morimoto.gx@renesas.com>
2015-07-31  7:43 ` Abouut adv7511 driver Lars-Peter Clausen
2015-07-31  8:21   ` Kuninori Morimoto
2015-08-01  9:44     ` Laurent Pinchart [this message]
2015-08-01 11:08       ` Mark Brown

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=2068811.mrRDGz9M57@avalon \
    --to=laurent.pinchart@ideasonboard.com \
    --cc=alsa-devel@alsa-project.org \
    --cc=alsa-devel@vger.kernel.org \
    --cc=broonie@kernel.org \
    --cc=broonie@linaro.org \
    --cc=lars@metafoo.de \
    --cc=magnus.damm@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.