<div dir="ltr">Hi Fran,<div><br></div><div>I will do my best to test it tomorrow.</div><div><br></div><div>Thanks,</div><div><br></div><div>Tomer</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, 4 Jun 2019 at 11:31, <<a href="mailto:franhsutw@gmail.com">franhsutw@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Hi Joel,<br>
    Let me check it and I will update a new patch soon. <br>
BTW, I had tried to merge these device tree patches into mainline (dev-5.2) but built fail.<br>
I will talk with Tomer to ask for help.<br>
<br>
Thanks,<br>
Fran<br>
<br>
> -----Original Message-----<br>
> From: Joel Stanley <<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>><br>
> Sent: Tuesday, June 4, 2019 1:29 PM<br>
> To: Benjamin Fair <<a href="mailto:benjaminfair@google.com" target="_blank">benjaminfair@google.com</a>><br>
> Cc: Fran Hsu <<a href="mailto:franhsutw@gmail.com" target="_blank">franhsutw@gmail.com</a>>; OpenBMC Maillist<br>
> <<a href="mailto:openbmc@lists.ozlabs.org" target="_blank">openbmc@lists.ozlabs.org</a>>; Fran Hsu <<a href="mailto:Fran.Hsu@quantatw.com" target="_blank">Fran.Hsu@quantatw.com</a>><br>
> Subject: Re: [PATCH dev-5.1 v5 1/4] ARM: dts: nuvoton: Add NPCM730 common<br>
> device tree include file.<br>
> <br>
> On Tue, 4 Jun 2019 at 05:23, Joel Stanley <<a href="mailto:joel@jms.id.au" target="_blank">joel@jms.id.au</a>> wrote:<br>
> ><br>
> > On Fri, 31 May 2019 at 15:28, Benjamin Fair <<a href="mailto:benjaminfair@google.com" target="_blank">benjaminfair@google.com</a>><br>
> wrote:<br>
> > > This looks great! I think it's ready to merge.<br>
> > ><br>
> > > On Fri, May 31, 2019 at 6:26 AM Fran Hsu <<a href="mailto:franhsutw@gmail.com" target="_blank">franhsutw@gmail.com</a>> wrote:<br>
> > > ><br>
> > > > Quanta GSJ BMC uses the Nuvoton NPCM730 BMC soc.<br>
> > > > This file describes the common setting of NPCM730 soc.<br>
> > > ><br>
> > > > Signed-off-by: Fran Hsu <<a href="mailto:Fran.Hsu@quantatw.com" target="_blank">Fran.Hsu@quantatw.com</a>><br>
> > ><br>
> > > Reviewed-by: Benjamin Fair <<a href="mailto:benjaminfair@google.com" target="_blank">benjaminfair@google.com</a>><br>
> ><br>
> > Thanks Benjamin for reviewing, and Fran for working to get these<br>
> > patches in a state that we can merge them. I've put them in the<br>
> > dev-5.1 tree with some small tweaks to the commit messages.<br>
> <br>
> I spoke too soon. I see these warnings when building:<br>
> <br>
> ../arch/arm/boot/dts/nuvoton-npcm730-gsj.dts:307.6-25: Warning<br>
> (i2c_bus_reg): /ahb/apb/i2c@8d000/ipmb@40000010:reg: I2C address must<br>
> be less than 10-bits, got "0x40000010"<br>
> ../arch/arm/boot/dts/nuvoton-npcm730-gsj.dts:320.6-25: Warning<br>
> (i2c_bus_reg): /ahb/apb/i2c@8e000/ipmb@40000012:reg: I2C address must<br>
> be less than 10-bits, got "0x40000012"<br>
> <br>
> Please send follow up patches to fix these warnings.<br>
> <br>
> The following warnings also exist when building the npcm kernel. If you could<br>
> also fix those it would be appreciated:<br>
> <br>
> ../arch/arm/boot/dts/nuvoton-common-npcm7xx.dtsi:222.22-231.5: Warning<br>
> (spi_bus_bridge): /ahb/fiu@fb000000: node name for SPI buses should be 'spi'<br>
>   also defined at ../arch/arm/boot/dts/nuvoton-npcm730-gsj.dts:81.22-126.5<br>
> arch/arm/boot/dts/nuvoton-npcm730-gsj.dtb: Warning (spi_bus_reg):<br>
> Failed prerequisite 'spi_bus_bridge'<br>
> ../arch/arm/boot/dts/nuvoton-common-npcm7xx.dtsi:222.22-231.5: Warning<br>
> (spi_bus_bridge): /ahb/fiu@fb000000: node name for SPI buses should be 'spi'<br>
>   also defined<br>
> at ../arch/arm/boot/dts/nuvoton-npcm750-evb.dts:164.22-221.5<br>
> ../arch/arm/boot/dts/nuvoton-common-npcm7xx.dtsi:233.22-244.5: Warning<br>
> (spi_bus_bridge): /ahb/fiu@c0000000: node name for SPI buses should be 'spi'<br>
>   also defined<br>
> at ../arch/arm/boot/dts/nuvoton-npcm750-evb.dts:223.22-242.5<br>
> <br>
> Cheers,<br>
> <br>
> Joel<br>
<br>
</blockquote></div>