Revisiting it again... Am Sun, 22 Jan 2023 10:08:52 +0100 schrieb Andreas Kemnade <andreas@xxxxxxxxxxxx>: > On Thu, 19 Jan 2023 09:30:20 +0200 > Tony Lindgren <tony@xxxxxxxxxxx> wrote: > > > * Tony Lindgren <tony@xxxxxxxxxxx> [230116 17:33]: > > > * Adam Ford <aford173@xxxxxxxxx> [230116 17:00]: > > > > Doesn't this imply the target-module stuff needs to be implemented for > > > > the drivers? It looks like a lot of the omap3 drivers are still using > > > > hwmods although some have target-modules. In this case, the mcspi > > > > drivers that Andreas is disabling don't appear to have target-module > > > > stuff configured. > > > > > > Sorry I don't remember if omap_device.c ignores status disabled or not. > > > But in any case, it should be trivial to update omap3.dtsi to configure > > > some of the devices like mcspi to probe with device tree data and ti-sysc > > > as needed. > > > > So as long as gta04 power management still behaves with this patch it > > should good to go. > > > # sleep 10 ; /usr/local/bin/idledump > CM_IDLEST1_CORE 00000042 RAM + SCM on, no issue, cross-checked, force-enabling mcspi brings more activity, so this an indication thet mcspi is off. > CM_IDLEST3_CORE 00000000 > CM_FCLKEN1_CORE 00000000 same here with force-enabling mcspi, brings in some bits set. > CM_FCLKEN3_CORE 00000002 see comments below. > CM_CLKSTST_CORE 00000003 > CM_IDLEST_CKGEN 00000209 this becomes 1 without this patch. > CM_IDLEST2_CKGEN 00000000 > CM_FCLKEN_DSS 00000000 > CM_IDLEST_DSS 00000000 > CM_FCLKEN_CAM 00000000 > CM_IDLEST_CAM 00000000 > CM_FCLKEN_PER 00000000 > CM_IDLEST_PER 00000000 > > > FCLKEN3_CORE becomes 0 after unbinding the bandgap sensor. > > but... > # cat /sys/kernel/debug/pm_debug/time > usbhost_pwrdm (ON),OFF:830267486567,RET:0,INA:0,ON:12202880865 > sgx_pwrdm (INA),OFF:0,RET:0,INA:841224365234,ON:1245971680 > core_pwrdm (ON),OFF:0,RET:0,INA:0,ON:842470336914 > per_pwrdm (ON),OFF:520406799328,RET:30043365464,INA:0,ON:292020111087 > > hmmm.... > > but does not look like anything related to mcspi*. > it does not look like, but it is, but how... There is something below my radar. Regards, Andreas
