Add hint to upstream documentation

Guard hacks for Linux < v6.0 with #ifdefs

Also comment on the upstream commits which made them unnecessary.

Align code style with first upstream in Linux v6.0

Add upstream note

Don't kfree_skb() after submitting the repurposed SKB

If can327 locks up your machine, but only in very specific situations,
this is probably why. The memory leak fix went too far, and I missed
the call to can327_feed_frame_to_netdev() before a return -ENODATA.

Fixes: 37111be717212b8c7779978c0385393c2d51747d

Rename elm327_* to can327_*

Don't count RTR DLC as sent bytes

RTR frames don't actually carry data, so bytes sent is 0.

Re-align buffers, make RX buffer a power of two

README: Shorten duplicate mention of replies not being used

Remove CAN specific LED code

This is now handled by common netdev LED events instead:

https://git.kernel.org/pub/scm/linux/kernel/git/netdev/net-next.git/commit/?id=6c1e423a3c84953edcf91ff03ab97829b287184a

elm327_init: Update old comment referencing bittiming_const

This has been gone since a44b237ce7e9 in 2019.

Remove fixed array size for can327_bitrate_const

No idea why this was even there.

Stop leaking SKBs in elm327_parse_frame()

SKBs need to be used or freed before returning, no exceptions.

Remove unused includes

README: Cleanup according to upstream requests

- Show communication example before known limitations
- Add missing :: to start code block
- Only mention baud rate once instead of twice
- Clarify AT CSM and AT MA
- Reflow sections touched upon

Increase ELM327_SIZE_RXBUF to support fat UART buffers

See: https://github.com/norly/elmcan/issues/8

There, a VM was used, and apparently buffers of over 256 bytes were
being piped in.

On RX buffer overflow, printk remaining input size

If can327_ldisc_rx() is called with a large 'count' because the UART
driver feeds us huge buffers, then the user should have a chance to
know and report this.

In this case, we'll have to increase ELM327_SIZE_RXBUF.

Fix compilation on Linux up to v5.4

mailbox_read()'s type signature was changed in 4e9c9484b085 which is
upstream since v5.5.

Update thanks

Accelerate parsing by not re-checking previous data

While waiting for a <CR> the code would search through prevois received
characters, too. Keep track of what we've already checked to speed this
up.

Speed up can327_is_valid_rx_char() with a LUT

Make elm327_rxbuf_cmp() return bool

Simplify/clarify branching and locking in several places

Don't ____cacheline_aligned the TX buffer

TTY drivers don't seem to care, and if they did, we'd likely have
to kmalloc() as before anyway.

Style and comments cleanup

Rename elmcan to can327

This is to clarify that this driver is not a product of ELM Electronics.

Restore WAKEUP set_bit vs. write() order in elm327_send()

Also, fix a silly spelling error.

Repack struct elmcan

Clear TTY_DO_WRITE_WAKEUP bit on uart_side_failure

checkpatch.pl style fixes

Fix lockdep_assert parameters

&elm->lock instead of elm->lock

Shorten the derivation comment at the start

Update comment about N_DEVELOPMENT

N_DEVELOPMENT has been officially set to 29 in Linux v5.18-rc1.

Only netif_wake_queue() when ELM327 is really idle

That is, when it is receiving, as that's the default state the driver
tries its best to keep it in.

Statically allocate TX buffer

And also optimise struct elmcan{}'s size with pahole.

Start/stop TTY on netdev open/close

This allows the ldisc to reset cleanly, and even after a UART error.
It also makes for a cleaner ldisc close().

Name enum elm327_to_to_do_bits in lowercase for kernel code style

Rename hw_failure to uart_side_failure for easier understanding

elm327_handle_prompt(): Explain size of local buffer

Also use snprintf() instead of sprintf()/strcpy(), to clarify the
size restriction.

Clarify memory/string comparisons

We really can't use strncmp() here. Sorry!

Remove .hangup()

We were just calling .close(), which the TTY layer does for us anyway.
And it does so with the correct locking.

Simplify TTY sending code and locking

The great get_elm() dance was unnecessary, since the TTY layer already
synchronises *all* calls to ldisc_ops with tty_ldisc_ref() and
tty_ldisc_lock().

All that we have to ensure is that the worker does not race against
close(), which is trivial if we clear TTY_DO_WRITE_WAKEUP and then flush
the worker.

Also, the worker no longer checks whether the netdev is up - all it
needs to do is to write out the TTY buffer, and wake up the netdev when
the buffer is empty.

Get rid of dummy mailbox_read() for rx_offload

Use can_rx_offload_add_manual() instead of can_rx_offload_add_fifo(),
starting with Linux 5.10.

Add can_dropped_invalid_skb() to elmcan_netdev_start_xmit()

Drop superflouous elm->can.state = CAN_STATE_STOPPED;

Minor cleanups

Convert comments to lockdep_assert_held()

Clean up types and unused vars in struct elmcan

Rename ELM_ to ELM327_STATE_

Unify indentation in struct elmcan

Move MODULE_* to end of file

Remove braces from pr_fmt

Re-add author names

Rework elm327_hw_failure()

Using es58x as a rough template

Minor cleanups

Replace ->can_dlc with ->len

can_frame.can_dlc has been deprecated in favour of can_frame.len and as
a new driver, this needs to stick to the new convention.

Also include the ugliest backwards hack #define I've ever written to
make this compile on Linux <= 5.10 which is currently LTS.

elmcan_netdev_start_xmit(): Remove unnecessary skb->len check

linux/net/can/af_can.c's can_send() guarantees this already.

Use alloc_can_skb()

Rename TODO_* to ELM327_TX_DO_*

Also rename elm->can_frame to elm->can_frame_to_send

Mark {_len,}_memstrcmp inline

Change MAGIC to DUMMY, and silence init

Remove switch() in elm327_parse_error()

This gets rid of the last hardcoded string lengths.

readme.rst: Add hint on upping the interface before use

Also name 500kbit/s as the most common OBD-II speed, to save newcomers
some headache.

Change N_ELMCAN to N_DEVELOPMENT

N_DEVELOPMENT is slated to be defined for out-of-tree ldisc modules.

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