Re: [PATCH v6 00/11] CEC update

Hans Verkuil
2015-06-16T07:09:26Z

Let me fill in the subject line so people know what it is about :-)

Hans

On 06/16/15 08:22, Hans Verkuil wrote:
> On 05/04/2015 07:32 PM, Kamil Debski wrote:
>> Hi,
>>
>> The sixth version of this patchset addresses recent comments on the mailing
>> list. Please see the changelog below for details.
>
> Just in case people are wondering what happened to this: about a month ago I
> took over from Kamil and I am working hard to get a v7 posted. I'm currently
> working on a utility to send messages over the cec line and once that's done
> I plan on posting a new version which should be close to the final version.
>
> One of the main problems is making sure that the framework covers the full
> CEC 2.0 functionality (or at least, making sure that any missing pieces can
> be added seamlessly later). CEC 2.0 is a lot more strict in what should and
> what shouldn't be implemented, so this takes time.
>
> Regards,
>
> Hans
>
>>
>> Best wishes,
>> Kamil Debski
>>
>> Changes since v5
>> ================
>> - drop struct cec_timeval in favour of a __u64 that keeps the timestamp in ns
>> - remove userspace documentation from Documentation/cec.txt as userspace API
>> is described in the DocBook
>> - add missing documentation for the passthrough mode to the DocBook
>> - add information about the number of events that can be queued
>> - fix misspelling of reply
>> - fix behaviour of posting an event in cec_received_msg, such that the behaviour
>> is consistent with the documentation
>>
>> Changes since v4
>> ================
>> - add sequence numbering to transmitted messages
>> - add sequence number handling to event hanlding
>> - add passthrough mode
>> - change reserved field sizes
>> - fixed CEC version defines and addec CEC 2.0 commands
>> - add DocBook documentation
>>
>> Changes since v3
>> ================
>> - remove the promiscuous mode
>> - rewrite the devicetree patches
>> - fixes, expansion and partial rewrite of the documentation
>> - reorder of API structures and addition of reserved fields
>> - use own struct to report time (32/64 bit safe)
>> - fix of handling events
>> - add cec.h to include/uapi/linux/Kbuild
>> - fixes in the adv76xx driver (add missing methods, change adv7604 to adv76xx)
>> - cleanup of debug messages in s5p-cec driver
>> - remove non necessary claiming of a gpio in the s5p-cec driver
>> - cleanup headers of the s5p-cec driver
>>
>> Changes since v2
>> ===============-
>> - added promiscuous mode
>> - added new key codes to the input framework
>> - add vendor ID reporting
>> - add the possibility to clear assigned logical addresses
>> - cleanup of the rc cec map
>>
>> Changes since v1
>> ================
>> - documentation edited and moved to the Documentation folder
>> - added key up/down message handling
>> - add missing CEC commands to the cec.h file
>>
>> Background
>> ==========
>>
>> The work on a common CEC framework was started over three years ago by Hans
>> Verkuil. Unfortunately the work has stalled. As I have received the task of
>> creating a driver for the CEC interface module present on the Exynos range of
>> SoCs, I got in touch with Hans. He replied that the work stalled due to his
>> lack of time.
>>
>> Original RFC by Hans Verkuil/Martin Bugge
>> =========================================
>> https://www.mail-archive.com/linux-media@vger.kernel.org/msg28735.html
>>
>>
>> Hans Verkuil (5):
>> cec: add HDMI CEC framework
>> DocBook/media: add CEC documentation
>> v4l2-subdev: add HDMI CEC ops
>> cec: adv7604: add cec support.
>> cec: adv7511: add cec support.
>>
>> Kamil Debski (6):
>> dts: exynos4*: add HDMI CEC pin definition to pinctrl
>> dts: exynos4: add node for the HDMI CEC device
>> dts: exynos4412-odroid*: enable the HDMI CEC device
>> HID: add HDMI CEC specific keycodes
>> rc: Add HDMI CEC protoctol handling
>> cec: s5p-cec: Add s5p-cec driver
>>
>> Documentation/DocBook/media/Makefile | 4 +-
>> Documentation/DocBook/media/v4l/biblio.xml | 10 +
>> Documentation/DocBook/media/v4l/cec-api.xml | 74 ++
>> Documentation/DocBook/media/v4l/cec-func-close.xml | 59 +
>> Documentation/DocBook/media/v4l/cec-func-ioctl.xml | 73 ++
>> Documentation/DocBook/media/v4l/cec-func-open.xml | 94 ++
>> Documentation/DocBook/media/v4l/cec-func-poll.xml | 89 ++
>> .../DocBook/media/v4l/cec-ioc-g-adap-log-addrs.xml | 275 +++++
>> .../DocBook/media/v4l/cec-ioc-g-adap-phys-addr.xml | 78 ++
>> .../DocBook/media/v4l/cec-ioc-g-adap-state.xml | 87 ++
>> Documentation/DocBook/media/v4l/cec-ioc-g-caps.xml | 173 +++
>> .../DocBook/media/v4l/cec-ioc-g-event.xml | 125 ++
>> .../DocBook/media/v4l/cec-ioc-g-passthrough.xml | 88 ++
>> .../DocBook/media/v4l/cec-ioc-g-vendor-id.xml | 70 ++
>> .../DocBook/media/v4l/cec-ioc-receive.xml | 185 +++
>> Documentation/DocBook/media_api.tmpl | 6 +-
>> Documentation/cec.txt | 165 +++
>> .../devicetree/bindings/media/s5p-cec.txt | 33 +
>> arch/arm/boot/dts/exynos4.dtsi | 12 +
>> arch/arm/boot/dts/exynos4210-pinctrl.dtsi | 7 +
>> arch/arm/boot/dts/exynos4412-odroid-common.dtsi | 4 +
>> arch/arm/boot/dts/exynos4x12-pinctrl.dtsi | 7 +
>> drivers/media/Kconfig | 6 +
>> drivers/media/Makefile | 2 +
>> drivers/media/cec.c | 1191 ++++++++++++++++++++
>> drivers/media/i2c/adv7511.c | 347 +++++-
>> drivers/media/i2c/adv7604.c | 207 +++-
>> drivers/media/platform/Kconfig | 10 +
>> drivers/media/platform/Makefile | 1 +
>> drivers/media/platform/s5p-cec/Makefile | 4 +
>> drivers/media/platform/s5p-cec/exynos_hdmi_cec.h | 37 +
>> .../media/platform/s5p-cec/exynos_hdmi_cecctrl.c | 208 ++++
>> drivers/media/platform/s5p-cec/regs-cec.h | 96 ++
>> drivers/media/platform/s5p-cec/s5p_cec.c | 283 +++++
>> drivers/media/platform/s5p-cec/s5p_cec.h | 76 ++
>> drivers/media/rc/keymaps/Makefile | 1 +
>> drivers/media/rc/keymaps/rc-cec.c | 144 +++
>> drivers/media/rc/rc-main.c | 1 +
>> include/media/adv7511.h | 6 +-
>> include/media/cec.h | 142 +++
>> include/media/rc-core.h | 1 +
>> include/media/rc-map.h | 5 +-
>> include/media/v4l2-subdev.h | 8 +
>> include/uapi/linux/Kbuild | 1 +
>> include/uapi/linux/cec.h | 332 ++++++
>> include/uapi/linux/input.h | 12 +
>> 46 files changed, 4824 insertions(+), 15 deletions(-)
>> create mode 100644 Documentation/DocBook/media/v4l/cec-api.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-func-close.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-func-ioctl.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-func-open.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-func-poll.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-log-addrs.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-phys-addr.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-adap-state.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-caps.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-event.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-passthrough.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-g-vendor-id.xml
>> create mode 100644 Documentation/DocBook/media/v4l/cec-ioc-receive.xml
>> create mode 100644 Documentation/cec.txt
>> create mode 100644 Documentation/devicetree/bindings/media/s5p-cec.txt
>> create mode 100644 drivers/media/cec.c
>> create mode 100644 drivers/media/platform/s5p-cec/Makefile
>> create mode 100644 drivers/media/platform/s5p-cec/exynos_hdmi_cec.h
>> create mode 100644 drivers/media/platform/s5p-cec/exynos_hdmi_cecctrl.c
>> create mode 100644 drivers/media/platform/s5p-cec/regs-cec.h
>> create mode 100644 drivers/media/platform/s5p-cec/s5p_cec.c
>> create mode 100644 drivers/media/platform/s5p-cec/s5p_cec.h
>> create mode 100644 drivers/media/rc/keymaps/rc-cec.c
>> create mode 100644 include/media/cec.h
>> create mode 100644 include/uapi/linux/cec.h
>>
>
> --
> To unsubscribe from this list: send the line "unsubscribe linux-media" in
> the body of a message to majordomo@vger.kernel.org
> More majordomo info at http://vger.kernel.org/majordomo-info.html
>