| Message ID | 20241120-topic-ufs-cleanup-v1-0-a5cef54b1cdc@linaro.org |
|---|---|
| Headers | show |
| Series | ufs: core: initial cleanup and defines sync with Linux v6.12 | expand |
Hi, On 20/11/24 2:52 pm, Neil Armstrong wrote: > This serie fixes the build and checkpatch warnings, and > does a split of the ufs.h into a separate ufshci.h which > helps syndhronizing the defines from Linux v6.12. > > No functional changes intended, tests on hardware > are welcome. > > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> > --- > Neil Armstrong (6): > ufs: core: include missing include/ufs.h > ufs: core: mark unexported functions as static > ufs: core: cosmetic fixups > ufs: core: move ufshci defines in a separate header > ufs: core: sync unipro.h with Linux v6.12 > ufs: core: sync ufshci.h with Linux v6.12 > > drivers/ufs/ufs.c | 43 +++-- > drivers/ufs/ufs.h | 335 +----------------------------------- > drivers/ufs/ufshci.h | 469 +++++++++++++++++++++++++++++++++++++++++++++++++++ > drivers/ufs/unipro.h | 128 ++++++++------ > 4 files changed, 571 insertions(+), 404 deletions(-) > --- > base-commit: dc1859f8d2ac3faaa5e2e1d465ec4bd8980520a5 > change-id: 20241120-topic-ufs-cleanup-23671e3fc9e9 > > Best regards, Verified these changes on AMD platform (amd_versal2_virt_defconfig) and didn't observe any issue with this series. Tested-by: Love Kumar <love.kumar@amd.com> Regards, Love Kumar
On 26/11/2024 07:43, Love Kumar wrote: > Hi, > > On 20/11/24 2:52 pm, Neil Armstrong wrote: >> This serie fixes the build and checkpatch warnings, and >> does a split of the ufs.h into a separate ufshci.h which >> helps syndhronizing the defines from Linux v6.12. >> >> No functional changes intended, tests on hardware >> are welcome. >> >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> --- >> Neil Armstrong (6): >> ufs: core: include missing include/ufs.h >> ufs: core: mark unexported functions as static >> ufs: core: cosmetic fixups >> ufs: core: move ufshci defines in a separate header >> ufs: core: sync unipro.h with Linux v6.12 >> ufs: core: sync ufshci.h with Linux v6.12 >> >> drivers/ufs/ufs.c | 43 +++-- >> drivers/ufs/ufs.h | 335 +----------------------------------- >> drivers/ufs/ufshci.h | 469 +++++++++++++++++++++++++++++++++++++++++++++++++++ >> drivers/ufs/unipro.h | 128 ++++++++------ >> 4 files changed, 571 insertions(+), 404 deletions(-) >> --- >> base-commit: dc1859f8d2ac3faaa5e2e1d465ec4bd8980520a5 >> change-id: 20241120-topic-ufs-cleanup-23671e3fc9e9 >> >> Best regards, > Verified these changes on AMD platform (amd_versal2_virt_defconfig) and didn't observe any issue with this series. > > Tested-by: Love Kumar <love.kumar@amd.com> Thanks for testing! Neil > > Regards, > Love Kumar
Hi, On 20/11/2024 10:22, Neil Armstrong wrote: > This serie fixes the build and checkpatch warnings, and > does a split of the ufs.h into a separate ufshci.h which > helps syndhronizing the defines from Linux v6.12. > > No functional changes intended, tests on hardware > are welcome. > > Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> > --- > Neil Armstrong (6): > ufs: core: include missing include/ufs.h > ufs: core: mark unexported functions as static > ufs: core: cosmetic fixups > ufs: core: move ufshci defines in a separate header > ufs: core: sync unipro.h with Linux v6.12 > ufs: core: sync ufshci.h with Linux v6.12 Gentle ping, could you have a quick look ? Thanks, Neil > > drivers/ufs/ufs.c | 43 +++-- > drivers/ufs/ufs.h | 335 +----------------------------------- > drivers/ufs/ufshci.h | 469 +++++++++++++++++++++++++++++++++++++++++++++++++++ > drivers/ufs/unipro.h | 128 ++++++++------ > 4 files changed, 571 insertions(+), 404 deletions(-) > --- > base-commit: dc1859f8d2ac3faaa5e2e1d465ec4bd8980520a5 > change-id: 20241120-topic-ufs-cleanup-23671e3fc9e9 > > Best regards,
Hi Neil On 04/12/24 17:59, Neil Armstrong wrote: > Hi, > > On 20/11/2024 10:22, Neil Armstrong wrote: >> This serie fixes the build and checkpatch warnings, and >> does a split of the ufs.h into a separate ufshci.h which >> helps syndhronizing the defines from Linux v6.12. >> >> No functional changes intended, tests on hardware >> are welcome. >> >> Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> >> --- >> Neil Armstrong (6): >> ufs: core: include missing include/ufs.h >> ufs: core: mark unexported functions as static >> ufs: core: cosmetic fixups >> ufs: core: move ufshci defines in a separate header >> ufs: core: sync unipro.h with Linux v6.12 >> ufs: core: sync ufshci.h with Linux v6.12 > > Gentle ping, could you have a quick look ? > Thanks for the ping, will take a look at it today. > Thanks, > Neil > >> >> drivers/ufs/ufs.c | 43 +++-- >> drivers/ufs/ufs.h | 335 +----------------------------------- >> drivers/ufs/ufshci.h | 469 +++++++++++++++++++++++++++++++++++++++++++++++++++ >> drivers/ufs/unipro.h | 128 ++++++++------ >> 4 files changed, 571 insertions(+), 404 deletions(-) >> --- >> base-commit: dc1859f8d2ac3faaa5e2e1d465ec4bd8980520a5 >> change-id: 20241120-topic-ufs-cleanup-23671e3fc9e9 >> >> Best regards, >
This serie fixes the build and checkpatch warnings, and does a split of the ufs.h into a separate ufshci.h which helps syndhronizing the defines from Linux v6.12. No functional changes intended, tests on hardware are welcome. Signed-off-by: Neil Armstrong <neil.armstrong@linaro.org> --- Neil Armstrong (6): ufs: core: include missing include/ufs.h ufs: core: mark unexported functions as static ufs: core: cosmetic fixups ufs: core: move ufshci defines in a separate header ufs: core: sync unipro.h with Linux v6.12 ufs: core: sync ufshci.h with Linux v6.12 drivers/ufs/ufs.c | 43 +++-- drivers/ufs/ufs.h | 335 +----------------------------------- drivers/ufs/ufshci.h | 469 +++++++++++++++++++++++++++++++++++++++++++++++++++ drivers/ufs/unipro.h | 128 ++++++++------ 4 files changed, 571 insertions(+), 404 deletions(-) --- base-commit: dc1859f8d2ac3faaa5e2e1d465ec4bd8980520a5 change-id: 20241120-topic-ufs-cleanup-23671e3fc9e9 Best regards,