[<prev] [next>] [<thread-prev] [day] [month] [year] [list]
Message-ID: <ea8001238622a81bfb89dbdc35a6604914a4d052.camel@mediatek.com>
Date: Sun, 29 Jan 2023 08:51:00 +0000
From: Yanchao Yang (杨彦超)
<Yanchao.Yang@...iatek.com>
To: "kuba@...nel.org" <kuba@...nel.org>
CC: Chris Feng (冯保林)
<Chris.Feng@...iatek.com>,
"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
"linux-mediatek@...ts.infradead.org"
<linux-mediatek@...ts.infradead.org>,
Mingliang Xu (徐明亮)
<mingliang.xu@...iatek.com>,
Min Dong (董敏) <min.dong@...iatek.com>,
"linuxwwan@...el.com" <linuxwwan@...el.com>,
"m.chetan.kumar@...el.com" <m.chetan.kumar@...el.com>,
Liang Lu (吕亮) <liang.lu@...iatek.com>,
Haijun Liu (刘海军)
<haijun.liu@...iatek.com>,
Haozhe Chang (常浩哲)
<Haozhe.Chang@...iatek.com>,
Hua Yang (杨华) <Hua.Yang@...iatek.com>,
"ryazanov.s.a@...il.com" <ryazanov.s.a@...il.com>,
Xiayu Zhang (张夏宇)
<Xiayu.Zhang@...iatek.com>,
"loic.poulain@...aro.org" <loic.poulain@...aro.org>,
"pabeni@...hat.com" <pabeni@...hat.com>,
"edumazet@...gle.com" <edumazet@...gle.com>,
Ting Wang (王挺) <ting.wang@...iatek.com>,
"johannes@...solutions.net" <johannes@...solutions.net>,
"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
Aiden Wang (王咏麒)
<Aiden.Wang@...iatek.com>,
Felix Chen (陈非) <Felix.Chen@...iatek.com>,
Lambert Wang (王伟)
<Lambert.Wang@...iatek.com>,
"davem@...emloft.net" <davem@...emloft.net>,
Mingchuang Qiao (乔明闯)
<Mingchuang.Qiao@...iatek.com>,
Guohao Zhang (张国豪)
<Guohao.Zhang@...iatek.com>
Subject: Re: [PATCH net-next v2 08/12] net: wwan: tmi: Add data plane
transaction layer
Hi Jakub,
sorry for late response, please check following reply.
On Thu, 2023-01-19 at 19:52 -0800, Jakub Kicinski wrote:
> On Wed, 18 Jan 2023 19:38:55 +0800 Yanchao Yang wrote:
> > drivers/net/wwan/mediatek/mtk_dpmaif.c | 4005
> > ++++++++++++++++++++
>
> clang detects out-of-bound memcpy/strcpy or such somewhere in this
> file.
> Please fix that.
Ok, we will check and fix it.
>
> Please try to make the series smaller than 17,770 :/
> Strip stuff down to minimal working version.
> I don't think anyone can review 17kLoC in one sitting :/
Thank your suggestion. I agree with you. Considering both the control
plane and data plane are the minimal function for TMI driver, other
functions can be removed and submitted after the initial version is
approved. The whole patch can be reduced about 2k lines. Is it ok? Or
do you have any more suggestions? Please help share it.
Many thanks.
yanchao.yang
Powered by blists - more mailing lists