From: Liberman Igal <Igal.Liberman@freescale.com>
To: Scott Wood <scottwood@freescale.com>,
        Madalin-Cristian Bucur <madalin.bucur@freescale.com>
CC: "netdev@vger.kernel.org" <netdev@vger.kernel.org>,
        "linux-kernel@vger.kernel.org" <linux-kernel@vger.kernel.org>,
        "linuxppc-dev@lists.ozlabs.org" <linuxppc-dev@lists.ozlabs.org>
Subject: RE: [PATCH 01/12] fsl/fman: Add the FMan FLIB headers
Thread-Topic: [PATCH 01/12] fsl/fman: Add the FMan FLIB headers
Thread-Index: AQHQo5FecOR+w2bcbEyghvBWm/vSsp2mFoQAgAq5nnA=
Date: Wed, 17 Jun 2015 14:59:37 +0000
Message-ID: <DM2PR03MB3830ADD2B503C6738BB27E7E6A60@DM2PR03MB383.namprd03.prod.outlook.com>
References: <1433949712-5648-1-git-send-email-madalin.bucur@freescale.com>
	 <1433949712-5648-2-git-send-email-madalin.bucur@freescale.com>
	 <1433949712-5648-3-git-send-email-madalin.bucur@freescale.com>
 <1433962446.2477.121.camel@freescale.com>
In-Reply-To: <1433962446.2477.121.camel@freescale.com>
Accept-Language: en-US
Content-Language: en-US
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
X-MS-Exchange-CrossTenant-originalarrivaltime: 17 Jun 2015 14:59:38.0209
 (UTC)
X-MS-Exchange-CrossTenant-fromentityheader: Hosted
X-MS-Exchange-CrossTenant-id: 710a03f5-10f6-4d38-9ff4-a80b81da590d
X-MS-Exchange-Transport-CrossTenantHeadersStamped: DM2PR03MB559
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org
Content-Transfer-Encoding: 8bit
X-MIME-Autoconverted: from base64 to 8bit by nfs id t5HFEKaI010494
Content-Length: 2253
Lines: 70


Regards,
Igal Liberman.

> -----Original Message-----
> From: Wood Scott-B07421
> Sent: Wednesday, June 10, 2015 9:54 PM
> To: Bucur Madalin-Cristian-B32716
> Cc: netdev@vger.kernel.org; linux-kernel@vger.kernel.org; linuxppc-
> dev@lists.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@freescale.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@freescale.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.

????{.n?+???????+%?????ݶ??w??{.n?+????{??G?????{ay?ʇڙ?,j??f???h?????????z_??(?階?ݢj"???m??????G????????????&???~???iO???z??v?^?m????????????I?