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=-8.2 required=3.0 tests=HEADER_FROM_DIFFERENT_DOMAINS, INCLUDES_PATCH,MAILING_LIST_MULTI,SIGNED_OFF_BY,SPF_HELO_NONE,SPF_PASS, URIBL_BLOCKED,USER_AGENT_SANE_1 autolearn=unavailable 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 68C14C3A5A7 for ; Mon, 2 Sep 2019 18:44:31 +0000 (UTC) Received: from vger.kernel.org (vger.kernel.org [209.132.180.67]) by mail.kernel.org (Postfix) with ESMTP id 4F80622D6D for ; Mon, 2 Sep 2019 18:44:31 +0000 (UTC) Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1726933AbfIBSo1 (ORCPT ); Mon, 2 Sep 2019 14:44:27 -0400 Received: from mail.kernel.org ([198.145.29.99]:60040 "EHLO mail.kernel.org" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1726805AbfIBSo1 (ORCPT ); Mon, 2 Sep 2019 14:44:27 -0400 Received: from localhost (83-86-89-107.cable.dynamic.v4.ziggo.nl [83.86.89.107]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPSA id CE3AC216C8; Mon, 2 Sep 2019 18:44:25 +0000 (UTC) Date: Mon, 2 Sep 2019 20:44:23 +0200 From: Greg KH To: Joe Perches Cc: Randy Dunlap , Stephen Rothwell , Linux Next Mailing List , Linux Kernel Mailing List , Valdis Kletnieks , "devel@driverdev.osuosl.org" , Linux FS Devel Subject: Re: linux-next: Tree for Sep 2 (exfat) Message-ID: <20190902184423.GA5697@kroah.com> References: <20190902224310.208575dc@canb.auug.org.au> <20190902174631.GB31445@kroah.com> <13e2db80-0c89-0f36-6876-f9639f0d30ab@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.12.1 (2019-06-15) Sender: linux-next-owner@vger.kernel.org Precedence: bulk List-ID: X-Mailing-List: linux-next@vger.kernel.org On Mon, Sep 02, 2019 at 11:11:45AM -0700, Joe Perches wrote: > On Mon, 2019-09-02 at 11:07 -0700, Randy Dunlap wrote: > > On 9/2/19 10:46 AM, Greg KH wrote: > > > On Mon, Sep 02, 2019 at 10:39:39AM -0700, Randy Dunlap wrote: > > > > On 9/2/19 5:43 AM, Stephen Rothwell wrote: > > > > > Hi all, > > > > > > > > > > News: I will only be doing 2 more releases before I leave for Kernel > > > > > Summit (there may be some reports on Thursday, but I doubt I will have > > > > > time to finish the full release) and then no more until Sept 30. > > > > > > > > > > Changes since 20190830: > > > > > > > > > > > > > Hi, > > > > I am seeing lots of exfat build errors when CONFIG_BLOCK is not set/enabled. > > > > Maybe its Kconfig should also say > > > > depends on BLOCK > > > > ? > > > > > > Here's what I committed to my tree: > > > > > > > > > From e2b880d3d1afaa5cad108c29be3e307b1917d195 Mon Sep 17 00:00:00 2001 > > > From: Greg Kroah-Hartman > > > Date: Mon, 2 Sep 2019 19:45:06 +0200 > > > Subject: staging: exfat: make exfat depend on BLOCK > > > > > > This should fix a build error in some configurations when CONFIG_BLOCK > > > is not selected. Also properly set the dependancy for no FAT support at > > > the same time. > > > > > > Reported-by: Randy Dunlap > > > Cc: Valdis Kletnieks > > > Signed-off-by: Greg Kroah-Hartman > > > > That works. Thanks. > > Acked-by: Randy Dunlap # build-tested > > > > > --- > > > drivers/staging/exfat/Kconfig | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/drivers/staging/exfat/Kconfig b/drivers/staging/exfat/Kconfig > > > index f52129c67f97..290dbfc7ace1 100644 > > > --- a/drivers/staging/exfat/Kconfig > > > +++ b/drivers/staging/exfat/Kconfig > > > @@ -1,11 +1,13 @@ > > > config EXFAT_FS > > > tristate "exFAT fs support" > > > + depends on BLOCK > > > select NLS > > > help > > > This adds support for the exFAT file system. > > > > > > config EXFAT_DONT_MOUNT_VFAT > > > bool "Prohibit mounting of fat/vfat filesysems by exFAT" > > > + depends on EXFAT_FS > > > default y > > > help > > > By default, the exFAT driver will only mount exFAT filesystems, and refuse > > I think this last one is backwards and should be > > config EXFAT_ALLOW_MOUNT_VFAT > and > default n This is just a "bandage" for now, it should be fixed properly by ripping out the code wherever possible and making this the default functionality anyway. This just keeps the code from breaking existing working systems if users accidentally build this module. thanks, greg k-h