Date: Thu, 14 Dec 2017 18:48:08 +0530
From: Vinod Koul <vinod.koul@intel.com>
To: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
Cc: Takashi <tiwai@suse.de>, LKML <linux-kernel@vger.kernel.org>,
        ALSA <alsa-devel@alsa-project.org>, Mark <broonie@kernel.org>,
        Pierre <pierre-louis.bossart@linux.intel.com>, patches.audio@intel.com,
        alan@linux.intel.com, Charles Keepax <ckeepax@opensource.cirrus.com>,
        Sagar Dharia <sdharia@codeaurora.org>, srinivas.kandagatla@linaro.org,
        plai@codeaurora.org, Sudheer Papothi <spapothi@codeaurora.org>
Subject: Re: [PATCH v6 00/14] soundwire: Add a new SoundWire subsystem
Message-ID: <20171214131808.GZ18649@localhost>
References: <1513230585-13882-1-git-send-email-vinod.koul@intel.com>
 <20171214073508.GA29585@kroah.com>
 <20171214105320.GX18649@localhost>
 <20171214124949.GA12935@kroah.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20171214124949.GA12935@kroah.com>
User-Agent: Mutt/1.5.24 (2015-08-30)
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 2160
Lines: 46

On Thu, Dec 14, 2017 at 01:49:49PM +0100, Greg Kroah-Hartman wrote:
> On Thu, Dec 14, 2017 at 04:23:20PM +0530, Vinod Koul wrote:
> > On Thu, Dec 14, 2017 at 08:35:08AM +0100, Greg Kroah-Hartman wrote:
> > > On Thu, Dec 14, 2017 at 11:19:31AM +0530, Vinod Koul wrote:
> > > > Changes in v6:
> > > >   - Add reviewed/acked tags from Philippe, Pierre, Takashi and Greg
> > > >   - Fix nitpicks from Takashi
> > > >   - Drop the sysfs patch for now
> > > 
> > > Wait, why drop the sysfs patch entirely?  You need those attributes,
> > > right?  You also need to document the existing sysfs files in
> > > Documentation/ABI/ for the class/device files you create in this patch
> > > series, so that needs to be done before this patch series can be
> > > accepted.
> > 
> > Well the sysfs patch is a standalone patch which contains the attributes. So
> > I dropped that "for this series" and plan to submit after reworking.
> > 
> > The rest of the code doesn't depend on it, so is fine.
> > 
> > I thought it is an okay kernel practice to drop patches with issues, merge the
> > rest and come back after fixing the issue. This is my plan for this as well.
> 
> Ok, but that feels a bit rushed, as I'm guessing those sysfs files are
> essencial for the use of the hardware.  To not include them feels odd to
> me :)

Ah, since folks acked and reviewed the rest of part, I felt we can merge the
big pile while I started changing the sysfs files, so sent the rest :)

This is basic IO patch series so stuff still works without the sysfs parts.
The second series has audio streaming parts that is yet to be published so
there is _still_ a lot of work left for soundwire and this is just a start.

> Anyway, what about the basic ABI documentation for the sysfs files you
> have in this series?  You need to have that patch in this series,
> otherwise it is really hard to get people to do it after the fact.

Sorry to disappoint but this series does not have any sysfs files. All the
sysfs file were in the sysfs patch which is now getting reworked as we speak
and added with ABI documentation. I should be able to post those in next
couple of days.

Thanks
-- 
~Vinod