All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@s-opensource.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org, dri-devel@lists.freedesktop.org,
	linux-samsung-soc@vger.kernel.org, linux-input@vger.kernel.org,
	lars@opdenkamp.eu, linux@arm.linux.org.uk,
	Hans Verkuil <hansverk@cisco.com>, Kamil Debski <kamil@wypas.org>,
	Hans Verkuil <hans.verkuil@cisco.com>
Subject: Re: [PATCHv16 07/13] cec.txt: add CEC framework documentation
Date: Thu, 16 Jun 2016 17:12:44 -0300	[thread overview]
Message-ID: <20160616171244.3ef5a7b4@recife.lan> (raw)
In-Reply-To: <1461937948-22936-8-git-send-email-hverkuil@xs4all.nl>

Em Fri, 29 Apr 2016 15:52:22 +0200
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> From: Hans Verkuil <hansverk@cisco.com>
> 
> Document the new HDMI CEC framework.

As we'll be moving documentation to Sphinx/Rst, it would be good if
you could make it work fine with sphinx, as this will likely be needed
for Kernel 4.9. Right now it most works, although several warnings are
produced: 

/devel/v4l/patchwork/tmp/cec.txt:40: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:40: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:40: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:40: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:40: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:42: WARNING: Definition list ends without a blank line; unexpected unindent.
/devel/v4l/patchwork/tmp/cec.txt:42: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:67: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:71: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:92: ERROR: Unexpected indentation.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:87: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:92: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:93: WARNING: Block quote ends without a blank line; unexpected unindent.
/devel/v4l/patchwork/tmp/cec.txt:93: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:93: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:95: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:97: WARNING: Definition list ends without a blank line; unexpected unindent.
/devel/v4l/patchwork/tmp/cec.txt:105: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:105: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:118: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:118: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:130: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:130: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:144: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:144: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:144: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:161: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:161: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:161: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:173: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:194: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:203: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:203: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:214: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:217: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:217: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:217: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:217: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:219: WARNING: Definition list ends without a blank line; unexpected unindent.
/devel/v4l/patchwork/tmp/cec.txt:224: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:224: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:224: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:238: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:238: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:243: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:258: WARNING: Inline emphasis start-string without end-string.
/devel/v4l/patchwork/tmp/cec.txt:258: WARNING: Inline emphasis start-string without end-string.


