lkml.org 
[lkml]   [2022]   [Jul]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
SubjectRe: [PATCH v1 1/2] gpio: realtek-otto: amend ctrl struct docs
From
Date
Hi,

On Fri, 2022-07-22 at 10:19 +0200, Sander Vanheule wrote:
> Commit 512c5be35223 ("gpio: realtek-otto: Support reversed port
> layouts") and commit 95fa6dbe58f2 ("gpio: realtek-otto: Support per-cpu
> interrupts") updated the realtek_gpio_ctrl struct with new fields, but
> the associated kernel-doc comment was not updated accordingly.
>
> Signed-off-by: Sander Vanheule <sander@svanheule.net>
> ---

I've just posted another patch to fix a driver issue, which conflicts with this
one. The fix should get merged first, we can come back to these patches later.

Best,
Sander

>  drivers/gpio/gpio-realtek-otto.c | 10 ++++++++++
>  1 file changed, 10 insertions(+)
>
> diff --git a/drivers/gpio/gpio-realtek-otto.c b/drivers/gpio/gpio-realtek-
> otto.c
> index 63dcf42f7c20..a352fbfc3c28 100644
> --- a/drivers/gpio/gpio-realtek-otto.c
> +++ b/drivers/gpio/gpio-realtek-otto.c
> @@ -43,9 +43,19 @@
>   *
>   * @gc: Associated gpio_chip instance
>   * @base: Base address of the register block for a GPIO bank
> + * @cpumask_base: Base address of the interrupt routing registers
> + * @cpu_irq_maskable: Mask of CPUs that can be individually masked for IRQs
>   * @lock: Lock for accessing the IRQ registers and values
>   * @intr_mask: Mask for interrupts lines
>   * @intr_type: Interrupt type selection
> + * @port_offset_u8: Get offset of an 8b port value
> + * @port_offset_u16: Get offset of a 16b port value
> + *
> + * The DIR, DATA, and ISR registers consist of four u8 port values, packed
> into
> + * a single 32b register. Use @port_offset_u8 to get the correct offset
> inside
> + * that register. The IMR register consists of four u16 port values, packed
> + * into two 32b registers. Use @port_offset_u16 to get the correct offset for
> + * the u16 value, starting from the first register.
>   *
>   * Because the interrupt mask register (IMR) combines the function of IRQ
> type
>   * selection and masking, two extra values are stored. @intr_mask is used to

\
 
 \ /
  Last update: 2022-07-23 11:53    [W:0.063 / U:1.820 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site