2022-09-28 14:52:51

by Wayne Chang

[permalink] [raw]
Subject: [PATCH 1/1] usb: gadget: tegra-xudc: Add Tegra234 SOC support

From: Sing-Han Chen <[email protected]>

This commit adds XUSB device mode controller support on Tegra234 SoC.
Tegra234 XUDC is very similar to the existing Tegra194 XUDC.

Signed-off-by: Sing-Han Chen <[email protected]>
Signed-off-by: Wayne Chang <[email protected]>
---
drivers/usb/gadget/udc/tegra-xudc.c | 17 +++++++++++++++++
1 file changed, 17 insertions(+)

diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c
index 3c37effdfa64..53277aa5a270 100644
--- a/drivers/usb/gadget/udc/tegra-xudc.c
+++ b/drivers/usb/gadget/udc/tegra-xudc.c
@@ -3656,6 +3656,19 @@ static struct tegra_xudc_soc tegra194_xudc_soc_data = {
.has_ipfs = false,
};

+static struct tegra_xudc_soc tegra234_xudc_soc_data = {
+ .clock_names = tegra186_xudc_clock_names,
+ .num_clks = ARRAY_SIZE(tegra186_xudc_clock_names),
+ .num_phys = 4,
+ .u1_enable = true,
+ .u2_enable = true,
+ .lpm_enable = true,
+ .invalid_seq_num = false,
+ .pls_quirk = false,
+ .port_reset_quirk = false,
+ .has_ipfs = false,
+};
+
static const struct of_device_id tegra_xudc_of_match[] = {
{
.compatible = "nvidia,tegra210-xudc",
@@ -3669,6 +3682,10 @@ static const struct of_device_id tegra_xudc_of_match[] = {
.compatible = "nvidia,tegra194-xudc",
.data = &tegra194_xudc_soc_data
},
+ {
+ .compatible = "nvidia,tegra234-xudc",
+ .data = &tegra234_xudc_soc_data
+ },
{ }
};
MODULE_DEVICE_TABLE(of, tegra_xudc_of_match);
--
2.25.1


2022-09-28 19:41:12

by Jon Hunter

[permalink] [raw]
Subject: Re: [PATCH 1/1] usb: gadget: tegra-xudc: Add Tegra234 SOC support


On 28/09/2022 14:55, Wayne Chang wrote:
> From: Sing-Han Chen <[email protected]>
>
> This commit adds XUSB device mode controller support on Tegra234 SoC.
> Tegra234 XUDC is very similar to the existing Tegra194 XUDC.
>
> Signed-off-by: Sing-Han Chen <[email protected]>
> Signed-off-by: Wayne Chang <[email protected]>
> ---
> drivers/usb/gadget/udc/tegra-xudc.c | 17 +++++++++++++++++
> 1 file changed, 17 insertions(+)
>
> diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c
> index 3c37effdfa64..53277aa5a270 100644
> --- a/drivers/usb/gadget/udc/tegra-xudc.c
> +++ b/drivers/usb/gadget/udc/tegra-xudc.c
> @@ -3656,6 +3656,19 @@ static struct tegra_xudc_soc tegra194_xudc_soc_data = {
> .has_ipfs = false,
> };
>
> +static struct tegra_xudc_soc tegra234_xudc_soc_data = {
> + .clock_names = tegra186_xudc_clock_names,
> + .num_clks = ARRAY_SIZE(tegra186_xudc_clock_names),
> + .num_phys = 4,
> + .u1_enable = true,
> + .u2_enable = true,
> + .lpm_enable = true,
> + .invalid_seq_num = false,
> + .pls_quirk = false,
> + .port_reset_quirk = false,
> + .has_ipfs = false,
> +};
> +
> static const struct of_device_id tegra_xudc_of_match[] = {
> {
> .compatible = "nvidia,tegra210-xudc",
> @@ -3669,6 +3682,10 @@ static const struct of_device_id tegra_xudc_of_match[] = {
> .compatible = "nvidia,tegra194-xudc",
> .data = &tegra194_xudc_soc_data
> },
> + {
> + .compatible = "nvidia,tegra234-xudc",
> + .data = &tegra234_xudc_soc_data
> + },


The device-tree binding documentation is missing for this compatible
string. Please send a patch to add this compatible string to the
appropriate binding doc.

Thanks!
Jon

--
nvpublic

2022-09-29 04:05:20

by Wayne Chang

[permalink] [raw]
Subject: Re: [PATCH 1/1] usb: gadget: tegra-xudc: Add Tegra234 SOC support

Hi Jonathan,

Thanks for the review.

On 9/29/22 03:04, Jonathan Hunter wrote:
>
> On 28/09/2022 14:55, Wayne Chang wrote:
>> From: Sing-Han Chen <[email protected]>
>>
>> This commit adds XUSB device mode controller support on Tegra234 SoC.
>> Tegra234 XUDC is very similar to the existing Tegra194 XUDC.
>>
>> Signed-off-by: Sing-Han Chen <[email protected]>
>> Signed-off-by: Wayne Chang <[email protected]>
>> ---
>> drivers/usb/gadget/udc/tegra-xudc.c | 17 +++++++++++++++++
>> 1 file changed, 17 insertions(+)
>>
>> diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c
>> index 3c37effdfa64..53277aa5a270 100644
>> --- a/drivers/usb/gadget/udc/tegra-xudc.c
>> +++ b/drivers/usb/gadget/udc/tegra-xudc.c
>> @@ -3656,6 +3656,19 @@ static struct tegra_xudc_soc tegra194_xudc_soc_data = {
>> .has_ipfs = false,
>> };
>>
>> +static struct tegra_xudc_soc tegra234_xudc_soc_data = {
>> + .clock_names = tegra186_xudc_clock_names,
>> + .num_clks = ARRAY_SIZE(tegra186_xudc_clock_names),
>> + .num_phys = 4,
>> + .u1_enable = true,
>> + .u2_enable = true,
>> + .lpm_enable = true,
>> + .invalid_seq_num = false,
>> + .pls_quirk = false,
>> + .port_reset_quirk = false,
>> + .has_ipfs = false,
>> +};
>> +
>> static const struct of_device_id tegra_xudc_of_match[] = {
>> {
>> .compatible = "nvidia,tegra210-xudc",
>> @@ -3669,6 +3682,10 @@ static const struct of_device_id tegra_xudc_of_match[] = {
>> .compatible = "nvidia,tegra194-xudc",
>> .data = &tegra194_xudc_soc_data
>> },
>> + {
>> + .compatible = "nvidia,tegra234-xudc",
>> + .data = &tegra234_xudc_soc_data
>> + },
>
>
> The device-tree binding documentation is missing for this compatible
> string. Please send a patch to add this compatible string to the
> appropriate binding doc.
Thanks. Sent out the change to add the compatible string to xudc binding
doc.

https://lore.kernel.org/all/[email protected]/T/#u

>
> Thanks!
> Jon
>

thanks,
Wayne.

2022-09-29 08:11:28

by Jon Hunter

[permalink] [raw]
Subject: Re: [PATCH 1/1] usb: gadget: tegra-xudc: Add Tegra234 SOC support


On 29/09/2022 04:46, Wayne Chang wrote:
> Hi Jonathan,
>
> Thanks for the review.
>
> On 9/29/22 03:04, Jonathan Hunter wrote:
>>
>> On 28/09/2022 14:55, Wayne Chang wrote:
>>> From: Sing-Han Chen <[email protected]>
>>>
>>> This commit adds XUSB device mode controller support on Tegra234 SoC.
>>> Tegra234 XUDC is very similar to the existing Tegra194 XUDC.
>>>
>>> Signed-off-by: Sing-Han Chen <[email protected]>
>>> Signed-off-by: Wayne Chang <[email protected]>
>>> ---
>>> drivers/usb/gadget/udc/tegra-xudc.c | 17 +++++++++++++++++
>>> 1 file changed, 17 insertions(+)
>>>
>>> diff --git a/drivers/usb/gadget/udc/tegra-xudc.c b/drivers/usb/gadget/udc/tegra-xudc.c
>>> index 3c37effdfa64..53277aa5a270 100644
>>> --- a/drivers/usb/gadget/udc/tegra-xudc.c
>>> +++ b/drivers/usb/gadget/udc/tegra-xudc.c
>>> @@ -3656,6 +3656,19 @@ static struct tegra_xudc_soc tegra194_xudc_soc_data = {
>>> .has_ipfs = false,
>>> };
>>>
>>> +static struct tegra_xudc_soc tegra234_xudc_soc_data = {
>>> + .clock_names = tegra186_xudc_clock_names,
>>> + .num_clks = ARRAY_SIZE(tegra186_xudc_clock_names),
>>> + .num_phys = 4,
>>> + .u1_enable = true,
>>> + .u2_enable = true,
>>> + .lpm_enable = true,
>>> + .invalid_seq_num = false,
>>> + .pls_quirk = false,
>>> + .port_reset_quirk = false,
>>> + .has_ipfs = false,
>>> +};
>>> +
>>> static const struct of_device_id tegra_xudc_of_match[] = {
>>> {
>>> .compatible = "nvidia,tegra210-xudc",
>>> @@ -3669,6 +3682,10 @@ static const struct of_device_id tegra_xudc_of_match[] = {
>>> .compatible = "nvidia,tegra194-xudc",
>>> .data = &tegra194_xudc_soc_data
>>> },
>>> + {
>>> + .compatible = "nvidia,tegra234-xudc",
>>> + .data = &tegra234_xudc_soc_data
>>> + },
>>
>>
>> The device-tree binding documentation is missing for this compatible
>> string. Please send a patch to add this compatible string to the
>> appropriate binding doc.
> Thanks. Sent out the change to add the compatible string to xudc binding
> doc.
>
> https://lore.kernel.org/all/[email protected]/T/#u

OK. Always best to send out these patches in a series together. I see
Krzysztof asking for the driver and DTS change. Given that we are
already at v6.0-rc7 this will not make v6.1, and so good to put all the
patches together in a series and resend.

Thanks
Jon

--
nvpublic