From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1424172AbdEYNra (ORCPT ); Thu, 25 May 2017 09:47:30 -0400 Received: from fllnx210.ext.ti.com ([198.47.19.17]:48742 "EHLO fllnx210.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S969144AbdEYNr1 (ORCPT ); Thu, 25 May 2017 09:47:27 -0400 Subject: Re: [PATCH v2 1/1] w1: Add subsystem kernel public interface To: Evgeniy Polyakov , Greg Kroah-Hartman References: <20170516200814.12360-1-afd@ti.com> <20170516200814.12360-2-afd@ti.com> <20170525130056.GB30447@kroah.com> <1399ebdb-337e-43b7-9e4d-a0c088bc8c7b@ti.com> <1499591495717767@web38g.yandex.ru> <299fe87d-d0fd-6f16-2f7e-ba56096e5021@ti.com> <1604131495718924@web38g.yandex.ru> CC: Sebastian Reichel , , From: "Andrew F. Davis" Message-ID: Date: Thu, 25 May 2017 08:47:19 -0500 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.8.0 MIME-Version: 1.0 In-Reply-To: <1604131495718924@web38g.yandex.ru> Content-Type: text/plain; charset="utf-8" Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 05/25/2017 08:28 AM, Evgeniy Polyakov wrote: > > > 25.05.2017, 16:22, "Andrew F. Davis" : > >>> Why does BQ27xxx need to move out of w1 tree? >> >> Currently we have to enable a pseudo-platform device driver in the >> power/supply BQ27xxx driver, then the w1 driver has to instantiate this >> platform device and then they connect and communicate by sharing >> callbacks. This is rather hacky. > > Why do you have to create a pseudo-platform device driver to connect w1 and power/supply? > > I'm not against creating w1 drivers in different places than drivers/w1, but so far > it was only power drivers which have problem with it (and they easily work it out), > and this rises a flag. > We could keep it in w1 if we really wanted, but then things like Kconfig will get difficult to manage (we will jump between menus and have odd dependencies). The other w1/slaves seem to mostly be simple EEPROMs and Gauges that would otherwise end up in misc/ so it is fine if they live in w1, but BQ27xxx does have a proper home in power/supplies and its i2c interface is already there, so moving the w1 interface there also makes sense to me. > I would rather move w1 header into include/linux, will it be enough? > That's what this patch does, we just also re-organize things a bit so only things that need to be public end up in include/linux. It seems to be all that is needed for my use-case at least.