Commit 45d96999 authored by Lyude Paul's avatar Lyude Paul

drm/dp: Fixup kernel docs for struct drm_dp_aux

* Make sure that struct members are referred to using @, otherwise they
  won't be formatted as such
* Make sure to refer to other struct types using & so they link back to
  each struct's definition
* Make sure to precede constant values with % so they're formatted
  correctly
Signed-off-by: default avatarLyude Paul <lyude@redhat.com>
Acked-by: default avatarRandy Dunlap <rdunlap@infradead.org>
Link: https://patchwork.freedesktop.org/patch/msgid/20210326203807.105754-2-lyude@redhat.com
parent be318fd8
...@@ -1839,34 +1839,34 @@ struct drm_dp_aux_cec { ...@@ -1839,34 +1839,34 @@ struct drm_dp_aux_cec {
* @crc_count: counter of captured frame CRCs * @crc_count: counter of captured frame CRCs
* @transfer: transfers a message representing a single AUX transaction * @transfer: transfers a message representing a single AUX transaction
* *
* The .dev field should be set to a pointer to the device that implements * The @dev field should be set to a pointer to the device that implements the
* the AUX channel. * AUX channel.
* *
* The .name field may be used to specify the name of the I2C adapter. If set to * The @name field may be used to specify the name of the I2C adapter. If set to
* NULL, dev_name() of .dev will be used. * %NULL, dev_name() of @dev will be used.
* *
* Drivers provide a hardware-specific implementation of how transactions * Drivers provide a hardware-specific implementation of how transactions are
* are executed via the .transfer() function. A pointer to a drm_dp_aux_msg * executed via the @transfer() function. A pointer to a &drm_dp_aux_msg
* structure describing the transaction is passed into this function. Upon * structure describing the transaction is passed into this function. Upon
* success, the implementation should return the number of payload bytes * success, the implementation should return the number of payload bytes that
* that were transferred, or a negative error-code on failure. Helpers * were transferred, or a negative error-code on failure. Helpers propagate
* propagate errors from the .transfer() function, with the exception of * errors from the @transfer() function, with the exception of the %-EBUSY
* the -EBUSY error, which causes a transaction to be retried. On a short, * error, which causes a transaction to be retried. On a short, helpers will
* helpers will return -EPROTO to make it simpler to check for failure. * return %-EPROTO to make it simpler to check for failure.
* *
* An AUX channel can also be used to transport I2C messages to a sink. A * An AUX channel can also be used to transport I2C messages to a sink. A
* typical application of that is to access an EDID that's present in the * typical application of that is to access an EDID that's present in the sink
* sink device. The .transfer() function can also be used to execute such * device. The @transfer() function can also be used to execute such
* transactions. The drm_dp_aux_register() function registers an I2C * transactions. The drm_dp_aux_register() function registers an I2C adapter
* adapter that can be passed to drm_probe_ddc(). Upon removal, drivers * that can be passed to drm_probe_ddc(). Upon removal, drivers should call
* should call drm_dp_aux_unregister() to remove the I2C adapter. * drm_dp_aux_unregister() to remove the I2C adapter. The I2C adapter uses long
* The I2C adapter uses long transfers by default; if a partial response is * transfers by default; if a partial response is received, the adapter will
* received, the adapter will drop down to the size given by the partial * drop down to the size given by the partial response for this transaction
* response for this transaction only. * only.
* *
* Note that the aux helper code assumes that the .transfer() function * Note that the aux helper code assumes that the @transfer() function only
* only modifies the reply field of the drm_dp_aux_msg structure. The * modifies the reply field of the &drm_dp_aux_msg structure. The retry logic
* retry logic and i2c helpers assume this is the case. * and i2c helpers assume this is the case.
*/ */
struct drm_dp_aux { struct drm_dp_aux {
const char *name; const char *name;
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment