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:   Fri, 21 Apr 2023 16:19:19 +0300
From:   Serge Semin <fancer.lancer@gmail.com>
To:     Jiaxun Yang <jiaxun.yang@flygoat.com>
Cc:     linux-kernel@vger.kernel.org,
        "linux-mips@vger.kernel.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: <20230421131919.3yy4smvzcrtjbnbu@mobilestation>
References: <bc6c86d.11bb9.1872c5aa2cb.Coremail.sunying@nj.iscas.ac.cn>
 <18c3fbb4-59db-d508-ddd6-080cc8944b39@opensource.wdc.com>
 <20230410232725.zwbccyryboxj3bh5@mobilestation>
 <011B19A8-1276-4603-B4D8-4CA7E9F07B3D@flygoat.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <011B19A8-1276-4603-B4D8-4CA7E9F07B3D@flygoat.com>
Precedence: bulk

On Fri, Apr 14, 2023 at 09:18:20AM +0100, Jiaxun Yang wrote:
> 
> 
> > 2023年4月11日 00:40，Serge Semin <fancer.lancer@gmail.com> 写道：
> > 
> > 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.
> 

> Hi Serge,
> 
> Is there any special support at MIPS arch level required by Baikal T1?
> 
> I think MIPS32R5 generic kernel should fit your purpose? You can easily add some “workaround”
> under generic kernel framework as well.

Alas there are multiple configs specific to our chip. So the MIPS
generic sub-module won't workout our needs at the current state. I'll
have a better look at it when time comes (whether it's possible to fix
it up somehow) but meanwhile it's better to retain the MIPS_BAIKAL_T1
config in the drivers.

-Serge(y)

> 
> Thanks
> Jiaxun
> 
> 
> 
>