From patchwork Wed Apr 26 08:03:08 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: "D. Starke" X-Patchwork-Id: 87711 Return-Path: Delivered-To: ouuuleilei@gmail.com Received: by 2002:a59:b0ea:0:b0:3b6:4342:cba0 with SMTP id b10csp79963vqo; Wed, 26 Apr 2023 01:10:47 -0700 (PDT) X-Google-Smtp-Source: AKy350YZz/u17hz0X3n6DCc4v6yk84gxRTUrShHEZaZ1816b2jwbJcs6BkJbkVvQRCEzTd+8Y/BN X-Received: by 2002:a05:6a00:21cf:b0:627:f0e1:4fbf with SMTP id t15-20020a056a0021cf00b00627f0e14fbfmr30510587pfj.33.1682496647415; Wed, 26 Apr 2023 01:10:47 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1682496647; cv=none; d=google.com; s=arc-20160816; b=VHivDj52pV3fKTRAFd2zZ9wUxNUFk5BreqjANThAycTMUiIgOuz9q/Y268PZ57V15/ zTFnacx9OK/XmpmfnS3o38ElkOs5kNrvj1ZxgfjHrdvX0FMoKW1R0/n+XfqYu7w2XJlI 6jD9Dk96SvrJL+RAunoR5f+WFE7kmP51QdxeyTVnL/DDw6u66/UBP3np7m1JAh24j9yq APCIDipyxRE+QcK2beeExIAuPoZ6T6tLJP/b6WFtyH1rCv72H36mQp36hpahZxUZHFK/ 5l7d7vwQvKcKAr1JO9XuJ95EC/Khd1+UFFRxz6X79dxzrCcA/QX8eDXlXHIyGHKy9PVx BG5w== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:feedback-id:content-transfer-encoding :mime-version:message-id:date:subject:cc:to:from:dkim-signature; bh=btnpjanps7GZuVDl1HyLhoOR3uwTdi+jFgZdJzn8lO8=; b=x6fvOgCssM4yE6yyCPJCtHwQ7eISy8zfX1I4i33Z6rdvo+IGu7T+efZkb1MtnHBj8I MDaX4AFCXoWfBA6rrqrvM0EUtDUbS7KqXQ4sgyACg8fniDtigSM6WWkxja7eH73dXStj cOEnn7LNhTveaeU2+ocLvXE+8L4M38KqyU/1VOUfvUUBtNwtXZR3fbOPxzwp1vZro8BT zp6+yUpqtc7QSDjZS2a8cyU/164svMOG/AFPBjHpRTusOw+6Xr7FTkLNxWwGl6b0QKJI 4oYS8CemqL3AxortVQ+uyTvSITiwWIfWrXszCQn/a/J0UiSv6CrufoIeREu0ui7ZVgh0 22nA== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@siemens.com header.s=fm1 header.b=W5lRVWft; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=siemens.com Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id z2-20020aa79902000000b0063d33dc49e8si15423962pff.363.2023.04.26.01.10.33; Wed, 26 Apr 2023 01:10:47 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@siemens.com header.s=fm1 header.b=W5lRVWft; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=REJECT sp=REJECT dis=NONE) header.from=siemens.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S239998AbjDZIDr (ORCPT + 99 others); Wed, 26 Apr 2023 04:03:47 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:57672 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S239686AbjDZIDg (ORCPT ); Wed, 26 Apr 2023 04:03:36 -0400 Received: from mta-65-227.siemens.flowmailer.net (mta-65-227.siemens.flowmailer.net [185.136.65.227]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 12EDF3C03 for ; Wed, 26 Apr 2023 01:03:32 -0700 (PDT) Received: by mta-65-227.siemens.flowmailer.net with ESMTPSA id 2023042608033045f25e7b2df527a7a6 for ; Wed, 26 Apr 2023 10:03:30 +0200 DKIM-Signature: v=1; a=rsa-sha256; q=dns/txt; c=relaxed/relaxed; s=fm1; d=siemens.com; i=daniel.starke@siemens.com; h=Date:From:Subject:To:Message-ID:MIME-Version:Content-Type:Content-Transfer-Encoding:Cc; bh=btnpjanps7GZuVDl1HyLhoOR3uwTdi+jFgZdJzn8lO8=; b=W5lRVWftuxQ4NVdnxx0NWP7jI7LdRTLWA1YNChTZ29wLWaE4GfdqQf5dZzFXItnI1kdeRh W3yvJuZ/H0lNGx+fSsEaHFHhL/zVScNUR5u78WS+aws60YBQzCDiw1f1kJxL+Db4wmFbbb+N lh/lSTsV82AlfnHts8V0yqXXxFsXw=; From: "D. Starke" To: linux-serial@vger.kernel.org, gregkh@linuxfoundation.org, jirislaby@kernel.org, ilpo.jarvinen@linux.intel.com Cc: linux-kernel@vger.kernel.org, Daniel Starke Subject: [PATCH v4 1/8] tty: n_gsm: add restart flag to DLC specific ioctl config Date: Wed, 26 Apr 2023 10:03:08 +0200 Message-Id: <20230426080315.7595-1-daniel.starke@siemens.com> MIME-Version: 1.0 X-Flowmailer-Platform: Siemens Feedback-ID: 519:519-314044:519-21489:flowmailer X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIMWL_WL_MED, DKIM_SIGNED,DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,RCVD_IN_MSPIKE_H2, SPF_HELO_PASS,SPF_PASS,T_SCC_BODY_TEXT_LINE,URIBL_BLOCKED autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org X-getmail-retrieved-from-mailbox: =?utf-8?q?INBOX?= X-GMAIL-THRID: =?utf-8?q?1764225604788707258?= X-GMAIL-MSGID: =?utf-8?q?1764225604788707258?= From: Daniel Starke Currently, changing the parameters of a DLCI gives no direct control to the user whether this should trigger a channel reset or not. The decision is solely made by the driver based on the assumption which parameter changes are compatible or not. Therefore, the user has no means to perform an automatic channel reset after parameter configuration for non-conflicting changes. Add the parameter 'flags' to 'gsm_dlci_config' to force a channel reset after ioctl setting regardless of whether the changes made require this or not by setting this to 'GSM_FL_RESTART'. Note that 'GSM_FL_RESTART' is currently the only allow flag to allow additions here. Signed-off-by: Daniel Starke --- drivers/tty/n_gsm.c | 4 ++++ include/uapi/linux/gsmmux.h | 13 ++++++++++++- 2 files changed, 16 insertions(+), 1 deletion(-) v3 -> v4: Changed gsm_dlci_config field name from 'restart' to 'flags' and introduced 'GSM_FL_RESTART' to set the restart flag. The patch description was changed accordingly. This was done as suggested during the review. The remarked kernel doc compatible field comment is done in patch 2/8. Link: https://lore.kernel.org/all/2023042453-dubbed-botany-2ed9@gregkh/ diff --git a/drivers/tty/n_gsm.c b/drivers/tty/n_gsm.c index b411a26cc092..66edcf65a4dd 100644 --- a/drivers/tty/n_gsm.c +++ b/drivers/tty/n_gsm.c @@ -2532,6 +2532,8 @@ static int gsm_dlci_config(struct gsm_dlci *dlci, struct gsm_dlci_config *dc, in return -EINVAL; if (dc->k > 7) return -EINVAL; + if (dc->flags & ~GSM_FL_RESTART) /* allow future extensions */ + return -EINVAL; /* * See what is needed for reconfiguration @@ -2546,6 +2548,8 @@ static int gsm_dlci_config(struct gsm_dlci *dlci, struct gsm_dlci_config *dc, in /* Requires care */ if (dc->priority != dlci->prio) need_restart = true; + if (dc->flags & GSM_FL_RESTART) + need_restart = true; if ((open && gsm->wait_config) || need_restart) need_open = true; diff --git a/include/uapi/linux/gsmmux.h b/include/uapi/linux/gsmmux.h index eb67884e5f38..958257af05ab 100644 --- a/include/uapi/linux/gsmmux.h +++ b/include/uapi/linux/gsmmux.h @@ -2,10 +2,20 @@ #ifndef _LINUX_GSMMUX_H #define _LINUX_GSMMUX_H +#include #include #include #include +/* + * flags definition for n_gsm + * + * Used by: + * struct gsm_dlci_config.flags + */ +/* Force DLCI channel reset? Always cleared on retrieval. */ +#define GSM_FL_RESTART _BITUL(0) + struct gsm_config { unsigned int adaption; @@ -58,7 +68,8 @@ struct gsm_dlci_config { __u32 priority; /* Priority (0 for default value) */ __u32 i; /* Frame type (1 = UIH, 2 = UI) */ __u32 k; /* Window size (0 for default value) */ - __u32 reserved[8]; /* For future use, must be initialized to zero */ + __u32 flags; /* DLCI specific flags. */ + __u32 reserved[7]; /* For future use, must be initialized to zero */ }; #define GSMIOC_GETCONF_DLCI _IOWR('G', 7, struct gsm_dlci_config)