Skip to content

Conversation

@astrunin
Copy link

  • Update rv3028 driver to report backup switchover indication when using RTC_VL_READ.
  • Add support for RTC_VL_CLR to clear this indication.

…g RTC_VL_READ. Add support for RTC_VL_CLR to clear this indication.
Copy link
Collaborator

@shmuelhazan shmuelhazan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@astrunin can you please retitle to:
DEV-ZYZ: do something

thanks

@astrunin astrunin changed the title Update rv3028 driver to report backup switchover indication when usin… VER-113: Update rv3028 driver to report backup switchover indication Sep 30, 2025
@astrunin
Copy link
Author

@astrunin can you please retitle to: DEV-ZYZ: do something

thanks

done

@shmuelhazan shmuelhazan changed the title VER-113: Update rv3028 driver to report backup switchover indication VER-113: rtc: rv3028: add RTC_VL_CLR & RTC_VL_BACKUP_SWITCH support Sep 30, 2025
@shmuelhazan shmuelhazan merged commit 242641e into tg-v6.12 Sep 30, 2025
return ret;

status = status & RV3028_STATUS_PORF ? RTC_VL_DATA_INVALID : 0;
status = ret & RV3028_STATUS_PORF ? RTC_VL_DATA_INVALID : 0;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't ret always zero at this point? the read value is in status, no?

}
return put_user(status, (unsigned int __user *)arg);
case RTC_VL_CLR:
status = regmap_read(rv3028->regmap, RV3028_STATUS, &status);

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

wrong usage of status, should be ret =

shmuelhazan pushed a commit that referenced this pull request Nov 6, 2025
)

- Update rv3028 driver to report backup switchover indication when using
RTC_VL_READ.
- Add support for RTC_VL_CLR to clear this indication.
adrian-nicolau added a commit that referenced this pull request Nov 13, 2025
)

- Update rv3028 driver to report backup switchover indication when using
RTC_VL_READ.
- Add support for RTC_VL_CLR to clear this indication.
adrian-nicolau added a commit that referenced this pull request Nov 28, 2025
)

- Update rv3028 driver to report backup switchover indication when using
RTC_VL_READ.
- Add support for RTC_VL_CLR to clear this indication.
adrian-nicolau pushed a commit that referenced this pull request Dec 1, 2025
A synchronous external abort occurs on the Renesas RZ/G3S SoC if unbind is
executed after the configuration sequence described above:

modprobe usb_f_ecm
modprobe libcomposite
modprobe configfs
cd /sys/kernel/config/usb_gadget
mkdir -p g1
cd g1
echo "0x1d6b" > idVendor
echo "0x0104" > idProduct
mkdir -p strings/0x409
echo "0123456789" > strings/0x409/serialnumber
echo "Renesas." > strings/0x409/manufacturer
echo "Ethernet Gadget" > strings/0x409/product
mkdir -p functions/ecm.usb0
mkdir -p configs/c.1
mkdir -p configs/c.1/strings/0x409
echo "ECM" > configs/c.1/strings/0x409/configuration

if [ ! -L configs/c.1/ecm.usb0 ]; then
        ln -s functions/ecm.usb0 configs/c.1
fi

echo 11e20000.usb > UDC
echo 11e20000.usb > /sys/bus/platform/drivers/renesas_usbhs/unbind

