From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1755054AbbHXRl0 (ORCPT ); Mon, 24 Aug 2015 13:41:26 -0400 Received: from mga14.intel.com ([192.55.52.115]:32241 "EHLO mga14.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751179AbbHXRlY (ORCPT ); Mon, 24 Aug 2015 13:41:24 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.15,739,1432623600"; d="scan'208";a="790021017" Date: Mon, 24 Aug 2015 11:40:42 -0600 From: Ross Zwisler To: Jeff Moyer Cc: Ross Zwisler , linux-kernel@vger.kernel.org, linux-nvdimm@ml01.01.org, "Rafael J. Wysocki" , linux-acpi@vger.kernel.org, Len Brown Subject: Re: [PATCH] nfit, nd_blk: BLK status register is only 32 bits Message-ID: <20150824174042.GA18754@linux.intel.com> References: <1440109658-29164-1-git-send-email-ross.zwisler@linux.intel.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Aug 24, 2015 at 01:02:17PM -0400, Jeff Moyer wrote: > Ross Zwisler writes: > > > Only read 32 bits for the BLK status register in read_blk_stat(). > > > > The format and size of this register is defined in the > > "NVDIMM Driver Writer's guide": > > > > http://pmem.io/documents/NVDIMM_Driver_Writers_Guide.pdf > > > > Signed-off-by: Ross Zwisler > > Reported-by: Nicholas Moulin > > Looks fine, > > Reviewed-by: Jeff Moyer > > However, now that you've drawn attention to that code, I'll note that > there is no checking of the pending or retry bits. In fact, > ACPI_NFIT_CONTROL_BUFFERED isn't even checked upon loading the tables. > Is this on a todo list somewhere? Yep, you're right, we do need to add support for each of these. I'm planning on doing it unless someone else gets to it first.