diff mbox series

scsi: ch: add include guard to chio.h

Message ID 20190728164643.16335-1-yamada.masahiro@socionext.com (mailing list archive)
State Deferred
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(-)

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
>
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?
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.
Martin K. Petersen Oct. 10, 2019, 2:31 a.m. UTC | #4
Masahiro,

>> Fine with me. Is it going through your tree or should I pick it up?
>
> Could you please apply it to your tree?

Applied to 5.5/scsi-queue, thanks!
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 */