All of lore.kernel.org
 help / color / mirror / Atom feed
From: Niklas Schnelle <schnelle@linux.ibm.com>
To: Arnd Bergmann <arnd@kernel.org>,
	Greg Kroah-Hartman <gregkh@linuxfoundation.org>,
	Jiri Slaby <jirislaby@kernel.org>,
	"Maciej W. Rozycki" <macro@orcam.me.uk>
Cc: Arnd Bergmann <arnd@arndb.de>,
	Jeff Johnson <quic_jjohnson@quicinc.com>,
	linux-kernel@vger.kernel.org, linux-serial@vger.kernel.org
Subject: Re: [PATCH] tty: serial: export serial_8250_warn_need_ioport
Date: Tue, 29 Oct 2024 17:14:57 +0100	[thread overview]
Message-ID: <e5e4b697118ac7fdff408665756a0adca36efda1.camel@linux.ibm.com> (raw)
In-Reply-To: <20241029152804.3318094-1-arnd@kernel.org>

On Tue, 2024-10-29 at 15:27 +0000, Arnd Bergmann wrote:
> From: Arnd Bergmann <arnd@arndb.de>
> 
> The newly added function is used from a loadable module, so it has
> to be exported the same way as the other function in this file:
> 
> ERROR: modpost: "serial_8250_warn_need_ioport" [drivers/tty/serial/8250/8250_pci.ko] undefined!
> 
> Fixes: 7c7e6c8924e7 ("tty: serial: handle HAS_IOPORT dependencies")
> Signed-off-by: Arnd Bergmann <arnd@arndb.de>
> ---
> The build regression is currently part of my asm-generic tree, so I've
> applied this fixup on top.
> 
>  drivers/tty/serial/8250/8250_pcilib.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/drivers/tty/serial/8250/8250_pcilib.c b/drivers/tty/serial/8250/8250_pcilib.c
> index ec4d04177802..3bdccf76f71d 100644
> --- a/drivers/tty/serial/8250/8250_pcilib.c
> +++ b/drivers/tty/serial/8250/8250_pcilib.c
> @@ -19,6 +19,7 @@ int serial_8250_warn_need_ioport(struct pci_dev *dev)
>  
>  	return -ENXIO;
>  }
> +EXPORT_SYMBOL_NS_GPL(serial_8250_warn_need_ioport, SERIAL_8250_PCI);
>  
>  int serial8250_pci_setup_port(struct pci_dev *dev, struct uart_8250_port *port,
>  		   u8 bar, unsigned int offset, int regshift)

Looks like I forgot to try the last version plus the temporarily
removed !S390 with allmodconfig instead of allyesconfig. 

Thanks for the fix!

Reviewed-by: Niklas Schnelle <schnelle@linux.ibm.com>

      reply	other threads:[~2024-10-29 16:15 UTC|newest]

Thread overview: 2+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-10-29 15:27 [PATCH] tty: serial: export serial_8250_warn_need_ioport Arnd Bergmann
2024-10-29 16:14 ` Niklas Schnelle [this message]

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=e5e4b697118ac7fdff408665756a0adca36efda1.camel@linux.ibm.com \
    --to=schnelle@linux.ibm.com \
    --cc=arnd@arndb.de \
    --cc=arnd@kernel.org \
    --cc=gregkh@linuxfoundation.org \
    --cc=jirislaby@kernel.org \
    --cc=linux-kernel@vger.kernel.org \
    --cc=linux-serial@vger.kernel.org \
    --cc=macro@orcam.me.uk \
    --cc=quic_jjohnson@quicinc.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.