From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S966061AbdEYNH7 (ORCPT ); Thu, 25 May 2017 09:07:59 -0400 Received: from fllnx210.ext.ti.com ([198.47.19.17]:47319 "EHLO fllnx210.ext.ti.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751159AbdEYNH5 (ORCPT ); Thu, 25 May 2017 09:07:57 -0400 Subject: Re: [PATCH v2 1/1] w1: Add subsystem kernel public interface To: Greg Kroah-Hartman References: <20170516200814.12360-1-afd@ti.com> <20170516200814.12360-2-afd@ti.com> <20170525130056.GB30447@kroah.com> CC: Evgeniy Polyakov , Sebastian Reichel , , From: "Andrew F. Davis" Message-ID: <1399ebdb-337e-43b7-9e4d-a0c088bc8c7b@ti.com> Date: Thu, 25 May 2017 08:07:50 -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: <20170525130056.GB30447@kroah.com> Content-Type: text/plain; charset="windows-1252" 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:00 AM, Greg Kroah-Hartman wrote: > On Tue, May 16, 2017 at 03:08:14PM -0500, Andrew F. Davis wrote: >> Like other subsystems we should be able to define slave devices outside >> of the w1 directory. To do this we move public facing interface >> definitions to include/linux/w1.h and rename the internal definition >> file to w1_internal.h. >> >> As w1_family.h and w1_int.h contained almost entirely public >> driver interface definitions we simply removed these files and >> moved the remaining definitions into w1_internal.h. >> >> With this we can now start to move slave devices out of w1/slaves and >> into the subsystem based on the function they implement, again like >> other drivers. > > Do you have a patch that does that? I don't want to accept this patch > if that never happens... > Yes, this reorganization stems from the need to move the BQ27xxx slave out, patches were posted and reviewed last cycle here[0] and will be reposted when this patch is accepted. Other w1/slaves can be moved out at their owners convenience if/when they wish. Thanks, Andrew [0] https://lkml.org/lkml/2017/3/16/688 > thanks, > > greg k-h >