> 
> Signed-off-by: Hans Verkuil <hansverk@cisco.com>
> [k.debski@samsung.com: add DocBook documentation by Hans Verkuil, with
> Signed-off-by: Kamil Debski <kamil@wypas.org>
> Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com>
> ---
>  Documentation/cec.txt | 267 ++++++++++++++++++++++++++++++++++++++++++++++++++
>  1 file changed, 267 insertions(+)
>  create mode 100644 Documentation/cec.txt
> 
> diff --git a/Documentation/cec.txt b/Documentation/cec.txt
> new file mode 100644
> index 0000000..75155fe
> --- /dev/null
> +++ b/Documentation/cec.txt
> @@ -0,0 +1,267 @@
> +CEC Kernel Support
> +==================
> +
> +The CEC framework provides a unified kernel interface for use with HDMI CEC
> +hardware. It is designed to handle a multiple types of hardware (receivers,
> +transmitters, USB dongles). The framework also gives the option to decide
> +what to do in the kernel driver and what should be handled by userspace
> +applications. In addition it integrates the remote control passthrough
> +feature into the kernel's remote control framework.
> +
> +
> +The CEC Protocol
> +----------------
> +
> +The CEC protocol enables consumer electronic devices to communicate with each
> +other through the HDMI connection. The protocol uses logical addresses in the
> +communication. The logical address is strictly connected with the functionality
> +provided by the device. The TV acting as the communication hub is always
> +assigned address 0. The physical address is determined by the physical
> +connection between devices.
> +
> +The CEC framework described here is up to date with the CEC 2.0 specification.
> +It is documented in the HDMI 1.4 specification with the new 2.0 bits documented
> +in the HDMI 2.0 specification. But for most of the features the freely available
> +HDMI 1.3a specification is sufficient:
> +
> +http://www.microprocessor.org/HDMISpecification13a.pdf
> +
> +
> +The Kernel Interface
> +====================
> +
> +CEC Adapter
> +-----------
> +
> +The struct cec_adapter represents the CEC adapter hardware. It is created by
> +calling cec_allocate_adapter() and deleted by calling cec_delete_adapter():
> +
> +struct cec_adapter *cec_allocate_adapter(const struct cec_adap_ops *ops,
> +	       void *priv, const char *name, u32 caps, u8 available_las,
> +	       struct device *parent);
> +void cec_delete_adapter(struct cec_adapter *adap);
> +
> +To create an adapter you need to pass the following information:
> +
> +ops: adapter operations which are called by the CEC framework and that you
> +have to implement.
> +
> +priv: will be stored in adap->priv and can be used by the adapter ops.
> +
> +name: the name of the CEC adapter. Note: this name will be copied.
> +
> +caps: capabilities of the CEC adapter. These capabilities determine the
> +	capabilities of the hardware and which parts are to be handled
> +	by userspace and which parts are handled by kernelspace. The
> +	capabilities are returned by CEC_ADAP_G_CAPS.
> +
> +available_las: the number of simultaneous logical addresses that this
> +	adapter can handle. Must be 1 <= available_las <= CEC_MAX_LOG_ADDRS.
> +
> +parent: the parent device.
> +
> +
> +To register the /dev/cecX device node and the remote control device (if
> +CEC_CAP_RC is set) you call:
> +
> +int cec_register_adapter(struct cec_adapter *adap);
> +
> +To unregister the devices call:
> +
> +void cec_unregister_adapter(struct cec_adapter *adap);
> +
> +Note: if cec_register_adapter() fails, then call cec_delete_adapter() to
> +clean up. But if cec_register_adapter() succeeded, then only call
> +cec_unregister_adapter() to clean up, never cec_delete_adapter(). The
> +unregister function will delete the adapter automatically once the last user
> +of that /dev/cecX device has closed its file handle.
> +
> +
> +Implementing the Low-Level CEC Adapter
> +--------------------------------------
> +
> +The following low-level adapter operations have to be implemented in
> +your driver:
> +
> +struct cec_adap_ops {
> +	/* Low-level callbacks */
> +	int (*adap_enable)(struct cec_adapter *adap, bool enable);
> +	int (*adap_monitor_all_enable)(struct cec_adapter *adap, bool enable);
> +	int (*adap_log_addr)(struct cec_adapter *adap, u8 logical_addr);
> +	int (*adap_transmit)(struct cec_adapter *adap, u8 attempts,
> +			     u32 signal_free_time, struct cec_msg *msg);
> +	void (*adap_log_status)(struct cec_adapter *adap);
> +
> +	/* High-level callbacks */
> +	...
> +};
> +
> +The three low-level ops deal with various aspects of controlling the CEC adapter
> +hardware:
> +
> +
> +To enable/disable the hardware:
> +
> +	int (*adap_enable)(struct cec_adapter *adap, bool enable);
> +
> +This callback enables or disables the CEC hardware. Enabling the CEC hardware
> +means powering it up in a state where no logical addresses are claimed. This
> +op assumes that the physical address (adap->phys_addr) is valid when enable is
> +true and will not change while the CEC adapter remains enabled. The initial
> +state of the CEC adapter after calling cec_allocate_adapter() is disabled.
> +
> +Note that adap_enable must return 0 if enable is false.
> +
> +
> +To enable/disable the 'monitor all' mode:
> +
> +	int (*adap_monitor_all_enable)(struct cec_adapter *adap, bool enable);
> +
> +If enabled, then the adapter should be put in a mode to also monitor messages
> +that not for us. Not all hardware supports this and this function is only
> +called if the CEC_CAP_MONITOR_ALL capability is set. This callback is optional
> +(some hardware may always be in 'monitor all' mode).
> +
> +Note that adap_monitor_all_enable must return 0 if enable is false.
> +
> +
> +To program a new logical address:
> +
> +	int (*adap_log_addr)(struct cec_adapter *adap, u8 logical_addr);
> +
> +If logical_addr == CEC_LOG_ADDR_INVALID then all programmed logical addresses
> +are to be erased. Otherwise the given logical address should be programmed.
> +If the maximum number of available logical addresses is exceeded, then it
> +should return -ENXIO. Once a logical address is programmed the CEC hardware
> +can receive directed messages to that address.
> +
> +Note that adap_log_addr must return 0 if logical_addr is CEC_LOG_ADDR_INVALID.
> +
> +
> +To transmit a new message:
> +
> +	int (*adap_transmit)(struct cec_adapter *adap, u8 attempts,
> +			     u32 signal_free_time, struct cec_msg *msg);
> +
> +This transmits a new message. The attempts argument is the suggested number of
> +attempts for the transmit.
> +
> +The signal_free_time is the number of data bit periods that the adapter should
> +wait when the line is free before attempting to send a message. This value
> +depends on whether this transmit is a retry, a message from a new initiator or
> +a new message for the same initiator. Most hardware will handle this
> +automatically, but in some cases this information is needed.
> +
> +The CEC_FREE_TIME_TO_USEC macro can be used to convert signal_free_time to
> +microseconds (one data bit period is 2.4 ms).
> +
> +
> +To log the current CEC hardware status:
> +
> +	void (*adap_status)(struct cec_adapter *adap, struct seq_file *file);
> +
> +This optional callback can be used to show the status of the CEC hardware.
> +The status is available through debugfs: cat /sys/kernel/debug/cec/cecX/status
> +
> +
> +Your adapter driver will also have to react to events (typically interrupt
> +driven) by calling into the framework in the following situations:
> +
> +When a transmit finished (successfully or otherwise):
> +
> +void cec_transmit_done(struct cec_adapter *adap, u8 status, u8 arb_lost_cnt,
> +		       u8 nack_cnt, u8 low_drive_cnt, u8 error_cnt);
> +
> +The status can be one of:
> +
> +CEC_TX_STATUS_OK: the transmit was successful.
> +CEC_TX_STATUS_ARB_LOST: arbitration was lost: another CEC initiator
> +took control of the CEC line and you lost the arbitration.
> +CEC_TX_STATUS_NACK: the message was nacked (for a directed message) or
> +acked (for a broadcast message). A retransmission is needed.
> +CEC_TX_STATUS_LOW_DRIVE: low drive was detected on the CEC bus. This
> +indicates that a follower detected an error on the bus and requested a
> +retransmission.
> +CEC_TX_STATUS_ERROR: some unspecified error occurred: this can be one of
> +the previous two if the hardware cannot differentiate or something else
> +entirely.
> +CEC_TX_STATUS_MAX_RETRIES: could not transmit the message after
> +trying multiple times. Should only be set by the driver if it has hardware
> +support for retrying messages. If set, then the framework assumes that it
> +doesn't have to make another attempt to transmit the message since the
> +hardware did that already.
> +
> +The *_cnt arguments are the number of error conditions that were seen.
> +This may be 0 if no information is available. Drivers that do not support
> +hardware retry can just set the counter corresponding to the transmit error
> +to 1, if the hardware does support retry then either set these counters to
> +0 if the hardware provides no feedback of which errors occurred and how many
> +times, or fill in the correct values as reported by the hardware.
> +
> +When a CEC message was received:
> +
> +void cec_received_msg(struct cec_adapter *adap, struct cec_msg *msg);
> +
> +Speaks for itself.
> +
> +Implementing the High-Level CEC Adapter
> +---------------------------------------
> +
> +The low-level operations drive the hardware, the high-level operations are
> +CEC protocol driven. The following high-level callbacks are available:
> +
> +struct cec_adap_ops {
> +	/* Low-level callbacks */
> +	...
> +
> +	/* High-level CEC message callback */
> +	int (*received)(struct cec_adapter *adap, struct cec_msg *msg);
> +};
> +
> +The received() callback allows the driver to optionally handle a newly
> +received CEC message
> +
> +	int (*received)(struct cec_adapter *adap, struct cec_msg *msg);
> +
> +If the driver wants to process a CEC message, then it can implement this
> +callback. If it doesn't want to handle this message, then it should return
> +-ENOMSG, otherwise the CEC framework assumes it processed this message and
> +it will not no anything with it.
> +
> +
> +CEC framework functions
> +-----------------------
> +
> +CEC Adapter drivers can call the following CEC framework functions:
> +
> +int cec_transmit_msg(struct cec_adapter *adap, struct cec_msg *msg,
> +		     bool block);
> +
> +Transmit a CEC message. If block is true, then wait until the message has been
> +transmitted, otherwise just queue it and return.
> +
> +void cec_s_phys_addr(struct cec_adapter *adap, u16 phys_addr, bool block);
> +
> +Change the physical address. This function will set adap->phys_addr and
> +send an event if it has changed. If cec_s_log_addrs() has been called and
> +the physical address has become valid, then the CEC framework will start
> +claiming the logical addresses. If block is true, then this function won't
> +return until this process has finished.
> +
> +When the physical address is set to a valid value the CEC adapter will
> +be enabled (see the adap_enable op). When it is set to CEC_PHYS_ADDR_INVALID,
> +then the CEC adapter will be disabled. If you change a valid physical address
> +to another valid physical address, then this function will first set the
> +address to CEC_PHYS_ADDR_INVALID before enabling the new physical address.
> +
> +int cec_s_log_addrs(struct cec_adapter *adap,
> +		    struct cec_log_addrs *log_addrs, bool block);
> +
> +Claim the CEC logical addresses. Should never be called if CEC_CAP_LOG_ADDRS
> +is set. If block is true, then wait until the logical addresses have been
> +claimed, otherwise just queue it and return. To unconfigure all logical
> +addresses call this function with log_addrs set to NULL or with
> +log_addrs->num_log_addrs set to 0. The block argument is ignored when
> +unconfiguring. This function will just return if the physical address is
> +invalid. Once the physical address becomes valid, then the framework will
> +attempt to claim these logical addresses.



Thanks,
Mauro

  reply	other threads:[~2016-06-16 20:12 UTC|newest]

Thread overview: 35+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-29 13:52 [PATCHv16 00/13] HDMI CEC framework Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 01/13] input.h: add BUS_CEC type Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 02/13] HID: add HDMI CEC specific keycodes Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 03/13] rc: Add HDMI CEC protocol handling Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 04/13] cec: add HDMI CEC framework Hans Verkuil
2016-06-16 16:00   ` Mauro Carvalho Chehab
2016-06-25 12:27     ` Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 05/13] cec/TODO: add TODO file so we know why this is still in staging Hans Verkuil
2016-04-29 13:52   ` Hans Verkuil
2016-06-16 19:53   ` Mauro Carvalho Chehab
2016-04-29 13:52 ` [PATCHv16 06/13] cec: add compat32 ioctl support Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 07/13] cec.txt: add CEC framework documentation Hans Verkuil
2016-04-29 13:52   ` Hans Verkuil
2016-06-16 20:12   ` Mauro Carvalho Chehab [this message]
2016-06-17  7:22     ` Hans Verkuil
2016-06-17  8:55       ` Mauro Carvalho Chehab
2016-04-29 13:52 ` [PATCHv16 08/13] DocBook/media: add CEC documentation Hans Verkuil
2016-06-16 21:09   ` Mauro Carvalho Chehab
2016-06-17  7:58     ` Hans Verkuil
2016-06-17  9:50       ` Mauro Carvalho Chehab
2016-06-17 11:09         ` Hans Verkuil
2016-06-17 11:37           ` Mauro Carvalho Chehab
2016-06-18 16:25             ` Dmitry Torokhov
2016-04-29 13:52 ` [PATCHv16 09/13] cec: adv7604: add cec support Hans Verkuil
2016-06-16 21:17   ` Mauro Carvalho Chehab
2016-06-17  8:03     ` Hans Verkuil
2016-06-17  9:53       ` Mauro Carvalho Chehab
2016-04-29 13:52 ` [PATCHv16 10/13] cec: adv7842: " Hans Verkuil
2016-06-16 21:22   ` Mauro Carvalho Chehab
2016-06-17  8:06     ` Hans Verkuil
2016-06-17  8:06       ` Hans Verkuil
2016-06-17 10:12       ` Mauro Carvalho Chehab
2016-04-29 13:52 ` [PATCHv16 11/13] cec: adv7511: " Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 12/13] cec: s5p-cec: Add s5p-cec driver Hans Verkuil
2016-04-29 13:52 ` [PATCHv16 13/13] vivid: add CEC emulation Hans Verkuil

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20160616171244.3ef5a7b4@recife.lan \
    --to=mchehab@s-opensource.com \
    --cc=dri-devel@lists.freedesktop.org \
    --cc=hans.verkuil@cisco.com \
    --cc=hansverk@cisco.com \
    --cc=hverkuil@xs4all.nl \
    --cc=kamil@wypas.org \
    --cc=lars@opdenkamp.eu \
    --cc=linux-input@vger.kernel.org \
    --cc=linux-media@vger.kernel.org \
    --cc=linux-samsung-soc@vger.kernel.org \
    --cc=linux@arm.linux.org.uk \
    --subject='Re: [PATCHv16 07/13] cec.txt: add CEC framework documentation' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.