lkml.org 
[lkml]   [2021]   [Jul]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH] Documentation: gpio: driver.rst: Remove gpiochip_irqchip_add mention
On Mon, Jul 5, 2021 at 2:44 AM Vincent Pelletier <plr.vincent@gmail.com> wrote:
>
> This function was removed in commit f1f37abbe6fc ("gpio: Retire the
> explicit gpio irqchip code") but this mention was left behind.
> Also, mention that .set_type() only has to set a line handler if the chip
> is cascaded, as opposed to hierarchical.
>
> Signed-off-by: Vincent Pelletier <plr.vincent@gmail.com>
> ---
> Documentation/driver-api/gpio/driver.rst | 11 ++++-------
> 1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/Documentation/driver-api/gpio/driver.rst b/Documentation/driver-api/gpio/driver.rst
> index d6b0d779859b..bbc53920d4dd 100644
> --- a/Documentation/driver-api/gpio/driver.rst
> +++ b/Documentation/driver-api/gpio/driver.rst
> @@ -547,13 +547,10 @@ To use the helpers please keep the following in mind:
> the irqchip can initialize. E.g. .dev and .can_sleep shall be set up
> properly.
>
> -- Nominally set all handlers to handle_bad_irq() in the setup call and pass
> - handle_bad_irq() as flow handler parameter in gpiochip_irqchip_add() if it is
> - expected for GPIO driver that irqchip .set_type() callback will be called
> - before using/enabling each GPIO IRQ. Then set the handler to
> - handle_level_irq() and/or handle_edge_irq() in the irqchip .set_type()
> - callback depending on what your controller supports and what is requested
> - by the consumer.
> +- Nominally set gpio_irq_chip.handler to handle_bad_irq. Then, if your irqchip
> + is cascaded, set the handler to handle_level_irq() and/or handle_edge_irq()
> + in the irqchip .set_type() callback depending on what your controller
> + supports and what is requested by the consumer.
>
>
> Locking IRQ usage
> --
> 2.32.0
>

Patch applied, thanks!

Bartosz

\
 
 \ /
  Last update: 2021-07-21 15:20    [W:0.043 / U:0.248 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site