On Fri, Feb 28, 2025 at 04:11:15PM +0100, 'Jonathan Neuschäfer via B4 Relay' via USB Mass Storage on Linux wrote: > While reading code, I noticed that some arrays in USB mass storage > drivers are declared static but not const, even though they are not > modified. This patchset marks them const. > > All patches were compile-tested. > > Signed-off-by: Jonathan Neuschäfer <j.ne@xxxxxxxxxx> > --- > Changes in v3: > - Elaborate *why* const is a good idea > - Link to v2: https://lore.kernel.org/r/20250226-misc-const-v2-0-ab655a4a29cc@xxxxxxxxxx > > Changes in v2: > - Add new patches 2-9 > - Use consistent authorship information > - Link to v1: https://lore.kernel.org/r/20250225-misc-const-v1-1-121ff3b86437@xxxxxxxxxx > > --- > Jonathan Neuschäfer (9): > usb: storage: jumpshot: Use const for constant arrays > usb: storage: transport: Use const for constant array > usb: storage: alauda: Use const for card ID array > usb: storage: datafab: Use const for constant arrays > usb: storage: initializers: Use const for constant array > usb: storage: realtek_cr: Use const for constant arrays > usb: storage: sddr09: Use const for constant arrays > usb: storage: sddr55: Use const for constant arrays > usb: storage: shuttle_usbat: Use const for constant array > > drivers/usb/storage/alauda.c | 8 ++++---- > drivers/usb/storage/datafab.c | 14 +++++++------- > drivers/usb/storage/initializers.c | 2 +- > drivers/usb/storage/jumpshot.c | 10 +++++----- > drivers/usb/storage/realtek_cr.c | 6 +++--- > drivers/usb/storage/sddr09.c | 14 +++++++------- > drivers/usb/storage/sddr55.c | 4 ++-- > drivers/usb/storage/shuttle_usbat.c | 2 +- > drivers/usb/storage/transport.c | 2 +- > 9 files changed, 31 insertions(+), 31 deletions(-) > --- > base-commit: 2014c95afecee3e76ca4a56956a936e23283f05b > change-id: 20240401-misc-const-e7b4cf20d5f9 > > Best regards, > -- > Jonathan Neuschäfer <j.ne@xxxxxxxxxx> For all 9 patches: Reviewed-by: Alan Stern <stern@xxxxxxxxxxxxxxxxxxx>
