On Wed, Feb 14, 2018 at 08:11:17PM +0200, Andy Shevchenko wrote: > There is no need to repeat the semantics of writel() and similar. > Moreover sparse complains about this: > > drivers/infiniband/hw/hns/hns_roce_hw_v1.c:1690:22: expected unsigned long long val > drivers/infiniband/hw/hns/hns_roce_hw_v1.c:1690:22: got restricted __le64 <noident> > > Fixing this by replacing __raw_write*(cpu_to_le*()) calls by plain > write*() ones. > > Note, write*() accessors are little endian by definition. > > Reported-by: kbuild test robot <fengguang.wu@xxxxxxxxx> > Signed-off-by: Andy Shevchenko <andriy.shevchenko@xxxxxxxxxxxxxxx> > drivers/infiniband/hw/hns/hns_roce_hw_v1.c | 8 ++++---- > drivers/infiniband/hw/hns/hns_roce_hw_v2.c | 8 ++++---- > 2 files changed, 8 insertions(+), 8 deletions(-) Yes, thanks, applied to -next Jason -- To unsubscribe from this list: send the line "unsubscribe linux-rdma" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
