2023-02-03 21:07:33

by Elliot Berman

[permalink] [raw]

Subject: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
This removes 1 of a few remaining Qualcomm-specific headers into a more
approciate subdirectory under include/.

Signed-off-by: Elliot Berman <[email protected]>
---
drivers/i2c/busses/i2c-qcom-geni.c | 2 +-
drivers/soc/qcom/qcom-geni-se.c | 2 +-
drivers/spi/spi-geni-qcom.c | 2 +-
drivers/tty/serial/qcom_geni_serial.c | 2 +-
include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} | 0
5 files changed, 4 insertions(+), 4 deletions(-)
rename include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} (100%)

diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c
index fd70794bfcee..80d586b0be7a 100644
--- a/drivers/i2c/busses/i2c-qcom-geni.c
+++ b/drivers/i2c/busses/i2c-qcom-geni.c
@@ -14,7 +14,7 @@
#include <linux/of.h>
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
-#include <linux/qcom-geni-se.h>
+#include <linux/soc/qcom/geni-se.h>
#include <linux/spinlock.h>

#define SE_I2C_TX_TRANS_LEN 0x26c
diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c
index f0475b93ca73..795a2e1d59b3 100644
--- a/drivers/soc/qcom/qcom-geni-se.c
+++ b/drivers/soc/qcom/qcom-geni-se.c
@@ -14,7 +14,7 @@
#include <linux/of_platform.h>
#include <linux/pinctrl/consumer.h>
#include <linux/platform_device.h>
-#include <linux/qcom-geni-se.h>
+#include <linux/soc/qcom/geni-se.h>

