Messages in this thread | | | From | Matthias Brugger <> | Subject | Re: [PATCH] arm64: dts: mt8173: Add afe device node | Date | Tue, 07 Jul 2015 09:46:55 +0200 |
| |
On Monday, July 06, 2015 09:02:38 PM Daniel Kurtz wrote: > On Mon, Jul 6, 2015 at 2:52 PM, Koro Chen <koro.chen@mediatek.com> wrote: > > This adds afe (audio front end) device node to the MT8173 dtsi file. > > > > This patch is based on Matthias's tree: > > https://github.com/mbgg/linux-mediatek > > branch: v4.2-next/arm64 > > > > Signed-off-by: Koro Chen <koro.chen@mediatek.com> > > --- > > > > arch/arm64/boot/dts/mediatek/mt8173.dtsi | 32 > > ++++++++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) > > > > diff --git a/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > b/arch/arm64/boot/dts/mediatek/mt8173.dtsi index 0696f8f..f8afab4 100644 > > --- a/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > +++ b/arch/arm64/boot/dts/mediatek/mt8173.dtsi > > @@ -14,6 +14,7 @@ > > > > #include <dt-bindings/clock/mt8173-clk.h> > > #include <dt-bindings/interrupt-controller/irq.h> > > #include <dt-bindings/interrupt-controller/arm-gic.h> > > > > +#include <dt-bindings/power/mt8173-power.h> > > > > #include <dt-bindings/reset-controller/mt8173-resets.h> > > #include "mt8173-pinfunc.h" > > > > @@ -393,6 +394,37 @@ > > > > #size-cells = <0>; > > status = "disabled"; > > > > }; > > > > + > > + afe: mt8173-afe-pcm@11220000 { > > I believe this should be something generic, like: > > afe: audio-controller@11220000 >
Yes, please find a name that better describes the component instead of just using the compatible string.
Thanks.
| |