2023-03-19 11:42:21

by Menna Mahmoud

Subject: [PATCH] staging: greybus: add blank line after struct

add blank line after struct for readability as
reported by checkpatch script

" CHECK: Please use a blank line after function/struct/union/enum
declarations"

Signed-off-by: Menna Mahmoud <[email protected]>
---
drivers/staging/greybus/gbphy.h | 2 ++
1 file changed, 2 insertions(+)

diff --git a/drivers/staging/greybus/gbphy.h b/drivers/staging/greybus/gbphy.h
index d4a225b76338..1de510499480 100644
--- a/drivers/staging/greybus/gbphy.h
+++ b/drivers/staging/greybus/gbphy.h
@@ -15,6 +15,7 @@ struct gbphy_device {
struct list_head list;
struct device dev;
};
+
#define to_gbphy_dev(d) container_of(d, struct gbphy_device, dev)

static inline void *gb_gbphy_get_data(struct gbphy_device *gdev)
@@ -43,6 +44,7 @@ struct gbphy_driver {

struct device_driver driver;
};
+
#define to_gbphy_driver(d) container_of(d, struct gbphy_driver, driver)

int gb_gbphy_register_driver(struct gbphy_driver *driver,
--
2.34.1

2023-03-19 11:42:25

by Menna Mahmoud

Subject: [PATCH] staging: greybus: remove unnecessary blank line

Remove unnecessary blank line before struct as reported
by checkpatch:

" CHECK: Please don't use multiple blank lines "

Signed-off-by: Menna Mahmoud <[email protected]>
---
drivers/staging/greybus/greybus_authentication.h | 1 -
1 file changed, 1 deletion(-)

diff --git a/drivers/staging/greybus/greybus_authentication.h b/drivers/staging/greybus/greybus_authentication.h
index 7edc7295b7ab..48b4a9794d3c 100644
--- a/drivers/staging/greybus/greybus_authentication.h
+++ b/drivers/staging/greybus/greybus_authentication.h
@@ -41,7 +41,6 @@
#define CAP_AUTH_RESULT_CR_NO_KEY 0x03
#define CAP_AUTH_RESULT_CR_SIG_FAIL 0x04

-
/* IOCTL support */
struct cap_ioc_get_endpoint_uid {
__u8 uid[8];
--
2.34.1

2023-03-19 13:20:39

by Menna Mahmoud

Subject: Re: [PATCH] staging: greybus: add blank line after struct

I am sorry, I resend this patch by mistake. please drop it.

Menna

On ١٩‏/٣‏/٢٠٢٣ ١٣:٤١, Menna Mahmoud wrote:
> add blank line after struct for readability as
> reported by checkpatch script
>
> " CHECK: Please use a blank line after function/struct/union/enum
> declarations"
>
> Signed-off-by: Menna Mahmoud <[email protected]>
> ---
> drivers/staging/greybus/gbphy.h | 2 ++
> 1 file changed, 2 insertions(+)
>
> diff --git a/drivers/staging/greybus/gbphy.h b/drivers/staging/greybus/gbphy.h
> index d4a225b76338..1de510499480 100644
> --- a/drivers/staging/greybus/gbphy.h
> +++ b/drivers/staging/greybus/gbphy.h
> @@ -15,6 +15,7 @@ struct gbphy_device {
> struct list_head list;
> struct device dev;
> };
> +
> #define to_gbphy_dev(d) container_of(d, struct gbphy_device, dev)
>
> static inline void *gb_gbphy_get_data(struct gbphy_device *gdev)
> @@ -43,6 +44,7 @@ struct gbphy_driver {
>
> struct device_driver driver;
> };
> +
> #define to_gbphy_driver(d) container_of(d, struct gbphy_driver, driver)
>
> int gb_gbphy_register_driver(struct gbphy_driver *driver,