Hello Hans, Hi Neil, (apologies in advance if any of this is wrong, I don't have any CEC capable TV so I can't test it) On Fri, Mar 23, 2018 at 1:59 PM, Hans Verkuil <hverkuil@xxxxxxxxx> wrote: > From: Hans Verkuil <hans.verkuil@xxxxxxxxx> > > Some boards (amlogic) have two CEC controllers: the DesignWare controller > and their own CEC controller (meson ao-cec). as far as I understand the Amlogic Meson SoCs have two domains: - AO (always-on, powered even in suspend mode) where meson-ao-cec can wake up the system from suspend - EE (everything else, not powered during suspend) where dw-hdmi-cec lives this far everything is OK > Since the CEC line is not hooked up to the DW controller we need a way > to disable that controller. This patch series adds the cec-disable > property for that purpose. drivers/pinctrl/meson/pinctrl-meson-gxbb.c has ao_cec_pins and ee_cec_pins, both use GPIOAO_12 drivers/pinctrl/meson/pinctrl-meson-gxl.c has ao_cec_pins and ee_cec_pins, both use GPIOAO_8 @Neil: do you know if the CEC signal routing is: ao_cec_pins -> meson-ao-cec ee_cec_pins -> dw-hdmi-cec I'm curious because if both CEC controllers can be used then it might be worth mentioning this in the cover-letter and patch description Regards Martin -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html