diff mbox series

scsi: ch: add include guard to chio.h

Message ID 20190728164643.16335-1-yamada.masahiro@socionext.com
State Accepted
Commit c51c4841f1571673d9075b8cf5efa6995ea91d0e
Headers show
Series scsi: ch: add include guard to chio.h | expand

Commit Message

Masahiro Yamada July 28, 2019, 4:46 p.m. UTC
Add a header include guard just in case.

Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

---

 include/uapi/linux/chio.h | 11 ++++-------
 1 file changed, 4 insertions(+), 7 deletions(-)

-- 
2.17.1

Comments

Masahiro Yamada Oct. 3, 2019, 2:30 a.m. UTC | #1
Hi,

On Mon, Jul 29, 2019 at 1:47 AM Masahiro Yamada
<yamada.masahiro@socionext.com> wrote:
>

> Add a header include guard just in case.

>

> Signed-off-by: Masahiro Yamada <yamada.masahiro@socionext.com>

> ---


Ping?


>  include/uapi/linux/chio.h | 11 ++++-------

>  1 file changed, 4 insertions(+), 7 deletions(-)

>

> diff --git a/include/uapi/linux/chio.h b/include/uapi/linux/chio.h

> index 689fc93fafda..e1cad4c319ee 100644

> --- a/include/uapi/linux/chio.h

> +++ b/include/uapi/linux/chio.h

> @@ -3,6 +3,9 @@

>   * ioctl interface for the scsi media changer driver

>   */

>

> +#ifndef _UAPI_LINUX_CHIO_H

> +#define _UAPI_LINUX_CHIO_H

> +

>  /* changer element types */

>  #define CHET_MT   0    /* media transport element (robot) */

>  #define CHET_ST   1    /* storage element (media slots) */

> @@ -160,10 +163,4 @@ struct changer_set_voltag {

>  #define CHIOSVOLTAG    _IOW('c',18,struct changer_set_voltag)

>  #define CHIOGVPARAMS   _IOR('c',19,struct changer_vendor_params)

>

> -/* ---------------------------------------------------------------------- */

> -

> -/*

> - * Local variables:

> - * c-basic-offset: 8

> - * End:

> - */

> +#endif /* _UAPI_LINUX_CHIO_H */

> --

> 2.17.1

>



-- 
Best Regards
Masahiro Yamada
Martin K. Petersen Oct. 4, 2019, 1:59 a.m. UTC | #2
Masahiro,

>> Add a header include guard just in case.


Fine with me. Is it going through your tree or should I pick it up?

-- 
Martin K. Petersen	Oracle Linux Engineering
Masahiro Yamada Oct. 4, 2019, 2:20 a.m. UTC | #3
Hi Martin,

On Fri, Oct 4, 2019 at 10:59 AM Martin K. Petersen
<martin.petersen@oracle.com> wrote:
>

>

> Masahiro,

>

> >> Add a header include guard just in case.

>

> Fine with me. Is it going through your tree or should I pick it up?




Could you please apply it to your tree?

Thanks.

-- 
Best Regards
Masahiro Yamada
diff mbox series

Patch

diff --git a/include/uapi/linux/chio.h b/include/uapi/linux/chio.h
index 689fc93fafda..e1cad4c319ee 100644
--- a/include/uapi/linux/chio.h
+++ b/include/uapi/linux/chio.h
@@ -3,6 +3,9 @@ 
  * ioctl interface for the scsi media changer driver
  */
 
+#ifndef _UAPI_LINUX_CHIO_H
+#define _UAPI_LINUX_CHIO_H
+
 /* changer element types */
 #define CHET_MT   0	/* media transport element (robot) */
 #define CHET_ST   1	/* storage element (media slots) */
@@ -160,10 +163,4 @@  struct changer_set_voltag {
 #define CHIOSVOLTAG    _IOW('c',18,struct changer_set_voltag)
 #define CHIOGVPARAMS   _IOR('c',19,struct changer_vendor_params)
 
-/* ---------------------------------------------------------------------- */
-
-/*
- * Local variables:
- * c-basic-offset: 8
- * End:
- */
+#endif /* _UAPI_LINUX_CHIO_H */