Re: [GIT PULL] TDA998x I2C driver development updates
On Thu, Aug 02, 2018 at 10:34:21AM +0100, Russell King wrote: > Hi David, > > The following changes since commit 4da1d4c751c9b1b713c13043bad7c4d27cd1418c: > > Merge commit 'refs/for-upstream/mali-dp' of git://linux-arm.org/linux-ld > into drm-next (2018-07-06 10:02:13 +1000) > > are available in the git repository at: > > git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel > > for you to fetch changes up to 4eb47188dd4915df8b3047e2256774598ac73834: > > drm/i2c: tda998x: correct PLL divider calculation (2018-08-02 10:27:15 > +0100) > > As per the patches posted, discussed and tested by Peter Rosin, this > converts TDA998x to a bridge driver, while still allowing Armada and > TI LCDC to continue using it as they always have done. It also gets > rid of the private .fill_modes function, and tweaks the TMDS divider > calculation to be more correct to the available information. Hi David, I noticed you haven't pulled this yet, and as 0-day found a build problem when CONFIG_OF is disabled, I've updated the branch. It's new head is 926a299c42e38bbe8bb909eae0146e676b66afe4, with this diffstat: drivers/gpu/drm/i2c/tda998x_drv.c | 372 +++--- 1 file changed, 189 insertions(+), 183 deletions(-) Thanks. -- RMK's Patch system: http://www.armlinux.org.uk/developer/patches/ FTTC broadband for 0.8mile line in suburbia: sync at 13.8Mbps down 630kbps up According to speedtest.net: 13Mbps down 490kbps up ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[GIT PULL] TDA998x I2C driver development updates
Hi David, The following changes since commit 4da1d4c751c9b1b713c13043bad7c4d27cd1418c: Merge commit 'refs/for-upstream/mali-dp' of git://linux-arm.org/linux-ld into drm-next (2018-07-06 10:02:13 +1000) are available in the git repository at: git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel for you to fetch changes up to 4eb47188dd4915df8b3047e2256774598ac73834: drm/i2c: tda998x: correct PLL divider calculation (2018-08-02 10:27:15 +0100) As per the patches posted, discussed and tested by Peter Rosin, this converts TDA998x to a bridge driver, while still allowing Armada and TI LCDC to continue using it as they always have done. It also gets rid of the private .fill_modes function, and tweaks the TMDS divider calculation to be more correct to the available information. Peter Rosin (2): drm/i2c: tda998x: find the drm_device via the drm_connector drm/i2c: tda998x: split tda998x_encoder_dpms into enable/disable Russell King (8): drm/i2c: tda998x: move tda998x_set_config() into tda998x_create() drm/i2c: tda998x: convert to bridge driver drm/i2c: tda998x: allocate tda998x_priv inside tda998x_create() drm/i2c: tda998x: cleanup from previous changes drm/i2c: tda998x: register bridge outside of component helper drm/i2c: tda998x: move mode_valid() to bridge drm/i2c: tda998x: get rid of private fill_modes function drm/i2c: tda998x: correct PLL divider calculation drivers/gpu/drm/i2c/tda998x_drv.c | 370 +++--- 1 file changed, 187 insertions(+), 183 deletions(-) ___ dri-devel mailing list dri-devel@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/dri-devel
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 9b2502b6ebc632ff49743b3639ea12d4f08808a5. These updates: * improve the robustness of the driver wrt races * improve the compliance for sending infoframes and audio * re-organise the function order in the driver to group like functions together. (This unfortunately causes a conflict with the change in drm-misc, but it should be trivial to solve, although it looks more scarey than it really is - sfr has already sent two reports about this, one earlier today.) * simplify tda998x_audio_get_eld and DPMS handling * power down sections of the chip that we never use * add some initial preparation for supporting the CEC driver This will update the following files: drivers/gpu/drm/i2c/tda998x_drv.c | 953 -- 1 file changed, 503 insertions(+), 450 deletions(-) through these changes: Colin Ian King (1): drm/i2c: tda998x: fix spelling mistake Russell King (16): drm/i2c: tda998x: move audio mutex initialisation drm/i2c: tda998x: avoid race in tda998x_encoder_mode_set() drm/i2c: tda998x: avoid racy access to mode clock drm/i2c: tda998x: avoid race when programming audio drm/i2c: tda998x: only configure infoframes and audio if supported drm/i2c: tda998x: only enable audio if supported by sink drm/i2c: tda998x: correct function name in comments drm/i2c: tda998x: move and rename tda998x_encoder_set_config() drm/i2c: tda998x: group connector functions and funcs together drm/i2c: tda998x: separate connector initialisation drm/i2c: tda998x: group audio functions together drm/i2c: tda998x: remove complexity from tda998x_audio_get_eld() drm/i2c: tda998x: switch to boolean is_on drm/i2c: tda998x: power down pre-filter and color conversion drm/i2c: tda998x: allow interrupt to be shared drm/i2c: tda998x: allow sharing of the CEC device accesses Many thanks.
[RESEND][GIT PULL] TDA998x I2C driver development updates
Sorry, my mistake - it was merged, and it's just that I seem to have accidentally rebased the branch, which is why the commits were still showing up in my tree. Sorry for the noise, please ignore. On Wed, Oct 19, 2016 at 05:37:18PM +0100, Russell King wrote: > David, > > Please incorporate the latest TDA998x I2C driver development updates, > which can be found at: > > git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel > > with SHA1 df0bd1e8f3c508bf4c3445f94b12e38289b65f13. > > This was sent on the 9th September, and I was hoping that it was going > to make 4.9-rc1 as TI want it in, but it appears that it was never > picked up. What's going on? Should I queue this (and my other DRM > changes) for Linus myself? > > This will update the following files: > > .../devicetree/bindings/display/bridge/tda998x.txt | 18 ++ > arch/arm/boot/dts/am335x-boneblack.dts | 71 - > drivers/gpu/drm/i2c/Kconfig| 1 + > drivers/gpu/drm/i2c/tda998x_drv.c | 297 > ++--- > include/drm/i2c/tda998x.h | 29 +- > include/dt-bindings/display/tda998x.h | 7 + > 6 files changed, 368 insertions(+), 55 deletions(-) > create mode 100644 include/dt-bindings/display/tda998x.h > > through these changes: > > Jyri Sarha (3): > drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata > drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding > ARM: dts: am335x-boneblack: Add HDMI audio support > > Russell King (1): > Merge commit 'efc9194bcff84' ("ASoC: hdmi-codec: callback function will > be called with private data") into drm-tda998x-devel > > Many thanks.
[RESEND][GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 df0bd1e8f3c508bf4c3445f94b12e38289b65f13. This was sent on the 9th September, and I was hoping that it was going to make 4.9-rc1 as TI want it in, but it appears that it was never picked up. What's going on? Should I queue this (and my other DRM changes) for Linus myself? This will update the following files: .../devicetree/bindings/display/bridge/tda998x.txt | 18 ++ arch/arm/boot/dts/am335x-boneblack.dts | 71 - drivers/gpu/drm/i2c/Kconfig| 1 + drivers/gpu/drm/i2c/tda998x_drv.c | 297 ++--- include/drm/i2c/tda998x.h | 29 +- include/dt-bindings/display/tda998x.h | 7 + 6 files changed, 368 insertions(+), 55 deletions(-) create mode 100644 include/dt-bindings/display/tda998x.h through these changes: Jyri Sarha (3): drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding ARM: dts: am335x-boneblack: Add HDMI audio support Russell King (1): Merge commit 'efc9194bcff84' ("ASoC: hdmi-codec: callback function will be called with private data") into drm-tda998x-devel Many thanks.
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://git.armlinux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 3e980591945eadbfdf4cbc05d56e5f44010a5a87. This adds the ASoC codec interfaces for TDA998x HDMI audio from Jyri Sarha. This will update the following files: .../devicetree/bindings/display/bridge/tda998x.txt | 18 ++ arch/arm/boot/dts/am335x-boneblack.dts | 71 - drivers/gpu/drm/i2c/Kconfig| 1 + drivers/gpu/drm/i2c/tda998x_drv.c | 297 ++--- include/drm/i2c/tda998x.h | 29 +- include/dt-bindings/display/tda998x.h | 7 + 6 files changed, 368 insertions(+), 55 deletions(-) create mode 100644 include/dt-bindings/display/tda998x.h through these changes: Jyri Sarha (3): drm/i2c: tda998x: Improve tda998x_configure_audio() audio related pdata drm/i2c: tda998x: Register ASoC hdmi-codec and add audio DT binding ARM: dts: am335x-boneblack: Add HDMI audio support Many thanks.
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://ftp.arm.linux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 9736e988d32807d5a186c722928a97f37346fec8. These changes from Liviu add support for atomic mode setting, add the TMDS clock limitation according to the device, and ensure that we correctly clean up in the unbind function. This will update the following files: drivers/gpu/drm/i2c/tda998x_drv.c | 12 ++-- 1 file changed, 10 insertions(+), 2 deletions(-) through these changes: Liviu Dudau (ARM) (3): drm/i2c: tda998x: unregister the connector in the unbind function drm/i2c: tda998x: increase the supported dotclock frequency to 165MHz for TDA19988 drm/i2c: tda998x: Add support for atomic modesetting Many thanks.
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://ftp.arm.linux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 9525c4dd923f8ffe38818f86cee523a5f7b19617. This series: * removes the now unused DRM slave encoder support, which all users have migrated away from, allowing us to simplify the code. * ensure all pending interrupts are processed together, rather than needing the handler to be re-entered each time. * use more HDMI helpers to setup the info frames. * fix EDID read handling by ensuring that we always wait the specified time before attempting to read the EDID, no matter where the EDID read request came from. This will update the following files: drivers/gpu/drm/i2c/tda998x_drv.c | 487 +++--- 1 file changed, 185 insertions(+), 302 deletions(-) through these changes: Russell King (11): drm/i2c: tda998x: remove useless NULL checks drm/i2c: tda998x: report whether we actually handled the IRQ drm/i2c: tda998x: re-implement "Fix EDID read timeout on HDMI connect" drm/i2c: tda998x: convert to u8/u16/u32 types drm/i2c: tda998x: handle all outstanding interrupts drm/i2c: tda998x: use more HDMI helpers drm/i2c: tda998x: remove DRM slave encoder support drm/i2c: tda998x: remove encoder pointer drm/i2c: tda998x: move connector into struct tda998x_priv drm/i2c: tda998x: kill struct tda998x_priv2 drm/i2c: tda998x: clean up after struct tda998x_priv2 removal Many thanks.
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://ftp.arm.linux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 5296b7f9d0e1ac31a9bc3044f04af18a521b9459. Just one fix for the new warning recently reported on 64-bit platforms. This will update the following files: drivers/gpu/drm/i2c/tda998x_drv.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) through these changes: Russell King (1): drm/i2c: tda998x: fix compiler warning for ssize_t Many thanks.
[GIT PULL] TDA998x I2C driver development updates
David, Please incorporate the latest TDA998x I2C driver development updates, which can be found at: git://ftp.arm.linux.org.uk/~rmk/linux-arm.git drm-tda998x-devel with SHA1 8c7a075da9f7980cc95ffcd7e6621d4a87f20f40. Just one commit to convert TDA998x to use the HDMI AVI infoframe helper. This will update the following files: drivers/gpu/drm/i2c/tda998x_drv.c | 26 ++ 1 file changed, 14 insertions(+), 12 deletions(-) through these changes: Russell King (1): drm/i2c: tda998x: use drm_hdmi_avi_infoframe_from_display_mode() Many thanks.