Re: Abouut adv7511 driver

Re: Abouut adv7511 driver

[Lars-Peter Clausen]

Cc: alsa-ml

On 07/31/2015 07:38 AM, Kuninori Morimoto wrote:
> 
> Hi Lars
> Cc Mark
> 
> I noticed Analog Devices has adv7511 support on
> http://wiki.analog.com/resources/tools-software/linux-drivers/drm/adv7511
> 
> 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.

- Lars

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

Re: Abouut adv7511 driver

[Kuninori Morimoto]

Hi Lars
Cc Magnus, Laurent

> > I noticed Analog Devices has adv7511 support on
> > http://wiki.analog.com/resources/tools-software/linux-drivers/drm/adv7511
> > 
> > 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.

Thank you for explaining this background.
I understand. feel free to contact me if you want me test.
--
To unsubscribe from this list: send the line "unsubscribe alsa-devel" in

Re: Abouut adv7511 driver

[Laurent Pinchart]

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

Re: Abouut adv7511 driver

[Mark Brown]

[-- Attachment #1.1: Type: text/plain, Size: 1691 bytes --]

On Sat, Aug 01, 2015 at 12:44:06PM +0300, Laurent Pinchart wrote:
> On Friday 31 July 2015 08:21:16 Kuninori Morimoto wrote:
> > Hi Lars

> > > 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.

It's a bit of a struggle to get anyone to consider the generic problems
for HDMI at all - Russell did some really nice work with the EDID
handling but that's been about it.  There was some talk about replacing
simple card with a graph based simple card but it hasn't been moving
quickly.

Personally I'm not 100% sure how relevant binding issues will be to the
HDMI integration - as far as I can tell the integration of audio and
video is generally happening within a single device (either an external
encoder or the SoC itself) so it's not something that needs to be
represented externally to that device and there's sometimes some
flexibility about the connections internal to the device so we don't
want to nail things up entirely in DT even if it was needed.  There may
be other kinds of hardware I'm not familiar with here where it does
become important though, or I may be missing things.

[-- Attachment #1.2: Digital signature --]
[-- Type: application/pgp-signature, Size: 473 bytes --]

[-- Attachment #2: Type: text/plain, Size: 0 bytes --]