From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from vger.kernel.org (vger.kernel.org [23.128.96.18]) by smtp.lore.kernel.org (Postfix) with ESMTP id 8F502C6FD1F for ; Wed, 22 Mar 2023 16:29:04 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S229839AbjCVQ3D (ORCPT ); Wed, 22 Mar 2023 12:29:03 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:35882 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S229990AbjCVQ3B (ORCPT ); Wed, 22 Mar 2023 12:29:01 -0400 Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 9782453739; Wed, 22 Mar 2023 09:28:29 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=intel.com; i=@intel.com; q=dns/txt; s=Intel; t=1679502509; x=1711038509; h=from:to:cc:subject:in-reply-to:references:date: message-id:mime-version; bh=KfUvgiz4DVLzFaYO7oEOLPOOhMl2JyBbYso8oKRbz/4=; b=iBLxbnttfPgbBL+Hn6mJCMQwPWgQpUpPd8Hu1O9g9KJ3fRzidD/eoUsX G1h8DhvoAZZGvVFzc4dLyYtAB1T87PHCnnsKqxkVCSJLHs0hWEJEkIu6b 3+YtPCHMfLH6JrboGeoay2c7wzI1E//R9skB6CHyUJ4zFpN+C2wzpnQhd kXeMePv2JwUGsEoz5qvh54z8gevBFFuXb8VN9JKVp6iP64zyH6KqswbrL v2wzPX5hLMAHaxYZwjlSfpZiErMoopSQJNbFdMkxZ4fJHKZUo+chgmWV+ QnvxLNVTIJg06YD+HFLMNg/M+F17ShfwMfZYgKLZBh71dlwb+6y5n4e8P g==; X-IronPort-AV: E=McAfee;i="6600,9927,10657"; a="339300029" X-IronPort-AV: E=Sophos;i="5.98,282,1673942400"; d="scan'208";a="339300029" Received: from fmsmga003.fm.intel.com ([10.253.24.29]) by fmsmga103.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Mar 2023 09:28:09 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=McAfee;i="6600,9927,10657"; a="771102926" X-IronPort-AV: E=Sophos;i="5.98,282,1673942400"; d="scan'208";a="771102926" Received: from jprokopo-mobl2.ger.corp.intel.com (HELO localhost) ([10.252.61.221]) by fmsmga003-auth.fm.intel.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 22 Mar 2023 09:27:59 -0700 From: Jani Nikula To: Andy Shevchenko , Pin-yen Lin Cc: Andrzej Hajda , Neil Armstrong , Robert Foss , Laurent Pinchart , Jonas Karlman , Jernej Skrabec , David Airlie , Daniel Vetter , Rob Herring , Krzysztof Kozlowski , Daniel Scally , Heikki Krogerus , Sakari Ailus , Greg Kroah-Hartman , "Rafael J . Wysocki" , Prashant Malani , Benson Leung , Guenter Roeck , Xin Ji , linux-kernel@vger.kernel.org, Thomas Zimmermann , linux-acpi@vger.kernel.org, Javier Martinez Canillas , AngeloGioacchino Del Regno , Hsin-Yi Wang , Lyude Paul , =?utf-8?Q?N=C3=ADco?= =?utf-8?Q?las_F_=2E_R_=2E_A_=2E_Prado?= , Allen Chen , dri-devel@lists.freedesktop.org, Marek Vasut , Stephen Boyd , chrome-platform@lists.linux.dev, devicetree@vger.kernel.org, Andi Shyti , Dmitry Baryshkov , Douglas Anderson , Imre Deak , Linus Walleij , YueHaibing Subject: Re: [PATCH v14 03/10] drm/display: Add Type-C switch helpers In-Reply-To: Organization: Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo References: <20230322104639.221402-1-treapking@chromium.org> <20230322104639.221402-4-treapking@chromium.org> Date: Wed, 22 Mar 2023 18:27:56 +0200 Message-ID: <87edpg7nub.fsf@intel.com> MIME-Version: 1.0 Content-Type: text/plain Precedence: bulk List-ID: X-Mailing-List: devicetree@vger.kernel.org On Wed, 22 Mar 2023, Andy Shevchenko wrote: > On Wed, Mar 22, 2023 at 06:46:32PM +0800, Pin-yen Lin wrote: >> +#ifdef CONFIG_DRM_DISPLAY_DP_TYPEC_HELPER > > Ah, maybe this should use IS_REACHABLE() ? Personally, I think IS_REACHABLE() is a build-time band-aid solution to a problem that should be solved in Kconfig. :p I think it always means there's a configuration combo that shouldn't exist, and it's a surprise to the user when they've configured something, Kconfig has deemed it a valid configuration, but they don't get the feature they want. As a user, how would they even debug that case? Double check configs, don't see anything wrong. BR, Jani. -- Jani Nikula, Intel Open Source Graphics Center