Message ID | 20221128143832.25584-1-allen-kh.cheng@mediatek.com |
---|---|
Headers | show |
Series | Add vcodec lat and core nodes for mt8192 | expand |
Il 28/11/22 15:38, Allen-KH Cheng ha scritto: > Add video-codec lat and core nodes for mt8192 SoC. > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: AngeloGioacchino Del Regno <angelogioacchino.delregno@collabora.com>
On Mon, Nov 28, 2022 at 10:38 PM Allen-KH Cheng <allen-kh.cheng@mediatek.com> wrote: > > This series is based on linux-next. > > Change from v4: > - Add patch for removing dma-ranges > - Update child names to video-codec > - Update patch title and message > > Change from v3: > - Drop `vcodec_lat:` and `vcodec_core:` > > Change from v2: > - Add a dt-bindings patch for changing "vcodec" to "video-codec" > - Rename child names for vcodec lat and core nodes > > Change from v1: > - Replace node names with video-codec > [Allen-KH Cheng <allen-kh.cheng@mediatek.com>] > > Allen-KH Cheng (3): > media: dt-bindings: media: mediatek: Rename child node names for > decoder > media: dt-bindings: media: mediatek: Remove "dma-ranges" property for > decoder > arm64: dts: mt8192: Add video-codec nodes Whole series is: Tested-by: Chen-Yu Tsai <wenst@chromium.org>
On Mon, Nov 28, 2022 at 10:38:32PM +0800, Allen-KH Cheng wrote: > Add video-codec lat and core nodes for mt8192 SoC. Even though the rule is to use generic names in the DT, here in the commit message and title I would have called them "video decoder nodes", to be more specific and differentiate from the encoder nodes. But the way you wrote is not wrong, so feel free to ignore this comment and leave it as it is. > > Signed-off-by: Allen-KH Cheng <allen-kh.cheng@mediatek.com> Reviewed-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> Tested-by: Nícolas F. R. A. Prado <nfraprado@collabora.com> Thanks, Nícolas