kernel/linux.git/drivers/char/ipmi/ipmi_devintf.c, branch v6.12.80

ipmi: make ipmi_class a static const structure

2023-06-20T14:49:08+00:00

Now that the driver core allows for struct class to be in read-only memory, move the ipmi_class structure to be declared at build time placing it into read-only memory, instead of having to be dynamically allocated at boot time. Cc: Corey Minyard Cc: openipmi-developer@lists.sourceforge.net Suggested-by: Greg Kroah-Hartman Signed-off-by: Ivan Orlov Signed-off-by: Greg Kroah-Hartman Message-Id: <20230620143701.577657-2-gregkh@linuxfoundation.org> Signed-off-by: Corey Minyard

driver core: class: remove module * from class_create()

2023-03-17T14:16:33+00:00

The module pointer in class_create() never actually did anything, and it shouldn't have been requred to be set as a parameter even if it did something. So just remove it and fix up all callers of the function in the kernel tree at the same time. Cc: "Rafael J. Wysocki" Acked-by: Benjamin Tissoires Link: https://lore.kernel.org/r/20230313181843.1207845-4-gregkh@linuxfoundation.org Signed-off-by: Greg Kroah-Hartman

ipmi:devintf: Return a proper error when recv buffer too small

2021-10-05T11:54:16+00:00

The right error message wasn't being set in one location, and it would return success on a failure. Signed-off-by: Corey Minyard

char: ipmi: remove unneeded break

2020-10-27T21:08:22+00:00

A break is not needed if it is preceded by a return Signed-off-by: Tom Rix Message-Id: <20201019194805.14996-1-trix@redhat.com> Signed-off-by: Corey Minyard

ipmi: Remove file from ipmi_file_private

2019-04-17T18:14:25+00:00

It's not used by anything, so just get rid of it. Reported-by: Al Viro Signed-off-by: Corey Minyard

ipmi: Fix return value when a message is truncated

2019-02-10T01:48:42+00:00

The user can ask the message to be returned even if it didn't supply enough memory for the data, and it will return an error but still fills in as much data as possible. However, the return value wasn't being set to an error, it was being overwritten. Create a second return value for that case. Signed-off-by: Corey Minyard

ipmi: Convert printk(KERN_ to pr_(

2018-09-18T21:15:33+00:00

Use the more common logging style. Miscellanea: o Convert old style continuation printks without KERN_CONT to pr_cont o Coalesce formats o Realign arguments o Remove unnecessary casts Signed-off-by: Joe Perches Signed-off-by: Corey Minyard

ipmi_devintf: Add an error return on invalid ioctls

2018-04-18T15:23:04+00:00

Signed-off-by: Corey Minyard

ipmi: Rename ipmi_user_t to struct ipmi_user *

2018-04-18T15:22:55+00:00

Get rid of that non-compliance in the user files. Include files will come later. Signed-off-by: Corey Minyard

ipmi_devintf: Small lock rework

2018-04-18T15:22:53+00:00

The mutex didn't really serve any useful purpose, from what I can tell, and it would just get in the way. So remove it. Removing that required a mutex around the default value setting and getting, so just use the receive mutex for that. Also pull the fasync stuff outside of the lock for adding the data to the queue, since it didn't need to be there. Signed-off-by: Corey Minyard