On Sat, Feb 17, 2024 at 8:30 AM Jon Cormier via
lists.yoctoproject.org <http://lists.yoctoproject.org>
<[email protected]
<mailto:[email protected]>> wrote:
I think the current issue is that we are bbappend-ing a systemd
recipe now
to install the udev rules. Perhaps there is a udev recipe that
should be
bbappended instead?
On Fri, Feb 16, 2024, 3:14 AM Aniket Limaye via
lists.yoctoproject.org <http://lists.yoctoproject.org>
<[email protected]
<mailto:[email protected]>> wrote:
> I assume the question being how do we plan to handle images where
> sysvinit is used instead of systemd?
>
> I do not have a lot of experience with sysvinit vs systemd or
udev rules
> for that matter. Do you have any suggestions to make these naming
> conventions apply to both?
>
> Regards,
> Aniket
>
>
> On 2/13/2024 11:10 PM, Denys Dmytriyenko wrote:
> > BTW, what about sysvinit support?
> >
> >
> > On Thu, Feb 08, 2024 at 01:28:47PM -0600, Andrew Davis via
> lists.yoctoproject.org <http://lists.yoctoproject.org> wrote:
> >> On 2/8/24 12:37 PM, Aniket Limaye via lists.yoctoproject.org
<http://lists.yoctoproject.org> wrote:
> >>> When multiple CAN's are present, then names that are getting
assigned
> >>> change after every boot even after providing alias in the
device
> >>> tree.
> >>> So add udev rules to specify the interface name to use for
> >>> each mcan device, to make it persistent across reboots.
> >>>
> >>> Signed-off-by: Bhavya Kapoor <[email protected]
<mailto:[email protected]>>
> >>> Signed-off-by: Aniket Limaye <[email protected]
<mailto:[email protected]>>
> >>> ---
> >>> v4:
> >>> - Reformat patch to avoid repeated code for k3 devices
> >>> - Add udev rules for dra7xx and ti33x too, as suggested
> >>> - Still keeping separate files per platform to avoid
potential
> >>> confusion created by merging all the rules into one
file
> >> You can keep the rules in separate files, but you need to
install
> >> them all unconditionally for all machines. Otherwise this
package
> >> becomes machine specific, which we should avoid.
> >>
> >> Andrew
> >>
> >>> v3: Fix typos in the filename being installed in v2
> >>> v2:
> >>> - Switch to udev rules instead of systemd network links,
as per
> >>> feedback from v1:
> https://lists.yoctoproject.org/g/meta-arago/message/15104
<https://lists.yoctoproject.org/g/meta-arago/message/15104>
> >>> - Also, extend to Sitara devices as well
> >>> ---
> >>> .../systemd/systemd/37-can-am62.rules | 4 ++
> >>> .../systemd/systemd/37-can-dra7.rules | 2 +
> >>> .../systemd/systemd/37-can-j7.rules | 20 ++++++++++
> >>> .../systemd/systemd/37-can-ti33x.rules | 2 +
> >>> .../recipes-core/systemd/systemd_%.bbappend | 37
> +++++++++++++++++++
> >>> 5 files changed, 65 insertions(+)
> >>> create mode 100644
> meta-arago-distro/recipes-core/systemd/systemd/37-can-am62.rules
> >>> create mode 100644
> meta-arago-distro/recipes-core/systemd/systemd/37-can-dra7.rules
> >>> create mode 100644
> meta-arago-distro/recipes-core/systemd/systemd/37-can-j7.rules
> >>> create mode 100644
> meta-arago-distro/recipes-core/systemd/systemd/37-can-ti33x.rules
> >>>
> >>> diff --git
>
a/meta-arago-distro/recipes-core/systemd/systemd/37-can-am62.rules
>
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-am62.rules
> >>> new file mode 100644
> >>> index 00000000..59dc8075
> >>> --- /dev/null
> >>> +++
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-am62.rules
> >>> @@ -0,0 +1,4 @@
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="4e08000.can",
> NAME="mcu_mcan0"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="4e18000.can",
> NAME="mcu_mcan1"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="20701000.can",
> NAME="main_mcan0"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="20711000.can",
> NAME="main_mcan1"
> >>> diff --git
>
a/meta-arago-distro/recipes-core/systemd/systemd/37-can-dra7.rules
>
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-dra7.rules
> >>> new file mode 100644
> >>> index 00000000..3b14150d
> >>> --- /dev/null
> >>> +++
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-dra7.rules
> >>> @@ -0,0 +1,2 @@
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="4ae3c000.can",
> NAME="main_dcan1"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="48480000.can",
> NAME="main_dcan2"
> >>> diff --git
> a/meta-arago-distro/recipes-core/systemd/systemd/37-can-j7.rules
> b/meta-arago-distro/recipes-core/systemd/systemd/37-can-j7.rules
> >>> new file mode 100644
> >>> index 00000000..f2804ecd
> >>> --- /dev/null
> >>> +++
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-j7.rules
> >>> @@ -0,0 +1,20 @@
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="40528000.can",
> NAME="mcu_mcan0"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="40568000.can",
> NAME="mcu_mcan1"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2701000.can",
> NAME="main_mcan0"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2711000.can",
> NAME="main_mcan1"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2721000.can",
> NAME="main_mcan2"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2731000.can",
> NAME="main_mcan3"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2741000.can",
> NAME="main_mcan4"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2751000.can",
> NAME="main_mcan5"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2761000.can",
> NAME="main_mcan6"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2771000.can",
> NAME="main_mcan7"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2781000.can",
> NAME="main_mcan8"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2791000.can",
> NAME="main_mcan9"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="27a1000.can",
> NAME="main_mcan10"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="27b1000.can",
> NAME="main_mcan11"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="27c1000.can",
> NAME="main_mcan12"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="27d1000.can",
> NAME="main_mcan13"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2681000.can",
> NAME="main_mcan14"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="2691000.can",
> NAME="main_mcan15"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="26a1000.can",
> NAME="main_mcan16"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="26b1000.can",
> NAME="main_mcan17"
> >>> diff --git
>
a/meta-arago-distro/recipes-core/systemd/systemd/37-can-ti33x.rules
>
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-ti33x.rules
> >>> new file mode 100644
> >>> index 00000000..62d650fe
> >>> --- /dev/null
> >>> +++
b/meta-arago-distro/recipes-core/systemd/systemd/37-can-ti33x.rules
> >>> @@ -0,0 +1,2 @@
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="481cc000.can",
> NAME="main_dcan0"
> >>> +SUBSYSTEM=="net", ACTION=="add", KERNELS=="481d0000.can",
> NAME="main_dcan1"
> >>> diff --git
a/meta-arago-distro/recipes-core/systemd/systemd_%.bbappend
> b/meta-arago-distro/recipes-core/systemd/systemd_%.bbappend
> >>> index f38606f2..f9919708 100644
> >>> ---
a/meta-arago-distro/recipes-core/systemd/systemd_%.bbappend
> >>> +++
b/meta-arago-distro/recipes-core/systemd/systemd_%.bbappend
> >>> @@ -2,6 +2,21 @@ PR:append = ".arago7"
> >>> FILESEXTRAPATHS:prepend := "${THISDIR}/${PN}:"
> >>> +CAN_RULES_SRC = ""
> >>> +CAN_RULES_SRC:j721e = "37-can-j7.rules"
> >>> +CAN_RULES_SRC:j7200 = "37-can-j7.rules"
> >>> +CAN_RULES_SRC:j721s2 = "37-can-j7.rules"
> >>> +CAN_RULES_SRC:j784s4 = "37-can-j7.rules"
> >>> +CAN_RULES_SRC:am65xx = "37-can-j7.rules"
> >>> +CAN_RULES_SRC:am62xx = "37-can-am62.rules"
> >>> +CAN_RULES_SRC:am62pxx = "37-can-am62.rules"
> >>> +CAN_RULES_SRC:am62axx = "37-can-am62.rules"
> >>> +CAN_RULES_SRC:am64xx = "37-can-am62.rules"
> >>> +CAN_RULES_SRC:j722s = "37-can-am62.rules"
> >>> +CAN_RULES_SRC:dra7xx = "37-can-dra7.rules"
> >>> +CAN_RULES_SRC:ti33x = "37-can-ti33x.rules"
> >>> +CAN_RULES_TGT = "${libdir}/udev/rules.d/37-can.rules"
> >>> +
> >>> SRC_URI:append = " \
> >>> file://local.rules \
> >>> file://usb1-rules.sh \
> >>> @@ -14,6 +29,10 @@ SRC_URI:append = " \
> >>> file://timesyncd.conf \
> >>> "
> >>> +SRC_URI:append:k3 = " file://${CAN_RULES_SRC}"
> >>> +SRC_URI:append:dra7xx = " file://${CAN_RULES_SRC}"
> >>> +SRC_URI:append:ti33x = " file://${CAN_RULES_SRC}"
> >>> +
> >>> do_install:append() {
> >>> install -d ${D}${sysconfdir}/udev/rules.d/
> >>> install -m 0644 ${WORKDIR}/local.rules
> ${D}${sysconfdir}/udev/rules.d/
> >>> @@ -43,3 +62,21 @@ do_install:append() {
> >>> install -d ${D}${sysconfdir}/systemd/
> >>> install -m 0644 ${WORKDIR}/timesyncd.conf
> ${D}${sysconfdir}/systemd/
> >>> }
> >>> +
> >>> +do_install:append:k3() {
> >>> + install -d ${D}${libdir}/udev/rules.d
> >>> + install -m 0644 ${WORKDIR}/${CAN_RULES_SRC}
${D}${CAN_RULES_TGT}
> >>> +}
> >>> +
> >>> +do_install:append:dra7xx() {
> >>> + install -d ${D}${libdir}/udev/rules.d
> >>> + install -m 0644 ${WORKDIR}/${CAN_RULES_SRC}
${D}${CAN_RULES_TGT}
> >>> +}
> >>> +
> >>> +do_install:append:ti33x() {
> >>> + install -d ${D}${libdir}/udev/rules.d
> >>> + install -m 0644 ${WORKDIR}/${CAN_RULES_SRC}
${D}${CAN_RULES_TGT}
> >>> +}
> >>> +
> >>> +FILES:udev += "${CAN_RULES_TGT}"
> >>> +
>
>
>
>
>
>
--
Jonathan Cormier
Software Engineer
Voice: 315.425.4045 x222
http://www.CriticalLink.com <http://www.criticallink.com/>
6712 Brooklawn Parkway, Syracuse, NY 13211
<https://www.linkedin.com/company/critical-link-llc>
<https://twitter.com/Critical_Link>
