| Message ID | 20200615095525.43414-3-codrin.ciubotariu@microchip.com (mailing list archive) |
|---|---|
| State | New, archived |
| Headers | show |
| Series | [1/3] ARM: dts: at91: sama5d2_xplained: classd: pull-down the R1 and R3 lines | expand |
Hi, The correct subject line prefix is "ARM: dts: at91:" On 15/06/2020 12:55:25+0300, Codrin Ciubotariu wrote: > There are no PDM microphones on SAMA5D2 Xplained, to exercize the > PDMIC. > > This reverts commit ca6349a8c51f2e3d6f2acdb36431e7d7328261f7. > > Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> > --- > arch/arm/boot/dts/at91-sama5d2_xplained.dts | 16 ---------------- > 1 file changed, 16 deletions(-) > This patch doesn't apply and I think you'll have to motivate the removal a bit more because this seems like a change of policy to me. > diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts > index 54d96649da77..c0a255bda477 100644 > --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts > +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts > @@ -109,16 +109,6 @@ timer1: timer@1 { > }; > }; > > - pdmic@f8018000 { > - pinctrl-names = "default"; > - pinctrl-0 = <&pinctrl_pdmic_default>; > - atmel,model = "PDMIC @ sama5d2_xplained"; > - atmel,mic-min-freq = <1000000>; > - atmel,mic-max-freq = <3246000>; > - atmel,mic-offset = <0x0>; > - status = "okay"; > - }; > - > uart1: serial@f8020000 { > pinctrl-names = "default"; > pinctrl-0 = <&pinctrl_uart1_default>; > @@ -533,12 +523,6 @@ pinctrl_macb0_phy_irq: macb0_phy_irq { > bias-disable; > }; > > - pinctrl_pdmic_default: pdmic_default { > - pinmux = <PIN_PB26__PDMIC_DAT>, > - <PIN_PB27__PDMIC_CLK>; > - bias-disable; > - }; > - > pinctrl_sdmmc0_default: sdmmc0_default { > cmd_data { > pinmux = <PIN_PA1__SDMMC0_CMD>, > -- > 2.25.1 >
On 18.06.2020 00:34, Alexandre Belloni wrote: > Hi, > > The correct subject line prefix is "ARM: dts: at91:" I just reverted the original patch. I can make it a normal commit if you want. > > On 15/06/2020 12:55:25+0300, Codrin Ciubotariu wrote: >> There are no PDM microphones on SAMA5D2 Xplained, to exercize the >> PDMIC. >> >> This reverts commit ca6349a8c51f2e3d6f2acdb36431e7d7328261f7. >> >> Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> >> --- >> arch/arm/boot/dts/at91-sama5d2_xplained.dts | 16 ---------------- >> 1 file changed, 16 deletions(-) >> > > This patch doesn't apply and I think you'll have to motivate the removal > a bit more because this seems like a change of policy to me. The PDMIC needs PDM microphones to work. sama5d2 xplained doesn't have such microphones, so there is no reason to enable PDMIC and take some pins since there is no-one using them. Isn't it the policy to enable only what is present on a board? > >> diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts >> index 54d96649da77..c0a255bda477 100644 >> --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts >> +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts >> @@ -109,16 +109,6 @@ timer1: timer@1 { >> }; >> }; >> >> - pdmic@f8018000 { >> - pinctrl-names = "default"; >> - pinctrl-0 = <&pinctrl_pdmic_default>; >> - atmel,model = "PDMIC @ sama5d2_xplained"; >> - atmel,mic-min-freq = <1000000>; >> - atmel,mic-max-freq = <3246000>; >> - atmel,mic-offset = <0x0>; >> - status = "okay"; >> - }; >> - >> uart1: serial@f8020000 { >> pinctrl-names = "default"; >> pinctrl-0 = <&pinctrl_uart1_default>; >> @@ -533,12 +523,6 @@ pinctrl_macb0_phy_irq: macb0_phy_irq { >> bias-disable; >> }; >> >> - pinctrl_pdmic_default: pdmic_default { >> - pinmux = <PIN_PB26__PDMIC_DAT>, >> - <PIN_PB27__PDMIC_CLK>; >> - bias-disable; >> - }; >> - >> pinctrl_sdmmc0_default: sdmmc0_default { >> cmd_data { >> pinmux = <PIN_PA1__SDMMC0_CMD>, >> -- >> 2.25.1 >> > > -- > Alexandre Belloni, Bootlin > Embedded Linux and Kernel engineering > https://bootlin.com >
diff --git a/arch/arm/boot/dts/at91-sama5d2_xplained.dts b/arch/arm/boot/dts/at91-sama5d2_xplained.dts index 54d96649da77..c0a255bda477 100644 --- a/arch/arm/boot/dts/at91-sama5d2_xplained.dts +++ b/arch/arm/boot/dts/at91-sama5d2_xplained.dts @@ -109,16 +109,6 @@ timer1: timer@1 { }; }; - pdmic@f8018000 { - pinctrl-names = "default"; - pinctrl-0 = <&pinctrl_pdmic_default>; - atmel,model = "PDMIC @ sama5d2_xplained"; - atmel,mic-min-freq = <1000000>; - atmel,mic-max-freq = <3246000>; - atmel,mic-offset = <0x0>; - status = "okay"; - }; - uart1: serial@f8020000 { pinctrl-names = "default"; pinctrl-0 = <&pinctrl_uart1_default>; @@ -533,12 +523,6 @@ pinctrl_macb0_phy_irq: macb0_phy_irq { bias-disable; }; - pinctrl_pdmic_default: pdmic_default { - pinmux = <PIN_PB26__PDMIC_DAT>, - <PIN_PB27__PDMIC_CLK>; - bias-disable; - }; - pinctrl_sdmmc0_default: sdmmc0_default { cmd_data { pinmux = <PIN_PA1__SDMMC0_CMD>,
There are no PDM microphones on SAMA5D2 Xplained, to exercize the PDMIC. This reverts commit ca6349a8c51f2e3d6f2acdb36431e7d7328261f7. Signed-off-by: Codrin Ciubotariu <codrin.ciubotariu@microchip.com> --- arch/arm/boot/dts/at91-sama5d2_xplained.dts | 16 ---------------- 1 file changed, 16 deletions(-)