From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753022AbbCJRj7 (ORCPT ); Tue, 10 Mar 2015 13:39:59 -0400 Received: from mail-pa0-f52.google.com ([209.85.220.52]:41179 "EHLO mail-pa0-f52.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752011AbbCJRj5 (ORCPT ); Tue, 10 Mar 2015 13:39:57 -0400 Date: Tue, 10 Mar 2015 10:39:48 -0700 From: Brian Norris To: Frans Klaver Cc: David Woodhouse , "linux-kernel@vger.kernel.org" , linux-mtd@lists.infradead.org Subject: Re: [PATCH 01/60] mtd: core: tone down suggestion that dev.parent should be set Message-ID: <20150310173948.GB4124@norris-Latitude-E6410> References: <1425418844-25177-1-git-send-email-fransklaver@gmail.com> <1425418844-25177-2-git-send-email-fransklaver@gmail.com> <20150309230250.GA13266@ld-irv-0074> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Tue, Mar 10, 2015 at 08:47:46AM +0100, Frans Klaver wrote: > On Tue, Mar 10, 2015 at 12:14 AM, Brian Norris > wrote: > > On Tue, Mar 03, 2015 at 10:39:45PM +0100, Frans Klaver wrote: > >> add_mtd_device() has a comment suggesting that the caller should have > >> set dev.parent. This is required to have the device show up in sysfs, > > > > What do you mean "have the device show up in sysfs"? AFAICT, this only > > has bearing on whether the *parent* device shows up as a sysfs symlink > > within the MTD device directory. i.e.: > > > > /sys/class/mtd/mtd*/device > > > > For instance, this sort of symlink: > > > > /sys/class/mtd/mtd0/device -> ../../../f03e2800.nand > > > > It might be good to clarify this in the commit message, since you make > > the problem sound worse than (I think) it is. > > I do? That was definitely not my intention. I'll look into it. Maybe it's just my bias when reading, since some people have complained loudly about this, seemingly without understanding that the problem really isn't that significant. So my question was really just to confirm my own understanding, that this only affects the 'device' symlink. BTW, it'd be nice if you don't respam with another 60 patches, if you're only changing a few of them. I can probably take most of them as-is, after you confirm there are no more compile failures. Thanks, Brian