diff mbox series

[net-next,1/2] tty: Add N_TASHTALK line discipline for TashTalk Localtalk serial driver

Message ID 20240817093258.9220-1-rwz@xhero.org (mailing list archive)
State Changes Requested
Delegated to: Netdev Maintainers
Headers show
Series pull-request appletalk 2024-08-17 | expand

Checks

Context Check Description
netdev/series_format success Posting correctly formatted
netdev/tree_selection success Clearly marked for net-next, async
netdev/ynl success Generated files up to date; no warnings/errors; no diff in generated;
netdev/fixes_present success Fixes tag not required for -next series
netdev/header_inline success No static functions without inline keyword in header files
netdev/build_32bit success Errors and warnings before: 21 this patch: 21
netdev/build_tools success Errors and warnings before: 0 this patch: 0
netdev/cc_maintainers success CCed 3 of 3 maintainers
netdev/build_clang success Errors and warnings before: 29 this patch: 29
netdev/verify_signedoff success Signed-off-by tag matches author and committer
netdev/deprecated_api success None detected
netdev/check_selftest success No net selftest shell script
netdev/verify_fixes success No Fixes tag
netdev/build_allmodconfig_warn success Errors and warnings before: 289 this patch: 289
netdev/checkpatch success total: 0 errors, 0 warnings, 0 checks, 10 lines checked
netdev/build_clang_rust success No Rust files in patch. Skipping build
netdev/kdoc success Errors and warnings before: 0 this patch: 0
netdev/source_inline success Was 0 now: 0

Commit Message

Rodolfo Zitellini Aug. 17, 2024, 9:32 a.m. UTC
This is the patch for the line discipline

Signed-off-by: Rodolfo Zitellini <rwz@xhero.org>

---
 include/uapi/linux/tty.h | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

Comments

Greg Kroah-Hartman Aug. 17, 2024, 11:03 a.m. UTC | #1
On Sat, Aug 17, 2024 at 11:32:58AM +0200, Rodolfo Zitellini wrote:
> This is the patch for the line discipline

Please look at the kernel documentation for best how to write changelog
texts.  This needs a bit of work.

thanks,

greg k-h
Rodolfo Zitellini Aug. 17, 2024, 1:15 p.m. UTC | #2
> Please look at the kernel documentation for best how to write changelog
> texts.  This needs a bit of work.

Hi Greg,
thanks for your review! I will absolutely improve the commit messages on both patches in the next version of the series.

Kind Regards,
Rodolfo
Jiri Slaby Aug. 19, 2024, 5:16 a.m. UTC | #3
Where is 2/2?

It cannot be seen:
https://lore.kernel.org/all/20240817093258.9220-1-rwz@xhero.org/#r

On 17. 08. 24, 11:32, Rodolfo Zitellini wrote:
> This is the patch for the line discipline
> 
> Signed-off-by: Rodolfo Zitellini <rwz@xhero.org>
> 
> ---
>   include/uapi/linux/tty.h | 3 ++-
>   1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/include/uapi/linux/tty.h b/include/uapi/linux/tty.h
> index 68aeae2addec..bac4c0af67e6 100644
> --- a/include/uapi/linux/tty.h
> +++ b/include/uapi/linux/tty.h
> @@ -39,8 +39,9 @@
>   #define N_MCTP		28	/* MCTP-over-serial */
>   #define N_DEVELOPMENT	29	/* Manual out-of-tree testing */
>   #define N_CAN327	30	/* ELM327 based OBD-II interfaces */
> +#define N_TASHTALK	31	/* TashTalk LocalTalk driver */
>   
>   /* Always the newest line discipline + 1 */
> -#define NR_LDISCS	31
> +#define NR_LDISCS	32
>   
>   #endif /* _UAPI_LINUX_TTY_H */
Jiri Slaby Aug. 19, 2024, 5:32 a.m. UTC | #4
On 19. 08. 24, 7:16, Jiri Slaby wrote:
> Where is 2/2?
> 
> It cannot be seen:
> https://lore.kernel.org/all/20240817093258.9220-1-rwz@xhero.org/#r

It's a separate <20240817093316.9239-1-rwz@xhero.org>. Please fix 
threading when sending next time.
diff mbox series

Patch

diff --git a/include/uapi/linux/tty.h b/include/uapi/linux/tty.h
index 68aeae2addec..bac4c0af67e6 100644
--- a/include/uapi/linux/tty.h
+++ b/include/uapi/linux/tty.h
@@ -39,8 +39,9 @@ 
 #define N_MCTP		28	/* MCTP-over-serial */
 #define N_DEVELOPMENT	29	/* Manual out-of-tree testing */
 #define N_CAN327	30	/* ELM327 based OBD-II interfaces */
+#define N_TASHTALK	31	/* TashTalk LocalTalk driver */
 
 /* Always the newest line discipline + 1 */
-#define NR_LDISCS	31
+#define NR_LDISCS	32
 
 #endif /* _UAPI_LINUX_TTY_H */