Re: [PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Laura Abbott
On 09/19/2017 04:55 AM, Benjamin Gaignard wrote: 2017-09-19 13:02 GMT+02:00 Greg KH : On Tue, Sep 19, 2017 at 12:25:38PM +0200, Benjamin Gaignard wrote: Instead a getting one common device "/dev/ion" for all the heaps this patch allow to create one device entry

[PATCH 1/6] staging: fsl-dpaa2/ethsw: Add APIs for DPSW object

2017-09-19 Thread Razvan Stefanescu
Add the command build/parse APIs for operating on DPSW objects through the DPAA2 Management Complex. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/Kconfig |8 + drivers/staging/fsl-dpaa2/Makefile |1 +

[PATCH 0/6] staging: Introduce DPAA2 Ethernet Switch driver

2017-09-19 Thread Razvan Stefanescu
This patchset introduces the Ethernet Switch Driver for Freescale/NXP SoCs with DPAA2 (DataPath Acceleration Architecture v2). The driver manages switch objects discovered on the fsl-mc bus. A description of the driver can be found in the associated README file. The patchset consists of: * A set

[PATCH 3/6] staging: fsl-dpaa2/ethsw: Add ethtool support

2017-09-19 Thread Razvan Stefanescu
Add driver information, link details and hardware statistics to be reported via ethtool -S. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/Makefile| 2 +- drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h | 13 ++

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

2017-09-19 Thread Razvan Stefanescu
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 --- drivers/staging/fsl-dpaa2/ethsw/Makefile |2 +-

[PATCH 4/6] staging: fsl-dpaa2/ethsw: Add maintainer for Ethernet Switch driver

2017-09-19 Thread Razvan Stefanescu
Signed-off-by: Razvan Stefanescu --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2281af4..cfd4f74 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4297,6 +4297,12 @@ L: linux-ker...@vger.kernel.org S:

[PATCH 5/6] staging: fsl-dpaa2/ethsw: Add README

2017-09-19 Thread Razvan Stefanescu
Add a README file describing the driver architecture, components and interfaces. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/README | 106 + 1 file changed, 106 insertions(+) create mode 100644

[PATCH 6/6] staging: fsl-dpaa2/ethsw: Add TODO

2017-09-19 Thread Razvan Stefanescu
Add a TODO file describing what needs to be added/changed before the driver can be moved out of staging. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/TODO | 14 ++ 1 file changed, 14 insertions(+) create mode 100644

[PATCH] [media] staging: atomisp: use clock framework for camera clocks

2017-09-19 Thread Pierre-Louis Bossart
The Atom ISP driver initializes and configures PMC clocks which are already handled by the clock framework. Remove all legacy vlv2_platform_clock stuff and move to the clk API to avoid conflicts, e.g. with audio machine drivers enabling the MCLK for external codecs Tested-by: Carlo Caione

Re: [PATCH] staging: ion: create one device entry per heap

2017-09-19 Thread Tomas Winkler
On Tue, Sep 19, 2017 at 1:07 PM, Benjamin Gaignard wrote: > 2017-09-19 11:40 GMT+02:00 Dan Carpenter : >> On Mon, Sep 18, 2017 at 04:58:46PM +0200, Benjamin Gaignard wrote: >>> -static int validate_ioctl_arg(unsigned int cmd, union

Re: [PATCH] staging: ion: create one device entry per heap

2017-09-19 Thread Greg Kroah-Hartman
On Tue, Sep 19, 2017 at 12:20:15PM +0200, Benjamin Gaignard wrote: > 2017-09-19 12:15 GMT+02:00 Tomas Winkler : > > On Tue, Sep 19, 2017 at 1:07 PM, Benjamin Gaignard > > wrote: > >> 2017-09-19 11:40 GMT+02:00 Dan Carpenter

Re: [PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Dan Carpenter
If you had spelled out how this patch breaks user space in the changelog then you wouldn't be catching so much flak for it now... You should fix that in v3. regards, dan carpenter ___ devel mailing list de...@linuxdriverproject.org

Re: [PATCH 0/6] staging: Introduce DPAA2 Ethernet Switch driver

2017-09-19 Thread Andrew Lunn
On Tue, Sep 19, 2017 at 12:01:32PM +0300, Razvan Stefanescu wrote: > This patchset introduces the Ethernet Switch Driver for Freescale/NXP SoCs > with DPAA2 (DataPath Acceleration Architecture v2). The driver manages > switch objects discovered on the fsl-mc bus. A description of the driver > can

[PATCH v2 0/2] staging: ion: get one device per heap

2017-09-19 Thread Benjamin Gaignard
version 2: - simplify ioctl check like propose by Dan - make sure that we don't register more than ION_DEV_MAX heaps Instead a getting one common device "/dev/ion" for all the heaps this patch allow to create one device entry ("/dev/ionX") per heap. Getting an entry per heap could allow to set

[PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Benjamin Gaignard
Instead a getting one common device "/dev/ion" for all the heaps this patch allow to create one device entry ("/dev/ionX") per heap. Getting an entry per heap could allow to set security rules per heap and global ones for all heaps. Allocation requests will be only allowed if the mask_id match

[PATCH v2 1/2] staging: ion: simplify ioctl args checking function

2017-09-19 Thread Benjamin Gaignard
Make arguments checking more easy to read. Signed-off-by: Benjamin Gaignard --- drivers/staging/android/ion/ion-ioctl.c | 11 +-- 1 file changed, 5 insertions(+), 6 deletions(-) diff --git a/drivers/staging/android/ion/ion-ioctl.c

Re: [PATCH] staging: ion: create one device entry per heap

2017-09-19 Thread Benjamin Gaignard
2017-09-19 11:40 GMT+02:00 Dan Carpenter : > On Mon, Sep 18, 2017 at 04:58:46PM +0200, Benjamin Gaignard wrote: >> -static int validate_ioctl_arg(unsigned int cmd, union ion_ioctl_arg *arg) >> +static int validate_ioctl_arg(struct file *filp, >> +

Re: [PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Greg KH
On Tue, Sep 19, 2017 at 12:25:38PM +0200, Benjamin Gaignard wrote: > Instead a getting one common device "/dev/ion" for > all the heaps this patch allow to create one device > entry ("/dev/ionX") per heap. > Getting an entry per heap could allow to set security rules > per heap and global ones for

Re: [PATCH 5/6] staging: fsl-dpaa2/ethsw: Add README

2017-09-19 Thread Andrew Lunn
On Tue, Sep 19, 2017 at 12:01:37PM +0300, Razvan Stefanescu wrote: > +Driver uses the switch device driver model and exposes each switch port as > +a network interface, which can be included in a bridge. Traffic switched > +between ports is offloaded into the hardware. Exposed network interfaces >

Re: [PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Greg KH
On Tue, Sep 19, 2017 at 01:55:36PM +0200, Benjamin Gaignard wrote: > >> + > >> spin_lock_init(>free_lock); > >> heap->free_list_size = 0; > >> > >> @@ -595,13 +610,9 @@ static int ion_device_create(void) > >> if (!idev) > >> return -ENOMEM; > >> > >> -

Re: [PATCH] staging: ion: create one device entry per heap

2017-09-19 Thread Benjamin Gaignard
2017-09-19 12:15 GMT+02:00 Tomas Winkler : > On Tue, Sep 19, 2017 at 1:07 PM, Benjamin Gaignard > wrote: >> 2017-09-19 11:40 GMT+02:00 Dan Carpenter : >>> On Mon, Sep 18, 2017 at 04:58:46PM +0200, Benjamin Gaignard wrote:

Re: [PATCH v2 04/11] usb: xhci: Add Intel cherrytrail extended cap / otg phy mux handling

2017-09-19 Thread Mathias Nyman
Hi, sorry about the long delay On 07.09.2017 18:49, Hans de Goede wrote: Hi, On 07-09-17 15:14, Mathias Nyman wrote: On 05.09.2017 19:42, Hans de Goede wrote: The Intel cherrytrail xhci controller has an extended cap mmio-range which contains registers to control the muxing to the xhci

Re: [PATCH] staging: ion: create one device entry per heap

2017-09-19 Thread Dan Carpenter
On Mon, Sep 18, 2017 at 04:58:46PM +0200, Benjamin Gaignard wrote: > -static int validate_ioctl_arg(unsigned int cmd, union ion_ioctl_arg *arg) > +static int validate_ioctl_arg(struct file *filp, > + unsigned int cmd, union ion_ioctl_arg *arg) > { > int ret = 0; >

Re: [PATCH v2 2/2] staging: ion: create one device entry per heap

2017-09-19 Thread Benjamin Gaignard
2017-09-19 13:02 GMT+02:00 Greg KH : > On Tue, Sep 19, 2017 at 12:25:38PM +0200, Benjamin Gaignard wrote: >> Instead a getting one common device "/dev/ion" for >> all the heaps this patch allow to create one device >> entry ("/dev/ionX") per heap. >> Getting an entry

Re: [PATCH v2 1/2] include: linux: sysfs: Add __ATTR_NAMED macro

2017-09-19 Thread Lars-Peter Clausen
On 09/13/2017 11:29 PM, Greg KH wrote: > On Wed, Sep 13, 2017 at 09:23:31PM +0200, Lars-Peter Clausen wrote: >> On 09/13/2017 08:58 PM, Greg KH wrote: >>> On Wed, Sep 13, 2017 at 06:03:10PM +0100, Jonathan Cameron wrote: On Wed, 13 Sep 2017 14:14:07 +0530 Himanshi Jain

[RESEND PATCH 1/6] staging: fsl-dpaa2/ethsw: Add APIs for DPSW object

2017-09-19 Thread Razvan Stefanescu
Add the command build/parse APIs for operating on DPSW objects through the DPAA2 Management Complex. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/Kconfig |8 + drivers/staging/fsl-dpaa2/Makefile |1 +

[RESEND PATCH 6/6] staging: fsl-dpaa2/ethsw: Add TODO

2017-09-19 Thread Razvan Stefanescu
Add a TODO file describing what needs to be added/changed before the driver can be moved out of staging. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/TODO | 14 ++ 1 file changed, 14 insertions(+) create mode 100644

[RESEND PATCH 4/6] staging: fsl-dpaa2/ethsw: Add maintainer for Ethernet Switch driver

2017-09-19 Thread Razvan Stefanescu
Signed-off-by: Razvan Stefanescu --- MAINTAINERS | 6 ++ 1 file changed, 6 insertions(+) diff --git a/MAINTAINERS b/MAINTAINERS index 2281af4..cfd4f74 100644 --- a/MAINTAINERS +++ b/MAINTAINERS @@ -4297,6 +4297,12 @@ L: linux-ker...@vger.kernel.org S:

[RESEND PATCH 5/6] staging: fsl-dpaa2/ethsw: Add README

2017-09-19 Thread Razvan Stefanescu
Add a README file describing the driver architecture, components and interfaces. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/README | 106 + 1 file changed, 106 insertions(+) create mode 100644

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

2017-09-19 Thread Razvan Stefanescu
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 --- drivers/staging/fsl-dpaa2/ethsw/Makefile |2 +-

[RESEND PATCH 0/6] staging: Introduce DPAA2 Ethernet Switch driver

2017-09-19 Thread Razvan Stefanescu
This patchset introduces the Ethernet Switch Driver for Freescale/NXP SoCs with DPAA2 (DataPath Acceleration Architecture v2). The driver manages switch objects discovered on the fsl-mc bus. A description of the driver can be found in the associated README file. The patchset consists of: * A set

[RESEND PATCH 3/6] staging: fsl-dpaa2/ethsw: Add ethtool support

2017-09-19 Thread Razvan Stefanescu
Add driver information, link details and hardware statistics to be reported via ethtool -S. Signed-off-by: Razvan Stefanescu --- drivers/staging/fsl-dpaa2/ethsw/Makefile| 2 +- drivers/staging/fsl-dpaa2/ethsw/dpsw-cmd.h | 13 ++

Re: [PATCH v2 05/11] mux: Add Intel Cherrytrail USB mux driver

2017-09-19 Thread Hans de Goede
Hi, Thank you for the reviews and patches! On 09/08/2017 05:45 PM, Peter Rosin wrote: On 2017-09-05 18:42, Hans de Goede wrote: Intel Cherrytrail SoCs have an internal USB mux for muxing the otg-port USB data lines between the xHCI host controller and the dwc3 gadget controller. On some

Re: [PATCH 1/2] mux: add mux_control_get_optional() API

2017-09-19 Thread Hans de Goede
Hi, On 09/08/2017 05:54 PM, Peter Rosin wrote: On 2017-09-08 17:45, Peter Rosin wrote: From: Stephen Boyd Sometimes drivers only use muxes under certain scenarios. For example, the chipidea usb controller may be connected to a usb switch on some platforms, and