The displayed trace is as follows:

 Internal error: synchronous external abort: 0000000096000010 [#1] SMP
 CPU: 0 UID: 0 PID: 188 Comm: sh Tainted: G M 6.17.0-rc7-next-20250922-00010-g41050493b2bd #55 PREEMPT
 Tainted: [M]=MACHINE_CHECK
 Hardware name: Renesas SMARC EVK version 2 based on r9a08g045s33 (DT)
 pstate: 604000c5 (nZCv daIF +PAN -UAO -TCO -DIT -SSBS BTYPE=--)
 pc : usbhs_sys_function_pullup+0x10/0x40 [renesas_usbhs]
 lr : usbhsg_update_pullup+0x3c/0x68 [renesas_usbhs]
 sp : ffff8000838b3920
 x29: ffff8000838b3920 x28: ffff00000d585780 x27: 0000000000000000
 x26: 0000000000000000 x25: 0000000000000000 x24: ffff00000c3e3810
 x23: ffff00000d5e5c80 x22: ffff00000d5e5d40 x21: 0000000000000000
 x20: 0000000000000000 x19: ffff00000d5e5c80 x18: 0000000000000020
 x17: 2e30303230316531 x16: 312d7968703a7968 x15: 3d454d414e5f4344
 x14: 000000000000002c x13: 0000000000000000 x12: 0000000000000000
 x11: ffff00000f358f38 x10: ffff00000f358db0 x9 : ffff00000b41f418
 x8 : 0101010101010101 x7 : 7f7f7f7f7f7f7f7f x6 : fefefeff6364626d
 x5 : 8080808000000000 x4 : 000000004b5ccb9d x3 : 0000000000000000
 x2 : 0000000000000000 x1 : ffff800083790000 x0 : ffff00000d5e5c80
 Call trace:
 usbhs_sys_function_pullup+0x10/0x40 [renesas_usbhs] (P)
 usbhsg_pullup+0x4c/0x7c [renesas_usbhs]
 usb_gadget_disconnect_locked+0x48/0xd4
 gadget_unbind_driver+0x44/0x114
 device_remove+0x4c/0x80
 device_release_driver_internal+0x1c8/0x224
 device_release_driver+0x18/0x24
 bus_remove_device+0xcc/0x10c
 device_del+0x14c/0x404
 usb_del_gadget+0x88/0xc0
 usb_del_gadget_udc+0x18/0x30
 usbhs_mod_gadget_remove+0x24/0x44 [renesas_usbhs]
 usbhs_mod_remove+0x20/0x30 [renesas_usbhs]
 usbhs_remove+0x98/0xdc [renesas_usbhs]
 platform_remove+0x20/0x30
 device_remove+0x4c/0x80
 device_release_driver_internal+0x1c8/0x224
 device_driver_detach+0x18/0x24
 unbind_store+0xb4/0xb8
 drv_attr_store+0x24/0x38
 sysfs_kf_write+0x7c/0x94
 kernfs_fop_write_iter+0x128/0x1b8
 vfs_write+0x2ac/0x350
 ksys_write+0x68/0xfc
 __arm64_sys_write+0x1c/0x28
 invoke_syscall+0x48/0x110
 el0_svc_common.constprop.0+0xc0/0xe0
 do_el0_svc+0x1c/0x28
 el0_svc+0x34/0xf0
 el0t_64_sync_handler+0xa0/0xe4
 el0t_64_sync+0x198/0x19c
 Code: 7100003f 1a9f07e1 531c6c22 f9400001 (79400021)
 ---[ end trace 0000000000000000 ]---
 note: sh[188] exited with irqs disabled
 note: sh[188] exited with preempt_count 1

The issue occurs because usbhs_sys_function_pullup(), which accesses the IP
registers, is executed after the USBHS clocks have been disabled. The
problem is reproducible on the Renesas RZ/G3S SoC starting with the
addition of module stop in the clock enable/disable APIs. With module stop
functionality enabled, a bus error is expected if a master accesses a
module whose clock has been stopped and module stop activated.

Disable the IP clocks at the end of remove.

Cc: stable <[email protected]>
Fixes: f1407d5 ("usb: renesas_usbhs: Add Renesas USBHS common code")
Signed-off-by: Claudiu Beznea <[email protected]>
Link: https://patch.msgid.link/[email protected]
Signed-off-by: Greg Kroah-Hartman <[email protected]>
adrian-nicolau added a commit that referenced this pull request Dec 1, 2025
)

- Update rv3028 driver to report backup switchover indication when using
RTC_VL_READ.
- Add support for RTC_VL_CLR to clear this indication.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants