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=-5.5 required=3.0 tests=DKIMWL_WL_HIGH,DKIM_SIGNED, DKIM_VALID,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 11004C34031 for ; Wed, 19 Feb 2020 02:03:02 +0000 (UTC) Received: from bombadil.infradead.org (bombadil.infradead.org [198.137.202.133]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by mail.kernel.org (Postfix) with ESMTPS id D2D6C206DB for ; Wed, 19 Feb 2020 02:03:01 +0000 (UTC) Authentication-Results: mail.kernel.org; dkim=pass (2048-bit key) header.d=lists.infradead.org header.i=@lists.infradead.org header.b="dPiI6CQ3"; dkim=fail reason="signature verification failed" (1024-bit key) header.d=kernel.org header.i=@kernel.org header.b="KCDvIX5E" DMARC-Filter: OpenDMARC Filter v1.3.2 mail.kernel.org D2D6C206DB Authentication-Results: mail.kernel.org; dmarc=fail (p=none dis=none) header.from=kernel.org Authentication-Results: mail.kernel.org; spf=none smtp.mailfrom=linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; d=lists.infradead.org; s=bombadil.20170209; h=Sender: Content-Transfer-Encoding:Content-Type:Cc:List-Subscribe:List-Help:List-Post: List-Archive:List-Unsubscribe:List-Id:In-Reply-To:MIME-Version:References: Message-ID:Subject:To:From:Date:Reply-To:Content-ID:Content-Description: Resent-Date:Resent-From:Resent-Sender:Resent-To:Resent-Cc:Resent-Message-ID: List-Owner; bh=nkvOJv2zsTyxeBdP3oU/dvH69omBZMZAHVQ2zPJ1sKA=; b=dPiI6CQ35smwuN ycaKbFMhj5GSj5mhWBZzsj8w0Wbfx4oORAJp+gj3DUg9bajGCtIY5ofYIyHA9sYdf+Q9iP5Hzfgf7 WlhZmtWOhAbKbcrE9ajCj7QuFykF1ebXMOnSAO76tkDQy81aqJJfeRQTYhyfLKYXyeHR8tYCBjPWU WXveoRvjJPYeRpdn6ydyNY21Mb41YUD9Ak+0CkNcG0/ZeklyBUV7A7cgMXGy1TvY+0jPZw+s6Pjov fLX08Nyr7RE05O5COlHFBxX+LfSnHBgymslCR7yNPM0cNOJZR8F3K1vEvuAoF2/gCt/k+021zP/Vm slVXDpMzetHbwChGnw/A==; Received: from localhost ([127.0.0.1] helo=bombadil.infradead.org) by bombadil.infradead.org with esmtp (Exim 4.92.3 #3 (Red Hat Linux)) id 1j4EhV-0002WT-0m; Wed, 19 Feb 2020 02:02:57 +0000 Received: from mail.kernel.org ([198.145.29.99]) by bombadil.infradead.org with esmtps (Exim 4.92.3 #3 (Red Hat Linux)) id 1j4EhS-0002VU-4N for linux-arm-kernel@lists.infradead.org; Wed, 19 Feb 2020 02:02:55 +0000 Received: from dragon (80.251.214.228.16clouds.com [80.251.214.228]) (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 458AC206DB; Wed, 19 Feb 2020 02:02:50 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=kernel.org; s=default; t=1582077771; bh=zZgKpAVTcS0HFTReo92vS7ROv9M5SlQjFcvItuZaouo=; h=Date:From:To:Cc:Subject:References:In-Reply-To:From; b=KCDvIX5ESx+/fv2GnQodDCuhBF3i33bgdk0P29bY4M65DN6YnkPaUL8Eq89/mUfa9 CHCnzIHCRkXPfC8R9b8lqwJyoRn5SpyfY3uDcf36X2cHocrVtDhl00zts6aYUhpUqg 8rZ+Sis0tKo+8K56bw/I3farjiXEOpmhwXx5QUfI= Date: Wed, 19 Feb 2020 10:02:45 +0800 From: Shawn Guo To: Leo Li Subject: Re: [PATCH 2/2] arm64: defconfig: enable additional drivers needed by NXP QorIQ boards Message-ID: <20200219020244.GG6075@dragon> References: <1581382559-18520-1-git-send-email-leoyang.li@nxp.com> <1581382559-18520-2-git-send-email-leoyang.li@nxp.com> <20200217053730.GB6042@dragon> MIME-Version: 1.0 Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.9.4 (2018-02-28) X-CRM114-Version: 20100106-BlameMichelson ( TRE 0.8.0 (BSD) ) MR-646709E3 X-CRM114-CacheID: sfid-20200218_180254_202788_8EBC6B4E X-CRM114-Status: GOOD ( 15.67 ) X-BeenThere: linux-arm-kernel@lists.infradead.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Cc: "linux-kernel@vger.kernel.org" , "linux-arm-kernel@lists.infradead.org" Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Sender: "linux-arm-kernel" Errors-To: linux-arm-kernel-bounces+infradead-linux-arm-kernel=archiver.kernel.org@lists.infradead.org On Tue, Feb 18, 2020 at 04:44:09PM +0000, Leo Li wrote: > > > > -----Original Message----- > > From: Shawn Guo > > Sent: Sunday, February 16, 2020 11:38 PM > > To: Leo Li > > Cc: linux-arm-kernel@lists.infradead.org; linux-kernel@vger.kernel.org > > Subject: Re: [PATCH 2/2] arm64: defconfig: enable additional drivers needed > > by NXP QorIQ boards > > > > On Mon, Feb 10, 2020 at 06:55:59PM -0600, Li Yang wrote: > > > This enables the following SoC device drivers for NXP/FSL QorIQ SoCs: > > > CONFIG_QORIQ_CPUFREQ=y > > > CONFIG_NET_SWITCHDEV=y > > > CONFIG_MSCC_OCELOT_SWITCH=y > > > CONFIG_CAN=m > > > CONFIG_CAN_FLEXCAN=m > > > CONFIG_FSL_MC_BUS=y > > > CONFIG_MTD_NAND_FSL_IFC=y > > > CONFIG_FSL_ENETC=y > > > CONFIG_FSL_ENETC_VF=y > > > CONFIG_SPI_FSL_LPSPI=y > > > CONFIG_SPI_FSL_QUADSPI=y > > > CONFIG_SPI_FSL_DSPI=y > > > CONFIG_GPIO_MPC8XXX=y > > > CONFIG_ARM_SBSA_WATCHDOG=y > > > CONFIG_DRM_MALI_DISPLAY=m > > > CONFIG_FSL_MC_DPIO=y > > > CONFIG_CRYPTO_DEV_FSL_DPAA2_CAAM=m > > > CONFIG_FSL_DPAA=y > > > CONFIG_FSL_FMAN=y > > > CONFIG_FSL_DPAA_ETH=y > > > CONFIG_FSL_DPAA2_ETH=y > > > > > > And the drivers for on-board devices for the upstreamed QorIQ > > > reference > > > boards: > > > CONFIG_MTD_CFI=y > > > CONFIG_MTD_CFI_ADV_OPTIONS=y > > > CONFIG_MTD_CFI_INTELEXT=y > > > CONFIG_MTD_CFI_AMDSTD=y > > > CONFIG_MTD_CFI_STAA=y > > > CONFIG_MTD_PHYSMAP=y > > > CONFIG_MTD_PHYSMAP_OF=y > > > CONFIG_MTD_DATAFLASH=y > > > CONFIG_MTD_SST25L=y > > > CONFIG_EEPROM_AT24=m > > > CONFIG_RTC_DRV_DS1307=y > > > CONFIG_RTC_DRV_PCF85363=y > > > CONFIG_RTC_DRV_PCF2127=y > > > CONFIG_E1000=y > > > CONFIG_AQUANTIA_PHY=y > > > CONFIG_MICROSEMI_PHY=y > > > CONFIG_VITESSE_PHY=y > > > CONFIG_MDIO_BUS_MUX_MULTIPLEXER=y > > > CONFIG_MUX_MMIO=y > > > > > > The following two options are implied by new options and removed from > > > defconfig: > > > CONFIG_CLK_QORIQ=y > > > CONFIG_MEMORY=y > > > > > > Signed-off-by: Li Yang > > > > This is too much change in a single patch. It should be split properly to make > > review and merge easier, considering arm-soc folks are cautious to those 'y' > > options. > > Ok. So probably separating them based on different subsystems will be good? It would be too many patches if I separate for each individual config option. Yep, subsystem should be good. And for those 'y' options, please state why they need necessarily to be 'y'. Shawn _______________________________________________ linux-arm-kernel mailing list linux-arm-kernel@lists.infradead.org http://lists.infradead.org/mailman/listinfo/linux-arm-kernel