Re: [PATCH 0/3 v2] Add I2S/ADV7511 audio support for ARC AXS10x boards
On Mon, Mar 28, 2016 at 03:36:08PM +0100, Jose Abreu wrote: > ARC AXS10x platforms consist of a mainboard with several peripherals. > One of those peripherals is an HDMI output port controlled by the ADV7511 > transmitter. I'm going to tell you the same thing I tell everyone else working on HDMI audio integration: you all need to talk to each other and review each other's code and unless there is a very good reason for it there should be at least some code sharing. For example take a look at the patches Jyri Sarha has been posting recently. I don't have detailed knowledge of HDMI and the range of hardware that's out there for it but I am seeing a number of different people posting patch serieses that look a lot like each other and like they should be sharing things. Please also try to keep your CC lists reasonable, the set of people you've copied on this stuff is enormous and I'm having trouble seeing why a lot of tehm are included. signature.asc Description: PGP signature
Re: [PATCH 0/3 v2] Add I2S/ADV7511 audio support for ARC AXS10x boards
On Mon, Mar 28, 2016 at 03:36:08PM +0100, Jose Abreu wrote: > ARC AXS10x platforms consist of a mainboard with several peripherals. > One of those peripherals is an HDMI output port controlled by the ADV7511 > transmitter. I'm going to tell you the same thing I tell everyone else working on HDMI audio integration: you all need to talk to each other and review each other's code and unless there is a very good reason for it there should be at least some code sharing. For example take a look at the patches Jyri Sarha has been posting recently. I don't have detailed knowledge of HDMI and the range of hardware that's out there for it but I am seeing a number of different people posting patch serieses that look a lot like each other and like they should be sharing things. Please also try to keep your CC lists reasonable, the set of people you've copied on this stuff is enormous and I'm having trouble seeing why a lot of tehm are included. signature.asc Description: PGP signature
[PATCH 0/3 v2] Add I2S/ADV7511 audio support for ARC AXS10x boards
ARC AXS10x platforms consist of a mainboard with several peripherals. One of those peripherals is an HDMI output port controlled by the ADV7511 transmitter. This patch set adds audio for the ADV7511 transmitter and I2S audio for the AXS10x platform. Changes v1 -> v2: * DT bindings moved to separate patch (as suggested by Alexey Brodkin) * Removed defconfigs entries (as suggested by Alexey Brodkin) Jose Abreu (3): drm/i2c/adv7511: Add audio support ASoC: dwc: Add I2S HDMI audio support arc: axs10x: Add support for Designware I2S on DT arch/arc/boot/dts/axs10x_mb.dtsi| 49 +- drivers/gpu/drm/i2c/Kconfig | 11 + drivers/gpu/drm/i2c/Makefile|2 + drivers/gpu/drm/i2c/adv7511.c | 1024 --- drivers/gpu/drm/i2c/adv7511.h | 41 ++ drivers/gpu/drm/i2c/adv7511_audio.c | 310 +++ drivers/gpu/drm/i2c/adv7511_core.c | 1005 ++ include/sound/soc-dai.h |1 + sound/soc/dwc/Kconfig |1 + sound/soc/dwc/designware_i2s.c | 385 - 10 files changed, 1788 insertions(+), 1041 deletions(-) delete mode 100644 drivers/gpu/drm/i2c/adv7511.c create mode 100644 drivers/gpu/drm/i2c/adv7511_audio.c create mode 100644 drivers/gpu/drm/i2c/adv7511_core.c -- 1.9.1
[PATCH 0/3 v2] Add I2S/ADV7511 audio support for ARC AXS10x boards
ARC AXS10x platforms consist of a mainboard with several peripherals. One of those peripherals is an HDMI output port controlled by the ADV7511 transmitter. This patch set adds audio for the ADV7511 transmitter and I2S audio for the AXS10x platform. Changes v1 -> v2: * DT bindings moved to separate patch (as suggested by Alexey Brodkin) * Removed defconfigs entries (as suggested by Alexey Brodkin) Jose Abreu (3): drm/i2c/adv7511: Add audio support ASoC: dwc: Add I2S HDMI audio support arc: axs10x: Add support for Designware I2S on DT arch/arc/boot/dts/axs10x_mb.dtsi| 49 +- drivers/gpu/drm/i2c/Kconfig | 11 + drivers/gpu/drm/i2c/Makefile|2 + drivers/gpu/drm/i2c/adv7511.c | 1024 --- drivers/gpu/drm/i2c/adv7511.h | 41 ++ drivers/gpu/drm/i2c/adv7511_audio.c | 310 +++ drivers/gpu/drm/i2c/adv7511_core.c | 1005 ++ include/sound/soc-dai.h |1 + sound/soc/dwc/Kconfig |1 + sound/soc/dwc/designware_i2s.c | 385 - 10 files changed, 1788 insertions(+), 1041 deletions(-) delete mode 100644 drivers/gpu/drm/i2c/adv7511.c create mode 100644 drivers/gpu/drm/i2c/adv7511_audio.c create mode 100644 drivers/gpu/drm/i2c/adv7511_core.c -- 1.9.1
