Project

General

Profile

Feature #33276

Add puc(4) driver to the kernel config

Added by Damon Marcella about 1 year ago. Updated about 1 year ago.

Status:
Done
Priority:
No priority
Assignee:
Alexander Motin
Category:
Hardware
Target version:
Estimated time:
Severity:
Low
Reason for Closing:
Reason for Blocked:
Needs QA:
No
Needs Doc:
No
Needs Merging:
No
Needs Automation:
No
Support Suite Ticket:
n/a
Hardware Configuration:

Description

By default, the uart(4) driver doesn't pull in the code in dev/uart/uart_bus_puc.c unless 'puc' is in the kernel config.
So that the uart(4) driver is able to make use of the puc(4) shim loaded as a kernel module, a FreeBSD forum post suggests removing 'puc' from the end of the line for uart_bus_puc.c in /sys/conf/files.

https://forums.freenas.org/index.php?threads/pci-serial-port-card.63590/

https://forums.freebsd.org/threads/pci-dual-serial-cards.8108/

Would it be acceptable or appropriate to either make the suggested change to /sys/conf/files or to include the puc(4) driver in the kernel config for the FreeNAS kernel?

puc.png (7.23 KB) puc.png Rishabh Chauhan, 06/11/2018 09:59 AM
18361

Associated revisions

Revision a0bed27f (diff)
Added by Alexander Motin about 1 year ago

Add puc(4) driver to the kernel configs. (#52)

Ticket: #33276

History

#1 Updated by Dru Lavigne about 1 year ago

  • Assignee changed from Release Council to Alexander Motin

#2 Updated by Alexander Motin about 1 year ago

  • Status changed from Unscreened to In Progress
  • Target version changed from Backlog to 11.2-RC2
  • Severity changed from New to Low

I see no any problem to add puc(4) driver into the kernel. I am running the test build right now.

#3 Updated by Alexander Motin about 1 year ago

  • Status changed from In Progress to Ready for Testing

Build has succeeded. puc(4) driver is in the kernel config for 11.2.

#4 Updated by Dru Lavigne about 1 year ago

  • Subject changed from uart(4) driver support for puc(4) kernel module in FreeNAS kernel to Add puc(4) driver to the kernel config
  • Needs Doc changed from Yes to No
  • Needs Merging changed from Yes to No

#5 Updated by Dru Lavigne about 1 year ago

  • Target version changed from 11.2-RC2 to 11.2-BETA1

#6 Updated by Rishabh Chauhan about 1 year ago

  • File puc.png puc.png added
  • Status changed from Ready for Testing to Passed Testing
  • Needs QA changed from Yes to No
18361

on a Mini: FreeNAS-11.2-MASTER-201806110410
After consulting Mav, I went and looked at PR: https://github.com/freenas/build/pull/52/files which states that puc has been added to the kernel
to confirm that on a machine, I tried #'kldstat -v | grep puc' to show the presence of puc (Refer attached screenshot puc.png)

#7 Updated by Dru Lavigne about 1 year ago

  • Status changed from Passed Testing to Done

Also available in: Atom PDF