Re: [PATCH v1 4/5] can: kvaser_usb: Use CAN_MHZ define in assignment of frequency

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

 



On 2021-11-19 14:19, Jimmy Assarsson wrote:
Use the CAN_MHZ define when assigning frequencies.

Maybe we should use the HZ_PER_MHZ define introduced in kernel 5.15 in
linux/units.h. It also got defines for KILO and MEGA, which we also got
in linux/can/bittiming.h.

What do you think?

...

  static const struct kvaser_usb_dev_cfg kvaser_usb_hydra_dev_cfg_kcan = {
  	.clock = {
-		.freq = 80000000,
+		.freq = 80 * CAN_MHZ,
  	},



[Index of Archives]     [Automotive Discussions]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [eCos]     [Asterisk Internet PBX]     [Linux API]     [CAN Bus]

  Powered by Linux