Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1754051AbdGJMee (ORCPT ); Mon, 10 Jul 2017 08:34:34 -0400 Received: from mail-lf0-f66.google.com ([209.85.215.66]:36418 "EHLO mail-lf0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1753907AbdGJMeb (ORCPT ); Mon, 10 Jul 2017 08:34:31 -0400 Date: Mon, 10 Jul 2017 14:34:27 +0200 From: Johan Hovold To: Anatolij Gustschin Cc: =?iso-8859-1?Q?Bj=F8rn?= Mork , Lee Jones , Linus Walleij , Alan Tull , Moritz Fischer , linux-gpio@vger.kernel.org, linux-fpga@vger.kernel.org, linux-usb@vger.kernel.org, linux-kernel@vger.kernel.org, Johan Hovold Subject: Re: [PATCH 1/3] mfd: Add support for FTDI FT232H devices Message-ID: <20170710123427.GJ29638@localhost> References: <1499374158-12388-1-git-send-email-agust@denx.de> <1499374158-12388-2-git-send-email-agust@denx.de> <87y3s0adr3.fsf@miraculix.mork.no> <20170707115329.06dcdb69@crub> MIME-Version: 1.0 Content-Type: text/plain; charset=iso-8859-1 Content-Disposition: inline Content-Transfer-Encoding: 8bit In-Reply-To: <20170707115329.06dcdb69@crub> User-Agent: Mutt/1.7.2 (2016-11-26) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1266 Lines: 33 On Fri, Jul 07, 2017 at 11:53:29AM +0200, Anatolij Gustschin wrote: > On Fri, 07 Jul 2017 09:48:48 +0200 > Bj?rn Mork bjorn@mork.no wrote: > > >[adding Johan on the CC list] > > > >Anatolij Gustschin writes: > > > >> +static struct usb_device_id ftdi_mfd_table[] = { > >> + { USB_DEVICE(0x0403, 0x6014) }, > >> + {} > >> +}; > >> +MODULE_DEVICE_TABLE(usb, ftdi_mfd_table); > > > >This device ID is currently handled by the ftdi_sio driver, so I believe > >you at least have to explain how you intend these two drivers to > >cooperate... > > these drivers cannot cooperate, the different ftdi function modes > use same device pins as in UART mode. So, you either can use the > device in UART interface mode or in some different mode. I do not > load the ftdi_sio module or do unbind the USB device from the > ftdio_sio driver and bind it to the mfd driver, e.g.: > > sh -c "echo -n "3-2:1.0" > /sys/bus/usb/drivers/ftdi_sio/unbind" > sh -c "echo -n "3-2:1.0" > /sys/bus/usb/drivers/ftdi-mfd/bind" I'm afraid that's not good enough. If we're going to support a non-UART mode through a separate driver, we need to have all drivers for these devices be able to retrieve the current mode during probe and only bind when the mode matches. Johan