lkml.org 
[lkml]   [2022]   [Oct]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH 6.0 726/862] i2c: designware-pci: Group AMD NAVI quirk parts together
    Date
    From: Andy Shevchenko <andriy.shevchenko@linux.intel.com>

    [ Upstream commit 65769162ae4b7f2d82e54998be446226b05fcd8f ]

    The code is ogranized in a way that all related parts
    to the certain platform quirk go together. This is not
    the case for AMD NAVI. Shuffle code to make it happen.

    While at it, drop the frequency definition and use
    hard coded value as it's done for other platforms and
    add a comment to the PCI ID list.

    Signed-off-by: Andy Shevchenko <andriy.shevchenko@linux.intel.com>
    Acked-by: Jarkko Nikula <jarkko.nikula@linux.intel.com>
    Signed-off-by: Wolfram Sang <wsa@kernel.org>
    Signed-off-by: Sasha Levin <sashal@kernel.org>
    ---
    drivers/i2c/busses/i2c-designware-pcidrv.c | 30 +++++++++++-----------
    1 file changed, 15 insertions(+), 15 deletions(-)

    diff --git a/drivers/i2c/busses/i2c-designware-pcidrv.c b/drivers/i2c/busses/i2c-designware-pcidrv.c
    index 608e61209455..ca368482b246 100644
    --- a/drivers/i2c/busses/i2c-designware-pcidrv.c
    +++ b/drivers/i2c/busses/i2c-designware-pcidrv.c
    @@ -27,7 +27,6 @@
    #include "i2c-ccgx-ucsi.h"

    #define DRIVER_NAME "i2c-designware-pci"
    -#define AMD_CLK_RATE_HZ 100000

    enum dw_pci_ctl_id_t {
    medfield,
    @@ -100,11 +99,6 @@ static u32 mfld_get_clk_rate_khz(struct dw_i2c_dev *dev)
    return 25000;
    }

    -static u32 navi_amd_get_clk_rate_khz(struct dw_i2c_dev *dev)
    -{
    - return AMD_CLK_RATE_HZ;
    -}
    -
    static int mfld_setup(struct pci_dev *pdev, struct dw_pci_controller *c)
    {
    struct dw_i2c_dev *dev = dev_get_drvdata(&pdev->dev);
    @@ -126,15 +120,6 @@ static int mfld_setup(struct pci_dev *pdev, struct dw_pci_controller *c)
    return -ENODEV;
    }

    -static int navi_amd_setup(struct pci_dev *pdev, struct dw_pci_controller *c)
    -{
    - struct dw_i2c_dev *dev = dev_get_drvdata(&pdev->dev);
    -
    - dev->flags |= MODEL_AMD_NAVI_GPU;
    - dev->timings.bus_freq_hz = I2C_MAX_STANDARD_MODE_FREQ;
    - return 0;
    -}
    -
    static int mrfld_setup(struct pci_dev *pdev, struct dw_pci_controller *c)
    {
    /*
    @@ -159,6 +144,20 @@ static u32 ehl_get_clk_rate_khz(struct dw_i2c_dev *dev)
    return 100000;
    }

    +static u32 navi_amd_get_clk_rate_khz(struct dw_i2c_dev *dev)
    +{
    + return 100000;
    +}
    +
    +static int navi_amd_setup(struct pci_dev *pdev, struct dw_pci_controller *c)
    +{
    + struct dw_i2c_dev *dev = dev_get_drvdata(&pdev->dev);
    +
    + dev->flags |= MODEL_AMD_NAVI_GPU;
    + dev->timings.bus_freq_hz = I2C_MAX_STANDARD_MODE_FREQ;
    + return 0;
    +}
    +
    static struct dw_pci_controller dw_pci_controllers[] = {
    [medfield] = {
    .bus_num = -1,
    @@ -389,6 +388,7 @@ static const struct pci_device_id i2_designware_pci_ids[] = {
    { PCI_VDEVICE(INTEL, 0x4bbe), elkhartlake },
    { PCI_VDEVICE(INTEL, 0x4bbf), elkhartlake },
    { PCI_VDEVICE(INTEL, 0x4bc0), elkhartlake },
    + /* AMD NAVI */
    { PCI_VDEVICE(ATI, 0x7314), navi_amd },
    { PCI_VDEVICE(ATI, 0x73a4), navi_amd },
    { PCI_VDEVICE(ATI, 0x73e4), navi_amd },
    --
    2.35.1


    \
     
     \ /
      Last update: 2022-10-19 13:12    [W:4.031 / U:0.184 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site