[PATCH v2 2/2] ARM: dts: add buffalo linkstation ls-wvl/vl
Andrew Lunn
andrew at lunn.ch
Mon Jun 29 06:43:43 PDT 2015
On Mon, Jun 29, 2015 at 03:22:56PM +0200, Gregory CLEMENT wrote:
> Hi Roger,
>
> On 22/06/2015 17:02, Roger Shimizu wrote:
> > Add dts file to support Buffalo Linkstation LS-WVL and LS-VL,
> > which are 3.5" HDD NAS in 2-bay and 1-bay respectively.
> >
> > Signed-off-by: Roger Shimizu <rogershimizu at gmail.com>
> > Reviewed-by: Nobuhiro Iwamatsu <iwamatsu at nigauri.org>
>
> it seems that you addressed Andrew's remarks too, so
>
> Acked-by: Gregory CLEMENT <gregory.clement at free-electrons.com>
>
>
> However I will wait for Andrew's formal acked-by before applying it.
Hi Gregory
One minor nit pick. Maybe you can fix it while committing?
> > + pmx_led_alarm: pmx-led-alarm {
> > + marvell,pins = "mpp36";
> > + marvell,function = "gpio";
> > + };
> > + pmx_led_function_red: pmx-led-function_red {
Second _red should be -red.
> > + marvell,pins = "mpp37";
> > + marvell,function = "gpio";
> > + };
Other than that,
Acked-by: Andrew Lunn <andrew at lunn.ch>
Thanks
Andrew
More information about the linux-arm-kernel
mailing list