diff mbox series

mmc: core: Remove struct mmc_context_info

Message ID 232106a8a6a374dee25feea9b94498361568c10b.1724246389.git.christophe.jaillet@wanadoo.fr (mailing list archive)
State New
Headers show
Series mmc: core: Remove struct mmc_context_info | expand

Commit Message

Christophe JAILLET Aug. 21, 2024, 1:24 p.m. UTC
The 'mmc_context_info' structure is unused.

It has been introduced in:

  - commit 2220eedfd7ae ("mmc: fix async request mechanism for sequential
    read scenarios")

in 2013-02 and its usages have been removed in:

  - commit 126b62700386 ("mmc: core: Remove code no longer needed after the
    switch to blk-mq")
  - commit 0fbfd1251830 ("mmc: block: Remove code no longer needed after
    the switch to blk-mq")

in 2017-12.

Now remove this unused structure.

Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
---
Compile tested in drivers/mmc/ only
---
 include/linux/mmc/host.h | 14 --------------
 1 file changed, 14 deletions(-)

Comments

Vladimir Zapolskiy Aug. 21, 2024, 2:05 p.m. UTC | #1
On 8/21/24 16:24, Christophe JAILLET wrote:
> The 'mmc_context_info' structure is unused.
> 
> It has been introduced in:
> 
>    - commit 2220eedfd7ae ("mmc: fix async request mechanism for sequential
>      read scenarios")
> 
> in 2013-02 and its usages have been removed in:
> 
>    - commit 126b62700386 ("mmc: core: Remove code no longer needed after the
>      switch to blk-mq")
>    - commit 0fbfd1251830 ("mmc: block: Remove code no longer needed after
>      the switch to blk-mq")
> 
> in 2017-12.
> 
> Now remove this unused structure.
> 
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>
> ---
> Compile tested in drivers/mmc/ only
> ---
>   include/linux/mmc/host.h | 14 --------------
>   1 file changed, 14 deletions(-)
> 
> diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
> index 49470188fca7..545bddfd7e1f 100644
> --- a/include/linux/mmc/host.h
> +++ b/include/linux/mmc/host.h
> @@ -292,20 +292,6 @@ struct mmc_slot {
>   	void *handler_priv;
>   };
>   
> -/**
> - * mmc_context_info - synchronization details for mmc context
> - * @is_done_rcv		wake up reason was done request
> - * @is_new_req		wake up reason was new request
> - * @is_waiting_last_req	mmc context waiting for single running request
> - * @wait		wait queue
> - */
> -struct mmc_context_info {
> -	bool			is_done_rcv;
> -	bool			is_new_req;
> -	bool			is_waiting_last_req;
> -	wait_queue_head_t	wait;
> -};
> -
>   struct regulator;
>   struct mmc_pwrseq;
>   

Reviewed-by: Vladimir Zapolskiy <vz@mleia.com>

--
Best wishes,
Vladimir
Ulf Hansson Aug. 26, 2024, 11:38 a.m. UTC | #2
On Wed, 21 Aug 2024 at 15:24, Christophe JAILLET
<christophe.jaillet@wanadoo.fr> wrote:
>
> The 'mmc_context_info' structure is unused.
>
> It has been introduced in:
>
>   - commit 2220eedfd7ae ("mmc: fix async request mechanism for sequential
>     read scenarios")
>
> in 2013-02 and its usages have been removed in:
>
>   - commit 126b62700386 ("mmc: core: Remove code no longer needed after the
>     switch to blk-mq")
>   - commit 0fbfd1251830 ("mmc: block: Remove code no longer needed after
>     the switch to blk-mq")
>
> in 2017-12.
>
> Now remove this unused structure.
>
> Signed-off-by: Christophe JAILLET <christophe.jaillet@wanadoo.fr>

Applied for next, thanks!

Kind regards
Uffe


> ---
> Compile tested in drivers/mmc/ only
> ---
>  include/linux/mmc/host.h | 14 --------------
>  1 file changed, 14 deletions(-)
>
> diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
> index 49470188fca7..545bddfd7e1f 100644
> --- a/include/linux/mmc/host.h
> +++ b/include/linux/mmc/host.h
> @@ -292,20 +292,6 @@ struct mmc_slot {
>         void *handler_priv;
>  };
>
> -/**
> - * mmc_context_info - synchronization details for mmc context
> - * @is_done_rcv                wake up reason was done request
> - * @is_new_req         wake up reason was new request
> - * @is_waiting_last_req        mmc context waiting for single running request
> - * @wait               wait queue
> - */
> -struct mmc_context_info {
> -       bool                    is_done_rcv;
> -       bool                    is_new_req;
> -       bool                    is_waiting_last_req;
> -       wait_queue_head_t       wait;
> -};
> -
>  struct regulator;
>  struct mmc_pwrseq;
>
> --
> 2.46.0
>
diff mbox series

Patch

diff --git a/include/linux/mmc/host.h b/include/linux/mmc/host.h
index 49470188fca7..545bddfd7e1f 100644
--- a/include/linux/mmc/host.h
+++ b/include/linux/mmc/host.h
@@ -292,20 +292,6 @@  struct mmc_slot {
 	void *handler_priv;
 };
 
-/**
- * mmc_context_info - synchronization details for mmc context
- * @is_done_rcv		wake up reason was done request
- * @is_new_req		wake up reason was new request
- * @is_waiting_last_req	mmc context waiting for single running request
- * @wait		wait queue
- */
-struct mmc_context_info {
-	bool			is_done_rcv;
-	bool			is_new_req;
-	bool			is_waiting_last_req;
-	wait_queue_head_t	wait;
-};
-
 struct regulator;
 struct mmc_pwrseq;