Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

overlays: seeed-can-fd-hat: clarify how to identify HAT version #4072

Conversation

marckleinebudde
Copy link
Contributor

@marckleinebudde marckleinebudde commented Jan 13, 2021

It turns out the used CAN SPI chip is not a good way to identify the version of
the CAN HAT.

There are two different board layouts of the Seeed Studio CAN BUS FD HAT. The
v1 board doesn't have a battery holder, while the v2 board has. Update the
overlay README accordingly.

Link: Seeed-Studio/seeed-linux-dtoverlays#13
Cc: Patrick Menschel menschel.p@posteo.de
Signed-off-by: Marc Kleine-Budde mkl@pengutronix.de

It turns out the used CAN SPI chip is not a good way to identify the version of
the CAN HAT.

There are two different board layouts of the Seeed Studio CAN BUS FD HAT. The
v1 board doesn't have a battery holder, while the v2 board has. Update the
overlay README accordinly.

Link: Seeed-Studio/seeed-linux-dtoverlays#13
Cc: Patrick Menschel <menschel.p@posteo.de>
Signed-off-by: Marc Kleine-Budde <mkl@pengutronix.de>
@pelwell pelwell merged commit bd20413 into raspberrypi:rpi-5.4.y Jan 13, 2021
@marckleinebudde marckleinebudde deleted the v5.4-rpi-overlay-seeed-studio-update branch January 13, 2021 21:12
@marckleinebudde
Copy link
Contributor Author

Wow - that was quick!

popcornmix added a commit to raspberrypi/firmware that referenced this pull request Jan 15, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle
popcornmix added a commit to Hexxeh/rpi-firmware that referenced this pull request Jan 15, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 16, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 16, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 16, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 18, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 19, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 20, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
paralin added a commit to skiffos/buildroot that referenced this pull request Jan 20, 2021
kernel: overlays: seeed-can-fd-hat: clarify how to identify HAT version
See: raspberrypi/linux#4072

kernel: Add a camera regulator node(s) to allow sharing on CM4 and remove firmware nastiness
See: raspberrypi/linux#4051

firmware: isp: Fix handling of different YUV colour spaces

firmware: poe_hat: Actually close the I2C handle

Signed-off-by: Christian Stewart <christian@paral.in>
bigbearishappy added a commit to Seeed-Studio/seeed-linux-dtoverlays that referenced this pull request Apr 13, 2022
Why delete?
Because the CAN-HAT driver is already merged into the main line
of raspberrypi kernel.
raspberrypi/linux#4072
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.

None yet

2 participants