diff mbox series

MAINTAINERS: Add include/linux/qcom* to ARM/QUALCOMM

Message ID 20230203183115.2836316-1-quic_eberman@quicinc.com
State New
Headers
Series MAINTAINERS: Add include/linux/qcom* to ARM/QUALCOMM |

Commit Message

Elliot Berman Feb. 3, 2023, 6:31 p.m. UTC 
  ARM/QUALCOMM support currently includes include/linux/*/qcom* but is
missing a few Qualcomm headers directly in include/linux/.

This effectively adds following headers directly under ARM/QUALCOMM.
 - include/linux/qcom-geni-se.h
 - include/linux/qcom_scm.h

Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>
---
 MAINTAINERS | 1 +
 1 file changed, 1 insertion(+)


base-commit: 3866989ec2c319341e2cf69ec6116269b634a271

Comments

Bjorn Andersson Feb. 3, 2023, 7:27 p.m. UTC | #1 
On Fri, Feb 03, 2023 at 10:31:14AM -0800, Elliot Berman wrote:
> ARM/QUALCOMM support currently includes include/linux/*/qcom* but is
> missing a few Qualcomm headers directly in include/linux/.
> 
> This effectively adds following headers directly under ARM/QUALCOMM.
>  - include/linux/qcom-geni-se.h
>  - include/linux/qcom_scm.h
> 

Seems odd to have these lone files here...

Would you be willing to push these two files into
include/linux/firmware/qcom/ and include/linux/soc/qcom/ instead?

Regards,
Bjorn

> Signed-off-by: Elliot Berman <quic_eberman@quicinc.com>
> ---
>  MAINTAINERS | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/MAINTAINERS b/MAINTAINERS
> index f2c3d471fe8f..752cddd73bf8 100644
> --- a/MAINTAINERS
> +++ b/MAINTAINERS
> @@ -2724,6 +2724,7 @@ F:	drivers/spi/spi-qup.c
>  F:	drivers/tty/serial/msm_serial.c
>  F:	drivers/usb/dwc3/dwc3-qcom.c
>  F:	include/dt-bindings/*/qcom*
> +F:	include/linux/qcom*
>  F:	include/linux/*/qcom*
>  F:	include/linux/soc/qcom/
>  
> 
> base-commit: 3866989ec2c319341e2cf69ec6116269b634a271
> -- 
> 2.39.1
>
Elliot Berman Feb. 3, 2023, 9:10 p.m. UTC | #2 
On 2/3/2023 11:27 AM, Bjorn Andersson wrote:
> On Fri, Feb 03, 2023 at 10:31:14AM -0800, Elliot Berman wrote:
>> ARM/QUALCOMM support currently includes include/linux/*/qcom* but is
>> missing a few Qualcomm headers directly in include/linux/.
>>
>> This effectively adds following headers directly under ARM/QUALCOMM.
>>   - include/linux/qcom-geni-se.h
>>   - include/linux/qcom_scm.h
>>
> 
> Seems odd to have these lone files here...
> 
> Would you be willing to push these two files into
> include/linux/firmware/qcom/ and include/linux/soc/qcom/ instead?
> 

Sure, this is also good approach to me.

qcom-geni-se.h: 
https://lore.kernel.org/all/20230203210133.3552796-1-quic_eberman@quicinc.com/

qcom_scm.h: 
https://lore.kernel.org/all/20230203210956.3580811-1-quic_eberman@quicinc.com/

Thanks,
Elliot
diff mbox series

Patch

diff --git a/MAINTAINERS b/MAINTAINERS
index f2c3d471fe8f..752cddd73bf8 100644
--- a/MAINTAINERS
+++ b/MAINTAINERS
@@ -2724,6 +2724,7 @@  F:	drivers/spi/spi-qup.c
 F:	drivers/tty/serial/msm_serial.c
 F:	drivers/usb/dwc3/dwc3-qcom.c
 F:	include/dt-bindings/*/qcom*
+F:	include/linux/qcom*
 F:	include/linux/*/qcom*
 F:	include/linux/soc/qcom/