| Message ID | 20160911134111.31141-2-martin.blumenstingl@googlemail.com (mailing list archive) |
|---|---|
| State | Superseded |
| Headers | show |
Hi John, Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: > From: Jerome Brunet <jbrunet@baylibre.com> > > Add compatible strings for amlogic Meson8b and GXBB SoCs with the > corresponding configuration parameters. > > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> Are you OK with adding this platform for v4.9? I know you mentioned you're working on new bindings to replace the current way, but since that hasn't been posted AFAICT, it would be nice to get this merged now and we can help test the new bindings when you're ready. If you're OK with that, and with your Ack, I can take merge the driver changes through the arm-soc tree along with the rest of the DT patches. Kevin > --- > Documentation/devicetree/bindings/usb/dwc2.txt | 2 ++ > drivers/usb/dwc2/platform.c | 34 ++++++++++++++++++++++++++ > 2 files changed, 36 insertions(+) > > diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt > index 20a68bf..2c30a54 100644 > --- a/Documentation/devicetree/bindings/usb/dwc2.txt > +++ b/Documentation/devicetree/bindings/usb/dwc2.txt > @@ -10,6 +10,8 @@ Required properties: > - "rockchip,rk3288-usb", "rockchip,rk3066-usb", "snps,dwc2": for rk3288 Soc; > - "lantiq,arx100-usb": The DWC2 USB controller instance in Lantiq ARX SoCs; > - "lantiq,xrx200-usb": The DWC2 USB controller instance in Lantiq XRX SoCs; > + - "amlogic,meson8b-usb": The DWC2 USB controller instance in Amlogic Meson8b SoCs; > + - "amlogic,meson-gxbb-usb": The DWC2 USB controller instance in Amlogic S905 SoCs; > - snps,dwc2: A generic DWC2 USB controller with default parameters. > - reg : Should contain 1 register range (address and length) > - interrupts : Should contain 1 interrupt > diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c > index fc6f525..8f7b34c 100644 > --- a/drivers/usb/dwc2/platform.c > +++ b/drivers/usb/dwc2/platform.c > @@ -181,6 +181,38 @@ static const struct dwc2_core_params params_ltq = { > .hibernation = -1, > }; > > +static const struct dwc2_core_params params_amlogic = { > + .otg_cap = DWC2_CAP_PARAM_NO_HNP_SRP_CAPABLE, > + .otg_ver = -1, > + .dma_enable = 1, > + .dma_desc_enable = 0, > + .dma_desc_fs_enable = 0, > + .speed = DWC2_SPEED_PARAM_HIGH, > + .enable_dynamic_fifo = 1, > + .en_multiple_tx_fifo = -1, > + .host_rx_fifo_size = 512, > + .host_nperio_tx_fifo_size = 500, > + .host_perio_tx_fifo_size = 500, > + .max_transfer_size = -1, > + .max_packet_count = -1, > + .host_channels = 16, > + .phy_type = DWC2_PHY_TYPE_PARAM_UTMI, > + .phy_utmi_width = -1, > + .phy_ulpi_ddr = -1, > + .phy_ulpi_ext_vbus = -1, > + .i2c_enable = -1, > + .ulpi_fs_ls = -1, > + .host_support_fs_ls_low_power = -1, > + .host_ls_low_power_phy_clk = -1, > + .ts_dline = -1, > + .reload_ctl = 1, > + .ahbcfg = GAHBCFG_HBSTLEN_INCR8 << > + GAHBCFG_HBSTLEN_SHIFT, > + .uframe_sched = 0, > + .external_id_pin_ctl = -1, > + .hibernation = -1, > +}; > + > /* > * Check the dr_mode against the module configuration and hardware > * capabilities. > @@ -464,6 +496,8 @@ static const struct of_device_id dwc2_of_match_table[] = { > { .compatible = "lantiq,xrx200-usb", .data = ¶ms_ltq }, > { .compatible = "snps,dwc2", .data = NULL }, > { .compatible = "samsung,s3c6400-hsotg", .data = NULL}, > + { .compatible = "amlogic,meson8b-usb", .data = ¶ms_amlogic }, > + { .compatible = "amlogic,meson-gxbb-usb", .data = ¶ms_amlogic }, > {}, > }; > MODULE_DEVICE_TABLE(of, dwc2_of_match_table);
On 9/14/2016 9:11 AM, Kevin Hilman wrote: > Hi John, > > Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: > >> From: Jerome Brunet <jbrunet@baylibre.com> >> >> Add compatible strings for amlogic Meson8b and GXBB SoCs with the >> corresponding configuration parameters. >> >> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> >> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > > Are you OK with adding this platform for v4.9? I know you mentioned > you're working on new bindings to replace the current way, but since > that hasn't been posted AFAICT, it would be nice to get this merged now > and we can help test the new bindings when you're ready. > > If you're OK with that, and with your Ack, I can take merge the driver > changes through the arm-soc tree along with the rest of the DT patches. > Sure. Unfortunately, I wasn't able to complete it for 4.9. You can add my acked-by on this: Acked-by: John Youn <johnyoun@synopsys.com> Regards, John > Kevin > >> --- >> Documentation/devicetree/bindings/usb/dwc2.txt | 2 ++ >> drivers/usb/dwc2/platform.c | 34 ++++++++++++++++++++++++++ >> 2 files changed, 36 insertions(+) >> >> diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt >> index 20a68bf..2c30a54 100644 >> --- a/Documentation/devicetree/bindings/usb/dwc2.txt >> +++ b/Documentation/devicetree/bindings/usb/dwc2.txt >> @@ -10,6 +10,8 @@ Required properties: >> - "rockchip,rk3288-usb", "rockchip,rk3066-usb", "snps,dwc2": for rk3288 Soc; >> - "lantiq,arx100-usb": The DWC2 USB controller instance in Lantiq ARX SoCs; >> - "lantiq,xrx200-usb": The DWC2 USB controller instance in Lantiq XRX SoCs; >> + - "amlogic,meson8b-usb": The DWC2 USB controller instance in Amlogic Meson8b SoCs; >> + - "amlogic,meson-gxbb-usb": The DWC2 USB controller instance in Amlogic S905 SoCs; >> - snps,dwc2: A generic DWC2 USB controller with default parameters. >> - reg : Should contain 1 register range (address and length) >> - interrupts : Should contain 1 interrupt >> diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c >> index fc6f525..8f7b34c 100644 >> --- a/drivers/usb/dwc2/platform.c >> +++ b/drivers/usb/dwc2/platform.c >> @@ -181,6 +181,38 @@ static const struct dwc2_core_params params_ltq = { >> .hibernation = -1, >> }; >> >> +static const struct dwc2_core_params params_amlogic = { >> + .otg_cap = DWC2_CAP_PARAM_NO_HNP_SRP_CAPABLE, >> + .otg_ver = -1, >> + .dma_enable = 1, >> + .dma_desc_enable = 0, >> + .dma_desc_fs_enable = 0, >> + .speed = DWC2_SPEED_PARAM_HIGH, >> + .enable_dynamic_fifo = 1, >> + .en_multiple_tx_fifo = -1, >> + .host_rx_fifo_size = 512, >> + .host_nperio_tx_fifo_size = 500, >> + .host_perio_tx_fifo_size = 500, >> + .max_transfer_size = -1, >> + .max_packet_count = -1, >> + .host_channels = 16, >> + .phy_type = DWC2_PHY_TYPE_PARAM_UTMI, >> + .phy_utmi_width = -1, >> + .phy_ulpi_ddr = -1, >> + .phy_ulpi_ext_vbus = -1, >> + .i2c_enable = -1, >> + .ulpi_fs_ls = -1, >> + .host_support_fs_ls_low_power = -1, >> + .host_ls_low_power_phy_clk = -1, >> + .ts_dline = -1, >> + .reload_ctl = 1, >> + .ahbcfg = GAHBCFG_HBSTLEN_INCR8 << >> + GAHBCFG_HBSTLEN_SHIFT, >> + .uframe_sched = 0, >> + .external_id_pin_ctl = -1, >> + .hibernation = -1, >> +}; >> + >> /* >> * Check the dr_mode against the module configuration and hardware >> * capabilities. >> @@ -464,6 +496,8 @@ static const struct of_device_id dwc2_of_match_table[] = { >> { .compatible = "lantiq,xrx200-usb", .data = ¶ms_ltq }, >> { .compatible = "snps,dwc2", .data = NULL }, >> { .compatible = "samsung,s3c6400-hsotg", .data = NULL}, >> + { .compatible = "amlogic,meson8b-usb", .data = ¶ms_amlogic }, >> + { .compatible = "amlogic,meson-gxbb-usb", .data = ¶ms_amlogic }, >> {}, >> }; >> MODULE_DEVICE_TABLE(of, dwc2_of_match_table); >
On Wed, Sep 14, 2016 at 11:12 AM, John Youn <John.Youn@synopsys.com> wrote: > On 9/14/2016 9:11 AM, Kevin Hilman wrote: >> Hi John, >> >> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >> >>> From: Jerome Brunet <jbrunet@baylibre.com> >>> >>> Add compatible strings for amlogic Meson8b and GXBB SoCs with the >>> corresponding configuration parameters. >>> >>> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> >>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> >> >> Are you OK with adding this platform for v4.9? I know you mentioned >> you're working on new bindings to replace the current way, but since >> that hasn't been posted AFAICT, it would be nice to get this merged now >> and we can help test the new bindings when you're ready. >> >> If you're OK with that, and with your Ack, I can take merge the driver >> changes through the arm-soc tree along with the rest of the DT patches. >> > > Sure. Unfortunately, I wasn't able to complete it for 4.9. > > You can add my acked-by on this: > > Acked-by: John Youn <johnyoun@synopsys.com> Great, thanks! I assume you're OK with PATCH 2/2 for the bindings as well (at least for now until you rework them?) Kevin
On 9/14/2016 11:17 AM, Kevin Hilman wrote: > On Wed, Sep 14, 2016 at 11:12 AM, John Youn <John.Youn@synopsys.com> wrote: >> On 9/14/2016 9:11 AM, Kevin Hilman wrote: >>> Hi John, >>> >>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >>> >>>> From: Jerome Brunet <jbrunet@baylibre.com> >>>> >>>> Add compatible strings for amlogic Meson8b and GXBB SoCs with the >>>> corresponding configuration parameters. >>>> >>>> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> >>>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> >>> >>> Are you OK with adding this platform for v4.9? I know you mentioned >>> you're working on new bindings to replace the current way, but since >>> that hasn't been posted AFAICT, it would be nice to get this merged now >>> and we can help test the new bindings when you're ready. >>> >>> If you're OK with that, and with your Ack, I can take merge the driver >>> changes through the arm-soc tree along with the rest of the DT patches. >>> >> >> Sure. Unfortunately, I wasn't able to complete it for 4.9. >> >> You can add my acked-by on this: >> >> Acked-by: John Youn <johnyoun@synopsys.com> > > Great, thanks! I assume you're OK with PATCH 2/2 for the bindings as > well (at least for now until you rework them?) > > Kevin > Do you mean PATCH 2/6 from this series? It doesn't seem to add or remove any bindings in dwc2 so I'm fine with it. Regards, John
On Wed, Sep 14, 2016 at 11:26 AM, John Youn <John.Youn@synopsys.com> wrote: > On 9/14/2016 11:17 AM, Kevin Hilman wrote: >> On Wed, Sep 14, 2016 at 11:12 AM, John Youn <John.Youn@synopsys.com> wrote: >>> On 9/14/2016 9:11 AM, Kevin Hilman wrote: >>>> Hi John, >>>> >>>> Martin Blumenstingl <martin.blumenstingl@googlemail.com> writes: >>>> >>>>> From: Jerome Brunet <jbrunet@baylibre.com> >>>>> >>>>> Add compatible strings for amlogic Meson8b and GXBB SoCs with the >>>>> corresponding configuration parameters. >>>>> >>>>> Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> >>>>> Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> >>>> >>>> Are you OK with adding this platform for v4.9? I know you mentioned >>>> you're working on new bindings to replace the current way, but since >>>> that hasn't been posted AFAICT, it would be nice to get this merged now >>>> and we can help test the new bindings when you're ready. >>>> >>>> If you're OK with that, and with your Ack, I can take merge the driver >>>> changes through the arm-soc tree along with the rest of the DT patches. >>>> >>> >>> Sure. Unfortunately, I wasn't able to complete it for 4.9. >>> >>> You can add my acked-by on this: >>> >>> Acked-by: John Youn <johnyoun@synopsys.com> >> >> Great, thanks! I assume you're OK with PATCH 2/2 for the bindings as >> well (at least for now until you rework them?) >> >> Kevin >> > > Do you mean PATCH 2/6 from this series? Yes. > It doesn't seem to add or remove any bindings in dwc2 so I'm fine with > it. Thanks. Kevin
On Sun, Sep 11, 2016 at 03:41:06PM +0200, Martin Blumenstingl wrote: > From: Jerome Brunet <jbrunet@baylibre.com> > > Add compatible strings for amlogic Meson8b and GXBB SoCs with the > corresponding configuration parameters. > > Signed-off-by: Jerome Brunet <jbrunet@baylibre.com> > Signed-off-by: Martin Blumenstingl <martin.blumenstingl@googlemail.com> > --- > Documentation/devicetree/bindings/usb/dwc2.txt | 2 ++ Acked-by: Rob Herring <robh@kernel.org> > drivers/usb/dwc2/platform.c | 34 ++++++++++++++++++++++++++ > 2 files changed, 36 insertions(+)
diff --git a/Documentation/devicetree/bindings/usb/dwc2.txt b/Documentation/devicetree/bindings/usb/dwc2.txt index 20a68bf..2c30a54 100644 --- a/Documentation/devicetree/bindings/usb/dwc2.txt +++ b/Documentation/devicetree/bindings/usb/dwc2.txt @@ -10,6 +10,8 @@ Required properties: - "rockchip,rk3288-usb", "rockchip,rk3066-usb", "snps,dwc2": for rk3288 Soc; - "lantiq,arx100-usb": The DWC2 USB controller instance in Lantiq ARX SoCs; - "lantiq,xrx200-usb": The DWC2 USB controller instance in Lantiq XRX SoCs; + - "amlogic,meson8b-usb": The DWC2 USB controller instance in Amlogic Meson8b SoCs; + - "amlogic,meson-gxbb-usb": The DWC2 USB controller instance in Amlogic S905 SoCs; - snps,dwc2: A generic DWC2 USB controller with default parameters. - reg : Should contain 1 register range (address and length) - interrupts : Should contain 1 interrupt diff --git a/drivers/usb/dwc2/platform.c b/drivers/usb/dwc2/platform.c index fc6f525..8f7b34c 100644 --- a/drivers/usb/dwc2/platform.c +++ b/drivers/usb/dwc2/platform.c @@ -181,6 +181,38 @@ static const struct dwc2_core_params params_ltq = { .hibernation = -1, }; +static const struct dwc2_core_params params_amlogic = { + .otg_cap = DWC2_CAP_PARAM_NO_HNP_SRP_CAPABLE, + .otg_ver = -1, + .dma_enable = 1, + .dma_desc_enable = 0, + .dma_desc_fs_enable = 0, + .speed = DWC2_SPEED_PARAM_HIGH, + .enable_dynamic_fifo = 1, + .en_multiple_tx_fifo = -1, + .host_rx_fifo_size = 512, + .host_nperio_tx_fifo_size = 500, + .host_perio_tx_fifo_size = 500, + .max_transfer_size = -1, + .max_packet_count = -1, + .host_channels = 16, + .phy_type = DWC2_PHY_TYPE_PARAM_UTMI, + .phy_utmi_width = -1, + .phy_ulpi_ddr = -1, + .phy_ulpi_ext_vbus = -1, + .i2c_enable = -1, + .ulpi_fs_ls = -1, + .host_support_fs_ls_low_power = -1, + .host_ls_low_power_phy_clk = -1, + .ts_dline = -1, + .reload_ctl = 1, + .ahbcfg = GAHBCFG_HBSTLEN_INCR8 << + GAHBCFG_HBSTLEN_SHIFT, + .uframe_sched = 0, + .external_id_pin_ctl = -1, + .hibernation = -1, +}; + /* * Check the dr_mode against the module configuration and hardware * capabilities. @@ -464,6 +496,8 @@ static const struct of_device_id dwc2_of_match_table[] = { { .compatible = "lantiq,xrx200-usb", .data = ¶ms_ltq }, { .compatible = "snps,dwc2", .data = NULL }, { .compatible = "samsung,s3c6400-hsotg", .data = NULL}, + { .compatible = "amlogic,meson8b-usb", .data = ¶ms_amlogic }, + { .compatible = "amlogic,meson-gxbb-usb", .data = ¶ms_amlogic }, {}, }; MODULE_DEVICE_TABLE(of, dwc2_of_match_table);