[U-Boot] [PATCH v3 0/4] Add Rock960 and Ficus 96Board support

[Tom Rini <trini@konsulko.com>]

On Thu, Dec 06, 2018 at 09:07:39PM +0530, Manivannan Sadhasivam wrote:
> On Thu, Dec 06, 2018 at 10:20:30AM -0500, Tom Rini wrote:
> > On Thu, Dec 06, 2018 at 08:53:33AM +0100, Alexander Graf wrote:
> > > 
> > > 
> > > On 26.10.18 20:00, Philipp Tomsich wrote:
> > > > 
> > > > 
> > > >> On 26.10.2018, at 19:17, Manivannan Sadhasivam <manivannan.sadhasivam@linaro.org> wrote:
> > > >>
> > > >> On Fri, Sep 28, 2018 at 12:32:57AM +0530, Manivannan Sadhasivam wrote:
> > > >>> This patchset adds support for Rock960 and Ficus 96Boards from Vamrs.
> > > >>> Since both boards share most of the configurations, a common Rock960
> > > >>> family support is added with common support and the actual boards are
> > > >>> based on this.
> > > >>>
> > > >>> The previous version of the patchseries were adding Rock960 [1] and
> > > >>> Ficus [2] board support individually, but this series fuses them
> > > >>> together based on the common board support as per Linux kernel.
> > > >>>
> > > >>> [1] https://patchwork.ozlabs.org/cover/963239/
> > > >>> [2] https://lists.denx.de/pipermail/u-boot/2018-August/339059.html
> > > >>>
> > > >>> This patchseries has been tested on Rock960 v1.2 board and expecting
> > > >>> Ezequiel to do the testing for Ficus.
> > > >>>
> > > >>> PS: I have explicitly removed the previous Ack's for the Ficus board
> > > >>> since there has been a heavy modification done on these patches.
> > > >>>
> > > >>> Thanks,
> > > >>> Mani
> > > >>>
> > > >>
> > > >> Hi,
> > > >>
> > > >> Is there any update on this patchset? AFAIK everyone is happy to get
> > > >> this merged!
> > > > 
> > > > It’s on pretty high up my list for the next merge window, but I didn’t get it into
> > > > the current version.
> > > 
> > > So there goes another merge window. Any progress here?
> > 
> > So, I'm going to call out here that this new board is entirely missing a
> > MAINTAINERS file.  Causing ./tools/genboardscfg.py to complain about new
> > defconfigs without a MAINTAINER.  I'm gonna fix it now, but, grumble.  I
> > know checkpatch.pl does complain "Hey, new file, do you need to update
> > MAINTAINERS" from Linux.
> >
> 
> Hi Tom,
> 
> Sorry for that! It was my bad to left MAINTAINERS entry for Ficus board.
> Since there are two boards (Rock960 and Ficus) sharing common board
> support, I forgot to update the later one.

Not a huge problem, really.  I've now become motivated enough to write a
.travis.yml test for this problem and thus pushing the problem visibly
up the chain like I should have months ago :)

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: not available
URL: <http://lists.denx.de/pipermail/u-boot/attachments/20181206/2eb351ec/attachment.sig>