Commit f0a816fb authored by Niklas Schnelle's avatar Niklas Schnelle Committed by Greg Kroah-Hartman
Browse files

/dev/port: don't compile file operations without CONFIG_DEVPORT 



In the future inb() and friends will not be available when compiling
with CONFIG_HAS_IOPORT=n so we must only try to access them here if
CONFIG_DEVPORT is set which depends on HAS_IOPORT.

Co-developed-by: default avatarArnd Bergmann <arnd@kernel.org>
Signed-off-by: default avatarArnd Bergmann <arnd@kernel.org>
Signed-off-by: default avatarNiklas Schnelle <schnelle@linux.ibm.com>
Link: https://lore.kernel.org/r/20240404114917.3627747-2-schnelle@linux.ibm.com


Signed-off-by: default avatarGreg Kroah-Hartman <gregkh@linuxfoundation.org>
parent 1d6b84d2

drivers/char/mem.c

+5 −1
Original line number Diff line number Diff line
@@ -383,6 +383,7 @@ static int mmap_mem(struct file *file, struct vm_area_struct *vma) 
	return 0;

}



#ifdef CONFIG_DEVPORT

static ssize_t read_port(struct file *file, char __user *buf,

			 size_t count, loff_t *ppos)

{
@@ -424,6 +425,7 @@ static ssize_t write_port(struct file *file, const char __user *buf, 
	*ppos = i;

	return tmp-buf;

}

#endif



static ssize_t read_null(struct file *file, char __user *buf,

			 size_t count, loff_t *ppos)
@@ -653,12 +655,14 @@ static const struct file_operations null_fops = { 
	.uring_cmd	= uring_cmd_null,

};



static const struct file_operations __maybe_unused port_fops = {

#ifdef CONFIG_DEVPORT

static const struct file_operations port_fops = {

	.llseek		= memory_lseek,

	.read		= read_port,

	.write		= write_port,

	.open		= open_port,

};

#endif



static const struct file_operations zero_fops = {

	.llseek		= zero_lseek,