On Thu, Sep 17, 2020 at 11:56:45PM +0100, Matthew Wilcox (Oracle) wrote: > iomap_set_range_uptodate() is the only caller of > iomap_iop_set_range_uptodate() and it makes future patches easier to > have it inline. > > Signed-off-by: Matthew Wilcox (Oracle) <willy@xxxxxxxxxxxxx> > --- > fs/iomap/buffered-io.c | 24 ++++++++++-------------- > 1 file changed, 10 insertions(+), 14 deletions(-) > > diff --git a/fs/iomap/buffered-io.c b/fs/iomap/buffered-io.c > index 897ab9a26a74..2a6492b3c4db 100644 > --- a/fs/iomap/buffered-io.c > +++ b/fs/iomap/buffered-io.c > @@ -135,8 +135,8 @@ iomap_adjust_read_range(struct inode *inode, struct iomap_page *iop, > *lenp = plen; > } > > -static void > -iomap_iop_set_range_uptodate(struct page *page, unsigned off, unsigned len) > +static > +void iomap_set_range_uptodate(struct page *page, unsigned off, unsigned len) Please don't use such weird formatting for the prototype, the existing one is perfectly fine, and the Linus approved version would be the static and the type on the same line.
