linux.git/drivers/staging/greybus/loopback.c, branch v5.8

staging: greybus: loopback: fix a spelling error.

2020-05-27T08:18:09+00:00

Successed -> succeeded.

Signed-off-by: Till Varoquaux 
Reviewed-by: Alex Elder 
Link: https://lore.kernel.org/r/20200518051314.1785567-1-till.varoquaux@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: greybus: move core include files to include/linux/greybus/

2019-08-27T17:02:59+00:00

With the goal of moving the core of the greybus code out of staging, the
include files need to be moved to include/linux/greybus.h and
include/linux/greybus/

Cc: Vaibhav Hiremath 
Cc: Johan Hovold 
Cc: Vaibhav Agarwal 
Cc: Rui Miguel Silva 
Cc: David Lin 
Cc: "Bryan O'Donoghue" 
Cc: greybus-dev@lists.linaro.org
Cc: devel@driverdev.osuosl.org
Acked-by: Mark Greer 
Acked-by: Viresh Kumar 
Acked-by: Alex Elder 
Link: https://lore.kernel.org/r/20190825055429.18547-8-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman

staging: greybus: loopback: Fix up some alignment checkpatch issues

2019-08-27T17:02:55+00:00

Some function prototypes do not match the expected alignment formatting
so fix that up so that checkpatch is happy.

Cc: "Bryan O'Donoghue" 
Cc: Johan Hovold 
Cc: greybus-dev@lists.linaro.org
Cc: devel@driverdev.osuosl.org
Acked-by: Viresh Kumar 
Acked-by: Alex Elder 
Link: https://lore.kernel.org/r/20190825055429.18547-7-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman

staging: greybus: convert to DEFINE_SHOW_ATTRIBUTE

2018-12-17T13:24:42+00:00

Use DEFINE_SHOW_ATTRIBUTE macro to simplify the code.

Signed-off-by: Yangtao Li 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: Added space between string concatenated

2018-12-05T08:47:02+00:00

Some concatenated strings are now spaced.

Signed-off-by: Cristian Sicilia 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: loopback.c: remove unused lists

2018-11-05T14:32:45+00:00

gb_loopback_device::list_op_async is never used except for the
LIST_INIT. The ::list field appears to have a few more uses, but on
closer inspection the linked list of struct gb_loopbacks that it heads
is never used for anything, so there's no reason to maintain it, much
less to keep it sorted.

Reviewed-by: Bryan O'Donoghue 
Signed-off-by: Rasmus Villemoes 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: loopback.c: remove unused gb_loopback::lbid

2018-10-11T10:10:32+00:00

It's not obvious how the code prevents adding more than 31 elements to
the list and thus invoking undefined behaviour in the 1 << new_lbid
expression, and in practice causing ->lbid values to repeat every 32
elements.

But the definition of struct gb_loopback is local to loopback.c, and the
lbid field is entirely unused outside of this function, so it seems we
can just drop it entirely.

Signed-off-by: Rasmus Villemoes 
Reviewed-by: Bryan O'Donoghue 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: Remove redundant license text

2017-11-11T13:46:21+00:00

Now that the SPDX tag is in all greybus files, that identifies the
license in a specific and legally-defined manner.  So the extra GPL text
wording can be removed as it is no longer needed at all.

This is done on a quest to remove the 700+ different ways that files in
the kernel describe the GPL license text.  And there's unneeded stuff
like the address (sometimes incorrect) for the FSF which is never
needed.

No copyright headers or other non-license-description text was removed.

Cc: Vaibhav Hiremath 
Reviewed-by: Alex Elder 
Acked-by: Vaibhav Agarwal 
Acked-by: David Lin 
Acked-by: Johan Hovold 
Acked-by: Viresh Kumar 
Acked-by: Mark Greer 
Acked-by: Rui Miguel Silva 
Acked-by: "Bryan O'Donoghue" 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: add SPDX identifiers to all greybus driver files

2017-11-11T13:46:20+00:00

It's good to have SPDX identifiers in all files to make it easier to
audit the kernel tree for correct licenses.

Update the drivers/staging/greybus files files with the correct SPDX
license identifier based on the license text in the file itself.  The
SPDX identifier is a legally binding shorthand, which can be used
instead of the full boiler plate text.

This work is based on a script and data from Thomas Gleixner, Philippe
Ombredanne, and Kate Stewart.

Cc: Vaibhav Hiremath 
Cc: "Bryan O'Donoghue" 
Cc: Thomas Gleixner 
Cc: Kate Stewart 
Cc: Philippe Ombredanne 
Acked-by: Vaibhav Agarwal 
Acked-by: David Lin 
Reviewed-by: Alex Elder 
Acked-by: Johan Hovold 
Acked-by: Viresh Kumar 
Acked-by: Mark Greer 
Acked-by: Rui Miguel Silva 
Signed-off-by: Greg Kroah-Hartman

staging: greybus: loopback: convert loopback to use generic async operations

2017-11-06T15:41:22+00:00

Loopback has its own internal method for tracking and timing out
asynchronous operations however previous patches make it possible to use
functionality provided by operation.c to do this instead. Using the code in
operation.c means we can completely subtract the timer, the work-queue, the
kref and the cringe-worthy 'pending' flag. The completion callback
triggered by operation.c will provide an authoritative result code -
including -ETIMEDOUT for asynchronous operations.

Signed-off-by: Bryan O'Donoghue 
Reviewed-by: Johan Hovold 
Cc: Alex Elder 
Cc: Kees Cook 
Cc: Mitch Tasman 
Cc: greybus-dev@lists.linaro.org
Cc: devel@driverdev.osuosl.org
Cc: linux-kernel@vger.kernel.org
Signed-off-by: Greg Kroah-Hartman