Re: 回复: [PATCH v3] bus: mhi: host: pci_generic: Add support for Quectel RM520N-GL modem

["mani@kernel.org" <mani@kernel.org>]

On Mon, Apr 03, 2023 at 01:21:29AM +0000, Duke Xin(辛安文) wrote:
> Hi Mani
> 
> I'm sorry. I might bother you. 
> 
> I want to know the review progress of [PATCH v3], it seems that you have forgotten my [PATCH v3]. You gave some modification suggestions in [PATCH v2], I revised them and submitted them to [PATCH v3].
> 

Still the patch has some concerns, so I shared some comments. Please fix them
before sending v4.

- Mani

> 
> 辛安文Duke Xin | Software Department IX Engineer | Quectel Wireless Solutions Co., Ltd. 
>             
> Mobile: +86-15375456183 | Email : Duke.xin@quectel.com  | Tel: +86-0551-65869386-8632
> Website: www.quectel.com  | QQ: 602659072 | Wechat: 15375456183
> 
> Building 1-C, China Speech Valley Area A, 3335 Xiyou Road, High-tech Zone, Hefei, Anhui 230088, China    
> 安徽省合肥市高新区习友路3335号中国（合肥）国际智能语音产业园A区1号中试楼 230088
> HQ: Building 5, Shanghai Business Park Phase III (Area B), No.1016 Tianlin Road, Minhang District, Shanghai 200233, China
> 总部：上海市闵行区田林路1016号科技绿洲3期（B区）5号楼  200233
> 
> -----邮件原件-----
> 发件人: Greg KH <gregkh@linuxfoundation.org> 
> 发送时间: 2023年1月31日 17:12
> 收件人: Duke Xin(辛安文) <duke_xinanwen@163.com>
> 抄送: mani@kernel.org; slark_xiao@163.com; loic.poulain@linaro.org; dnlplm@gmail.com; yonglin.tan@outlook.com; fabio.porcedda@gmail.com; mhi@lists.linux.dev; linux-arm-msm@vger.kernel.org; inux-kernel@vger.kernel.org; Jerry Meng(蒙杰) <jerry.meng@quectel.com>; Duke Xin(辛安文) <duke.xin@quectel.com>
> 主题: Re: [PATCH v3] bus: mhi: host: pci_generic: Add support for Quectel RM520N-GL modem
> 
> On Fri, Jan 27, 2023 at 06:47:54PM -0800, Duke Xin(辛安文) wrote:
> > The project is based on Qualcomm's sdx6x chips for laptop,so the mhi 
> > interface definition and enumeration align with previous Quectel sdx24 
> > configuration
> > 
> > Signed-off-by: Duke Xin(辛安文) <duke_xinanwen@163.com>
> > ---
> >  drivers/bus/mhi/host/pci_generic.c | 13 +++++++++++++
> >  1 file changed, 13 insertions(+)
> > 
> 
> Hi,
> 
> This is the friendly patch-bot of Greg Kroah-Hartman.  You have sent him a patch that has triggered this response.  He used to manually respond to these common problems, but in order to save his sanity (he kept writing the same thing over and over, yet to different people), I was created.  Hopefully you will not take offence and will fix the problem in your patch and resubmit it so that it can be accepted into the Linux kernel tree.
> 
> You are receiving this message because of the following common error(s) as indicated below:
> 
> - This looks like a new version of a previously submitted patch, but you
>   did not list below the --- line any changes from the previous version.
>   Please read the section entitled "The canonical patch format" in the
>   kernel file, Documentation/process/submitting-patches.rst for what
>   needs to be done here to properly describe this.
> 
> If you wish to discuss this problem further, or you have questions about how to resolve this issue, please feel free to respond to this email and Greg will reply once he has dug out from the pending patches received from other developers.
> 
> thanks,
> 
> greg k-h's patch email bot

-- 
மணிவண்ணன் சதாசிவம்