From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753196Ab2GPCqL (ORCPT ); Sun, 15 Jul 2012 22:46:11 -0400 Received: from void.printf.net ([89.145.121.20]:40606 "EHLO void.printf.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750860Ab2GPCqI (ORCPT ); Sun, 15 Jul 2012 22:46:08 -0400 From: Chris Ball To: Muthu Kumar Cc: merez@codeaurora.org, linux-mmc@vger.kernel.org, linux-arm-msm@vger.kernel.org, DOCUMENTATION , open list Subject: Re: [PATCH v2 1/1] mmc: block: Add write packing control References: <1338576911-17089-1-git-send-email-merez@codeaurora.org> <1338576911-17089-2-git-send-email-merez@codeaurora.org> <87629qkvgp.fsf@octavius.laptop.org> Date: Sun, 15 Jul 2012 22:46:01 -0400 In-Reply-To: (Muthu Kumar's message of "Sun, 15 Jul 2012 18:49:09 -0700") Message-ID: <87pq7wjudy.fsf@octavius.laptop.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.0.97 (gnu/linux) MIME-Version: 1.0 Content-Type: text/plain Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi, On Sun, Jul 15 2012, Muthu Kumar wrote: >> I've already replied to a later version of the patch, but just to get >> this comment in at the appropriate point of the discussion as well: >> >> Even though it would result in a cleaner sysfs, I don't want to do >> this now because it will break userspace scripts that are depending >> on the current locations of these attributes. > > Maya is adding a new sysfs attribute with that patch. So, there should > not be any user space stuff that depends on it. In the later patchset, Maya's "[PATCH v4 1/2] mmc: card: Move MMC specific attributes to mmc sub-directory" moves the existing attributes into the mmc/ directory. It's that move that I'm objecting to, rather than the creation of a new directory -- although since we're going to leave the current attributes where they are, it might not make sense to add the new directory. We'd be creating two places that people have to look for mmc-related attributes, which is arguably less clean than having one place to look even though it's mixed in with the other block device attributes. Thanks, - Chris. -- Chris Ball One Laptop Per Child