linux.git/drivers/staging/pi433, branch v6.9

staging: pi433: Move FIFO_THRESHOLD define to source file

2024-03-05T14:11:26+00:00

FIFO_THRESHOLD is only being used in pi433_if.c source files.

Signed-off-by: Shahar Avidar 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240227221204.534131-6-ikobh7@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Remove the unused FREQUENCY define

2024-03-05T14:11:26+00:00

FREQUENCY is not being used, delete its comment.

Signed-off-by: Shahar Avidar 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240227221204.534131-5-ikobh7@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Redefine F_OSC using units.h macro

2024-03-05T14:11:25+00:00

Use HZ_PER_MHZ for a more robust definition.

Suggested-by: Andy Shevchenko 
Signed-off-by: Shahar Avidar 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240227221204.534131-4-ikobh7@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Remove a duplicated F_OSC define

2024-03-05T14:11:25+00:00

F_OSC is already defined & only used by rf69.c source file.

Signed-off-by: Shahar Avidar 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240227221204.534131-3-ikobh7@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Remove a duplicated FIFO_SIZE define

2024-03-05T14:11:25+00:00

FIFO_SIZE is being used in both rf69.c & pi433_if.c source files.
It is also already defined in rf69.h header file.

Signed-off-by: Shahar Avidar 
Reviewed-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20240227221204.534131-2-ikobh7@gmail.com
Signed-off-by: Greg Kroah-Hartman

staging: pi433: make pi433_class constant

2023-10-06T13:38:34+00:00

Now that the driver core allows for struct class to be in read-only
memory, making all 'class' structures to be declared at build time
placing them into read-only memory, instead of having to be dynamically
allocated at load time.

Link: https://lore.kernel.org/r/2023100512-sweat-abruptly-2445@gregkh
Signed-off-by: Greg Kroah-Hartman

staging: Explicitly include correct DT includes

2023-07-27T08:01:07+00:00

The DT of_device.h and of_platform.h date back to the separate
of_platform_bus_type before it as merged into the regular platform bus.
As part of that merge prepping Arm DT support 13 years ago, they
"temporarily" include each other. They also include platform_device.h
and of.h. As a result, there's a pretty much random mix of those include
files used throughout the tree. In order to detangle these headers and
replace the implicit includes with struct declarations, users need to
explicitly include the correct includes.

Signed-off-by: Rob Herring 
Reviewed-by: Hans Verkuil 
Reviewed-by: Luca Ceresoli  # tegra-video
Acked-by: Parthiban Veerasooran 
Acked-by: Alex Elder 
Link: https://lore.kernel.org/r/20230714175002.4064428-1-robh@kernel.org
Signed-off-by: Greg Kroah-Hartman

staging: pi433: Remove stray gpiod_unexport() call

2023-05-28T14:07:51+00:00

There is no gpiod_export() and gpiod_unexport() looks pretty much stray.
If user space tools somehow belong to that, they should be fixed, the
gpiod_export() and gpiod_unexport() shouldn't be used in a new code,
GPIO sysfs is deprecated. That said, and taking into account staging
state of the driver, simply drop the stray call.

Signed-off-by: Andy Shevchenko 
Link: https://lore.kernel.org/r/20230528140147.32427-1-andriy.shevchenko@linux.intel.com
Signed-off-by: Greg Kroah-Hartman

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

staging: pi433: fix memory leak with using debugfs_lookup()

2023-02-06T09:45:34+00:00

When calling debugfs_lookup() the result must have dput() called on it,
otherwise the memory will leak over time.  To make things simpler, just
call debugfs_lookup_and_remove() instead which handles all of the logic
at once.  This requires saving off the root directory dentry to make
creation of individual device subdirectories easier.

Cc: Paulo Miguel Almeida 
Cc: Dan Carpenter 
Cc: Sidong Yang 
Cc: Liu Shixin 
Cc: "Uwe Kleine-König" 
Link: https://lore.kernel.org/r/20230202141138.2291946-1-gregkh@linuxfoundation.org
Signed-off-by: Greg Kroah-Hartman