Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1760946Ab2J2Uuy (ORCPT ); Mon, 29 Oct 2012 16:50:54 -0400 Received: from void.printf.net ([89.145.121.20]:57000 "EHLO void.printf.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751753Ab2J2Uuv (ORCPT ); Mon, 29 Oct 2012 16:50:51 -0400 From: Chris Ball To: "Kirill A. Shutemov" Cc: Anton Vorontsov , Colin Cross , Kees Cook , Tony Luck , linux-kernel@vger.kernel.org, Adrian Hunter , Octavian Purdila , Irina Tirdea , linux-mmc@vger.kernel.org Subject: Re: [PATCH 00/26] pstore, mmc: add mmc as backend for pstore References: <1351000104-13015-1-git-send-email-irina.tirdea@intel.com> <20121029202820.GA21660@shutemov.name> Date: Mon, 29 Oct 2012 16:50:42 -0400 In-Reply-To: <20121029202820.GA21660@shutemov.name> (Kirill A. Shutemov's message of "Mon, 29 Oct 2012 22:28:20 +0200") Message-ID: <87390xroil.fsf@octavius.laptop.org> User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/24.1 (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 Content-Length: 1266 Lines: 35 Hi, On Mon, Oct 29 2012, Kirill A. Shutemov wrote: > On Tue, Oct 23, 2012 at 04:47:58PM +0300, Irina Tirdea wrote: >> These patches enable using the mmc card to store panic information. >> >> They include changes for pstore and mmc: >> - add block device backend for pstore >> - add logic in mmc for writing in panic mode > > Chris, any feedback on mmc/sdhci part? Looks like the patchset hasn't been sent to linux-mmc@vger.kernel.org yet -- could you send a v2 with that address included, please? It would also be great to see descriptive commit messages, rather than just patch subject lines. Documentation-wise, the patchset mentions that host controller support is required for this to work, but doesn't explain what kind of support that is, and the sdhci-pci support seems unconditional on type of host controller -- why wouldn't every controller work? Is it (just) because other controller drivers use msleep? Thanks, - Chris. -- Chris Ball One Laptop Per Child -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/