On Mon, Sep 3, 2018 at 11:05 AM Chen-Yu Tsai <wens@xxxxxxxx> wrote: > > On Fri, Aug 31, 2018 at 6:17 PM Maxime Ripard <maxime.ripard@xxxxxxxxxxx> wrote: > > > > On Thu, Aug 30, 2018 at 04:09:23PM +0800, Chen-Yu Tsai wrote: > > > Hi, > > > > > > Allwinner's H5 SoC is pin compatible with the H3 SoC. As such, some > > > vendors produce H3 and H5 variants for the same device. Such is the > > > case with Libre Computer's ALL-H3-CC, and the Bananapi M2 Plus. > > > > > > This series follows that of the ALL-H3-CC, splitting out a common > > > board dtsi, and then two SoC-specific dts files that include the > > > SoC level and common board dtsi's, as well as putting in the board > > > name. The first patch is a minor fix that I think should be done > > > before the migration. > > > > > > Please have a look. > > > > Acked-by: Maxime Ripard <maxime.ripard@xxxxxxxxxxx> Applied. > > > Also, on a related matter, Bananapi recently released revision v1.2 > > > of the M2 Plus. The original commercially available version was v1.1. > > > v1.2 adds a GPIO control that can change the CPU cores' supply voltage > > > between 1.1V and 1.3V. Do we want two extra dts files for this? Put > > > them in the existing dts files regardless? Or let people handle this > > > via overlays? > > > > If that the sole change, I'd be inclined to merge it as a separate DT > > for that particular version. Is there any way to detect it at runtime? > > It seems there isn't. The pin in both new and old revisions have an > external pull-up with the same value, so it isn't possible to detect > whether the pin is used or not. And there are no other changes between > the two. > > ChenYu