* [PATCH] platform/surface: aggregator: Fix kernel-doc references
@ 2021-01-14 15:08 Maximilian Luz
2021-01-18 18:22 ` Hans de Goede
0 siblings, 1 reply; 2+ messages in thread
From: Maximilian Luz @ 2021-01-14 15:08 UTC (permalink / raw
To: linux-doc, Jonathan Corbet
Cc: Maximilian Luz, Mauro Carvalho Chehab, Hans de Goede, Mark Gross,
linux-kernel, platform-driver-x86
Both, ssh_rtl_rx_start() and ssh_rtl_tx_start() functions, do not exist
and have been consolidated into ssh_rtl_start(). Nevertheless,
kernel-doc references the former functions. Replace those references
with references to ssh_rtl_start().
Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
---
Note: This patch does not change the kernel doc at the ssh_rtl_start()
itself, as there is already another patch for it:
"platform/surface: aggregator: fix a kernel-doc markup"
https://lore.kernel.org/patchwork/patch/1364953/
---
drivers/platform/surface/aggregator/ssh_request_layer.c | 5 ++---
1 file changed, 2 insertions(+), 3 deletions(-)
diff --git a/drivers/platform/surface/aggregator/ssh_request_layer.c b/drivers/platform/surface/aggregator/ssh_request_layer.c
index bb1c862411a2..4fbe58265e31 100644
--- a/drivers/platform/surface/aggregator/ssh_request_layer.c
+++ b/drivers/platform/surface/aggregator/ssh_request_layer.c
@@ -1004,9 +1004,8 @@ int ssh_request_init(struct ssh_request *rqst, enum ssam_request_flags flags,
*
* Initializes the given request transport layer and associated packet
* transport layer. Transmitter and receiver threads must be started
- * separately via ssh_rtl_tx_start() and ssh_rtl_rx_start(), after the
- * request-layer has been initialized and the lower-level serial device layer
- * has been set up.
+ * separately via ssh_rtl_start(), after the request-layer has been
+ * initialized and the lower-level serial device layer has been set up.
*
* Return: Returns zero on success and a nonzero error code on failure.
*/
--
2.30.0
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [PATCH] platform/surface: aggregator: Fix kernel-doc references
2021-01-14 15:08 [PATCH] platform/surface: aggregator: Fix kernel-doc references Maximilian Luz
@ 2021-01-18 18:22 ` Hans de Goede
0 siblings, 0 replies; 2+ messages in thread
From: Hans de Goede @ 2021-01-18 18:22 UTC (permalink / raw
To: Maximilian Luz, linux-doc, Jonathan Corbet
Cc: Mauro Carvalho Chehab, Mark Gross, linux-kernel,
platform-driver-x86
Hi,
On 1/14/21 4:08 PM, Maximilian Luz wrote:
> Both, ssh_rtl_rx_start() and ssh_rtl_tx_start() functions, do not exist
> and have been consolidated into ssh_rtl_start(). Nevertheless,
> kernel-doc references the former functions. Replace those references
> with references to ssh_rtl_start().
>
> Signed-off-by: Maximilian Luz <luzmaximilian@gmail.com>
Thank you for your patch, I've applied this patch to my review-hans
branch:
https://git.kernel.org/pub/scm/linux/kernel/git/pdx86/platform-drivers-x86.git/log/?h=review-hans
Note it will show up in my review-hans branch once I've pushed my
local branch there, which might take a while.
Once I've run some tests on this branch the patches there will be
added to the platform-drivers-x86/for-next branch and eventually
will be included in the pdx86 pull-request to Linus for the next
merge-window.
Regards,
Hans
> ---
>
> Note: This patch does not change the kernel doc at the ssh_rtl_start()
> itself, as there is already another patch for it:
>
> "platform/surface: aggregator: fix a kernel-doc markup"
> https://lore.kernel.org/patchwork/patch/1364953/
>
> ---
> drivers/platform/surface/aggregator/ssh_request_layer.c | 5 ++---
> 1 file changed, 2 insertions(+), 3 deletions(-)
>
> diff --git a/drivers/platform/surface/aggregator/ssh_request_layer.c b/drivers/platform/surface/aggregator/ssh_request_layer.c
> index bb1c862411a2..4fbe58265e31 100644
> --- a/drivers/platform/surface/aggregator/ssh_request_layer.c
> +++ b/drivers/platform/surface/aggregator/ssh_request_layer.c
> @@ -1004,9 +1004,8 @@ int ssh_request_init(struct ssh_request *rqst, enum ssam_request_flags flags,
> *
> * Initializes the given request transport layer and associated packet
> * transport layer. Transmitter and receiver threads must be started
> - * separately via ssh_rtl_tx_start() and ssh_rtl_rx_start(), after the
> - * request-layer has been initialized and the lower-level serial device layer
> - * has been set up.
> + * separately via ssh_rtl_start(), after the request-layer has been
> + * initialized and the lower-level serial device layer has been set up.
> *
> * Return: Returns zero on success and a nonzero error code on failure.
> */
>
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2021-01-18 18:24 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-01-14 15:08 [PATCH] platform/surface: aggregator: Fix kernel-doc references Maximilian Luz
2021-01-18 18:22 ` Hans de Goede
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).