Re: [PATCH v1 1/2] mmc: sdhci-omap: Use of_device_get_match_data() helper

From: Ulf Hansson
Date: Tue Apr 26 2022 - 09:56:09 EST


On Sun, 24 Apr 2022 at 00:16, Bean Huo <huobean@xxxxxxxxx> wrote:
>
> From: Bean Huo <beanhuo@xxxxxxxxxx>
>
> Only the device data is needed, not the entire struct of_device_id.
> Use of_device_get_match_data() instead of open coding of_match_device().
>
> Signed-off-by: Bean Huo <beanhuo@xxxxxxxxxx>

Applied for next, thanks!

Kind regards
Uffe


> ---
> drivers/mmc/host/sdhci-omap.c | 9 ++-------
> 1 file changed, 2 insertions(+), 7 deletions(-)
>
> diff --git a/drivers/mmc/host/sdhci-omap.c b/drivers/mmc/host/sdhci-omap.c
> index 64e27c2821f9..86e867ffbb10 100644
> --- a/drivers/mmc/host/sdhci-omap.c
> +++ b/drivers/mmc/host/sdhci-omap.c
> @@ -1219,16 +1219,11 @@ static int sdhci_omap_probe(struct platform_device *pdev)
> struct sdhci_pltfm_host *pltfm_host;
> struct sdhci_omap_host *omap_host;
> struct mmc_host *mmc;
> - const struct of_device_id *match;
> - struct sdhci_omap_data *data;
> + const struct sdhci_omap_data *data;
> const struct soc_device_attribute *soc;
> struct resource *regs;
>
> - match = of_match_device(omap_sdhci_match, dev);
> - if (!match)
> - return -EINVAL;
> -
> - data = (struct sdhci_omap_data *)match->data;
> + data = of_device_get_match_data(&pdev->dev);
> if (!data) {
> dev_err(dev, "no sdhci omap data\n");
> return -EINVAL;
> --
> 2.34.1
>