/**
* DOC: Overview
diff --git a/drivers/spi/spi-geni-qcom.c b/drivers/spi/spi-geni-qcom.c
index 4e83cc5b445d..e05e40a49294 100644
--- a/drivers/spi/spi-geni-qcom.c
+++ b/drivers/spi/spi-geni-qcom.c
@@ -12,7 +12,7 @@
#include <linux/platform_device.h>
#include <linux/pm_opp.h>
#include <linux/pm_runtime.h>
-#include <linux/qcom-geni-se.h>
+#include <linux/soc/qcom/geni-se.h>
#include <linux/spi/spi.h>
#include <linux/spinlock.h>

diff --git a/drivers/tty/serial/qcom_geni_serial.c b/drivers/tty/serial/qcom_geni_serial.c
index b487823f0e61..52f3a0184577 100644
--- a/drivers/tty/serial/qcom_geni_serial.c
+++ b/drivers/tty/serial/qcom_geni_serial.c
@@ -16,7 +16,7 @@
#include <linux/platform_device.h>
#include <linux/pm_runtime.h>
#include <linux/pm_wakeirq.h>
-#include <linux/qcom-geni-se.h>
+#include <linux/soc/qcom/geni-se.h>
#include <linux/serial.h>
#include <linux/serial_core.h>
#include <linux/slab.h>
diff --git a/include/linux/qcom-geni-se.h b/include/linux/soc/qcom/geni-se.h
similarity index 100%
rename from include/linux/qcom-geni-se.h
rename to include/linux/soc/qcom/geni-se.h

base-commit: 3866989ec2c319341e2cf69ec6116269b634a271
--
2.39.1

2023-02-03 21:57:19

by Guru Das Srinagesh

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On Feb 03 2023 13:01, Elliot Berman wrote:
> Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> This removes 1 of a few remaining Qualcomm-specific headers into a more
> approciate subdirectory under include/.
>
> Signed-off-by: Elliot Berman <[email protected]>

Reviewed-by: Guru Das Srinagesh <[email protected]>

2023-02-04 07:05:01

by Wolfram Sang

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On Fri, Feb 03, 2023 at 01:01:32PM -0800, Elliot Berman wrote:
> Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> This removes 1 of a few remaining Qualcomm-specific headers into a more
> approciate subdirectory under include/.
>
> Signed-off-by: Elliot Berman <[email protected]>

Acked-by: Wolfram Sang <[email protected]> # for I2C

---

Attachments:

(No filename) (365.00 B)
signature.asc (833.00 B)
Download all attachments

2023-02-04 07:52:11

by Greg Kroah-Hartman

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On Fri, Feb 03, 2023 at 01:01:32PM -0800, Elliot Berman wrote:
> Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> This removes 1 of a few remaining Qualcomm-specific headers into a more
> approciate subdirectory under include/.
>
> Signed-off-by: Elliot Berman <[email protected]>
> ---
> drivers/i2c/busses/i2c-qcom-geni.c | 2 +-
> drivers/soc/qcom/qcom-geni-se.c | 2 +-
> drivers/spi/spi-geni-qcom.c | 2 +-
> drivers/tty/serial/qcom_geni_serial.c | 2 +-
> include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} | 0
> 5 files changed, 4 insertions(+), 4 deletions(-)
> rename include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} (100%)

Who do you want to take this patch? Through what tree? I can take it
through the tty tree if no one objects, otherwise if someone else is:

Acked-by: Greg Kroah-Hartman <[email protected]>

2023-02-06 18:17:55

by Elliot Berman

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On 2/3/2023 1:01 PM, Elliot Berman wrote:
> Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> This removes 1 of a few remaining Qualcomm-specific headers into a more
> approciate subdirectory under include/.

Need to s/approciate/appropriate/ (Thanks Mukesh!)

>
> Signed-off-by: Elliot Berman <[email protected]>
> ---
> drivers/i2c/busses/i2c-qcom-geni.c | 2 +-
> drivers/soc/qcom/qcom-geni-se.c | 2 +-
> drivers/spi/spi-geni-qcom.c | 2 +-
> drivers/tty/serial/qcom_geni_serial.c | 2 +-
> include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} | 0
> 5 files changed, 4 insertions(+), 4 deletions(-)
> rename include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} (100%)
>
> diff --git a/drivers/i2c/busses/i2c-qcom-geni.c b/drivers/i2c/busses/i2c-qcom-geni.c
> index fd70794bfcee..80d586b0be7a 100644
> --- a/drivers/i2c/busses/i2c-qcom-geni.c
> +++ b/drivers/i2c/busses/i2c-qcom-geni.c
> @@ -14,7 +14,7 @@
> #include <linux/of.h>
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> -#include <linux/qcom-geni-se.h>
> +#include <linux/soc/qcom/geni-se.h>
> #include <linux/spinlock.h>
>
> #define SE_I2C_TX_TRANS_LEN 0x26c
> diff --git a/drivers/soc/qcom/qcom-geni-se.c b/drivers/soc/qcom/qcom-geni-se.c
> index f0475b93ca73..795a2e1d59b3 100644
> --- a/drivers/soc/qcom/qcom-geni-se.c
> +++ b/drivers/soc/qcom/qcom-geni-se.c
> @@ -14,7 +14,7 @@
> #include <linux/of_platform.h>
> #include <linux/pinctrl/consumer.h>
> #include <linux/platform_device.h>
> -#include <linux/qcom-geni-se.h>
> +#include <linux/soc/qcom/geni-se.h>
>
> /**
> * DOC: Overview
> diff --git a/drivers/spi/spi-geni-qcom.c b/drivers/spi/spi-geni-qcom.c
> index 4e83cc5b445d..e05e40a49294 100644
> --- a/drivers/spi/spi-geni-qcom.c
> +++ b/drivers/spi/spi-geni-qcom.c
> @@ -12,7 +12,7 @@
> #include <linux/platform_device.h>
> #include <linux/pm_opp.h>
> #include <linux/pm_runtime.h>
> -#include <linux/qcom-geni-se.h>
> +#include <linux/soc/qcom/geni-se.h>
> #include <linux/spi/spi.h>
> #include <linux/spinlock.h>
>
> diff --git a/drivers/tty/serial/qcom_geni_serial.c b/drivers/tty/serial/qcom_geni_serial.c
> index b487823f0e61..52f3a0184577 100644
> --- a/drivers/tty/serial/qcom_geni_serial.c
> +++ b/drivers/tty/serial/qcom_geni_serial.c
> @@ -16,7 +16,7 @@
> #include <linux/platform_device.h>
> #include <linux/pm_runtime.h>
> #include <linux/pm_wakeirq.h>
> -#include <linux/qcom-geni-se.h>
> +#include <linux/soc/qcom/geni-se.h>
> #include <linux/serial.h>
> #include <linux/serial_core.h>
> #include <linux/slab.h>
> diff --git a/include/linux/qcom-geni-se.h b/include/linux/soc/qcom/geni-se.h
> similarity index 100%
> rename from include/linux/qcom-geni-se.h
> rename to include/linux/soc/qcom/geni-se.h
>
> base-commit: 3866989ec2c319341e2cf69ec6116269b634a271

2023-02-09 03:13:01

by Bjorn Andersson

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On Sat, Feb 04, 2023 at 08:51:59AM +0100, Greg Kroah-Hartman wrote:
> On Fri, Feb 03, 2023 at 01:01:32PM -0800, Elliot Berman wrote:
> > Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> > This removes 1 of a few remaining Qualcomm-specific headers into a more
> > approciate subdirectory under include/.
> >
> > Signed-off-by: Elliot Berman <[email protected]>
> > ---
> > drivers/i2c/busses/i2c-qcom-geni.c | 2 +-
> > drivers/soc/qcom/qcom-geni-se.c | 2 +-
> > drivers/spi/spi-geni-qcom.c | 2 +-
> > drivers/tty/serial/qcom_geni_serial.c | 2 +-
> > include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} | 0
> > 5 files changed, 4 insertions(+), 4 deletions(-)
> > rename include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} (100%)
>
> Who do you want to take this patch? Through what tree? I can take it
> through the tty tree if no one objects, otherwise if someone else is:
>
> Acked-by: Greg Kroah-Hartman <[email protected]>

I'm happy with that, please go ahead Greg.

Acked-by: Bjorn Andersson <[email protected]>

Regards,
Bjorn

2023-02-09 12:28:09

by Greg Kroah-Hartman

[permalink] [raw]

Subject: Re: [PATCH] soc: qcom: geni-se: Move qcom-geni-se.h to linux/soc/qcom/geni-se.h

---

On Wed, Feb 08, 2023 at 07:14:40PM -0800, Bjorn Andersson wrote:
> On Sat, Feb 04, 2023 at 08:51:59AM +0100, Greg Kroah-Hartman wrote:
> > On Fri, Feb 03, 2023 at 01:01:32PM -0800, Elliot Berman wrote:
> > > Move include/linux/qcom-geni-se.h to include/linux/soc/qcom/geni-se.h.
> > > This removes 1 of a few remaining Qualcomm-specific headers into a more
> > > approciate subdirectory under include/.
> > >
> > > Signed-off-by: Elliot Berman <[email protected]>
> > > ---
> > > drivers/i2c/busses/i2c-qcom-geni.c | 2 +-
> > > drivers/soc/qcom/qcom-geni-se.c | 2 +-
> > > drivers/spi/spi-geni-qcom.c | 2 +-
> > > drivers/tty/serial/qcom_geni_serial.c | 2 +-
> > > include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} | 0
> > > 5 files changed, 4 insertions(+), 4 deletions(-)
> > > rename include/linux/{qcom-geni-se.h => soc/qcom/geni-se.h} (100%)
> >
> > Who do you want to take this patch? Through what tree? I can take it
> > through the tty tree if no one objects, otherwise if someone else is:
> >
> > Acked-by: Greg Kroah-Hartman <[email protected]>
>
> I'm happy with that, please go ahead Greg.
>
> Acked-by: Bjorn Andersson <[email protected]>

Great, now queued up, thanks.

greg k-h