Hi Simon, On Wed, Jul 19, 2017 at 11:05 AM, Simon Glass <s...@chromium.org> wrote: > Hi Mario, > > On 14 July 2017 at 05:55, Mario Six <mario....@gdsys.cc> wrote: >> This patch adds a simple transmitter uclass meant for DVI or DisplayPort >> transmitters. The only driver functionality implemented are power_on and >> power_off methods for powering the transmitter device on and off, >> respectively. > > Could we use UCLASS_DISPLAY for this? We are using that for things > like DisplayPort and HDMI. > > If so you could add any missing operation and move your tests to that? >
OK, I thought the display uclass consisted of literal displays (as in LCD screens and such). I'll use that uclass in v2 then. >> >> Signed-off-by: Mario Six <mario....@gdsys.cc> >> --- >> >> arch/sandbox/dts/sandbox.dts | 4 ++ >> arch/sandbox/dts/test.dts | 4 ++ >> configs/sandbox_defconfig | 2 + >> drivers/Kconfig | 2 + >> drivers/Makefile | 1 + >> drivers/transmitter/Kconfig | 16 +++++++ >> drivers/transmitter/Makefile | 9 ++++ >> drivers/transmitter/sandbox_transmitter.c | 74 >> +++++++++++++++++++++++++++++++ >> drivers/transmitter/transmitter-uclass.c | 36 +++++++++++++++ >> include/dm/uclass-id.h | 1 + >> include/transmitter.h | 49 ++++++++++++++++++++ >> test/dm/Makefile | 1 + >> test/dm/transmitter.c | 31 +++++++++++++ >> 13 files changed, 230 insertions(+) >> create mode 100644 drivers/transmitter/Kconfig >> create mode 100644 drivers/transmitter/Makefile >> create mode 100644 drivers/transmitter/sandbox_transmitter.c >> create mode 100644 drivers/transmitter/transmitter-uclass.c >> create mode 100644 include/transmitter.h >> create mode 100644 test/dm/transmitter.c >> > > Regards, > Simon Best regards, Mario _______________________________________________ U-Boot mailing list U-Boot@lists.denx.de https://lists.denx.de/listinfo/u-boot