[oe] [PATCH 3/5 v2] linux-xilinx-ml507: update kernel version

Stefan Schmidt stefan at datenfreihafen.org
Mon May 24 19:35:34 UTC 2010


Hello.

On Mon, 2010-05-24 at 11:24, Adrian Alonso wrote:
> 
> On Mon, May 24, 2010 at 8:55 AM, Stefan Schmidt
> <stefan at datenfreihafen.org>wrote:
> >
> > I'm build testing this combination right now. One question that is still
> > open is
> > about the USE_VT in your first patch. Why did you comment it out? Something
> > like
> > this should be mentioned in the changelog.
> >
> > Yep I forgot to comment why i disable this option, it disables virtual
> terminal, i think in
> kernel options (not sure); disabling USE_VT I can have xserver running
> otherwise fails
> config files for machines that have by default graphical support don't
> include this option
> and i saw that only machines that doesn't have graphical support use this
> option to
> explicitly disable it.
> 
> Since we are in a dynamic architecture where display support can be included
> or not
> in the hardware model, i think that leaving the option in the config file is
> a good idea
> and just add some comments mentioning when this option is useful.

I added a note about this into the commit message. I also changed a bit in the
other messages. Please keep in mind to make them in a proper way. When I do a
review it is always the first part I read and try to understand, afterwards I'm
checking if the message matches the change itself and then I'm going to review
the change.

Every problem you can avoid before sending the patches off is time that is safed
in review cycles and in the end until it is pushed into the repo.

I found no problems during some build tests and pushed them into the repo.
Thanks.

regards
Stefan Schmidt




More information about the Openembedded-devel mailing list