On Thu, 11 Apr 2024 at 06:19, Bjorn Andersson <andersson@xxxxxxxxxx> wrote: > > On Wed, Apr 10, 2024 at 01:12:15AM +0300, Dmitry Baryshkov wrote: > > On Tue, 9 Apr 2024 at 23:02, Konrad Dybcio <konrad.dybcio@xxxxxxxxxx> wrote: > > > On 4/9/24 20:27, Dmitry Baryshkov wrote: > > > > Enable CONFIG_INTERCONNECT_QCOM_SM6115 as module to enable the > > > > interconnect driver for the SoC used on Qualcomm Robotics RB2 board. > > > > > > > > Signed-off-by: Dmitry Baryshkov <dmitry.baryshkov@xxxxxxxxxx> > > > > --- > > > > > > =y for console? > > > > I think with pinctrl being set to m it won't reach the console anyway. > > And earlycon should work w/o ICC driver if I'm not mistaken. > > > > Several other Qualcomm platforms also have interconnects built as > > modules in defconfig. I really suppose that we should move all such > > drivers to =m and force using initrd. But this sounds like a topic for > > plumbers. > > > > That currently does not work very well, because if you probe defer the > UART into the future e.g. systemd will open /dev/console before it > exist - with no interest in reopening that later. > > So, if you care about UART, that is suboptimal. > > Resolve this, and I'd be happy to see them all go =m. Doesn't /dev/console handle switching between earlycon and actual console? I'll take a look at some point in the future. But I can't help but notice that currently we have 5 ICC drivers built as modules (out of 21 mentioned in the defconfig). Should we fix them too? CONFIG_INTERCONNECT_QCOM_MSM8916=m CONFIG_INTERCONNECT_QCOM_MSM8996=m CONFIG_INTERCONNECT_QCOM_QCS404=m CONFIG_INTERCONNECT_QCOM_SM8150=m CONFIG_INTERCONNECT_QCOM_SM8350=m -- With best wishes Dmitry
