re: CVS import: xsrc/external/mit/fontconfig/dist

[matthew green]

matthew green writes:
> co...@sdf.org writes:
> > Now I get:
> > $ fc-match anything
> > Fontconfig warning: line 148: blank doesn't take any effect anymore. ple=
> ase remove it from your fonts.conf
> > Vera.ttf: "Bitstream Vera Sans" "Roman"
> > =
> 
> > I can't find where the  lines in fonts.conf come from. any ideas?
> 
> /etc/fonts/fonts.conf:88:   
> /etc/fonts/fonts.conf:148:  
> 
> guess we need an update for it in postinstall?

i commited the updated fonts.conf.


.mrg.

re: CVS commit: src

[matthew green]

Izumi Tsutsui writes:
> > Module Name:src
> > Committed By:   mrg
> > Date:   Tue Aug 29 07:53:54 UTC 2017
> > 
> > Modified Files:
> > src/distrib/sets/lists/xbase: mi
> > src/etc/mtree: NetBSD.dist.Xorg
> > src/external/mit/xorg/lib/xkeyboard-config/geometry: Makefile
> > src/external/mit/xorg/lib/xkeyboard-config/symbols: Makefile
> > 
> > Log Message:
> > update for xkeyboard-config 2.21.
> > prepare for libdrm data files.
> 
> src/external/mit/xorg/lib/xkeyboard-config/rules/base etc.
> should also be regen (as Makefile says)?

thanks.  i knew i was missing a step somewhere...


.mrg.

re: CVS import: xsrc/external/mit/fontconfig/dist

[matthew green]

co...@sdf.org writes:
> Now I get:
> $ fc-match anything
> Fontconfig warning: line 148: blank doesn't take any effect anymore. please 
> remove it from your fonts.conf
> Vera.ttf: "Bitstream Vera Sans" "Roman"
> 
> I can't find where the  lines in fonts.conf come from. any ideas?

/etc/fonts/fonts.conf:88:   
/etc/fonts/fonts.conf:148:  

guess we need an update for it in postinstall?


.mrg.

Re: CVS commit: src/sys/dev/i2c

[Jared McNeill]

On Tue, 29 Aug 2017, Manuel Bouyer wrote:


But it depends on the order in which the devices are probed/attached,
isn't it ?


The MD FDT code hooks up the powerdown handler very early, in initarm:

https://nxr.netbsd.org/xref/src/sys/arch/evbarm/fdt/fdt_machdep.c#392

re: CVS commit: xsrc/external/mit/libpciaccess/dist

[matthew green]

co...@sdf.org writes:
> On Tue, Aug 29, 2017 at 05:52:34AM +, matthew green wrote:
> > @@ -175,6 +179,9 @@ insert( uint16_t vendor )
> > struct pci_id_node * child =
> > calloc( 1, sizeof( struct pci_id_node ) );
> >  
> > +   if ( tree == NULL )
> > +   return NULL;
> > +
> > child->bits = 4;
> >  
> > n->children[ idx ] = child;
> > @@ -183,6 +190,9 @@ insert( uint16_t vendor )
> > struct pci_id_leaf * leaf =
> > calloc( 1, sizeof( struct pci_id_leaf ) );
> >  
> > +   if ( tree == NULL )
> > +   return NULL;
> > +
> > leaf->vendor = vendor;
> 
> should check child & leaf instead, I think

seems not entirely unreasonable.  can you look at it, i was
just porting this change forward..  thanks!


.mrg.

Re: CVS commit: src/sys/dev/i2c

[Manuel Bouyer]

On Tue, Aug 29, 2017 at 02:59:58PM -0300, Jared McNeill wrote:
> I'm not opposed to having a compile time option or sysctl -- but if we do I
> think we should try to make it MI. In theory an ACPI capable x86 board could
> have this problem as well..

Sure. The problem is more which poweroff callback to choose when there are
several available.

> 
> For axp specifically I think the right place to put it would be as a
> properly for the PMIC node in the device tree to disable poweroff.

Yes, I though about this too, and I agree.

> This way
> we can set it on a per-board basis. I didn't see one in the current binding
> docs, so we'd want to coordinate with the linux-sunxi folks on that.

sure.

> > For a personnal project (for those curious, available at
> > https://github.com/mbouyer/marine_chartplotter) I embeeded a olimex lime2
> > in a case, along with a power/interface board. I did develop a driver for
> > the external power controller, and in such a case I don't want the
> > power hook of the axp20x to be called. I don't think this use case is
> > uncommon ...
> 
> Looking at your driver:
> 
> https://github.com/mbouyer/marine_chartplotter/blob/master/software/NetBSD/driver/picbmc.c#L137
> 
> By taking over cpu_powerdown_address this will bypass the FDT hooks, so you
> should be ok here.

But it depends on the order in which the devices are probed/attached,
isn't it ?

-- 
Manuel Bouyer 
 NetBSD: 26 ans d'experience feront toujours la difference
--

Re: CVS commit: src/sys/dev/i2c

[Jared McNeill]

On Tue, 29 Aug 2017, Manuel Bouyer wrote:


I'd like to have a way to disable this (a compile-time option would be OK
for me).

