On Wed, Aug 09, 2017 at 05:18:00PM +0800, Chunfeng Yun wrote: > reduce the boilerplate code to get the specific data > > Signed-off-by: Chunfeng Yun <chunfeng....@mediatek.com> > --- > drivers/phy/tegra/xusb.c | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-)
You're not reducing by much here. =) > diff --git a/drivers/phy/tegra/xusb.c b/drivers/phy/tegra/xusb.c > index 3cbcb25..25cdfb1 100644 > --- a/drivers/phy/tegra/xusb.c > +++ b/drivers/phy/tegra/xusb.c > @@ -848,7 +848,6 @@ static int tegra_xusb_padctl_probe(struct platform_device > *pdev) > struct device_node *np = of_node_get(pdev->dev.of_node); > const struct tegra_xusb_padctl_soc *soc; > struct tegra_xusb_padctl *padctl; > - const struct of_device_id *match; > struct resource *res; > int err; > > @@ -861,8 +860,9 @@ static int tegra_xusb_padctl_probe(struct platform_device > *pdev) > > of_node_put(np); > > - match = of_match_node(tegra_xusb_padctl_of_match, pdev->dev.of_node); > - soc = match->data; > + soc = of_device_get_match_data(&pdev->dev); > + if (!soc) > + return -EINVAL; This is never going to happen. There will always be a match, otherwise this function would never be called. Also, if you omit these last two lines, then you actually do reduce the boilerplate. Thierry
signature.asc
Description: PGP signature