* [PATCH] i2c: designware: Add support for AMD Seattle I2C @ 2015-12-15 21:55 Suravee Suthikulpanit 2015-12-16 2:14 ` Suravee Suthikulpanit 2016-01-04 19:25 ` Wolfram Sang 0 siblings, 2 replies; 9+ messages in thread From: Suravee Suthikulpanit @ 2015-12-15 21:55 UTC (permalink / raw) To: mika.westerberg, wsa Cc: linux-i2c, linux-acpi, linux-kernel, Suravee Suthikulpanit Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> --- drivers/i2c/busses/i2c-designware-platdrv.c | 1 + 1 file changed, 1 insertion(+) diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c index 57f623b..a027154 100644 --- a/drivers/i2c/busses/i2c-designware-platdrv.c +++ b/drivers/i2c/busses/i2c-designware-platdrv.c @@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { { "80860F41", 0 }, { "808622C1", 0 }, { "AMD0010", 0 }, + { "AMDI0510", 0 }, { } }; MODULE_DEVICE_TABLE(acpi, dw_i2c_acpi_match); -- 2.5.0 ^ permalink raw reply related [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-15 21:55 [PATCH] i2c: designware: Add support for AMD Seattle I2C Suravee Suthikulpanit @ 2015-12-16 2:14 ` Suravee Suthikulpanit 2015-12-16 9:16 ` Mika Westerberg 2016-01-04 19:25 ` Wolfram Sang 1 sibling, 1 reply; 9+ messages in thread From: Suravee Suthikulpanit @ 2015-12-16 2:14 UTC (permalink / raw) To: mika.westerberg, wsa; +Cc: linux-i2c, linux-acpi, linux-kernel Hi Mika, On 12/15/15 15:55, Suravee Suthikulpanit wrote: > Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform > > Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> > --- > drivers/i2c/busses/i2c-designware-platdrv.c | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c > index 57f623b..a027154 100644 > --- a/drivers/i2c/busses/i2c-designware-platdrv.c > +++ b/drivers/i2c/busses/i2c-designware-platdrv.c > @@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { > { "80860F41", 0 }, > { "808622C1", 0 }, > { "AMD0010", 0 }, > + { "AMDI0510", 0 }, > { } Since this driver seems to be used by several SOCs, and we have been adding the HID from various SOC vendors. Do you think it would be better to assign a CID so that each SOC vendor can specify in their ACPI DSDT and we can match them here? Then, we can also associate the FMCN and SSCN along with the CID, and guarantee compatibility. Thanks, Suravee > }; > MODULE_DEVICE_TABLE(acpi, dw_i2c_acpi_match); > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-16 2:14 ` Suravee Suthikulpanit @ 2015-12-16 9:16 ` Mika Westerberg 2015-12-16 14:29 ` Suravee Suthikulpanit 0 siblings, 1 reply; 9+ messages in thread From: Mika Westerberg @ 2015-12-16 9:16 UTC (permalink / raw) To: Suravee Suthikulpanit; +Cc: wsa, linux-i2c, linux-acpi, linux-kernel On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: > Hi Mika, > > On 12/15/15 15:55, Suravee Suthikulpanit wrote: > >Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform > > > >Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> > >--- > > drivers/i2c/busses/i2c-designware-platdrv.c | 1 + > > 1 file changed, 1 insertion(+) > > > >diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c > >index 57f623b..a027154 100644 > >--- a/drivers/i2c/busses/i2c-designware-platdrv.c > >+++ b/drivers/i2c/busses/i2c-designware-platdrv.c > >@@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { > > { "80860F41", 0 }, > > { "808622C1", 0 }, > > { "AMD0010", 0 }, > >+ { "AMDI0510", 0 }, > > { } > > Since this driver seems to be used by several SOCs, and we have been adding > the HID from various SOC vendors. Do you think it would be better to assign > a CID so that each SOC vendor can specify in their ACPI DSDT and we can > match them here? Sure _CID would work here. > Then, we can also associate the FMCN and SSCN along with the CID, and > guarantee compatibility. Well, the driver checks those everytime it finds that the device has ACPI companion regardless of _HID/_CID. ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-16 9:16 ` Mika Westerberg @ 2015-12-16 14:29 ` Suravee Suthikulpanit 2015-12-16 14:54 ` Mika Westerberg 0 siblings, 1 reply; 9+ messages in thread From: Suravee Suthikulpanit @ 2015-12-16 14:29 UTC (permalink / raw) To: Mika Westerberg; +Cc: wsa, linux-i2c, linux-acpi, linux-kernel On 12/16/2015 03:16 AM, Mika Westerberg wrote: > On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: >> Hi Mika, >> >> On 12/15/15 15:55, Suravee Suthikulpanit wrote: >>> Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform >>> >>> Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> >>> --- >>> drivers/i2c/busses/i2c-designware-platdrv.c | 1 + >>> 1 file changed, 1 insertion(+) >>> >>> diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c >>> index 57f623b..a027154 100644 >>> --- a/drivers/i2c/busses/i2c-designware-platdrv.c >>> +++ b/drivers/i2c/busses/i2c-designware-platdrv.c >>> @@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { >>> { "80860F41", 0 }, >>> { "808622C1", 0 }, >>> { "AMD0010", 0 }, >>> + { "AMDI0510", 0 }, >>> { } >> >> Since this driver seems to be used by several SOCs, and we have been adding >> the HID from various SOC vendors. Do you think it would be better to assign >> a CID so that each SOC vendor can specify in their ACPI DSDT and we can >> match them here? > > Sure _CID would work here. Do you know if Synopsys has already provided a CID that we can use for this? If not, who do you think should provide this? Also, do you think the FMCN and SSCN should be documented somewhere in the spec so that FW and OSes can agree upon going forward? > >> Then, we can also associate the FMCN and SSCN along with the CID, and >> guarantee compatibility. > > Well, the driver checks those everytime it finds that the device has > ACPI companion regardless of _HID/_CID. Not sure what you mean by "device has ACPI companion". Do you mean the driver check those for every matched devices here? Thanks, Suravee > -- > To unsubscribe from this list: send the line "unsubscribe linux-acpi" in > the body of a message to majordomo@vger.kernel.org > More majordomo info at http://vger.kernel.org/majordomo-info.html > ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-16 14:29 ` Suravee Suthikulpanit @ 2015-12-16 14:54 ` Mika Westerberg 2015-12-17 0:49 ` Suravee Suthikulanit 0 siblings, 1 reply; 9+ messages in thread From: Mika Westerberg @ 2015-12-16 14:54 UTC (permalink / raw) To: Suravee Suthikulpanit; +Cc: wsa, linux-i2c, linux-acpi, linux-kernel On Wed, Dec 16, 2015 at 08:29:38AM -0600, Suravee Suthikulpanit wrote: > > > On 12/16/2015 03:16 AM, Mika Westerberg wrote: > >On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: > >>Hi Mika, > >> > >>On 12/15/15 15:55, Suravee Suthikulpanit wrote: > >>>Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform > >>> > >>>Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> > >>>--- > >>> drivers/i2c/busses/i2c-designware-platdrv.c | 1 + > >>> 1 file changed, 1 insertion(+) > >>> > >>>diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c > >>>index 57f623b..a027154 100644 > >>>--- a/drivers/i2c/busses/i2c-designware-platdrv.c > >>>+++ b/drivers/i2c/busses/i2c-designware-platdrv.c > >>>@@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { > >>> { "80860F41", 0 }, > >>> { "808622C1", 0 }, > >>> { "AMD0010", 0 }, > >>>+ { "AMDI0510", 0 }, > >>> { } > >> > >>Since this driver seems to be used by several SOCs, and we have been adding > >>the HID from various SOC vendors. Do you think it would be better to assign > >>a CID so that each SOC vendor can specify in their ACPI DSDT and we can > >>match them here? > > > >Sure _CID would work here. > > Do you know if Synopsys has already provided a CID that we can use for this? No. > If not, who do you think should provide this? Why can't you make _CID for AMD part only? For Intel we are going to get new IDs for every major SoC release no matter what. > Also, do you think the FMCN > and SSCN should be documented somewhere in the spec so that FW and OSes can > agree upon going forward? Since this is designware I2C specific thing it should not be part of the ACPI spec, I think. But, yes I agree should be documented *somewhere* ;-) > >>Then, we can also associate the FMCN and SSCN along with the CID, and > >>guarantee compatibility. > > > >Well, the driver checks those everytime it finds that the device has > >ACPI companion regardless of _HID/_CID. > > Not sure what you mean by "device has ACPI companion". Do you mean the > driver check those for every matched devices here? That's right: if (has_acpi_companion(&pdev->dev)) dw_i2c_acpi_configure(pdev); ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-16 14:54 ` Mika Westerberg @ 2015-12-17 0:49 ` Suravee Suthikulanit 2016-01-03 18:45 ` Wolfram Sang 0 siblings, 1 reply; 9+ messages in thread From: Suravee Suthikulanit @ 2015-12-17 0:49 UTC (permalink / raw) To: Mika Westerberg; +Cc: wsa, linux-i2c, linux-acpi, linux-kernel Mika, On 12/16/2015 8:54 AM, Mika Westerberg wrote: > On Wed, Dec 16, 2015 at 08:29:38AM -0600, Suravee Suthikulpanit wrote: >> > >> > >> >On 12/16/2015 03:16 AM, Mika Westerberg wrote: >>> > >On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: >>>> > >>Hi Mika, >>>> > >> >>>> > >>On 12/15/15 15:55, Suravee Suthikulpanit wrote: >>>>> > >>>Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform >>>>> > >>> >>>>> > >>>Signed-off-by: Suravee Suthikulpanit<Suravee.Suthikulpanit@amd.com> >>>>> > >>>--- >>>>> > >>> drivers/i2c/busses/i2c-designware-platdrv.c | 1 + >>>>> > >>> 1 file changed, 1 insertion(+) >>>>> > >>> >>>>> > >>>diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c >>>>> > >>>index 57f623b..a027154 100644 >>>>> > >>>--- a/drivers/i2c/busses/i2c-designware-platdrv.c >>>>> > >>>+++ b/drivers/i2c/busses/i2c-designware-platdrv.c >>>>> > >>>@@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { >>>>> > >>> { "80860F41", 0 }, >>>>> > >>> { "808622C1", 0 }, >>>>> > >>> { "AMD0010", 0 }, >>>>> > >>>+ { "AMDI0510", 0 }, >>>>> > >>> { } >>>> > >> >>>> > >>Since this driver seems to be used by several SOCs, and we have been adding >>>> > >>the HID from various SOC vendors. Do you think it would be better to assign >>>> > >>a CID so that each SOC vendor can specify in their ACPI DSDT and we can >>>> > >>match them here? >>> > > >>> > >Sure _CID would work here. >> > >> >Do you know if Synopsys has already provided a CID that we can use for this? > No. > >> >If not, who do you think should provide this? > Why can't you make _CID for AMD part only? For Intel we are going to get > new IDs for every major SoC release no matter what. > Actually, after discussed with the team. We have decided to go with the AMDI0510 at this point, and we will reuse this as CID in future SOC if it contains compatible I2C controller. Thanks, Suravee ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-17 0:49 ` Suravee Suthikulanit @ 2016-01-03 18:45 ` Wolfram Sang 2016-01-03 22:54 ` Suravee Suthikulpanit 0 siblings, 1 reply; 9+ messages in thread From: Wolfram Sang @ 2016-01-03 18:45 UTC (permalink / raw) To: Suravee Suthikulanit; +Cc: Mika Westerberg, linux-i2c, linux-acpi, linux-kernel [-- Attachment #1: Type: text/plain, Size: 2171 bytes --] On Wed, Dec 16, 2015 at 06:49:59PM -0600, Suravee Suthikulanit wrote: > Mika, > > On 12/16/2015 8:54 AM, Mika Westerberg wrote: > >On Wed, Dec 16, 2015 at 08:29:38AM -0600, Suravee Suthikulpanit wrote: > >>> > >>> > >>>On 12/16/2015 03:16 AM, Mika Westerberg wrote: > >>>> >On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: > >>>>> >>Hi Mika, > >>>>> >> > >>>>> >>On 12/15/15 15:55, Suravee Suthikulpanit wrote: > >>>>>> >>>Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform > >>>>>> >>> > >>>>>> >>>Signed-off-by: Suravee Suthikulpanit<Suravee.Suthikulpanit@amd.com> > >>>>>> >>>--- > >>>>>> >>> drivers/i2c/busses/i2c-designware-platdrv.c | 1 + > >>>>>> >>> 1 file changed, 1 insertion(+) > >>>>>> >>> > >>>>>> >>>diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c > >>>>>> >>>index 57f623b..a027154 100644 > >>>>>> >>>--- a/drivers/i2c/busses/i2c-designware-platdrv.c > >>>>>> >>>+++ b/drivers/i2c/busses/i2c-designware-platdrv.c > >>>>>> >>>@@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { > >>>>>> >>> { "80860F41", 0 }, > >>>>>> >>> { "808622C1", 0 }, > >>>>>> >>> { "AMD0010", 0 }, > >>>>>> >>>+ { "AMDI0510", 0 }, > >>>>>> >>> { } > >>>>> >> > >>>>> >>Since this driver seems to be used by several SOCs, and we have been adding > >>>>> >>the HID from various SOC vendors. Do you think it would be better to assign > >>>>> >>a CID so that each SOC vendor can specify in their ACPI DSDT and we can > >>>>> >>match them here? > >>>> > > >>>> >Sure _CID would work here. > >>> > >>>Do you know if Synopsys has already provided a CID that we can use for this? > >No. > > > >>>If not, who do you think should provide this? > >Why can't you make _CID for AMD part only? For Intel we are going to get > >new IDs for every major SoC release no matter what. > > > Actually, after discussed with the team. We have decided to go with the > AMDI0510 at this point, and we will reuse this as CID in future SOC if it > contains compatible I2C controller. So, can I take the patch as is? [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2016-01-03 18:45 ` Wolfram Sang @ 2016-01-03 22:54 ` Suravee Suthikulpanit 0 siblings, 0 replies; 9+ messages in thread From: Suravee Suthikulpanit @ 2016-01-03 22:54 UTC (permalink / raw) To: Wolfram Sang; +Cc: Mika Westerberg, linux-i2c, linux-acpi, linux-kernel Hi Wolfram, On 01/03/2016 12:45 PM, Wolfram Sang wrote: > On Wed, Dec 16, 2015 at 06:49:59PM -0600, Suravee Suthikulanit wrote: >> Mika, >> >> On 12/16/2015 8:54 AM, Mika Westerberg wrote: >>> On Wed, Dec 16, 2015 at 08:29:38AM -0600, Suravee Suthikulpanit wrote: >>>>> >>>>> >>>>> On 12/16/2015 03:16 AM, Mika Westerberg wrote: >>>>>>> On Tue, Dec 15, 2015 at 08:14:34PM -0600, Suravee Suthikulpanit wrote: >>>>>>>>> Hi Mika, >>>>>>>>> >>>>>>>>> On 12/15/15 15:55, Suravee Suthikulpanit wrote: >>>>>>>>>>> Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform >>>>>>>>>>> >>>>>>>>>>> Signed-off-by: Suravee Suthikulpanit<Suravee.Suthikulpanit@amd.com> >>>>>>>>>>> --- >>>>>>>>>>> drivers/i2c/busses/i2c-designware-platdrv.c | 1 + >>>>>>>>>>> 1 file changed, 1 insertion(+) >>>>>>>>>>> >>>>>>>>>>> diff --git a/drivers/i2c/busses/i2c-designware-platdrv.c b/drivers/i2c/busses/i2c-designware-platdrv.c >>>>>>>>>>> index 57f623b..a027154 100644 >>>>>>>>>>> --- a/drivers/i2c/busses/i2c-designware-platdrv.c >>>>>>>>>>> +++ b/drivers/i2c/busses/i2c-designware-platdrv.c >>>>>>>>>>> @@ -117,6 +117,7 @@ static const struct acpi_device_id dw_i2c_acpi_match[] = { >>>>>>>>>>> { "80860F41", 0 }, >>>>>>>>>>> { "808622C1", 0 }, >>>>>>>>>>> { "AMD0010", 0 }, >>>>>>>>>>> + { "AMDI0510", 0 }, >>>>>>>>>>> { } >>>>>>>>> >>>>>>>>> Since this driver seems to be used by several SOCs, and we have been adding >>>>>>>>> the HID from various SOC vendors. Do you think it would be better to assign >>>>>>>>> a CID so that each SOC vendor can specify in their ACPI DSDT and we can >>>>>>>>> match them here? >>>>>>> >>>>>>> Sure _CID would work here. >>>>> >>>>> Do you know if Synopsys has already provided a CID that we can use for this? >>> No. >>> >>>>> If not, who do you think should provide this? >>> Why can't you make _CID for AMD part only? For Intel we are going to get >>> new IDs for every major SoC release no matter what. >>> >> Actually, after discussed with the team. We have decided to go with the >> AMDI0510 at this point, and we will reuse this as CID in future SOC if it >> contains compatible I2C controller. > > So, can I take the patch as is? > Yes, please pull this as is. Thank you, Suravee ^ permalink raw reply [flat|nested] 9+ messages in thread
* Re: [PATCH] i2c: designware: Add support for AMD Seattle I2C 2015-12-15 21:55 [PATCH] i2c: designware: Add support for AMD Seattle I2C Suravee Suthikulpanit 2015-12-16 2:14 ` Suravee Suthikulpanit @ 2016-01-04 19:25 ` Wolfram Sang 1 sibling, 0 replies; 9+ messages in thread From: Wolfram Sang @ 2016-01-04 19:25 UTC (permalink / raw) To: Suravee Suthikulpanit Cc: mika.westerberg, linux-i2c, linux-acpi, linux-kernel [-- Attachment #1: Type: text/plain, Size: 261 bytes --] On Tue, Dec 15, 2015 at 03:55:53PM -0600, Suravee Suthikulpanit wrote: > Add device HID AMDI0510 to match the I2C controlers on AMD Seattle platform > > Signed-off-by: Suravee Suthikulpanit <Suravee.Suthikulpanit@amd.com> Applied to for-next, thanks! [-- Attachment #2: Digital signature --] [-- Type: application/pgp-signature, Size: 819 bytes --] ^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2016-01-04 19:25 UTC | newest] Thread overview: 9+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2015-12-15 21:55 [PATCH] i2c: designware: Add support for AMD Seattle I2C Suravee Suthikulpanit 2015-12-16 2:14 ` Suravee Suthikulpanit 2015-12-16 9:16 ` Mika Westerberg 2015-12-16 14:29 ` Suravee Suthikulpanit 2015-12-16 14:54 ` Mika Westerberg 2015-12-17 0:49 ` Suravee Suthikulanit 2016-01-03 18:45 ` Wolfram Sang 2016-01-03 22:54 ` Suravee Suthikulpanit 2016-01-04 19:25 ` Wolfram Sang
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for NNTP newsgroup(s).