>> In my opinion will be better generate only parameters which >> you want not all. >> That smells with unusable parameters. > >In the long term, this may be true. In the short term: >1) dtb size is not the key problem Yes of course >2) making sure that everything works is a key problem. >3) The code that generates the dts should be as simple as possible, >so that we can easily document what it does. Yes but you must document every parameter which your generate do. The better way is document only parameters which you want use.
>In the long term, I'm all for optimizing the device tree that gets >built, >assuming that it appears to be a problem in real systems. We can optimize now. I made new version of my generator u-boot_v3_00_a (monstr.eu) where you can simply add parameter which you want to use. If you want use any parameter add it. For microblaze cpu - line 1184. And for others peripherals lines 926-980. (The last parameter of function call). Which parameters do you want for PPC405, Grant? Regards, Michal Simek This is example of generator. / { model = "mONStR"; chosen { linux,platform = <600>; bootargs = "root=/dev/xsysace/disc0/part2"; } ; cpus { #size-cells = <0>; #cpus = < 0 >; #address-cells = <1>; microblaze_0,5.00.c { 32-bit; linux,boot-cpu; device_type = "cpu"; reg = <0>; clock-frequency = <5f5e1000>; timebase-frequency = <1FCA055>; i-cache-line-size = <2000>; i-cache-size = <10>; d-cache-line-size = <2000>; d-cache-size = <10>; xilinx,pvr = <0>; xilinx,debug-enabled = <1>; xilinx,fsl-links = <0>; } ; } ; [EMAIL PROTECTED] { compatible = "opb_mdm_2.00.a\0opb_mdm"; name = "debug_module"; reg = < 41400000 10000 >; device_type = "opb_mdm"; xilinx,mb-dbg-ports = <1>; xilinx,uart-width = <8>; xilinx,use-uart = <1>; } ; [EMAIL PROTECTED] { compatible = "opb_uartlite_1.00.b\0opb_uartlite"; name = "RS232_Uart_1"; interrupts = < 4 0 >; reg = < 40600000 10000 >; device_type = "opb_uartlite"; xilinx,baudrate = <2580>; xilinx,data-bits = <8>; xilinx,clk-freq = <5f5e100>; xilinx,odd-parity = <0>; xilinx,use-parity = <0>; } ; [EMAIL PROTECTED] { compatible = "opb_ethernet_1.04.a\0opb_ethernet"; name = "Ethernet_MAC"; interrupts = < 3 0 >; reg = < 40c00000 10000 >; device_type = "opb_ethernet"; xilinx,cam-exist = <0>; xilinx,dev-blk-id = <1>; xilinx,dev-mir-enable = <1>; xilinx,dma-present = <1>; xilinx,include-dev-pencoder = <1>; xilinx,ipif-rdfifo-depth = <8000>; xilinx,ipif-wrfifo-depth = <8000>; xilinx,mii-exist = <1>; } ; [EMAIL PROTECTED] { compatible = "opb_sysace_1.00.c\0opb_sysace"; name = "SysACE_CompactFlash"; reg = < 41820000 10000 >; device_type = "opb_sysace"; xilinx,mem-width = <10>; } ; [EMAIL PROTECTED] { compatible = "opb_gpio_3.01.b\0opb_gpio"; name = "LEDs_4Bit"; reg = < 40000000 10000 >; device_type = "opb_gpio"; xilinx,gpio-width = <4>; xilinx,is-dual = <0>; } ; [EMAIL PROTECTED] { edk_name = "DDR_256MB_32MX64_rank1_row13_col10_cl2_5"; compatible = "opb_ddr"; memreg:reg = < 30000000 10000000 >; device_type = "memory"; } ; [EMAIL PROTECTED] { compatible = "opb_ps2_dual_ref_1.00.a\0opb_ps2_dual_ref"; name = "PS2_Ports"; interrupts = < 2 0 >; interrupts = < 1 0 >; reg = < 7a400000 10000 >; device_type = "opb_ps2_dual_ref"; } ; [EMAIL PROTECTED] { compatible = "opb_timer_1.00.b\0opb_timer"; name = "opb_timer_1"; interrupts = < 0 0 >; reg = < 41c00000 10000 >; device_type = "opb_timer"; xilinx,count-width = <20>; xilinx,one-timer-only = <0>; } ; [EMAIL PROTECTED] { compatible = "opb_intc_1.00.c\0opb_intc"; name = "opb_intc_0"; reg = < 41200000 10000 >; device_type = "opb_intc"; } ; [EMAIL PROTECTED] { compatible = "opb_uart16550_1.00.d\0opb_uart16550"; name = "opb_uart16550_0"; reg = < 40400000 10000 >; device_type = "opb_uart16550"; } ; [EMAIL PROTECTED] { compatible = "opb_sysace_1.00.c\0opb_sysace"; name = "opb_sysace_0"; reg = < 41800000 10000 >; device_type = "opb_sysace"; xilinx,mem-width = <10>; } ; } ; _______________________________________________ Linuxppc-dev mailing list Linuxppc-dev@ozlabs.org https://ozlabs.org/mailman/listinfo/linuxppc-dev