[PATCH v4 2/9] dts: add developer tools

Tu, Lijuan lijuan.tu at intel.com
Tue Sep 20 14:22:09 CEST 2022



> -----Original Message-----
> From: Juraj Linkeš <juraj.linkes at pantheon.tech>
> Sent: Tuesday, September 20, 2022 8:14 PM
> To: Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>;
> thomas at monjalon.net; Richardson, Bruce <bruce.richardson at intel.com>
> Cc: david.marchand at redhat.com; Randles, Ronan <ronan.randles at intel.com>;
> ohilyard at iol.unh.edu; Tu, Lijuan <lijuan.tu at intel.com>; dev at dpdk.org; nd
> <nd at arm.com>; nd <nd at arm.com>
> Subject: RE: [PATCH v4 2/9] dts: add developer tools
> 
> 
> 
> > -----Original Message-----
> > From: Honnappa Nagarahalli <Honnappa.Nagarahalli at arm.com>
> > Sent: Wednesday, September 14, 2022 9:08 PM
> > To: thomas at monjalon.net; Juraj Linkeš <juraj.linkes at pantheon.tech>;
> > Bruce Richardson <bruce.richardson at intel.com>
> > Cc: david.marchand at redhat.com; ronan.randles at intel.com;
> > ohilyard at iol.unh.edu; lijuan.tu at intel.com; dev at dpdk.org; nd
> > <nd at arm.com>; nd <nd at arm.com>
> > Subject: RE: [PATCH v4 2/9] dts: add developer tools
> >
> > <snip>
> >
> > >
> > > 14/09/2022 15:13, Bruce Richardson:
> > > > On Wed, Sep 14, 2022 at 12:45:00PM +0000, Juraj Linkeš wrote:
> > > > > From: Honnappa Nagarahalli
> > > > > > > > On Fri, Jul 29, 2022 at 10:55:43AM +0000, Juraj Linkeš wrote:
> > > > <snip>
> > > > > > > > > diff --git a/dts/format.sh b/dts/format.sh new file mode
> > > > > > > > > 100755
> > > > > > Should this be in dpdk/devtools directory? If yes, need a
> > > > > > different name for the script, dts-fix-format.sh?
> > > > >
> > > > > We should decide where we'll put it, either to dpdk/devtools or
> > > > > dpdk/dts/devtools. So far I have it in dpdk/dts/devtools, but it
> > > > > may make more sense to put it into dpdk/devtools.
> > > >
> > > > Third option of "devtools/dts". I'd like to have it either in the
> > > > devtools directory directly, or in a subdirectory of devtools.
> > >
> > > I am OK with devtools/dts/ or direcly in dts/ DTS *is* a developer tool, right?
> > > So why some scripts should be in a different directory?
> >
> > +1 for devtools/dts
> > >
> 
> I like this as well, I'll move them there.

Do we consider all python files follow the same format ? if yes, prefer devtools, else prefer devtools/dts .


More information about the dev mailing list