All of lore.kernel.org
 help / color / mirror / Atom feed
From: Jyri Sarha <jsarha@ti.com>
To: dri-devel@lists.freedesktop.org, devicetree@vger.kernel.org
Cc: fabrizio.castro@bp.renesas.com, Songjun.Wu@atmel.com,
	tony@atomide.com, peter.ujfalusi@ti.com, tomi.valkeinen@ti.com,
	laurent.pinchart@ideasonboard.com, voice.shen@atmel.com
Subject: [PATCH v2 0/5] drm/bridge: sii902x: HDMI-audio support and some fixes
Date: Wed, 27 Feb 2019 23:54:18 +0200	[thread overview]
Message-ID: <cover.1551303673.git.jsarha@ti.com> (raw)

Changes since first version:
- Moved reviewed patches to front:
  - drm/bridge: sii902x: add input_bus_flags
  - drm/bridge: sii902x: Set output mode to HDMI or DVI according to EDID
  - drm/bridge: sii902x: pixel clock unit is 10kHz instead of 1kHz
- Added a new fix:
  - drm/bridge: sii902x: Select I2C_MUX
- Applied some review suggestions to
  - drm/bridge: sii902x: Implement HDMI audio support
    - use clock-names property to name mclk
    - move comment describing added mutex to struct sii902x and improve it
    - cleanup sii902x_mute()
    - cleanup sii902x_select_mclk_div()
    - fix condition for checking ENABLE_BIT from i2s_fifo_routing in
      sii902x_audio_codec_init()

Still to do

- Agree on i2s wires to HDMI audio fifo routing in dts. 

  The current scheme is quite straight forward, but there is maybe
  there is even more straight forward solutions like:

  audio-fifo-enable = <1 1 1 1>;
  audio-i2s-pin-to-fifo = <0 1 2 3>;

  Meaning that all fifos are enabled and SD0 is routed to fifo 0, SD1
  to fifo 1, etc. I am not sure if the channel swap functionality
  should show in dts binding.

Jyri Sarha (4):
  drm/bridge: sii902x: Set output mode to HDMI or DVI according to EDID
  drm/bridge: sii902x: pixel clock unit is 10kHz instead of 1kHz
  drm/bridge: sii902x: Select I2C_MUX
  drm/bridge: sii902x: Implement HDMI audio support

Tomi Valkeinen (1):
  drm/bridge: sii902x: add input_bus_flags

 .../bindings/display/bridge/sii902x.txt       |  36 +-
 drivers/gpu/drm/bridge/Kconfig                |   1 +
 drivers/gpu/drm/bridge/sii902x.c              | 472 +++++++++++++++++-
 include/dt-bindings/sound/sii902x-audio.h     |  11 +
 4 files changed, 512 insertions(+), 8 deletions(-)
 create mode 100644 include/dt-bindings/sound/sii902x-audio.h

-- 
Texas Instruments Finland Oy, Porkkalankatu 22, 00180 Helsinki. Y-tunnus/Business ID: 0615521-4. Kotipaikka/Domicile: Helsinki

_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

             reply	other threads:[~2019-02-27 21:54 UTC|newest]

Thread overview: 18+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-02-27 21:54 Jyri Sarha [this message]
2019-02-27 21:54 ` [PATCH v2 1/5] drm/bridge: sii902x: add input_bus_flags Jyri Sarha
2019-03-04 12:48   ` Laurent Pinchart
2019-02-27 21:54 ` [PATCH v2 2/5] drm/bridge: sii902x: Set output mode to HDMI or DVI according to EDID Jyri Sarha
2019-03-04 12:52   ` Laurent Pinchart
2019-03-04 14:15     ` Jyri Sarha
2019-02-27 21:54 ` [PATCH v2 3/5] drm/bridge: sii902x: pixel clock unit is 10kHz instead of 1kHz Jyri Sarha
2019-03-04 15:59   ` Laurent Pinchart
2019-02-27 21:54 ` [PATCH v2 4/5] drm/bridge: sii902x: Select I2C_MUX Jyri Sarha
2019-03-04  8:54   ` Andrzej Hajda
2019-03-04 16:05   ` Laurent Pinchart
2019-02-27 21:54 ` [PATCH v2 5/5] drm/bridge: sii902x: Implement HDMI audio support Jyri Sarha
2019-03-04  9:26   ` Andrzej Hajda
2019-03-12 16:11   ` Rob Herring
2019-03-04 12:42 ` [PATCH v2 0/5] drm/bridge: sii902x: HDMI-audio support and some fixes Laurent Pinchart
2019-03-04 14:29   ` Jyri Sarha
2019-03-04 16:10     ` Laurent Pinchart
2019-03-06 14:18     ` Olivier MOYSAN

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=cover.1551303673.git.jsarha@ti.com \
    --to=jsarha@ti.com \
    --cc=Songjun.Wu@atmel.com \
    --cc=devicetree@vger.kernel.org \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=fabrizio.castro@bp.renesas.com \
    --cc=laurent.pinchart@ideasonboard.com \
    --cc=peter.ujfalusi@ti.com \
    --cc=tomi.valkeinen@ti.com \
    --cc=tony@atomide.com \
    --cc=voice.shen@atmel.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.