From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1752864AbdDCM0o (ORCPT ); Mon, 3 Apr 2017 08:26:44 -0400 Received: from mga05.intel.com ([192.55.52.43]:47701 "EHLO mga05.intel.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1752163AbdDCM0m (ORCPT ); Mon, 3 Apr 2017 08:26:42 -0400 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.36,270,1486454400"; d="scan'208";a="841228369" From: Andy Shevchenko To: Philipp Zabel , linux-kernel@vger.kernel.org, Mika Westerberg Cc: Andy Shevchenko , Ramiro Oliveira Subject: [PATCH v1] reset: Make optional stuff optional for all users Date: Mon, 3 Apr 2017 15:26:38 +0300 Message-Id: <20170403122638.88263-1-andriy.shevchenko@linux.intel.com> X-Mailer: git-send-email 2.11.0 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org There is Device Tree oriented check for optional resource. Of course it will fail on non-DT platforms. Remove this check to make things optional for all users. Fixes: bb475230b8e5 ("reset: make optional functions really optional") Cc: Ramiro Oliveira Signed-off-by: Andy Shevchenko --- The reset framework is too Device Tree oriented, and who knows what the logic was behind the commit which introduced devm_reset_*() functions without thinking out of the DT box. This commit fixes almost all Intel newest boards that have no legacy UART since UART driver started using this DT-centric framework. drivers/reset/core.c | 3 --- 1 file changed, 3 deletions(-) diff --git a/drivers/reset/core.c b/drivers/reset/core.c index f1e5e65388bb..62314e663f29 100644 --- a/drivers/reset/core.c +++ b/drivers/reset/core.c @@ -331,9 +331,6 @@ struct reset_control *__of_reset_control_get(struct device_node *node, int rstc_id; int ret; - if (!node) - return ERR_PTR(-EINVAL); - if (id) { index = of_property_match_string(node, "reset-names", id); -- 2.11.0