Re: [PATCH v5 5/8] thunderbolt: Networking state machine

2016-07-28 Thread Lukas Wunner
On Thu, Jul 28, 2016 at 11:15:18AM +0300, Amir Levy wrote: > +static void nhi_handle_notification_msg(struct tbt_nhi_ctxt *nhi_ctxt, > + const u8 *msg) > +{ > + struct port_net_dev *port; > + u8 port_num; > + > +#define INTER_DOMAIN_LINK_SHIFT 0 >

rst htmldocs on kernel.org

2016-08-11 Thread Lukas Wunner
Hi Jonathan, not sure if this is already on your radar or if you're at all the right person to contact, but I just noticed that the gpu htmldocs are gone from https://www.kernel.org/doc/htmldocs/gpu/ Actually all the rst-formatted docs are missing. It would be great if they could be reinstated

Re: rst htmldocs on kernel.org

2016-08-16 Thread Lukas Wunner
Hi Jonathan, [+cc Konstantin] On Mon, Aug 15, 2016 at 04:21:15PM -0600, Jonathan Corbet wrote: > On Thu, 11 Aug 2016 21:13:29 +0200 Lukas Wunner <lu...@wunner.de> wrote: > > not sure if this is already on your radar or if you're at all the > > right person to contac

Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs

2016-10-04 Thread Lukas Wunner
On Tue, Oct 04, 2016 at 02:23:13PM +0200, Lukas Wunner wrote: > On Tue, Oct 04, 2016 at 01:25:23PM +0200, Daniel Vetter wrote: > > On Tue, Oct 4, 2016 at 12:10 PM, Jon Hunter <jonath...@nvidia.com> wrote: > > > Looks like crtc is a errno in the above case. I see this

Re: [PATCH v8 2/4] drm: Add API for capturing frame CRCs

2016-10-04 Thread Lukas Wunner
On Tue, Oct 04, 2016 at 01:25:23PM +0200, Daniel Vetter wrote: > On Tue, Oct 4, 2016 at 12:10 PM, Jon Hunter wrote: > > Looks like crtc is a errno in the above case. I see this function is > > called by looping through all the crtc and we never check to see if > > they are

Re: [Resend][PATCH 2/3] PM / core / docs: Convert sleep states API document to reST

2017-01-09 Thread Lukas Wunner
Hi Rafael, I've perused devices.rst up until section "Entering System Suspend" so far, about half of the document. Here are my comments, I'll read the remainder of the document later. On Fri, Jan 06, 2017 at 02:41:38AM +0100, Rafael J. Wysocki wrote: > +.. |struct| replace:: :c:type:`struct`

Re: [PATCH 0/3] PM / docs: linux/pm.h kerneldocs update and conversion of two docs to reST

2017-01-08 Thread Lukas Wunner
On Fri, Jan 06, 2017 at 02:38:13AM +0100, Rafael J. Wysocki wrote: > I sent patches [1-2/3] previosly a couple of weeks ago and there have not > been any comments since then, so either they are fine by everybody or the > timing was particularly bad and no one had the time to look at them. So far

Re: [PATCH 1/2] PM: docs: Describe high-level PM strategies and sleep states

2017-08-20 Thread Lukas Wunner
e idle" would mean that a component is runtime active, but idling and could thus be transitioned to runtime suspended state. However above it says that if it's idle, it's already "in low-power states and may not be accessible". For someone reading this it may be difficult to reconcil

Re: [PATCH 01/12] PM / core: Add NEVER_SKIP and SMART_PREPARE driver flags

2017-10-15 Thread Lukas Wunner
On Mon, Oct 16, 2017 at 03:29:02AM +0200, Rafael J. Wysocki wrote: > + :c:func:`dev_pm_set_driver_flags` helper function.] If the first of > + tese flags is set, the PM core will not apply the direct-complete ^ these > + proceudre described above to the given device