Date:   Fri, 3 Feb 2023 11:27:18 -0800
From:   Bjorn Andersson <andersson@kernel.org>
To:     Elliot Berman <quic_eberman@quicinc.com>
Cc:     Andy Gross <agross@kernel.org>,
        Konrad Dybcio <konrad.dybcio@linaro.org>,
        linux-arm-msm@vger.kernel.org, linux-kernel@vger.kernel.org
Subject: Re: [PATCH] MAINTAINERS: Add include/linux/qcom* to ARM/QUALCOMM
Message-ID: <20230203192718.eyuvmi7kniwulv63@ripper>
References: <20230203183115.2836316-1-quic_eberman@quicinc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20230203183115.2836316-1-quic_eberman@quicinc.com>
Precedence: bulk

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
>