Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751509AbdFGIT1 (ORCPT ); Wed, 7 Jun 2017 04:19:27 -0400 Received: from smtp.codeaurora.org ([198.145.29.96]:34328 "EHLO smtp.codeaurora.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751464AbdFGITZ (ORCPT ); Wed, 7 Jun 2017 04:19:25 -0400 DMARC-Filter: OpenDMARC Filter v1.3.2 smtp.codeaurora.org 381AB60850 Authentication-Results: pdx-caf-mail.web.codeaurora.org; dmarc=none (p=none dis=none) header.from=codeaurora.org Authentication-Results: pdx-caf-mail.web.codeaurora.org; spf=none smtp.mailfrom=architt@codeaurora.org Subject: Re: [PATCH v3] drm/bridge: Build the panel wrapper in drm_kms_helper To: Eric Anholt , Maarten Lankhorst , Andrzej Hajda , Laurent Pinchart , Thierry Reding References: <20170602202514.11900-1-eric@anholt.net> <2ab4e84f-9834-1dce-d16a-14f9cc4083e9@codeaurora.org> <2cbcbf54-debf-10a1-390d-128171d81f91@linux.intel.com> <3fd262cf-1db6-4335-320c-af92f9014502@linux.intel.com> <87fufd9f60.fsf@eliezer.anholt.net> Cc: linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org From: Archit Taneja Message-ID: <9be9f59f-3d6e-d68b-490e-68fa325a38fe@codeaurora.org> Date: Wed, 7 Jun 2017 13:49:19 +0530 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: <87fufd9f60.fsf@eliezer.anholt.net> Content-Type: text/plain; charset=windows-1252; format=flowed Content-Transfer-Encoding: 7bit Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Length: 1012 Lines: 30 On 06/06/2017 11:22 PM, Eric Anholt wrote: > Maarten Lankhorst writes: > >> This fixes the following depmod error when building drm as a module: >> depmod: ERROR: Found 6 modules in dependency cycles! >> depmod: ERROR: Cycle detected: drm -> drm_kms_helper -> drm >> >> Fixes: 13dfc0540a57 ("drm/bridge: Refactor out the panel wrapper from the lvds-encoder bridge.") >> Signed-off-by: Maarten Lankhorst > > Looks plausible to me. Given how much trouble I've had getting the > kbuild right, anyone else want to chime in? > The fix looks good to me too. depmod doesn't complain after this patch. I can queue it to drm-misc-next if no one else has any comments on it in the next few hours or so. I'll make sure I check with 'make module_install' before pushing, I didn't know depmod wasn't called otherwise. Thanks, Archit -- Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum, a Linux Foundation Collaborative Project