From mboxrd@z Thu Jan 1 00:00:00 1970 From: Andrew Lunn Subject: Re: [PATCH v3 2/4] bus: fsl-mc: add restool userspace support Date: Thu, 5 Apr 2018 14:48:10 +0200 Message-ID: <20180405124810.GE12178@lunn.ch> References: <20180404010549.GA12443@lunn.ch> <20180404124246.GA20869@lunn.ch> <5AC5FAA8.80409@nxp.com> <20180405114736.GA12178@lunn.ch> <5AC61393.7090509@nxp.com> Mime-Version: 1.0 Content-Type: text/plain; charset=us-ascii Cc: Stuart Yoder , Arnd Bergmann , Ioana Ciornei , gregkh , Linux Kernel Mailing List , Ruxandra Ioana Ciocoi Radulescu , Razvan Stefanescu , Roy Pledge , Networking To: Laurentiu Tudor Return-path: Received: from vps0.lunn.ch ([185.16.172.187]:51653 "EHLO vps0.lunn.ch" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751097AbeDEMsQ (ORCPT ); Thu, 5 Apr 2018 08:48:16 -0400 Content-Disposition: inline In-Reply-To: <5AC61393.7090509@nxp.com> Sender: netdev-owner@vger.kernel.org List-ID: > > Hi Laurentiu > > > > So i can use switchdev without it? I can modprobe the switchdev > > driver, all the physical interfaces will appear, and i can use ip addr > > add etc. I do not need to use a user space tool at all in order to use > > the network functionality? > > Absolutely! Great. Then the easiest way forwards is to simply drop the IOCTL code for the moment. Get the basic support for the hardware into the kernel first. Then come back later to look at dynamic behaviour which needs some form of configuration. > In normal use cases the system designer, depending on the requirements, > configures the various devices that it desires through a firmware > configuration (think something like a device tree). The devices > configured are presented on the mc-bus and probed normally by the > kernel. The standard networking linux tools can be used as expected. So what you should probably do is start a discussion on what this device tree binding looks like. But you need to be careful even here. Device tree describes the hardware, not how you configure the hardware. So maybe DT does not actually fit. Andrew