* [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct
@ 2015-10-12 8:07 David Henningsson
2015-10-12 8:17 ` David Henningsson
0 siblings, 1 reply; 5+ messages in thread
From: David Henningsson @ 2015-10-12 8:07 UTC (permalink / raw)
To: tiwai, intel-gfx, alsa-devel, daniel, jani.nikula, libin.yang
Cc: David Henningsson
To make kernel-doc happy, the i915_audio_component_audio_ops struct
cannot be nested.
Signed-off-by: David Henningsson <david.henningsson@canonical.com>
---
Documentation/DocBook/drm.tmpl | 1 +
include/drm/i915_component.h | 92 ++++++++++++++++++++++++++++++------------
2 files changed, 67 insertions(+), 26 deletions(-)
diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
index 9ddf8c6..f16e4e8 100644
--- a/Documentation/DocBook/drm.tmpl
+++ b/Documentation/DocBook/drm.tmpl
@@ -4051,6 +4051,7 @@ int num_ioctls;</synopsis>
<title>High Definition Audio</title>
!Pdrivers/gpu/drm/i915/intel_audio.c High Definition Audio over HDMI and Display Port
!Idrivers/gpu/drm/i915/intel_audio.c
+!Iinclude/drm/i915_component.h
</sect2>
<sect2>
<title>Panel Self Refresh PSR (PSR/SRD)</title>
diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h
index 89dc7d6..76c10c8 100644
--- a/include/drm/i915_component.h
+++ b/include/drm/i915_component.h
@@ -30,38 +30,78 @@
*/
#define MAX_PORTS 5
+/**
+ * struct i915_audio_component_ops - Ops implemented by i915 driver, called by hda driver
+ */
+struct i915_audio_component_ops {
+ /**
+ * @owner: i915 module
+ */
+ struct module *owner;
+ /**
+ * @get_power: Request that power well is to be turned on
+ */
+ void (*get_power)(struct device *);
+ /**
+ * @put_power: Allow the power well to be turned off
+ */
+ void (*put_power)(struct device *);
+ /**
+ * @codec_wake_override: Force the audio codec to stay awake
+ */
+ void (*codec_wake_override)(struct device *, bool enable);
+ /**
+ * @get_cdclk_freq: Query the i915 driver about the current cdclk frequency
+ */
+ int (*get_cdclk_freq)(struct device *);
+ /**
+ * @sync_audio_rate: set n/cts based on the sample rate
+ *
+ * Called from audio driver. After audio driver sets the
+ * sample rate, it will call this function to set n/cts
+ */
+ int (*sync_audio_rate)(struct device *, int port, int rate);
+};
+
+/**
+ * struct i915_audio_component_audio_ops - Ops implemented by hda driver, called by i915 driver
+ */
+struct i915_audio_component_audio_ops {
+ /**
+ * @audio_ptr: Pointer to be used in call to pin_eld_notify
+ */
+ void *audio_ptr;
+ /**
+ * @pin_eld_notify: Notify the HDA driver that pin sense and/or ELD information has changed
+ *
+ * Called when the i915 driver has set up audio pipeline or has just
+ * begun to tear it down. This allows the HDA driver to update its
+ * status accordingly (even when the HDA controller is in power save
+ * mode).
+ */
+ void (*pin_eld_notify)(void *audio_ptr, int port);
+};
+
+/**
+ * struct i915_audio_component - Used for direct communication between i915 and hda drivers
+ */
struct i915_audio_component {
+ /**
+ * @dev: i915 device, used as parameter for ops
+ */
struct device *dev;
/**
* @aud_sample_rate: the array of audio sample rate per port
*/
int aud_sample_rate[MAX_PORTS];
-
- const struct i915_audio_component_ops {
- struct module *owner;
- void (*get_power)(struct device *);
- void (*put_power)(struct device *);
- void (*codec_wake_override)(struct device *, bool enable);
- int (*get_cdclk_freq)(struct device *);
- /**
- * @sync_audio_rate: set n/cts based on the sample rate
- *
- * Called from audio driver. After audio driver sets the
- * sample rate, it will call this function to set n/cts
- */
- int (*sync_audio_rate)(struct device *, int port, int rate);
- } *ops;
-
- const struct i915_audio_component_audio_ops {
- void *audio_ptr;
- /**
- * Call from i915 driver, notifying the HDA driver that
- * pin sense and/or ELD information has changed.
- * @audio_ptr: HDA driver object
- * @port: Which port has changed (PORTA / PORTB / PORTC etc)
- */
- void (*pin_eld_notify)(void *audio_ptr, int port);
- } *audio_ops;
+ /**
+ * @ops: Ops implemented by i915 driver, called by hda driver
+ */
+ const struct i915_audio_component_ops *ops;
+ /**
+ * @audio_ops: Ops implemented by hda driver, called by i915 driver
+ */
+ const struct i915_audio_component_audio_ops *audio_ops;
};
#endif /* _I915_COMPONENT_H_ */
--
1.9.1
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply related [flat|nested] 5+ messages in thread* Re: [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct
2015-10-12 8:07 [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct David Henningsson
@ 2015-10-12 8:17 ` David Henningsson
2015-10-13 9:40 ` Takashi Iwai
0 siblings, 1 reply; 5+ messages in thread
From: David Henningsson @ 2015-10-12 8:17 UTC (permalink / raw)
To: tiwai, intel-gfx, alsa-devel, daniel, jani.nikula, libin.yang
On 2015-10-12 10:07, David Henningsson wrote:
> To make kernel-doc happy, the i915_audio_component_audio_ops struct
> cannot be nested.
>
> Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> ---
Changes since v1:
* Added a notice about when pin_eld_notify is called
* Uses new inline struct member style
Verified with "make htmldocs", looks fine to me.
Also, applies to Takashi's tree (master branch), maybe we should take it
through Takashi's tree to avoid conflicts, especially as there might be
more stuff coming that way (dp mst support from Mengdong, and perhaps
info retrieval from me).
> Documentation/DocBook/drm.tmpl | 1 +
> include/drm/i915_component.h | 92 ++++++++++++++++++++++++++++++------------
> 2 files changed, 67 insertions(+), 26 deletions(-)
>
> diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
> index 9ddf8c6..f16e4e8 100644
> --- a/Documentation/DocBook/drm.tmpl
> +++ b/Documentation/DocBook/drm.tmpl
> @@ -4051,6 +4051,7 @@ int num_ioctls;</synopsis>
> <title>High Definition Audio</title>
> !Pdrivers/gpu/drm/i915/intel_audio.c High Definition Audio over HDMI and Display Port
> !Idrivers/gpu/drm/i915/intel_audio.c
> +!Iinclude/drm/i915_component.h
> </sect2>
> <sect2>
> <title>Panel Self Refresh PSR (PSR/SRD)</title>
> diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h
> index 89dc7d6..76c10c8 100644
> --- a/include/drm/i915_component.h
> +++ b/include/drm/i915_component.h
> @@ -30,38 +30,78 @@
> */
> #define MAX_PORTS 5
>
> +/**
> + * struct i915_audio_component_ops - Ops implemented by i915 driver, called by hda driver
> + */
> +struct i915_audio_component_ops {
> + /**
> + * @owner: i915 module
> + */
> + struct module *owner;
> + /**
> + * @get_power: Request that power well is to be turned on
> + */
> + void (*get_power)(struct device *);
> + /**
> + * @put_power: Allow the power well to be turned off
> + */
> + void (*put_power)(struct device *);
> + /**
> + * @codec_wake_override: Force the audio codec to stay awake
> + */
> + void (*codec_wake_override)(struct device *, bool enable);
> + /**
> + * @get_cdclk_freq: Query the i915 driver about the current cdclk frequency
> + */
> + int (*get_cdclk_freq)(struct device *);
> + /**
> + * @sync_audio_rate: set n/cts based on the sample rate
> + *
> + * Called from audio driver. After audio driver sets the
> + * sample rate, it will call this function to set n/cts
> + */
> + int (*sync_audio_rate)(struct device *, int port, int rate);
> +};
> +
> +/**
> + * struct i915_audio_component_audio_ops - Ops implemented by hda driver, called by i915 driver
> + */
> +struct i915_audio_component_audio_ops {
> + /**
> + * @audio_ptr: Pointer to be used in call to pin_eld_notify
> + */
> + void *audio_ptr;
> + /**
> + * @pin_eld_notify: Notify the HDA driver that pin sense and/or ELD information has changed
> + *
> + * Called when the i915 driver has set up audio pipeline or has just
> + * begun to tear it down. This allows the HDA driver to update its
> + * status accordingly (even when the HDA controller is in power save
> + * mode).
> + */
> + void (*pin_eld_notify)(void *audio_ptr, int port);
> +};
> +
> +/**
> + * struct i915_audio_component - Used for direct communication between i915 and hda drivers
> + */
> struct i915_audio_component {
> + /**
> + * @dev: i915 device, used as parameter for ops
> + */
> struct device *dev;
> /**
> * @aud_sample_rate: the array of audio sample rate per port
> */
> int aud_sample_rate[MAX_PORTS];
> -
> - const struct i915_audio_component_ops {
> - struct module *owner;
> - void (*get_power)(struct device *);
> - void (*put_power)(struct device *);
> - void (*codec_wake_override)(struct device *, bool enable);
> - int (*get_cdclk_freq)(struct device *);
> - /**
> - * @sync_audio_rate: set n/cts based on the sample rate
> - *
> - * Called from audio driver. After audio driver sets the
> - * sample rate, it will call this function to set n/cts
> - */
> - int (*sync_audio_rate)(struct device *, int port, int rate);
> - } *ops;
> -
> - const struct i915_audio_component_audio_ops {
> - void *audio_ptr;
> - /**
> - * Call from i915 driver, notifying the HDA driver that
> - * pin sense and/or ELD information has changed.
> - * @audio_ptr: HDA driver object
> - * @port: Which port has changed (PORTA / PORTB / PORTC etc)
> - */
> - void (*pin_eld_notify)(void *audio_ptr, int port);
> - } *audio_ops;
> + /**
> + * @ops: Ops implemented by i915 driver, called by hda driver
> + */
> + const struct i915_audio_component_ops *ops;
> + /**
> + * @audio_ops: Ops implemented by hda driver, called by i915 driver
> + */
> + const struct i915_audio_component_audio_ops *audio_ops;
> };
>
> #endif /* _I915_COMPONENT_H_ */
>
--
David Henningsson, Canonical Ltd.
https://launchpad.net/~diwic
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct
2015-10-12 8:17 ` David Henningsson
@ 2015-10-13 9:40 ` Takashi Iwai
2015-10-13 15:39 ` Daniel Vetter
0 siblings, 1 reply; 5+ messages in thread
From: Takashi Iwai @ 2015-10-13 9:40 UTC (permalink / raw)
To: David Henningsson; +Cc: intel-gfx, alsa-devel
On Mon, 12 Oct 2015 10:17:51 +0200,
David Henningsson wrote:
>
>
>
> On 2015-10-12 10:07, David Henningsson wrote:
> > To make kernel-doc happy, the i915_audio_component_audio_ops struct
> > cannot be nested.
> >
> > Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> > ---
>
> Changes since v1:
>
> * Added a notice about when pin_eld_notify is called
>
> * Uses new inline struct member style
>
> Verified with "make htmldocs", looks fine to me.
>
> Also, applies to Takashi's tree (master branch), maybe we should take it
> through Takashi's tree to avoid conflicts, especially as there might be
> more stuff coming that way (dp mst support from Mengdong, and perhaps
> info retrieval from me).
I'm fine to take it, just let me know (and give acks).
thanks,
Takashi
>
>
> > Documentation/DocBook/drm.tmpl | 1 +
> > include/drm/i915_component.h | 92 ++++++++++++++++++++++++++++++------------
> > 2 files changed, 67 insertions(+), 26 deletions(-)
> >
> > diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
> > index 9ddf8c6..f16e4e8 100644
> > --- a/Documentation/DocBook/drm.tmpl
> > +++ b/Documentation/DocBook/drm.tmpl
> > @@ -4051,6 +4051,7 @@ int num_ioctls;</synopsis>
> > <title>High Definition Audio</title>
> > !Pdrivers/gpu/drm/i915/intel_audio.c High Definition Audio over HDMI and Display Port
> > !Idrivers/gpu/drm/i915/intel_audio.c
> > +!Iinclude/drm/i915_component.h
> > </sect2>
> > <sect2>
> > <title>Panel Self Refresh PSR (PSR/SRD)</title>
> > diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h
> > index 89dc7d6..76c10c8 100644
> > --- a/include/drm/i915_component.h
> > +++ b/include/drm/i915_component.h
> > @@ -30,38 +30,78 @@
> > */
> > #define MAX_PORTS 5
> >
> > +/**
> > + * struct i915_audio_component_ops - Ops implemented by i915 driver, called by hda driver
> > + */
> > +struct i915_audio_component_ops {
> > + /**
> > + * @owner: i915 module
> > + */
> > + struct module *owner;
> > + /**
> > + * @get_power: Request that power well is to be turned on
> > + */
> > + void (*get_power)(struct device *);
> > + /**
> > + * @put_power: Allow the power well to be turned off
> > + */
> > + void (*put_power)(struct device *);
> > + /**
> > + * @codec_wake_override: Force the audio codec to stay awake
> > + */
> > + void (*codec_wake_override)(struct device *, bool enable);
> > + /**
> > + * @get_cdclk_freq: Query the i915 driver about the current cdclk frequency
> > + */
> > + int (*get_cdclk_freq)(struct device *);
> > + /**
> > + * @sync_audio_rate: set n/cts based on the sample rate
> > + *
> > + * Called from audio driver. After audio driver sets the
> > + * sample rate, it will call this function to set n/cts
> > + */
> > + int (*sync_audio_rate)(struct device *, int port, int rate);
> > +};
> > +
> > +/**
> > + * struct i915_audio_component_audio_ops - Ops implemented by hda driver, called by i915 driver
> > + */
> > +struct i915_audio_component_audio_ops {
> > + /**
> > + * @audio_ptr: Pointer to be used in call to pin_eld_notify
> > + */
> > + void *audio_ptr;
> > + /**
> > + * @pin_eld_notify: Notify the HDA driver that pin sense and/or ELD information has changed
> > + *
> > + * Called when the i915 driver has set up audio pipeline or has just
> > + * begun to tear it down. This allows the HDA driver to update its
> > + * status accordingly (even when the HDA controller is in power save
> > + * mode).
> > + */
> > + void (*pin_eld_notify)(void *audio_ptr, int port);
> > +};
> > +
> > +/**
> > + * struct i915_audio_component - Used for direct communication between i915 and hda drivers
> > + */
> > struct i915_audio_component {
> > + /**
> > + * @dev: i915 device, used as parameter for ops
> > + */
> > struct device *dev;
> > /**
> > * @aud_sample_rate: the array of audio sample rate per port
> > */
> > int aud_sample_rate[MAX_PORTS];
> > -
> > - const struct i915_audio_component_ops {
> > - struct module *owner;
> > - void (*get_power)(struct device *);
> > - void (*put_power)(struct device *);
> > - void (*codec_wake_override)(struct device *, bool enable);
> > - int (*get_cdclk_freq)(struct device *);
> > - /**
> > - * @sync_audio_rate: set n/cts based on the sample rate
> > - *
> > - * Called from audio driver. After audio driver sets the
> > - * sample rate, it will call this function to set n/cts
> > - */
> > - int (*sync_audio_rate)(struct device *, int port, int rate);
> > - } *ops;
> > -
> > - const struct i915_audio_component_audio_ops {
> > - void *audio_ptr;
> > - /**
> > - * Call from i915 driver, notifying the HDA driver that
> > - * pin sense and/or ELD information has changed.
> > - * @audio_ptr: HDA driver object
> > - * @port: Which port has changed (PORTA / PORTB / PORTC etc)
> > - */
> > - void (*pin_eld_notify)(void *audio_ptr, int port);
> > - } *audio_ops;
> > + /**
> > + * @ops: Ops implemented by i915 driver, called by hda driver
> > + */
> > + const struct i915_audio_component_ops *ops;
> > + /**
> > + * @audio_ops: Ops implemented by hda driver, called by i915 driver
> > + */
> > + const struct i915_audio_component_audio_ops *audio_ops;
> > };
> >
> > #endif /* _I915_COMPONENT_H_ */
> >
>
> --
> David Henningsson, Canonical Ltd.
> https://launchpad.net/~diwic
>
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct
2015-10-13 9:40 ` Takashi Iwai
@ 2015-10-13 15:39 ` Daniel Vetter
2015-10-13 15:41 ` Daniel Vetter
0 siblings, 1 reply; 5+ messages in thread
From: Daniel Vetter @ 2015-10-13 15:39 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel, intel-gfx, David Henningsson
On Tue, Oct 13, 2015 at 11:40:23AM +0200, Takashi Iwai wrote:
> On Mon, 12 Oct 2015 10:17:51 +0200,
> David Henningsson wrote:
> >
> >
> >
> > On 2015-10-12 10:07, David Henningsson wrote:
> > > To make kernel-doc happy, the i915_audio_component_audio_ops struct
> > > cannot be nested.
> > >
> > > Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> > > ---
> >
> > Changes since v1:
> >
> > * Added a notice about when pin_eld_notify is called
> >
> > * Uses new inline struct member style
> >
> > Verified with "make htmldocs", looks fine to me.
> >
> > Also, applies to Takashi's tree (master branch), maybe we should take it
> > through Takashi's tree to avoid conflicts, especially as there might be
> > more stuff coming that way (dp mst support from Mengdong, and perhaps
> > info retrieval from me).
>
> I'm fine to take it, just let me know (and give acks).
There's other doc patches floating around for this, and I've already
pulled in your branch. Soproabbyl simpler if I pull this into
drm-intel.git
-Daniel
>
>
> thanks,
>
> Takashi
>
>
> >
> >
> > > Documentation/DocBook/drm.tmpl | 1 +
> > > include/drm/i915_component.h | 92 ++++++++++++++++++++++++++++++------------
> > > 2 files changed, 67 insertions(+), 26 deletions(-)
> > >
> > > diff --git a/Documentation/DocBook/drm.tmpl b/Documentation/DocBook/drm.tmpl
> > > index 9ddf8c6..f16e4e8 100644
> > > --- a/Documentation/DocBook/drm.tmpl
> > > +++ b/Documentation/DocBook/drm.tmpl
> > > @@ -4051,6 +4051,7 @@ int num_ioctls;</synopsis>
> > > <title>High Definition Audio</title>
> > > !Pdrivers/gpu/drm/i915/intel_audio.c High Definition Audio over HDMI and Display Port
> > > !Idrivers/gpu/drm/i915/intel_audio.c
> > > +!Iinclude/drm/i915_component.h
> > > </sect2>
> > > <sect2>
> > > <title>Panel Self Refresh PSR (PSR/SRD)</title>
> > > diff --git a/include/drm/i915_component.h b/include/drm/i915_component.h
> > > index 89dc7d6..76c10c8 100644
> > > --- a/include/drm/i915_component.h
> > > +++ b/include/drm/i915_component.h
> > > @@ -30,38 +30,78 @@
> > > */
> > > #define MAX_PORTS 5
> > >
> > > +/**
> > > + * struct i915_audio_component_ops - Ops implemented by i915 driver, called by hda driver
> > > + */
> > > +struct i915_audio_component_ops {
> > > + /**
> > > + * @owner: i915 module
> > > + */
> > > + struct module *owner;
> > > + /**
> > > + * @get_power: Request that power well is to be turned on
> > > + */
> > > + void (*get_power)(struct device *);
> > > + /**
> > > + * @put_power: Allow the power well to be turned off
> > > + */
> > > + void (*put_power)(struct device *);
> > > + /**
> > > + * @codec_wake_override: Force the audio codec to stay awake
> > > + */
> > > + void (*codec_wake_override)(struct device *, bool enable);
> > > + /**
> > > + * @get_cdclk_freq: Query the i915 driver about the current cdclk frequency
> > > + */
> > > + int (*get_cdclk_freq)(struct device *);
> > > + /**
> > > + * @sync_audio_rate: set n/cts based on the sample rate
> > > + *
> > > + * Called from audio driver. After audio driver sets the
> > > + * sample rate, it will call this function to set n/cts
> > > + */
> > > + int (*sync_audio_rate)(struct device *, int port, int rate);
> > > +};
> > > +
> > > +/**
> > > + * struct i915_audio_component_audio_ops - Ops implemented by hda driver, called by i915 driver
> > > + */
> > > +struct i915_audio_component_audio_ops {
> > > + /**
> > > + * @audio_ptr: Pointer to be used in call to pin_eld_notify
> > > + */
> > > + void *audio_ptr;
> > > + /**
> > > + * @pin_eld_notify: Notify the HDA driver that pin sense and/or ELD information has changed
> > > + *
> > > + * Called when the i915 driver has set up audio pipeline or has just
> > > + * begun to tear it down. This allows the HDA driver to update its
> > > + * status accordingly (even when the HDA controller is in power save
> > > + * mode).
> > > + */
> > > + void (*pin_eld_notify)(void *audio_ptr, int port);
> > > +};
> > > +
> > > +/**
> > > + * struct i915_audio_component - Used for direct communication between i915 and hda drivers
> > > + */
> > > struct i915_audio_component {
> > > + /**
> > > + * @dev: i915 device, used as parameter for ops
> > > + */
> > > struct device *dev;
> > > /**
> > > * @aud_sample_rate: the array of audio sample rate per port
> > > */
> > > int aud_sample_rate[MAX_PORTS];
> > > -
> > > - const struct i915_audio_component_ops {
> > > - struct module *owner;
> > > - void (*get_power)(struct device *);
> > > - void (*put_power)(struct device *);
> > > - void (*codec_wake_override)(struct device *, bool enable);
> > > - int (*get_cdclk_freq)(struct device *);
> > > - /**
> > > - * @sync_audio_rate: set n/cts based on the sample rate
> > > - *
> > > - * Called from audio driver. After audio driver sets the
> > > - * sample rate, it will call this function to set n/cts
> > > - */
> > > - int (*sync_audio_rate)(struct device *, int port, int rate);
> > > - } *ops;
> > > -
> > > - const struct i915_audio_component_audio_ops {
> > > - void *audio_ptr;
> > > - /**
> > > - * Call from i915 driver, notifying the HDA driver that
> > > - * pin sense and/or ELD information has changed.
> > > - * @audio_ptr: HDA driver object
> > > - * @port: Which port has changed (PORTA / PORTB / PORTC etc)
> > > - */
> > > - void (*pin_eld_notify)(void *audio_ptr, int port);
> > > - } *audio_ops;
> > > + /**
> > > + * @ops: Ops implemented by i915 driver, called by hda driver
> > > + */
> > > + const struct i915_audio_component_ops *ops;
> > > + /**
> > > + * @audio_ops: Ops implemented by hda driver, called by i915 driver
> > > + */
> > > + const struct i915_audio_component_audio_ops *audio_ops;
> > > };
> > >
> > > #endif /* _I915_COMPONENT_H_ */
> > >
> >
> > --
> > David Henningsson, Canonical Ltd.
> > https://launchpad.net/~diwic
> >
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread* Re: [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct
2015-10-13 15:39 ` Daniel Vetter
@ 2015-10-13 15:41 ` Daniel Vetter
0 siblings, 0 replies; 5+ messages in thread
From: Daniel Vetter @ 2015-10-13 15:41 UTC (permalink / raw)
To: Takashi Iwai; +Cc: alsa-devel, intel-gfx, David Henningsson
On Tue, Oct 13, 2015 at 05:39:18PM +0200, Daniel Vetter wrote:
> On Tue, Oct 13, 2015 at 11:40:23AM +0200, Takashi Iwai wrote:
> > On Mon, 12 Oct 2015 10:17:51 +0200,
> > David Henningsson wrote:
> > >
> > >
> > >
> > > On 2015-10-12 10:07, David Henningsson wrote:
> > > > To make kernel-doc happy, the i915_audio_component_audio_ops struct
> > > > cannot be nested.
> > > >
> > > > Signed-off-by: David Henningsson <david.henningsson@canonical.com>
> > > > ---
> > >
> > > Changes since v1:
> > >
> > > * Added a notice about when pin_eld_notify is called
> > >
> > > * Uses new inline struct member style
> > >
> > > Verified with "make htmldocs", looks fine to me.
> > >
> > > Also, applies to Takashi's tree (master branch), maybe we should take it
> > > through Takashi's tree to avoid conflicts, especially as there might be
> > > more stuff coming that way (dp mst support from Mengdong, and perhaps
> > > info retrieval from me).
> >
> > I'm fine to take it, just let me know (and give acks).
>
> There's other doc patches floating around for this, and I've already
> pulled in your branch. Soproabbyl simpler if I pull this into
> drm-intel.git
Worse, it conflicts all over with the docbook I have already. But I like
yours better, so can you please rebase and merge with what we have?
Thanks, Daniel
--
Daniel Vetter
Software Engineer, Intel Corporation
http://blog.ffwll.ch
_______________________________________________
Intel-gfx mailing list
Intel-gfx@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/intel-gfx
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2015-10-13 15:38 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-10-12 8:07 [PATCH v2] drm/i915: Improve kernel-doc for i915_audio_component struct David Henningsson
2015-10-12 8:17 ` David Henningsson
2015-10-13 9:40 ` Takashi Iwai
2015-10-13 15:39 ` Daniel Vetter
2015-10-13 15:41 ` Daniel Vetter
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox