[PATCH] staging: fsl-mc: Add missing header

'its_fsl_mc_msi_cleanup' was not declared. Since these are properly declared, but the header is not included, add it in the source files. This way the symbol is properly exported. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- Sent against staging-testing. drivers/staging/fsl-mc/bus/

[PATCH 1/4] staging: android/ion: Fix line over 80 characters

Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/android/ion/ion-ioctl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion-ioctl.c b/drivers/staging/android/ion/ion-ioctl.c index 7e7431d..e28fffb

[PATCH 2/4] staging: android/ion: Don't use return in void function

Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/android/ion/ion_of.c | 1 - 1 file changed, 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_of.c b/drivers/staging/android/ion/ion_of.c index 46b2bb9..7791c70 100644 --- a/drivers/staging/andro

[PATCH 4/4] staging: android/ion: Use variable names in header

Populate header function signatures with variable names as well, not just variable types. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/android/ion/ion_priv.h | 38 +- 1 file changed, 19 insertions(+), 19 deletions(-)

[PATCH 3/4] staging: android/ion: Align comment mark

Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/android/ion/ion_priv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/android/ion/ion_priv.h b/drivers/staging/android/ion/ion_priv.h index 3c3b324..00d8b53

[PATCH] staging: android/ion: Fix coding style issues

Fix following checkpatch warnings: - Lines over 80 characters - void function with return statement - Unaligned comment mark - Header function prototypes missing variable names Sent against staging-next. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/a

Re: [PATCH 1/4] staging: android/ion: Fix line over 80 characters

On Thu, Jan 12, 2017 at 11:56 AM, Greg KH <gre...@linuxfoundation.org> wrote: > On Thu, Jan 12, 2017 at 12:32:26PM +0300, Dan Carpenter wrote: >> On Thu, Jan 12, 2017 at 09:54:16AM +0200, Bogdan Purcareata wrote: >> > Signed-off-by: Bogdan Purcareata <bogdan.purcare...@g

[PATCH v2 3/4] staging: android/ion: Align comment mark

Fix missing space right before multiple line comment closing. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- v1 -> v2 - add changelog drivers/staging/android/ion/ion_priv.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/drivers/staging/an

[PATCH v2 4/4] staging: android/ion: Use variable names in header

Populate header function signatures with variable names as well, not just variable types. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- v1 -> v2: - fix CHECK report from checkpatch drivers/staging/android/ion/ion_priv.h | 38 +

[PATCH v2 1/4] staging: android/ion: Split function call

Fix checkpatch error of line exceeding 80 characters. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- v1 -> v2: - add changelog - fix CHECK reports from checkpatch drivers/staging/android/ion/ion-ioctl.c | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) d

[PATCH v2 2/4] staging: android/ion: Don't use return in void function

Return statements are superfluous in void functions, and checkpatch complains about them too. Remove offending return. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- v1 -> v2 - add changelog drivers/staging/android/ion/ion_of.c | 1 - 1 file changed, 1 deletion

[PATCH] staging: android/ion: Fix coding style issues

Fix following checkpatch warnings: - Lines over 80 characters - void function with return statement - Unaligned comment mark - Header function prototypes missing variable names Signed-off-by: Bogdan Purcareata <bogdan.purcare...@gmail.com> --- drivers/staging/android/ion/ion-ioctl.

[PATCH 2/2] staging: fsl-dpaa2/eth: Error report format fixes

Fix mishaps in error format strings. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c b/drivers/staging/fsl

RE: [PATCH 1/2] staging: fsl-dpaa2/eth: Fix skb use after free

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Thursday, July 20, 2017 2:18 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Ruxandra Ioana Radulescu <ruxandra.radule...@nxp.com>; > gre...@linuxfoundation.org

[PATCH 1/2] staging: fsl-dpaa2/eth: Fix skb use after free

Once a Tx frame descriptor is enqueued, an interrupt might be triggered to process the Tx confirmation and free the skb, hitting a memory use after free when updating the tx_bytes statistic based on skb->len. Use the frame descriptor length instead. Signed-off-by: Bogdan Purcare

[PATCH] staging: fsl-mc/dpio: Skip endianness conversion in portal config

Writing to the register using writel does the CPU to LE conversion down the line, so it's not required here. Doing it breaks portal configuration on big endian kernels. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/staging/fsl-mc/bus/dpio/qbman-portal.

RE: [PATCH] staging: fsl-dpaa2/eth: Remove dead code

ists.infradead.org; Bogdan > Purcareata <bogdan.purcare...@nxp.com>; stuyo...@gmail.com; Laurentiu Tudor > <laurentiu.tu...@nxp.com> > Subject: [PATCH] staging: fsl-dpaa2/eth: Remove dead code > > All possible values of the switch statement are explicitly > hand

RE: [PATCH] staging: fsl-mc/dpio: Propagate error code

ists.infradead.org; Bogdan > Purcareata <bogdan.purcare...@nxp.com>; stuyo...@gmail.com; Laurentiu Tudor > <laurentiu.tu...@nxp.com>; Ruxandra Ioana Radulescu > <ruxandra.radule...@nxp.com>; Roy Pledge <roy.ple...@nxp.com>; Haiying Wang > <haiying.w...@nxp.com&

RE: [RESEND PATCH 2/6] staging: fsl-dpaa2/ethsw: Add Freescale DPAA2 Ethernet Switch driver

> Introduce the DPAA2 Ethernet Switch driver, which manages Datapath Switch > (DPSW) objects discovered on the MC bus. > > Suggested-by: Alexandru Marginean > Signed-off-by: Razvan Stefanescu > --- >

[PATCH v2 3/5] staging: fsl-dpaa2/eth: Don't account SWA in tx_data_offset

When configuring the Tx buffer layout, the software annotation size is mentioned, and MC accounts for it when configuring the frame tx_data_offset. No need to handle it in the driver as well. This results in 64B less memory allocated per frame. Signed-off-by: Bogdan Purcareata <bogdan.purc

[PATCH v2 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> --- v1 -> v2: - changed *_RAW_BUF_SIZE macro to inline function drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 18 + drivers/staging/fsl-dpaa2/ethernet/d

[PATCH v2 2/5] staging: fsl-dpaa2/eth: Split function

From: Ioana Radulescu <ruxandra.radule...@nxp.com> Since setup_dpni() became a bit too long, move the buffer layout configuration to a separate function. Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.co

[PATCH v2 1/5] staging: fsl-dpaa2/eth: Label cleanup

From: Ioana Radulescu <ruxandra.radule...@nxp.com> Clean up goto labels in a couple of functions, by removing/renaming redundant ones. Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- v1 -> v2: -

[PATCH v2 5/5] staging: fsl-dpaa2/eth: Extra headroom in RX buffers

. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> --- v1 -> v2: - changed *_RX_HEAD_ROOM from macro to inline function - since the patch is touching the area, did the same for *_NEEDED_HEADROOM drivers/staging/fsl

[PATCH v2 0/5] staging: fsl-dpaa2/eth: Frame buffer work

to prevent netstack reallocations in forwarding scenarios. Patchset sent against staging-next. v1 -> v2: - clarified one commit message - changed some macros to inline functions - more comments per individual patches Bogdan Purcareata (3): staging: fsl-dpaa2/eth: Don't account SWA in tx_data_off

RE: [PATCH 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Monday, October 30, 2017 10:56 AM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: de...@driverdev.osuosl.org; gre...@linuxfoundation.org; linux- > ker...@vger.kernel.org &

[PATCH 3/5] staging: fsl-dpaa2/eth: Don't account SWA in tx_data_offset

When configuring the Tx buffer layout, the software annotation size is mentioned, and MC accounts for it when configuring the frame tx_data_offset. No need to handle it in the driver as well. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/staging/fsl-dpaa2/et

[PATCH 1/5] staging: fsl-dpaa2/eth: Label cleanup

From: Ioana Radulescu <ruxandra.radule...@nxp.com> Clean up goto labels in a couple of functions, by removing/renaming redundant ones. Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/staging/

[PATCH 2/5] staging: fsl-dpaa2/eth: Split function

From: Ioana Radulescu <ruxandra.radule...@nxp.com> Since setup_dpni() became a bit too long, move the buffer layout configuration to a separate function. Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --

[PATCH 0/5] staging: fsl-dpaa2/eth: Frame buffer work

scenarios. Patchset sent against staging-next. Bogdan Purcareata (3): staging: fsl-dpaa2/eth: Don't account SWA in tx_data_offset staging: fsl-dpaa2/eth: Change RX buffer alignment staging: fsl-dpaa2/eth: Extra headroom in RX buffers Ioana Radulescu (2): staging: fsl-dpaa2/eth: Label cleanup

RE: [PATCH 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Friday, October 27, 2017 5:30 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Ruxandra Ioana Radulescu <ruxandra.radule...@nxp.com>; > gre...@linuxfoundation.org

[PATCH 4/5] staging: fsl-dpaa2/eth: Change RX buffer alignment

-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 18 ++ drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h | 14 +++--- 2 files changed, 25 inse

[PATCH 5/5] staging: fsl-dpaa2/eth: Extra headroom in RX buffers

. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> --- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.c | 79 +++--- drivers/staging/fsl-dpaa2/ethernet/dpaa2-eth.h | 32 +++ 2 files changed, 6

RE: [PATCH 3/5] staging: fsl-dpaa2/eth: Don't account SWA in tx_data_offset

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Friday, October 27, 2017 5:27 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Ruxandra Ioana Radulescu <ruxandra.radule...@nxp.com>; > gre...@linuxfoundation.org

RE: [PATCH 3/5] staging: fsl-dpaa2/eth: Don't account SWA in tx_data_offset

> -Original Message- > From: Dan Carpenter [mailto:dan.carpen...@oracle.com] > Sent: Friday, October 27, 2017 5:34 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Ruxandra Ioana Radulescu <ruxandra.radule...@nxp.com>; > gre...@linuxfoundation.org

RE: [PATCH 3/8] staging: fsl-mc: Add SPDX license identifiers

> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Tuesday, January 16, 2018 5:13 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Laurentiu Tudor <laurentiu.tu...@nxp.com>; Ruxandra Ioana Ciocoi Radulescu > &l

[PATCH v2] staging: fsl-mc: Add SPDX license identifiers

The fsl-mc bus and dpio driver files use a combination of GPL-2.0 and GPL-2.0+ OR BSD-3-Clause licenses. Add SPDX tags and delete the full license text, keeping the existing license for each file. Add GPL-2.0 for Kconfig / Makefile, where not present. Signed-off-by: Bogdan Purcareata

[PATCH 0/8] staging: fsl-mc: Cleanup

process (.rst). [1] https://patchwork.kernel.org/patch/10081731/ Bogdan Purcareata (3): staging: fsl-mc: Cleanup dprc and dpmcp header files staging: fsl-mc: Consolidate bus core header files staging: fsl-mc: Add SPDX license identifiers Ioana Radulescu (5): staging: fsl-mc: Remove dead code

[PATCH 2/8] staging: fsl-mc: Consolidate bus core header files

The DPAA2 objects involved (DPMNG, DPMCP, DPRC) are used by the fsl-mc bus infrastructure only, so group the APIs and structs into fsl-mc-private.h. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Reviewed-by: Laurentiu Tudor <laurentiu.tu...@nxp.com> --- drivers/stagin

[PATCH 3/8] staging: fsl-mc: Add SPDX license identifiers

The fsl-mc bus and dpio driver files use a combination of GPL-2.0 and GPL-2.0+ OR BSD-3-Clause licenses. Update to latter for all files. Add SPDX tags and delete the full license text. Update copyright information. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Re

[PATCH 6/8] staging: fsl-mc: Update include header

From: Ioana Radulescu With the declarations of its_fsl_mc_msi_init/cleanup() now removed from fsl-mc-private.h, irq-gic-v3-its-fsl-mc-msi.c only needs a couple of definitions from mc.h, so include that header directly. Signed-off-by: Ioana Radulescu

[PATCH 1/8] staging: fsl-mc: Cleanup dprc and dpmcp header files

A lot of API and associated structures are not used by current code, so remove them. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Reviewed-by: Laurentiu Tudor <laurentiu.tu...@nxp.com> --- drivers/staging/fsl-mc/bus/dpmcp-cmd.h | 2 - drivers/staging/fsl-mc

[PATCH 5/8] staging: fsl-mc: Remove unnecessary dependency

From: Ioana Radulescu The function that enables fsl-mc msi interrupts doesn't need to be explicitly called from the fsl-mc bus driver initialization routine. Mark it to be independently called at system init; this is in line with how things are handled by other GICv3

[PATCH 4/8] staging: fsl-mc: Remove dead code

From: Ioana Radulescu Remove one unused function from irq-gic-v3-its-fsl-mc-msi.c Signed-off-by: Ioana Radulescu Reviewed-by: Laurentiu Tudor --- drivers/staging/fsl-mc/bus/fsl-mc-private.h | 2 --

[PATCH v5 2/2] staging: fsl-mc: Move irqchip code out of staging

Tudor <laurentiu.tu...@nxp.com> [rebased, split irqchip to separate patch] Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Jason Cooper <ja...@lakedaemon.net> Cc: Marc Zyngier <marc.zyng...@arm.com> --- Notes:

[PATCH v5 0/2] staging: fsl-mc: Move bus driver out of staging

with the rest (Marc Zyngier) Bogdan Purcareata (2): staging: fsl-mc: Move core bus out of staging staging: fsl-mc: Move irqchip code out of staging Documentation/networking/dpaa2/index.rst | 8 + Documentation/networking/dpaa2/overview.rst| 404 + Documentation/networking

RE: [PATCH v5 1/2] staging: fsl-mc: Move core bus out of staging

> -Original Message- > From: Greg KH [mailto:gre...@linuxfoundation.org] > Sent: Friday, January 26, 2018 4:06 PM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com> > Cc: Laurentiu Tudor <laurentiu.tu...@nxp.com>; Ruxandra Ioana Ciocoi Radulescu > <r

[PATCH 7/8] staging: fsl-mc: README cleanup

From: Ioana Radulescu Minor cosmetic changes to the DPAA2 overview documentation file. Add a reference to the mc-bus driver sysfs documentation. Signed-off-by: Ioana Radulescu Reviewed-by: Laurentiu Tudor ---

[PATCH 8/8] staging: fsl-mc: Convert documentation to rst format

From: Ioana Radulescu Update the doc file to comply with the rst format. It's not integrated into the documentation build structure yet, since it's still located in drivers/staging. Signed-off-by: Ioana Radulescu Reviewed-by: Laurentiu

[PATCH v6 0/2] staging: fsl-mc: Move bus driver out of staging

: - split irqchip glue code to separate patch (GregKH) - integrate doc with the kernel build system -v4: - regenerated patch with renames detection disabled (Andrew Lunn) -v3: - rebased -v2: - group irqchip gic its glue code together with the rest (Marc Zyngier) Bogdan

[PATCH v6 2/2] staging: fsl-mc: Move irqchip code out of staging

] Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> [add Kconfig dependency on ARM_GIC_V3_ITS] Signed-off-by: Ioana Radulescu <ruxandra.radule...@nxp.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Jason Cooper <ja...@lakedaemon.net> Cc: Marc Zyngier <m

[PATCH v6 1/2] staging: fsl-mc: Move core bus out of staging

pdates] Signed-off-by: Laurentiu Tudor <laurentiu.tu...@nxp.com> [rebased, split irqchip to separate patch] Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Cc: Thomas Gleixner <t...@linutronix.de> Cc: Jason Cooper <ja...@lakedaemon.net> Cc: Marc Zyngier <marc.zy

RE: [PATCH v6 1/2] staging: fsl-mc: Move core bus out of staging

> -Original Message- > From: Horia Geantă > Sent: Tuesday, February 06, 2018 9:18 AM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com>; gre...@linuxfoundation.org; > Laurentiu Tudor <laurentiu.tu...@nxp.com>; Ruxandra Ioana Ciocoi Radulescu > <ruxand

[PATCH 0/3] staging: fsl-mc: Move DPBP and DPCON out of staging

. Patch 1 removes dead code from the involved source files. Patches 2-3 handle moving out the mentioned objects, one at a time. Bogdan Purcareata (3): staging: fsl-mc: Cleanup dpbp and dpcon API staging: fsl-mc: Move DPBP out of staging staging: fsl-mc: Move DPCON out of staging drivers/

[PATCH 2/3] staging: fsl-mc: Move DPBP out of staging

, exposing the public API Update references in the dpaa2-eth staging driver. DPBP stands for Data Path Buffer Pool - you can read more about the object in Documentation/networking/dpaa2/overview.rst Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/bus/fsl-mc/Ma

[PATCH 3/3] staging: fsl-mc: Move DPCON out of staging

, exposing the public API Update references in the dpaa2-eth staging driver. DPCON stands for Data Path Concentrator - an interface between DPIO (Data Path IO) and its users (e.g. dpaa2-eth). You can read more about DPIO in Documentation/networking/dpaa2/overview.rst Signed-off-by: Bogdan

[PATCH 1/3] staging: fsl-mc: Cleanup dpbp and dpcon API

Some functions and associated structures are not used by current code, so remove them. Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> --- drivers/staging/fsl-mc/bus/dpbp-cmd.h | 10 -- drivers/staging/fsl-mc/bus/dpbp.c | 67 --- d

[PATCH v2 1/2] staging: fsl-mc: Move DPBP out of staging

, exposing the public API Update references in the dpaa2-eth staging driver. DPBP stands for Data Path Buffer Pool - you can read more about the object in Documentation/networking/dpaa2/overview.rst Signed-off-by: Bogdan Purcareata <bogdan.purcare...@nxp.com> Reviewed-by: Laurentiu Tudor <lau

RE: [PATCH 2/3] staging: fsl-mc: Move DPBP out of staging

> -Original Message- > From: Laurentiu Tudor > Sent: Friday, March 02, 2018 11:18 AM > To: Bogdan Purcareata <bogdan.purcare...@nxp.com>; gre...@linuxfoundation.org; > Ruxandra Ioana Ciocoi Radulescu <ruxandra.radule...@nxp.com> > Cc: stuyo...@gmail.

[PATCH v2 0/2] staging: fsl-mc: Move DPBP and DPCON out of staging

(Data Path Buffer Pool) and DPCON (Data Path Concentrator). The patches handle moving out the mentioned objects, one at a time. v1 -> v2: - previous cleanup patch already applied, so removed from patchset - properly refactor #include line (Laurentiu) Bogdan Purcareata (2): staging: fsl-mc: