Restart on error instead of stopping

readme.rst: Add missing word

Add thanks for DKMS config

out-of-tree.rst: Add porting info

readme.rst: Split off out-of-tree documentation

This simplifies syncing changes with a future upstreamed version.

readme.rst: Remove module parameter documentation

Whoops, forgot to remove the accept_flaky_uart documentation.

Remove accept_flaky_uart module option.

Maybe we really shouldn't *too* lenient - the hexdump parsing code bails
on the first strange character anyway.

Introduce _memstrcmp() to beautify elm327_parse_error()

Also used in elm327_parse_frame() and elm327_parse_line()

Improve error handling in case of stray characters

Remove superfluous lines

Clean up comments

Forward port to Linux v5.17-rc3

...while keeping backwards compatibility, tested on v5.10.

Clean up more strings

Change WARN_ON into WARN_ON_ONCE

readme.rst: Update title

readme.rst: Remove empty To-Do list

Add BSD-3-Clause licence

Convert RX path to NAPI via can_rx_offload_*

Clarify some error texts and return values

readme.rst: Clarify DKMS install command

The backticks around `pwd` weren't rendered in GitHub's view of the
readme, and eventually led to a misunderstanding. See issue #6.

Remove TTY_LDISC_MAGIC

This is in sync with upstream 981b22b8777df7de070be1803f6d7ed4f634a43c
which removed .magic from struct tty_ldisc_ops in Linux 5.13.

Merge pull request #4 from tomaszduda23/master

Adds dkms.conf to sign kernel module

Adds dkms.conf to sign kernel module

Add thanks

Style: According to linux-5.2-rc4 checkpatch.pl

Flush worker earlier, resolving the last to-do item.

Undo buffer rework for TTY RX

Actually, we fully control the RX buffer, so DMA shouldn't be a concern.

Fixes: 40ac32acdae89383ae44f32e612bf37fa0493631

Style: Fix one more block comment

Separate buffers from struct elm

This avoids trouble with CPU caches racing DMA accesses on ARM.

Style: Fix big comments

readme.rst: Clarifications

Document dependency on can-dev

Silence unused parameter in elmcan_do_set_bittiming()

Extract RX sanity checker into separate function

Comments

Avoid return in function bodies

Use can_change_mtu() instead of hand-made function

readme.rst: Document minimum Linux version

readme.rst: Add note on termination, minor fixes

Comments and minor style

Turn BUG_ON() into WARN_ON()

Style: Move elm327_is_ready_char() further down to where it is used

Simplify init script sending

My problems with initializing the chip should now be solved by the
'accept_flaky_uart' module parameter.

ioctl(SIOCGIFNAME): Copy at most IFNAMSIZ bytes, but less if possible

Thus, we don't leak any trailing bytes that may be in the name buffer.

Always copy IFNAMSIZ bytes on ioctl(SIOCGIFNAME)

Just in case dev->name is ever not NUL terminated

Remove stray spin_lock_bh()

Style fixes

Synchronize email addresses

Fix up stray comments now that we have elm327_is_ready_char()

Sanity check TTY input and bail on problems

Also, introduce module parameter 'accept_flaky_uart'.

If your adapter or its connection is unreliable, set this option to
true to try and make the best of a bad situation, but undefined
behavior is prone to occur.

Ignore tty RX after HW failure, locking fix

readme.rst: Add rationale behind the chosen configuration

Plus minor fixes.

README.txt: Add limitations for 'v1.5' clones

README.rst: We actually use can_bus_off() now

README.rst: Forgot to bump the ldisc number

Work around hardware bug when waiting for '>' prompt

Sometimes the ELM327 sets 0x80 and/or 0x40 when sending '>' to indicate
that it is ready to receive the next command.

Masking these two bits out seems to take care of most or all hangs
during initialization.

Shut down ELM327's 'protocol' before reconfiguring CAN

According to ELM327's manual, AT PC should be sent before issuing AT PB.

Drop fake bittimings in favor of bitrate table

The fake bittiming table produces wrong results in Linux 4.19's bitrate
based parameter calculation.

SocketCAN support for fixed bitrates was introduced in 431af779256c in
Jan 2017, and as of Linux 4.18 the only driver using it is mcba_usb.
Bitrates are also fixed in ELM327, and none of the other parameters can
be tuned, so using this API makes sense here as well.

Improve readability of elm327_handle_prompt()

Null stack variables before use

Don't try to send garbage after each init script entry

Previously, we'd try to send the local txbuf even though it had not been
written to. Thus strlen() would overrun the buffer, which recent Linux
versions catch in a BUG_ON().

Use more netdev_err() instead of pr_err()

...and remove a stray comment

Return cleanly in functions calling elm327_hw_failure()

Don't reset on HW fault, but take the interface down.

There is currently no interface to try again, other than detaching and
re-attaching the line discipline. It wouldn't make much sense either,
unless we've run into an unexpected state in the ELM327's firmware.

elmcan_ldisc_ioctl(): Use elm->dev->name again

We now have proper locking, so dev->name is guaranteed to exist.

Use netdev_* prints throughout

Rename elm327_panic() to elm327_hw_failure() to avoid confusion

Handle tty->ops->write() returning error

This was already the case in elmcan_ldisc_tx_worker(), but was missing
in elm327_send().

Replace '>' with constant ELM327_READY_CHAR

Bump N_ELMCAN to avoid conflicts with future in-tree line disciplines.

We can't go higher than 29 because of NR_LDISCS == 30 in Linux 5.0.
We can't go lower than 28 because of N_NULL == 27 in Linux 5.0.

Fix race between ldisc_close() and ldisc_*()

There was a chance that something needed the elm object after it was
freed. Proper locking stops this.

Avoid ldisc_ioctl() racing unregister_candev()

When getting the CAN interface's name via the SIOCGIFNAME ioctl,
elm->dev->name may no longer exist because the interface is being shut
down.

Also, print the name of the interface to dmesg.

Comments

Import first public commit

Initial commit