On evaluation boards (like cubieboard, or olimex lime2), you may have
a power button on the board itself but you don't have a way to add an external
power button (e.g. if you put the board in a case), so once powered off
you have to power-cycle it.


I'm not opposed to having a compile time option or sysctl -- but if we do 
I think we should try to make it MI. In theory an ACPI capable x86 board 
could have this problem as well..


For axp specifically I think the right place to put it would be as a 
properly for the PMIC node in the device tree to disable poweroff. This 
way we can set it on a per-board basis. I didn't see one in the current 
binding docs, so we'd want to coordinate with the linux-sunxi folks on that.



For a personnal project (for those curious, available at
https://github.com/mbouyer/marine_chartplotter) I embeeded a olimex lime2
in a case, along with a power/interface board. I did develop a driver for
the external power controller, and in such a case I don't want the
power hook of the axp20x to be called. I don't think this use case is
uncommon ...


Looking at your driver:

https://github.com/mbouyer/marine_chartplotter/blob/master/software/NetBSD/driver/picbmc.c#L137

By taking over cpu_powerdown_address this will bypass the FDT hooks, so 
you should be ok here.


Cheers,
Jared

Re: CVS commit: src

[Izumi Tsutsui]

> Module Name:  src
> Committed By: mrg
> Date: Tue Aug 29 07:53:54 UTC 2017
> 
> Modified Files:
>   src/distrib/sets/lists/xbase: mi
>   src/etc/mtree: NetBSD.dist.Xorg
>   src/external/mit/xorg/lib/xkeyboard-config/geometry: Makefile
>   src/external/mit/xorg/lib/xkeyboard-config/symbols: Makefile
> 
> Log Message:
> update for xkeyboard-config 2.21.
> prepare for libdrm data files.

src/external/mit/xorg/lib/xkeyboard-config/rules/base etc.
should also be regen (as Makefile says)?

---
Izumi Tsutsui

Re: CVS import: xsrc/external/mit/fontconfig/dist

[coypu]

Now I get:
$ fc-match anything
Fontconfig warning: line 148: blank doesn't take any effect anymore. please 
remove it from your fonts.conf
Vera.ttf: "Bitstream Vera Sans" "Roman"

I can't find where the  lines in fonts.conf come from. any ideas?

Re: CVS commit: src/sys/dev/i2c

[Manuel Bouyer]

On Tue, Aug 29, 2017 at 10:10:54AM +, Jared D. McNeill wrote:
> Module Name:  src
> Committed By: jmcneill
> Date: Tue Aug 29 10:10:54 UTC 2017
> 
> Modified Files:
>   src/sys/dev/i2c: axp20x.c axp20xvar.h
> 
> Log Message:
> Add powerdown support and hook it in to FDT

I'd like to have a way to disable this (a compile-time option would be OK
for me).

On evaluation boards (like cubieboard, or olimex lime2), you may have
a power button on the board itself but you don't have a way to add an external
power button (e.g. if you put the board in a case), so once powered off
you have to power-cycle it.

For a personnal project (for those curious, available at
https://github.com/mbouyer/marine_chartplotter) I embeeded a olimex lime2
in a case, along with a power/interface board. I did develop a driver for
the external power controller, and in such a case I don't want the
power hook of the axp20x to be called. I don't think this use case is
uncommon ...

-- 
Manuel Bouyer 
 NetBSD: 26 ans d'experience feront toujours la difference
--

Re: CVS commit: xsrc/external/mit/libpciaccess/dist

[coypu]

On Tue, Aug 29, 2017 at 05:52:34AM +, matthew green wrote:
> @@ -175,6 +179,9 @@ insert( uint16_t vendor )
>   struct pci_id_node * child =
>   calloc( 1, sizeof( struct pci_id_node ) );
>  
> + if ( tree == NULL )
> + return NULL;
> +
>   child->bits = 4;
>  
>   n->children[ idx ] = child;
> @@ -183,6 +190,9 @@ insert( uint16_t vendor )
>   struct pci_id_leaf * leaf =
>   calloc( 1, sizeof( struct pci_id_leaf ) );
>  
> + if ( tree == NULL )
> + return NULL;
> +
>   leaf->vendor = vendor;
>  


should check child & leaf instead, I think