[PATCH] arm: dts: aspeed: witherspoon: Add devices
Matt Spinler
mspinler at linux.vnet.ibm.com
Tue Jan 17 06:43:32 AEDT 2017
On 1/15/2017 6:02 PM, Joel Stanley wrote:
> On Sat, Jan 14, 2017 at 6:23 AM, Matt Spinler
> <mspinler at linux.vnet.ibm.com> wrote:
>> From: Matt Spinler <spinler at us.ibm.com>
>>
>> Add more I2C devices and the rear LEDs.
> Please make the subject line more descriptive. "add i2c and led devices".
Sure. Was just trying to fit in the 50 characters.
>> Signed-off-by: Matt Spinler <spinler at us.ibm.com>
>> ---
>> arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts | 39 ++++++++++++++++++++++--
>> 1 file changed, 36 insertions(+), 3 deletions(-)
>>
>> diff --git a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
>> index 9badf3e94599..11de7a957738 100644
>> --- a/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
>> +++ b/arch/arm/boot/dts/aspeed-bmc-opp-witherspoon.dts
>> @@ -19,6 +19,22 @@
>> memory {
>> reg = <0x80000000 0x40000000>;
>> };
>> +
>> + leds {
>> + compatible = "gpio-leds";
>> +
>> + rear-fault-led {
> Follow the naming of the other machines. This should be "rear-fault".
>
> Are there other fault leds? If not, call it "fault".
There's also a set of front LEDs for these functions. They're off of a
9552 and just I haven't gotten to their representation yet.
When I eventually get the autogeneration of the device tree working,
these names will be whatever they show up as in the MRW.
I will change these to 'rear-fault', etc.
>
>> + gpios = <&gpio ASPEED_GPIO(N, 2) GPIO_ACTIVE_LOW>;
> Did you attempt to compile this patch? It doesn't build for me, with
> the first error on this line.
Doh! I merged it from another tree where I tested it, but missed
updating the gpio include to the aspeed version.
>> + };
>> +
>> + rear-id-led {
> Same here. Call it "identify". as in the other dts.
Will change it to rear-identify as mentioned above.
>
>> + gpios = <&gpio ASPEED_GPIO(N, 4) GPIO_ACTIVE_LOW>;
>> + };
>> +
>> + rear-power-led {
> power.
Will be rear-power.
>
>> + gpios = <&gpio ASPEED_GPIO(N, 3) GPIO_ACTIVE_LOW>;
>> + };
>> + };
>> };
>>
>> &fmc {
>> @@ -115,25 +131,38 @@
>> compatible = "bosch,bmp180";
>> reg = <0x77>;
>> };
>> +
>> + max31785 at 52 {
>> + compatible = "maxim,max31785";
>> + reg = <0x52>;
>> + };
>> };
>>
>> &i2c4 {
>> status = "okay";
>>
>> - /* CP1 TMP423A */
>> /* CP1 VDN & VDDR */
>> /* CP1 VDD & VCS */
>> /* CP1 VIO */
>> +
>> + tmp423a at 4c {
>> + compatible = "ti,tmp423";
>> + reg = <0x4c>;
>> + };
>> };
>>
>>
>> &i2c5 {
>> status = "okay";
>>
>> - /* CP1 TMP423A */
>> /* CP1 VDN & VDDR */
>> /* CP1 VDD & VCS */
>> /* CP1 VIO */
>> +
>> + tmp423a at 4c {
>> + compatible = "ti,tmp423";
>> + reg = <0x4c>;
>> + };
>> };
>>
>> &i2c6 {
>> @@ -152,13 +181,17 @@
>> &i2c9 {
>> status = "okay";
>>
>> - /* Tmp275 */
>>
>> /* MUX
>> * -> PCIe Slot 3
>> * -> PCIe Slot 4
>> * -> PCA9517 -> PEX8725
>> */
>> +
>> + tmp275 at 4a {
>> + compatible = "ti,tmp275";
>> + reg = <0x4a>;
>> + };
>> };
>>
>> &i2c10 {
>> --
>> 2.11.0
>>
>> _______________________________________________
>> openbmc mailing list
>> openbmc at lists.ozlabs.org
>> https://lists.ozlabs.org/listinfo/openbmc
More information about the openbmc
mailing list