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 X-Spam-Level: X-Spam-Status: No, score=-2.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, MAILING_LIST_MULTI,SPF_PASS,URIBL_BLOCKED,USER_AGENT_MUTT autolearn=ham autolearn_force=no version=3.4.0 Received: from mail.kernel.org (mail.kernel.org [198.145.29.99]) by smtp.lore.kernel.org (Postfix) with ESMTP id A9D0DC04ABB for ; Tue, 11 Sep 2018 08:21:17 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 5D9D520854 for ; Tue, 11 Sep 2018 08:21:17 +0000 (UTC) DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org 5D9D520854 Authentication-Results: mail.kernel.org; dmarc=none (p=none dis=none) header.from=linuxfoundation.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-kernel-owner@vger.kernel.org Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726780AbeIKNTZ (ORCPT ); Tue, 11 Sep 2018 09:19:25 -0400 Received: from mail.linuxfoundation.org ([140.211.169.12]:39944 "EHLO mail.linuxfoundation.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726407AbeIKNTZ (ORCPT ); Tue, 11 Sep 2018 09:19:25 -0400 Received: from localhost (ip-213-127-74-90.ip.prioritytelecom.net [213.127.74.90]) by mail.linuxfoundation.org (Postfix) with ESMTPSA id DD049D24; Tue, 11 Sep 2018 08:21:13 +0000 (UTC) Date: Tue, 11 Sep 2018 10:21:10 +0200 From: gregkh To: Ajay Singh Cc: Arnd Bergmann , driverdevel , linux-wireless , Linux Kernel Mailing List , ganesh.krishna@microchip.com, aditya.shankar@microchip.com Subject: Re: [PATCH] staging: wilc1000: revert "fix TODO to compile spi and sdio components in single module" Message-ID: <20180911082110.GB17267@kroah.com> References: <20180813212108.2442868-1-arnd@arndb.de> <20180814105237.64b01528@ajaysk-VirtualBox> <20180814084335.GB7208@kroah.com> <20180910160836.24efb0b9@ajaysk-VirtualBox> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20180910160836.24efb0b9@ajaysk-VirtualBox> User-Agent: Mutt/1.10.1 (2018-07-13) Sender: linux-kernel-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On Mon, Sep 10, 2018 at 04:08:36PM +0530, Ajay Singh wrote: > Hi Greg, > > On Tue, 14 Aug 2018 10:43:35 +0200 > gregkh wrote: > > > On Tue, Aug 14, 2018 at 10:34:27AM +0200, Arnd Bergmann wrote: > > > On Tue, Aug 14, 2018 at 7:22 AM Ajay Singh > > > wrote: > > > > > > > > Hi Arnd, > > > > > > > > On Mon, 13 Aug 2018 23:20:33 +0200 > > > > Arnd Bergmann wrote: > > > > > > > > > The TODO item named "make spi and sdio components coexist in one > > > > > build" was apparently addressed a long time ago, but never > > > > > removed from the TODO file. However, the new patch that tries > > > > > to address it actually makes it worse again by duplicating the > > > > > common parts of the driver into two separate modules rather > > > > > than sharing them. This also introduces a build regression when > > > > > one of the two is built-in while the other is a loadable module: > > > > > > > > Thanks for sharing your inputs and submitting patch. > > > > I have also submitted a patch to address the compilation error[1]. > > > > We can ignore my patch and proceed with your changes. > > > > > > > > [1].https://patchwork.kernel.org/patch/10563873/ > > > > > > That patch seems useful regardless, as it removes dead code, > > > but I'd still prefer to revert staging-linusthe 9abc44ba4e2f ("staging: wilc1000: > > > fix TODO to compile spi and sdio components in single module") > > > commit for the other reasons I explained. > > > > I agree, I'll queue it up soon, I have other patches to get to Linus > > first. > > > > This patch is applied only to 'staging-linus' in commit f45b893 and > it's not applied to 'staging-testing/next' tree. > Please apply this patch to 'staging-next' tree also. Because I have one > patch to submit on top of these changes to delete the 'wilc_debug.c' > unused file. I will merge the staging-linus branch into staging-testing once Linus pulls them in. Should happen next Monday or so. You can send me patches that build on top of this now, I'll just wait until then to apply them to my tree. thanks, greg k-h