[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <Pine.LNX.4.44L0.1904171511510.1400-100000@iolanthe.rowland.org>
Date: Wed, 17 Apr 2019 15:14:14 -0400 (EDT)
From: Alan Stern <stern@...land.harvard.edu>
To: Raul E Rangel <rrangel@...omium.org>
cc: linux-usb@...r.kernel.org, <groeck@...omium.org>,
<oneukum@...e.com>, <djkurtz@...omium.org>, <zwisler@...omium.org>,
Sebastian Andrzej Siewior <bigeasy@...utronix.de>,
Martin Blumenstingl <martin.blumenstingl@...glemail.com>,
Dmitry Torokhov <dtor@...omium.org>,
<linux-kernel@...r.kernel.org>,
"Gustavo A. R. Silva" <gustavo@...eddedor.com>,
Miquel Raynal <miquel.raynal@...tlin.com>,
Johan Hovold <johan@...nel.org>,
Greg Kroah-Hartman <gregkh@...uxfoundation.org>,
Mathias Nyman <mathias.nyman@...ux.intel.com>
Subject: Re: [PATCH v3] usb/hcd: Send a uevent signaling that the host
controller had died
On Wed, 17 Apr 2019, Raul E Rangel wrote:
> +/* Workqueue routine for when the root-hub has died. */
> +static void hcd_died_work(struct work_struct *work)
> +{
> + struct usb_hcd *hcd = container_of(work, struct usb_hcd, died_work);
> + char *env[] = {
> + "ERROR=DEAD",
> + NULL
> + };
This can now be
static const char *env[] = ...
right? There's no need for the array to be reinitialized every time
the routine runs.
Alan Stern
Powered by blists - more mailing lists