2023-05-12 12:17:25

by Javier Martinez Canillas

Subject: [PATCH] drm/ssd130x: Fix include guard name

This is a leftover from an early iteration of the driver when it was still
named ssd1307 instead of ssd130x. Change it for consistency with the rest.

Signed-off-by: Javier Martinez Canillas <[email protected]>
---

drivers/gpu/drm/solomon/ssd130x.h | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/drivers/gpu/drm/solomon/ssd130x.h b/drivers/gpu/drm/solomon/ssd130x.h
index 03038c1b6476..db03ee5db392 100644
--- a/drivers/gpu/drm/solomon/ssd130x.h
+++ b/drivers/gpu/drm/solomon/ssd130x.h
@@ -10,8 +10,8 @@
* Copyright 2012 Free Electrons
*/

-#ifndef __SSD1307X_H__
-#define __SSD1307X_H__
+#ifndef __SSD130X_H__
+#define __SSD130X_H__

#include <drm/drm_connector.h>
#include <drm/drm_crtc.h>
@@ -94,4 +94,4 @@ struct ssd130x_device *ssd130x_probe(struct device *dev, struct regmap *regmap);
void ssd130x_remove(struct ssd130x_device *ssd130x);
void ssd130x_shutdown(struct ssd130x_device *ssd130x);

-#endif /* __SSD1307X_H__ */
+#endif /* __SSD130X_H__ */
--
2.40.1

2023-05-12 13:54:30

by Sam Ravnborg

Subject: Re: [PATCH] drm/ssd130x: Fix include guard name

On Fri, May 12, 2023 at 02:02:31PM +0200, Javier Martinez Canillas wrote:
> This is a leftover from an early iteration of the driver when it was still
> named ssd1307 instead of ssd130x. Change it for consistency with the rest.
>
> Signed-off-by: Javier Martinez Canillas <[email protected]>
Reviewed-by: Sam Ravnborg <[email protected]>
> ---
>
> drivers/gpu/drm/solomon/ssd130x.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/solomon/ssd130x.h b/drivers/gpu/drm/solomon/ssd130x.h
> index 03038c1b6476..db03ee5db392 100644
> --- a/drivers/gpu/drm/solomon/ssd130x.h
> +++ b/drivers/gpu/drm/solomon/ssd130x.h
> @@ -10,8 +10,8 @@
> * Copyright 2012 Free Electrons
> */
>
> -#ifndef __SSD1307X_H__
> -#define __SSD1307X_H__
> +#ifndef __SSD130X_H__
> +#define __SSD130X_H__
>
> #include <drm/drm_connector.h>
> #include <drm/drm_crtc.h>
> @@ -94,4 +94,4 @@ struct ssd130x_device *ssd130x_probe(struct device *dev, struct regmap *regmap);
> void ssd130x_remove(struct ssd130x_device *ssd130x);
> void ssd130x_shutdown(struct ssd130x_device *ssd130x);
>
> -#endif /* __SSD1307X_H__ */
> +#endif /* __SSD130X_H__ */
> --
> 2.40.1

2023-05-13 13:46:44

by Sui Jingfeng

Subject: Re: drm/ssd130x: Fix include guard name

Reviewed-by: Sui Jingfeng <[email protected]>


On 2023/5/12 20:02, Javier Martinez Canillas wrote:
> This is a leftover from an early iteration of the driver when it was still
> named ssd1307 instead of ssd130x. Change it for consistency with the rest.
>
> Signed-off-by: Javier Martinez Canillas <[email protected]>
> Reviewed-by: Sam Ravnborg <[email protected]>
> ---
>
> drivers/gpu/drm/solomon/ssd130x.h | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/gpu/drm/solomon/ssd130x.h b/drivers/gpu/drm/solomon/ssd130x.h
> index 03038c1b6476..db03ee5db392 100644
> --- a/drivers/gpu/drm/solomon/ssd130x.h
> +++ b/drivers/gpu/drm/solomon/ssd130x.h
> @@ -10,8 +10,8 @@
> * Copyright 2012 Free Electrons
> */
>
> -#ifndef __SSD1307X_H__
> -#define __SSD1307X_H__
> +#ifndef __SSD130X_H__
> +#define __SSD130X_H__
>
> #include <drm/drm_connector.h>
> #include <drm/drm_crtc.h>
> @@ -94,4 +94,4 @@ struct ssd130x_device *ssd130x_probe(struct device *dev, struct regmap *regmap);
> void ssd130x_remove(struct ssd130x_device *ssd130x);
> void ssd130x_shutdown(struct ssd130x_device *ssd130x);
>
> -#endif /* __SSD1307X_H__ */
> +#endif /* __SSD130X_H__ */

2023-05-15 12:35:39

by Javier Martinez Canillas

Subject: Re: drm/ssd130x: Fix include guard name

Sui Jingfeng <[email protected]> writes:

> Reviewed-by: Sui Jingfeng <[email protected]>
>
>

Pushed to drm-misc (drm-misc-next). Thanks!

--
Best regards,

Javier Martinez Canillas
Core Platforms
Red Hat