linux.git/drivers/staging/greybus/loopback.c, branch v6.7

drivers: remove struct module * setting from struct class

2023-03-17T14:16:27+00:00

There is no need to manually set the owner of a struct class, as the
registering function does it automatically, so remove all of the
explicit settings from various drivers that did so as it is unneeded.

This allows us to remove this pointer entirely from this structure going
forward.

Cc: "Rafael J. Wysocki" 
Link: https://lore.kernel.org/r/20230313181843.1207845-2-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman

staging: greybus: loopback: fix a spelling error.

2022-06-07T11:04:42+00:00

asynchronus -> asynchronous

Signed-off-by: Steve Lemuel 
Link: https://lore.kernel.org/r/SYBP282MB347864E247D2EC70BF46F9A3A0A29@SYBP282MB3478.AUSP282.PROD.OUTLOOK.COM
Signed-off-by: Greg Kroah-Hartman

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