From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753433AbdBTNgA (ORCPT ); Mon, 20 Feb 2017 08:36:00 -0500 Received: from mail-pg0-f66.google.com ([74.125.83.66]:35942 "EHLO mail-pg0-f66.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752726AbdBTNf4 (ORCPT ); Mon, 20 Feb 2017 08:35:56 -0500 Reply-To: minyard@acm.org Subject: Re: [PATCH] ipmi: bt-bmc: Use a regmap for register access References: <20161206025715.2002-1-andrew@aj.id.au> <36014e0c-82f8-eeaf-ee2a-6c1e413b957d@kaod.org> <1487565939.3779.4.camel@aj.id.au> To: Andrew Jeffery , =?UTF-8?Q?C=c3=a9dric_Le_Goater?= Cc: Joel Stanley , openipmi-developer@lists.sourceforge.net, linux-arm-kernel@lists.infradead.org, linux-kernel@vger.kernel.org From: Corey Minyard Message-ID: <1fe8fe1f-e035-8ef1-cecd-d3d29bcda075@acm.org> Date: Mon, 20 Feb 2017 07:35:16 -0600 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:45.0) Gecko/20100101 Thunderbird/45.7.0 MIME-Version: 1.0 In-Reply-To: <1487565939.3779.4.camel@aj.id.au> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 02/19/2017 10:45 PM, Andrew Jeffery wrote: > Hi Cory, > > On Tue, 2016-12-06 at 16:02 +0100, Cédric Le Goater wrote: >> [ this is a resend bc of some mailing list issues] >> >> On 12/06/2016 03:57 AM, Andrew Jeffery wrote: >>> The registers for the bt-bmc device live under the Aspeed LPC >>> controller. Devicetree bindings have recently been introduced for the >>> LPC controller where the "host" portion of the LPC register space is >>> described as a syscon device. Future devicetrees describing the bt-bmc >>> device should nest its node under the appropriate "simple-mfd", "syscon" >>> compatible node. >>> >>> This change allows the bt-bmc driver to function with both syscon and >>> non-syscon- based devicetree descriptions by always using a regmap for >>> register access, either retrieved from the parent syscon device or >>> instantiated if none exists. >>> >>> The patch has been tested on an OpenPOWER Palmetto machine, successfully >>> booting, rebooting and powering down the host. >>> >>> Signed-off-by: Andrew Jeffery >> It would be nice to have an example of the associated binding. >> I did not see it. A part from that : >> >> Reviewed-by: Cédric Le Goater > > Will this make it into 4.11? I thought you were doing a v2 with a few little fixes. Get it to me quickly, if you can. -corey > Cheers, > > Andrew