Hi Simon,
On Mon, Nov 25, 2019 at 12:12 PM Simon Glass <s...@chromium.org> wrote:
>
> For Apollo Lake we need to take the I2C bus controller out of reset before
> using this. Add this functionality to the driver.
>
> Signed-off-by: Simon Glass <s...@chromium.org>
> Reviewed-by: Heiko Schocher <h...@denx.de>
> ---
>
> Changes in v5: None
> Changes in v4:
> - Drop unrelated change metioned by Heiko
> - apollolake -> Apollo Lake
>
> Changes in v3:
> - Add a weak function to avoid errors on other platforms
>
> Changes in v2: None
>
> drivers/i2c/designware_i2c_pci.c | 22 ++++++++++++++++++++++
> 1 file changed, 22 insertions(+)
>
> diff --git a/drivers/i2c/designware_i2c_pci.c
> b/drivers/i2c/designware_i2c_pci.c
> index 2bf90eaa4b..5114883923 100644
> --- a/drivers/i2c/designware_i2c_pci.c
> +++ b/drivers/i2c/designware_i2c_pci.c
> @@ -8,8 +8,14 @@
> #include <common.h>
> #include <dm.h>
> #include <spl.h>
> +#include <asm/lpss.h>
> #include "designware_i2c.h"
>
> +enum {
> + VANILLA = 0,
What's this?
> + INTEL_APL,
> +};
> +
> /* BayTrail HCNT/LCNT/SDA hold time */
> static struct dw_scl_sda_cfg byt_config = {
> .ss_hcnt = 0x200,
> @@ -62,6 +68,15 @@ static int designware_i2c_pci_ofdata_to_platdata(struct
> udevice *dev)
>
> static int designware_i2c_pci_probe(struct udevice *dev)
> {
> + struct dw_i2c *priv = dev_get_priv(dev);
> +
> + if (dev_get_driver_data(dev) == INTEL_APL) {
> + /* Ensure controller is in D0 state */
> + lpss_set_power_state(dev, STATE_D0);
> +
> + lpss_reset_release(priv->regs);
> + }
> +
> return designware_i2c_probe(dev);
> }
>
> @@ -91,6 +106,7 @@ static int designware_i2c_pci_bind(struct udevice *dev)
>
> static const struct udevice_id designware_i2c_pci_ids[] = {
> { .compatible = "snps,designware-i2c-pci" },
> + { .compatible = "intel,apl-i2c", INTEL_APL },
nits: .data = INTEL_APL to make it clear
> { }
> };
>
> @@ -116,6 +132,12 @@ static struct pci_device_id designware_pci_supported[] =
> {
> { PCI_VDEVICE(INTEL, 0x0f45) },
> { PCI_VDEVICE(INTEL, 0x0f46) },
> { PCI_VDEVICE(INTEL, 0x0f47) },
> + { PCI_VDEVICE(INTEL, 0x5aac), .driver_data = INTEL_APL },
> + { PCI_VDEVICE(INTEL, 0x5aae), .driver_data = INTEL_APL },
> + { PCI_VDEVICE(INTEL, 0x5ab0), .driver_data = INTEL_APL },
> + { PCI_VDEVICE(INTEL, 0x5ab2), .driver_data = INTEL_APL },
> + { PCI_VDEVICE(INTEL, 0x5ab4), .driver_data = INTEL_APL },
> + { PCI_VDEVICE(INTEL, 0x5ab6), .driver_data = INTEL_APL },
> {},
> };
>
> --
Regards,
Bin
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
