> > > Hi, > > > > On 02.09.19 23:16, Andy Shevchenko wrote: > > > On Mon, Sep 2, 2019 at 11:58 PM Rafael J. Wysocki > > > <rafael@xxxxxxxxxx> > > wrote: > > >> > > >> On Thu, Jul 11, 2019 at 12:35 PM Chuanhua Han > > >> <chuanhua.han@xxxxxxx> > > wrote: > > >>> > > >>> Enable NXP i2c controller to boot with ACPI > > >>> > > >>> Signed-off-by: Meenakshi Aggarwal <meenakshi.aggarwal@xxxxxxx> > > >>> Signed-off-by: Udit Kumar <udit.kumar@xxxxxxx> > > >>> Signed-off-by: Chuanhua Han <chuanhua.han@xxxxxxx> > > >> > > >> Wolfram, any objections to this from the i2c side? > > > > > > May I propose amendment(s)? > > > > > >>> @@ -44,6 +44,7 @@ > > >>> #include <linux/pm_runtime.h> > > >>> #include <linux/sched.h> > > >>> #include <linux/slab.h> > > > > > >>> +#include <linux/acpi.h> > > > > > > If it's kept in order, better to go with it. (Yes, it is as I have > > > checked) However, property.h should be included instead, see below. Ok, got it. I will fix it in v2. > > > > > >>> const struct of_device_id *of_id = > > of_match_device(i2c_imx_dt_ids, > > >>> > > >>> &pdev->dev); > > >>> + const struct acpi_device_id *acpi_id = > > >>> + acpi_match_device(i2c_imx_acpi_ids, > > >>> + &pdev->dev); > > > > > > > > >>> if (of_id) > > >>> i2c_imx->hwdata = of_id->data; > > >>> + else if (acpi_id) > > >>> + i2c_imx->hwdata = (struct imx_i2c_hwdata *) > > >>> + acpi_id->driver_data; > > > > > > > > > The above altogher may be replaced with > > > > > > const struct imx_i2c_hwdata *match; > > > ... > > > match = device_get_match_data(&pdev->dev); > > > if (match) > > > i2c_imx->hwdata = match; > > > else > > > ... > > Ok, I will correct it in v2. > > Instead of "may be replaced", I would say: it should be replaced :) > > > > >>> + .acpi_match_table = ACPI_PTR(i2c_imx_acpi_ids), > > > > > > Since there is no #ifdef guard no need to use ACPI_PTR(). > > > > > > > What iMX/(other NXP?) SoCs are with ACPI support? Where I can get > > one? I would like to know more about it. > - Nxp has variety Socs, include i.MX, Layerscape, etc. > - You can get one from here > https://www.nxp.com/design/qoriq-developer-resources/qoriq-lx2160a-develo > pment-board:LX2160A-RDB > > > > > Kind regards, > > Oleksij Rempel > > > > -- > > Pengutronix e.K. | > > | > > Industrial Linux Solutions | > > https://eur01.safelinks.protection.outlook.com/?url=http%3A%2F%2Fwww.p > > e > ngutronix.de%2F&data=02%7C01%7Cmeenakshi.aggarwal%40nxp.com% > > > 7C640eb015a91f4959d3b508d7303168fb%7C686ea1d3bc2b4c6fa92cd99c5c > > > 301635%7C0%7C0%7C637030861076879938&sdata=sPWtkVtHHDvoRR > > ZmWJuipCO%2BEwG%2BcupgZvcIV1%2BrlEY%3D&reserved=0 | > Peiner Str. > > 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0 > > | > > Amtsgericht Hildesheim, HRA 2686 | Fax: > > +49-5121-206917-5555 |
