[PATCH] ARM: dts: stm32: Add alternate pinmux for DCMI pins

[Marek Vasut]

On 8/28/20 5:56 PM, Alexandre Torgue wrote:
> Hi,

Hi,

> On 8/28/20 2:52 PM, Marek Vasut wrote:
>> On 8/28/20 2:47 PM, Alexandre Torgue wrote:
>>> Hi Marek
>>
>> Hi,
>>
>> [...]
>>
>>>> +        pins {
>>>> +            pinmux = <STM32_PINMUX('A', 4,  AF13)>,/* DCMI_HSYNC */
>>>> +                 <STM32_PINMUX('B', 7,  AF13)>,/* DCMI_VSYNC */
>>>> +                 <STM32_PINMUX('A', 6,  AF13)>,/* DCMI_PIXCLK */
>>>> +                 <STM32_PINMUX('A', 9,  AF13)>,/* DCMI_D0 */
>>>> +                 <STM32_PINMUX('H', 10, AF13)>,/* DCMI_D1 */
>>>> +                 <STM32_PINMUX('E', 0, AF13)>,/* DCMI_D2 */
>>>> +                 <STM32_PINMUX('E', 1, AF13)>,/* DCMI_D3 */
>>>> +                 <STM32_PINMUX('H', 14, AF13)>,/* DCMI_D4 */
>>>> +                 <STM32_PINMUX('I', 4,  AF13)>,/* DCMI_D5 */
>>>> +                 <STM32_PINMUX('I', 6,  AF13)>,/* DCMI_D6 */
>>>> +                 <STM32_PINMUX('E', 6,  AF13)>,/* DCMI_D7 */
>>>> +                 <STM32_PINMUX('I', 1,  AF13)>,/* DCMI_D8 */
>>>> +                 <STM32_PINMUX('H', 7,  AF13)>;/* DCMI_D9 */
>>>> +            bias-pull-up;
>>>> +        };
>>>> +    };
>>>> +
>>>> +    dcmi_sleep_pins_b: dcmi-sleep-1 {
>>>> +        pins {
>>>> +            pinmux = <STM32_PINMUX('A', 4,  ANALOG)>,/* DCMI_HSYNC */
>>>> +                 <STM32_PINMUX('B', 7,  ANALOG)>,/* DCMI_VSYNC */
>>>> +                 <STM32_PINMUX('A', 6,  ANALOG)>,/* DCMI_PIXCLK */
>>>> +                 <STM32_PINMUX('A', 9,  ANALOG)>,/* DCMI_D0 */
>>>> +                 <STM32_PINMUX('H', 10, ANALOG)>,/* DCMI_D1 */
>>>> +                 <STM32_PINMUX('E', 0, ANALOG)>,/* DCMI_D2 */
>>>> +                 <STM32_PINMUX('E', 1, ANALOG)>,/* DCMI_D3 */
>>>> +                 <STM32_PINMUX('H', 14, ANALOG)>,/* DCMI_D4 */
>>>> +                 <STM32_PINMUX('I', 4,  ANALOG)>,/* DCMI_D5 */
>>>> +                 <STM32_PINMUX('I', 6,  ANALOG)>,/* DCMI_D6 */
>>>> +                 <STM32_PINMUX('E', 6,  ANALOG)>,/* DCMI_D7 */
>>>> +                 <STM32_PINMUX('I', 1,  ANALOG)>,/* DCMI_D8 */
>>>> +                 <STM32_PINMUX('H', 7,  ANALOG)>;/* DCMI_D9 */
>>>> +        };
>>>> +    };
>>>> +
>>>
>>> Defined but not used. Do you have another patch in pipe ?
>>
>> Yeah, for camera on the AV96, but that depends on DCMI media patches to
>> be usable. Same for the RCC patches.
> 
> Ok so I prefer to wait for other DCMI patches before merging those ones.

Fine by me.