Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753340AbbKYMQF (ORCPT ); Wed, 25 Nov 2015 07:16:05 -0500 Received: from mga03.intel.com ([134.134.136.65]:42690 "EHLO mga03.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752411AbbKYMQD (ORCPT ); Wed, 25 Nov 2015 07:16:03 -0500 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,342,1444719600"; d="scan'208";a="846802141" Date: Wed, 25 Nov 2015 14:12:52 +0200 From: Mika Westerberg To: Arnd Bergmann Cc: Andy Shevchenko , "Rafael J. Wysocki" , Greg Kroah-Hartman , Jarkko Nikula , linux-i2c@vger.kernel.org, linux-acpi@vger.kernel.org, linux-kernel@vger.kernel.org, Lee Jones , Kevin Fenzi , Wolfram Sang Subject: Re: [PATCH v1 11/13] mfd: intel-lpss: Pass HSUART configuration via properties Message-ID: <20151125121252.GY1587@lahna.fi.intel.com> References: <1448360579-79260-1-git-send-email-andriy.shevchenko@linux.intel.com> <1448360579-79260-12-git-send-email-andriy.shevchenko@linux.intel.com> <6720400.mTdslW7yXO@wuerfel> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <6720400.mTdslW7yXO@wuerfel> Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo 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 Content-Length: 946 Lines: 21 On Tue, Nov 24, 2015 at 08:53:04PM +0100, Arnd Bergmann wrote: > On Tuesday 24 November 2015 12:22:57 Andy Shevchenko wrote: > > +static struct property_entry uart_properties[] = { > > + PROPERTY_ENTRY_U32("reg-io-width", 4), > > + PROPERTY_ENTRY_U32("reg-shift", 2), > > + PROPERTY_ENTRY_U8("snps,uart-16550-compatible", 1), > > + { }, > > > > If I read the binding correctly, the "snps,uart-16550-compatible" property > is meant to be boolean, meaning true if present and zero-length or false > if absent. Using a u8 propert instead feels wrong. > > Maybe we can have a PROPERTY_ENTRY_BOOL() for that? That's a good idea. We'll add it to the next revision of the series. -- 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/