[PATCH 7/7] ARM: dts: at91: sama5d4: add vdec0 component

Emil Velikov emil.l.velikov at gmail.com
Mon Mar 8 15:57:26 GMT 2021


On Mon, 8 Mar 2021 at 13:21, Nicolas Ferre <nicolas.ferre at microchip.com> wrote:
>
> Hi Emil,
>
Greetings Nicolas,

> So nice to see this support! Thank you so much for handling that.
>
> Little comments below...
>

> Nitpicking: I would use "microchip,sama5d4-vdec". We tend to use the
> microchip name for new DT bidings and compatibility strings.
>
Should i use Microchip (instead of Atmel) only for the DT bindings or
throughout the series?

> I'm fine with having a "staging" component. Maybe add the hantro vdec as
> a module instead.
>
Ack, will do for v2.

Thanks
Emil



More information about the Linux-rockchip mailing list