06:20 <narmstrong> xdarklight: thx

11:46 <Darkmatter66> mjourdan, Hi, were you able to boot your odroid-c2 board ? and were you able to reproduce the issue in u-boot ?

11:48 <mjourdan> Darkmatter66: Yeah I got into u-boot, but couldn't get a probe on the vpu driver when I left it on sunday. I'll resume this evening

11:50 <Darkmatter66> mjourdan, do you mean the same issue as me or you had a different error ?

11:50 <Darkmatter66> mjourdan, I remember i had to enable a few config options to get the driver to probe

11:51 <mjourdan> yeah, I enabled them all as well, but the probe isn't called. Pretty sure yours do with the behaviour of your screen you described

11:53 <Darkmatter66> mjourdan, this is the config i used saved using "make savedefconfig" https://0x0.st/zmy2.txt

11:53 <Darkmatter66> mjourdan, you just need to change CONFIG_SYS_BOARD and CONFIG_DEFAULT_DEVICE_TREE to the ordoid-c2

13:38 <mjourdan> Darkmatter66: got slightly further, the probe of the hdmi controller failed (meson_dw_hdmi) because of a lack of hdmi-supply on odroid-c2, and the check was mandatory. But bypassing it gives me a timeout on HPD signal check so there's still something wrong.

13:39 <mjourdan> Darkmatter66: looking at p201 dts, looks like you're in the same case (no hdmi-supply)

17:23 <xdarklight> mjourdan: thank you for the Reviewed-by!

17:32 <Darkmatter66> Darkmatter66, Maybe this is my fault! I remember when i added support for 201 in u-boot narmstrong gave some notes on modifying the dts ( I remember one of them had to do with hdmi_tx or hdmi-supply ) maybe I didn't add what narmstrong asked for

17:33 <Darkmatter66> mjourdan, ^

18:14 <Darkmatter66> mjourdan, I added the hdmi-supply to the hdmi_tx node and I still get the same problem, the hdmi_5v i just copied from the libretech-cc file here https://github.com/u-boot/u-boot/blob/e1a2ed7180adeefb6164239a18249dca5701319d/arch/arm/dts/meson-gxl-s905x-libretech-cc.dts#L75-L85

18:54 <mjourdan> Darkmatter66: I don't think you're supposed to add one, especially not copied from another dts, these are board specific. The bug is in the driver that makes a mandatory check on the supply while it should be optional.

18:59 <mjourdan> xdarklight: no problem! btw, how is HDMI on M8 going ?

19:14 <xdarklight> mjourdan: I can read the EDID (most of the time without CRC errors). that's where I stopped because I want to get CVBS output working first (but I'm still waiting for a _working_ replacement for my broken USB video grabber)

20:36 <xdarklight> narmstrong: there seems to be some other SPDX conversion going on: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=2c1212de6f9794a7becba5f219fa6ce8a8222c90 :(

20:45 <xdarklight> narmstrong: also I would like you to have a quick look at the PWM driver cleanups that I will send this weekend: https://github.com/xdarklight/linux/compare/1806e6f48ae65aaf869badcad5929f157c8f31b8...269d7e6a02a0d52d28e7c7d7d23873ce484d0c71

20:46 <xdarklight> I have tried to implement Uwe's suggestion in this commit, but I'm not sure it improves readability: https://github.com/xdarklight/linux/commit/68c2595725ef4afae031859505ec12028be44539 - I'd like to get your feedback on this one before I send everything to the list

20:46 <xdarklight> (if we are uncertain that it'll make the code easier to read then I'll put those patches at the end of the series so they can be dropped easily)

20:56 <narmstrong> Wtf are these changes

20:57 <narmstrong> (I was talking about spdx)

20:59 <narmstrong> xdarklight: I’ll have a look !!