Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20;
Date:   Tue, 11 Apr 2023 02:40:16 +0300
From:   Serge Semin <fancer.lancer@gmail.com>
To:     Damien Le Moal <damien.lemoal@opensource.wdc.com>,
        sunying@nj.iscas.ac.cn
Cc:     Damien Le Moal <damien.lemoal@opensource.wdc.com>,
        linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org,
        greybus-dev@lists.linaro.org,
        Linux ARM <linux-arm-kernel@lists.infradead.org>,
        linux-mips@vger.kernel.org
Subject: Re: Multiple undefined configuration options are dependent in
 Kconfig under the v6.3-rc4 drivers directory
Message-ID: <20230410232725.zwbccyryboxj3bh5@mobilestation>
References: <bc6c86d.11bb9.1872c5aa2cb.Coremail.sunying@nj.iscas.ac.cn>
 <18c3fbb4-59db-d508-ddd6-080cc8944b39@opensource.wdc.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <18c3fbb4-59db-d508-ddd6-080cc8944b39@opensource.wdc.com>
Precedence: bulk

On Thu, Mar 30, 2023 at 07:34:58AM +0900, Damien Le Moal wrote:
> On 3/29/23 16:52, 孙滢 wrote:
> > It has been discovered that the following configuration options are undefined in the current latest version, v6.3-rc4, yet they are being relied upon by other configuration options in multiple Kconfig files:
> > 
> > MIPS_BAIKAL_T1 is undefined, used as a 'depends on' condition in multiple files such as drivers/ata/Kconfig, drivers/hwmon/Kconfig, drivers/bus/Kconfig, and drivers/memory/Kconfig.
> > MFD_MAX597X is undefined, used as a 'depends on' condition in Kconfig file drivers/regulator/Kconfig.
> > MFD_SM5703 is undefined, used as a 'depends on' condition in Kconfig file drivers/regulator/Kconfig.
> > ARCH_THUNDERBAY is undefined, used as a 'depends on' condition in Kconfig files drivers/pinctrl/Kconfig and drivers/phy/intel/Kconfig.
> > ARCH_BCM4908 is undefined, used as a 'depends on' condition in Kconfig file drivers/leds/blink/Kconfig.
> > MFD_TN48M_CPLD is undefined, used as a 'depends on' condition in Kconfig files drivers/gpio/Kconfig and drivers/reset/Kconfig. 
> > USB_HSIC_USB3613 is undefined, used as a 'depends on' condition in drivers/staging/greybus/Kconfig and drivers/staging/greybus/arche-platform.c.

Please, don't drop the MIPS_BAIKAL_T1 config. It will be defined and
thus utilized after I submit the SoC CSP support to the MIPS arch.

-Serge(y)

> > 
> > If these 7 configuration options are deprecated, it is recommended to remove the dependencies on them in the Kconfig files. 
> > If they are still useful, it is recommended to define them.
> 
> + linux-arm & linux-mips
> 
> What about you send patches to fix this ?
> 
> > 
> > 
> > Best regards,
> > Ying Sun
> > Pengpeng Hou
> > Yanjie Ren
> 
> -- 
> Damien Le Moal
> Western Digital Research
>