Received-SPF: pass (google.com: best guess record for domain of linux-ext4-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Wed, 12 Feb 2020 09:02:36 -0800
From:   "Darrick J. Wong" <darrick.wong@oracle.com>
To:     Eric Whitney <enwlinux@gmail.com>
Cc:     linux-ext4@vger.kernel.org, tytso@mit.edu
Subject: Re: [PATCH] ext4:  delete declaration for ext4_split_extent()
Message-ID: <20200212170236.GA6863@magnolia>
References: <20200212162141.22381-1-enwlinux@gmail.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20200212162141.22381-1-enwlinux@gmail.com>
User-Agent: Mutt/1.9.4 (2018-02-28)
Sender: linux-ext4-owner@vger.kernel.org
Precedence: bulk

On Wed, Feb 12, 2020 at 11:21:41AM -0500, Eric Whitney wrote:
> There are no forward references for ext4_split_extent() in extents.c,
> so delete its unnecessary declaration.
> 
> Signed-off-by: Eric Whitney <enwlinux@gmail.com>

Looks good to me,
Reviewed-by: Darrick J. Wong <darrick.wong@oracle.com>

--D

> ---
>  fs/ext4/extents.c | 7 -------
>  1 file changed, 7 deletions(-)
> 
> diff --git a/fs/ext4/extents.c b/fs/ext4/extents.c
> index 89aa9c7ae293..a5338a8da2ab 100644
> --- a/fs/ext4/extents.c
> +++ b/fs/ext4/extents.c
> @@ -83,13 +83,6 @@ static void ext4_extent_block_csum_set(struct inode *inode,
>  	et->et_checksum = ext4_extent_block_csum(inode, eh);
>  }
>  
> -static int ext4_split_extent(handle_t *handle,
> -				struct inode *inode,
> -				struct ext4_ext_path **ppath,
> -				struct ext4_map_blocks *map,
> -				int split_flag,
> -				int flags);
> -
>  static int ext4_split_extent_at(handle_t *handle,
>  			     struct inode *inode,
>  			     struct ext4_ext_path **ppath,
> -- 
> 2.11.0
>