Re: [PATCH v1 1/2] clocksource: loongson2_hpet: add hpet driver support

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 





在 2022/9/30 下午10:43, WANG Xuerui 写道:
Hi,

On 9/30/22 17:35, Yinbo Zhu wrote:
HPET (High Precision Event Timer) defines a new set of timers, which
are used by the operating system to schedule threads, interrupt the
kernel and interrupt the multimedia timer server. The operating
system can assign different timers to different applications. By
configuration, each timer can generate interrupt independently.

The loongson2 HPET module includes a main count and three comparators
, all of which are 32 bits wide. Among the three comparators, only
one comparator supports periodic interrupt, all three comparators
support non periodic interrupts.

Signed-off-by: Yinbo Zhu <zhuyinbo@xxxxxxxxxxx>
---
  MAINTAINERS                          |   6 +
  arch/loongarch/kernel/time.c         |   3 +
  drivers/clocksource/Kconfig          |   9 +
  drivers/clocksource/Makefile         |   1 +
  drivers/clocksource/loongson2_hpet.c | 332 +++++++++++++++++++++++++++
  5 files changed, 351 insertions(+)
  create mode 100644 drivers/clocksource/loongson2_hpet.c

Thanks for the contribution, but as HPET is a fairly common peripheral that have well standardized behavior, why not simply add DT support to the drivers/char/hpet.c and be done with it?
At present, the drivers/char/hpet.c doesn't support DT(dts parse) and
loongson2 soc wasn't belong to X86 or IA64 architecture, Although it is
an option to add DT support for this driver, I consider that the hpet
design of loongson2 series socs may not be universal. In addition, I
notice that mips and x86 have their own hpet drivers. In order to reduce
the impact on the public driver and make more flexible modifications to
the loongson2 hpet driver under its own architecture, a separate
loongson2 hpet driver was submitted.

config HPET
        bool "HPET - High Precision Event Timer" if (X86 || IA64)
        default n
        depends on ACPI
        help
If you say Y here, you will have a miscdevice named "/dev/hpet/". Each open selects one of the timers supported by the HPET. The timers are
          non-periodic and/or periodic.


BRs
Yinbo Zhu.






[Index of Archives]     [Device Tree Compilter]     [Device Tree Spec]     [Linux Driver Backports]     [Video for Linux]     [Linux USB Devel]     [Linux PCI Devel]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [XFree86]     [Yosemite Backpacking]


  Powered by Linux