| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |
| Who | What | When |
|---|---|---|
| hverkuil | benjiG: posted v5 of the HPD notifier, now renamed to CEC notifier. Can you test it with your HW? | [14:20] |
| benjiG | hverkuil: I will do that asap | [14:27] |
| hverkuil | benjiG: thanks! Just a warning, I don't think I actually compile-tested it for sti, so you may encounter compiler warnings/errors.
shame on me. | [14:30] |
| benjiG | hverkuil: could I use exynos4-cec3 branch for testing ? | [14:31] |
| hverkuil | no, use exynos4-cec2 | [14:31] |
| benjiG | ok | [14:31] |
| hverkuil | (the cec3 branch contains exynos-specific stuff) | [14:31] |
| ..... (idle for 24mn) | ||
| benjiG | hverkuil: cec-core.c doesn't compile on my side because 'struct cec_adapter' has no member named 'notifier'
I guess it is because it call cec_notifier_register(adap->notifier..) instead of cec_notifier_register(notifier...) | [14:55] |
| ........ (idle for 35mn) | ||
| hverkuil | benjiG: I've rebased exynos4-cec2. Can you try again?
I accidentally deleted the notifier from struct cec_adapter AND I didn't compile it afterwards for the exynos or I would have seen the compile error :-( | [15:34] |
| benjiG | hverkuil: adap->notifier = notifier; was missing in cec_register_cec_notifier
#ifdef CONFIG_MEDIA_CEC_NOTIFIER struct cec_notifier *notifier; #endif in struct cec_adapter with it complie and works on my side | [15:35] |
| hverkuil | I'd sworn I added adap->notifier = notifier. Apparently didn't commit it. Hmm. | [15:37] |
| benjiG | hverkuil: let's do another force update :-) | [15:37] |
| hverkuil | rebased again. Should work now :-) | [15:39] |
| benjiG | hverkuil it is up and running on my side | [15:42] |
| hverkuil | benjiG: thanks for testing! | [15:44] |
| benjiG | hverkuil: I do not have all the setup to perform a full cec-compliance but /dev/cec0 is present and cec-ctl provide correct values | [15:45] |
| hverkuil | Fingers crossed. I think I covered all of Daniels comments, so hopefully we'll see some progress now.
I also like it that cec-notifier is now in drivers/media. Much easier to maintain in the future. | [15:45] |
| benjiG | yes that sound good
I will work on cec for stm32 when you will some ack for this version | [15:46] |
| hverkuil | nice! | [15:47] |
| narmstrong | hverkuil: I'll be happy to test it on dw-hdmi when russel send it's patchset | [15:55] |
| *** | benjiG has left | [15:55] |
| narmstrong | hverkuil: sailus: For the YUV 4:2:0 bus format, should I send back my patch with the _0_5X24/30/36/48 suffix ? | [15:56] |
| .................. (idle for 1h29mn) | ||
| hverkuil | narmstrong: yes please. | [17:25] |
| ↑back Search ←Prev date Next date→ Show only urls | (Click on time to select a line by its url) |