netdev - RE: [PATCH 01/12] fsl/fman: Add the FMan FLIB headers

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Message-ID: <DM2PR03MB3830ADD2B503C6738BB27E7E6A60@DM2PR03MB383.namprd03.prod.outlook.com>
Date:	Wed, 17 Jun 2015 14:59:37 +0000
From:	Liberman Igal <Igal.Liberman@...escale.com>
To:	Scott Wood <scottwood@...escale.com>,
	Madalin-Cristian Bucur <madalin.bucur@...escale.com>
CC:	"netdev@...r.kernel.org" <netdev@...r.kernel.org>,
	"linux-kernel@...r.kernel.org" <linux-kernel@...r.kernel.org>,
	"linuxppc-dev@...ts.ozlabs.org" <linuxppc-dev@...ts.ozlabs.org>
Subject: RE: [PATCH 01/12] fsl/fman: Add the FMan FLIB headers



Regards,
Igal Liberman.

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Wednesday, June 10, 2015 9:54 PM
> To: Bucur Madalin-Cristian-B32716
> Cc: netdev@...r.kernel.org; linux-kernel@...r.kernel.org; linuxppc-
> dev@...ts.ozlabs.org; Liberman Igal-B31950
> Subject: Re: [PATCH 01/12] fsl/fman: Add the FMan FLIB headers
> 
> On Wed, 2015-06-10 at 18:21 +0300, Madalin Bucur wrote:
> > From: Igal Liberman <Igal.Liberman@...escale.com>
> >
> > This patch presents the FMan Foundation Libraries (FLIB) headers.
> > The FMan FLib provides the basic API used by the FMan drivers to
> > configure and control the FMan hardware.
> >
> > Signed-off-by: Igal Liberman <Igal.Liberman@...escale.com>
> > ---
> >  .../ethernet/freescale/fman/flib/common/general.h  |  41 ++
> >  .../net/ethernet/freescale/fman/flib/fsl_fman.h    | 609
> > +++++++++++++++++++++
> >  2 files changed, 650 insertions(+)
> >  create mode 100644
> > drivers/net/ethernet/freescale/fman/flib/common/general.h
> >  create mode 100644
> > drivers/net/ethernet/freescale/fman/flib/fsl_fman.h
> 
> Why do we need separate patches just for headers?
> 

We wanted to make the patches smaller, it's the main reason for this separation.

> What does "common" refer to?
> 

I removed ./flib/common.

> What does the flib directory mean, in the context of Linux?  If someone were
> to add code to this driver, how do they know if the code should go into the
> flib directory or not?
> 
> >
> > +#define iowrite32be(val, addr)               out_be32(&(*addr), val)
> > +#define ioread32be(addr)             in_be32(&(*addr))
> 
> iowrite32be()/ioread32be() are already defined for all relevant architectures.
> Why are you redefining them into something PPC- specific?
> 

Removed those. 

> > +/* do not change! if changed, must be disabled for rev1 ! */ #define
> > +DEFAULT_HALT_ON_EXTERNAL_ACTIVATION  false
> > +/* do not change! if changed, must be disabled for rev1 ! */ #define
> > +DEFAULT_HALT_ON_UNRECOVERABLE_ECC_ERROR false
> 
> rev1 of what chip?
> 

P4080. I'll update the comments. 

> -Scott

Igal.