From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <linux-kernel-owner@vger.kernel.org>
Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand
	id S932816AbcC3IYs (ORCPT <rfc822;w@1wt.eu>);
	Wed, 30 Mar 2016 04:24:48 -0400
Received: from mailout2.w1.samsung.com ([210.118.77.12]:14363 "EHLO
	mailout2.w1.samsung.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org
	with ESMTP id S932594AbcC3IYp (ORCPT
	<rfc822;linux-kernel@vger.kernel.org>);
	Wed, 30 Mar 2016 04:24:45 -0400
X-AuditID: cbfec7f4-f796c6d000001486-eb-56fb8d4996e0
Subject: Re: [RESEND v2] phy: hi6220: Add missing MFD_SYSCON dependency on
 HAS_IOMEM
To: Lee Jones <lee.jones@linaro.org>
References: <1459298943-20458-1-git-send-email-k.kozlowski@samsung.com>
 <20160330080837.GA3323@x1> <20160330081057.GD3323@x1>
Cc: Kishon Vijay Abraham I <kishon@ti.com>, linux-kernel@vger.kernel.org
From: Krzysztof Kozlowski <k.kozlowski@samsung.com>
Message-id: <56FB8D46.3030506@samsung.com>
Date: Wed, 30 Mar 2016 17:24:38 +0900
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101
 Thunderbird/38.6.0
MIME-version: 1.0
In-reply-to: <20160330081057.GD3323@x1>
Content-type: text/plain; charset=utf-8
Content-transfer-encoding: 7bit
X-Brightmail-Tracker: H4sIAAAAAAAAA+NgFjrELMWRmVeSWpSXmKPExsVy+t/xK7qevb/DDGbuFbN4/cLQ4sLTHjaL
	+1+PMlpc3jWHzYHF4861PWwefVtWMXocv7GdyePzJrkAligum5TUnMyy1CJ9uwSujJkN+QW3
	OSoWrVrH2MA4m72LkZNDQsBEYvmT9awQtpjEhXvr2boYuTiEBJYySrxd8wzKecoo8eLiJGaQ
	KmGBMIkHp58AdXBwiAioSJx7Yw4SFhKol2j/1ccIYjMLeEgcmt0DtoBNwFhi8/IlbCA2r4CW
	xPaNc8BqWARUJSb23QSLiwpESDyZe5IRokZQ4sfkeywg4zkF1CX2nWUEMZmBzClTciGmy0ts
	XvOWeQKjwCwkDbMQqmYhqVrAyLyKUTS1NLmgOCk911CvODG3uDQvXS85P3cTIyRkv+xgXHzM
	6hCjAAejEg/vBunfYUKsiWXFlbmHGCU4mJVEeEV7gEK8KYmVValF+fFFpTmpxYcYpTlYlMR5
	5+56HyIkkJ5YkpqdmlqQWgSTZeLglGpg1JXIcsxmfHOfuX3unb1c4iwfmfzXKjo4nJkk7Nyy
	gvtFgMbUcHNv09tFcVN3uXKaxb72Vkyf5BsS5Wv19dmP2/m7r7iW9c0/8ZB7uaGHGouhUbvd
	du9VSV/3rq/8EXDgo9DjyfbLFsid83lsV32o4sz2/IXmHWZKEyPMnP4Xli3L82TZufeHEktx
	RqKhFnNRcSIA9hiErlUCAAA=
Sender: linux-kernel-owner@vger.kernel.org
List-ID: <linux-kernel.vger.kernel.org>
X-Mailing-List: linux-kernel@vger.kernel.org

On 30.03.2016 17:10, Lee Jones wrote:
> On Wed, 30 Mar 2016, Lee Jones wrote:
> 
>> On Wed, 30 Mar 2016, Krzysztof Kozlowski wrote:
>>
>>> The MFD_SYSCON depends on HAS_IOMEM so when selecting it avoid unmet
>>> direct dependencies.
>>>
>>> This fixes warning on allyesconfig on ARCH=um:
>>>
>>> warning: (ST_IRQCHIP && HIP04_ETH && STMMAC_PLATFORM && DWMAC_IPQ806X &&
>>> 	DWMAC_LPC18XX && DWMAC_ROCKCHIP && DWMAC_SOCFPGA && DWMAC_STI && TI_CPSW
>>> 	&& PINCTRL_ROCKCHIP && PINCTRL_DOVE && POWER_RESET_KEYSTONE &&
>>> 	S3C2410_WATCHDOG && VIDEO_OMAP3 && VIDEO_S5P_FIMC && USB_XHCI_MTK &&
>>> 	RTC_DRV_AT91SAM9 && LPC18XX_DMAMUX && VIDEO_OMAP4 && HWSPINLOCK_QCOM &&
>>> 	ATMEL_ST && QCOM_GSBI && PHY_HI6220_USB)
>>> selects MFD_SYSCON which has unmet direct dependencies (HAS_IOMEM)
>>>
>>> Signed-off-by: Krzysztof Kozlowski <k.kozlowski@samsung.com>
>>> Acked-by: Lee Jones <lee.jones@linaro.org>
>>
>> Holding off on this until the discussion reaches some kind of
>> consensus.
> 
> FYI: https://lkml.org/lkml/2016/3/29/716

Yes, Rob's solution looks better to me.

BR,
Krzysztof