Hi Simon,
Am 01.06.2017 um 05:11 schrieb Simon Glass:
> On 26 May 2017 at 07:55, Wadim Egorov <w.ego...@phytec.de> wrote:
>> Hello Simon,
>>
>>
>> Am 24.05.2017 um 02:44 schrieb Simon Glass:
>>> Hi Wadim,
>>>
>>> On 15 May 2017 at 08:19, Wadim Egorov <w.ego...@phytec.de> wrote:
>>>> The phyCORE-RK3288 is a SoM (System on Module) containing a RK3288 SoC.
>>>> The module can be connected to different carrier boards.
>>>> It can be also equipped with different RAM, SPI flash and eMMC variants.
>>>> The Rapid Development Kit option is using the following setup:
>>>>
>>>> - 1 GB DDR3 RAM (2 Banks)
>>>> - 1x 4 KB EEPROM
>>>> - DP83867 Gigabit Ethernet PHY
>>>> - 16 MB SPI Flash
>>>> - 4 GB eMMC Flash
>>>>
>>>> Add basic support for the PCM-947 carrier board, a RK3288 based development
>>>> board made by PHYTEC. This board works in a combination with
>>>> the phyCORE-RK3288 System on Module.
>>>>
>>>> Signed-off-by: Wadim Egorov <w.ego...@phytec.de>
>>>> Reviewed-by: Simon Glass <s...@chromium.org>
>>>> ---
>>>> Changes in v2:
>>>> - Move phycore initialization to an own function
>>>> - Use of_machine_is_compatible() instead of #ifdef
>>>> - Drop board_boot_order() and use spl_boot_device()
>>>> - Added Reviewed-by: Simon Glass <s...@chromium.org>
>>>>
>>>> ---
>>>> arch/arm/dts/Makefile | 1 +
>>>> arch/arm/dts/rk3288-phycore-rdk.dts | 294 ++++++++++++++++
>>>> arch/arm/dts/rk3288-phycore-som.dtsi | 503
>>>> +++++++++++++++++++++++++++
>>>> arch/arm/mach-rockchip/rk3288-board-spl.c | 37 ++
>>>> arch/arm/mach-rockchip/rk3288/Kconfig | 10 +
>>>> board/phytec/phycore_rk3288/Kconfig | 15 +
>>>> board/phytec/phycore_rk3288/MAINTAINERS | 6 +
>>>> board/phytec/phycore_rk3288/Makefile | 8 +
>>>> board/phytec/phycore_rk3288/phycore-rk3288.c | 8 +
>>>> configs/phycore-rk3288_defconfig | 69 ++++
>>>> include/configs/phycore_rk3288.h | 23 ++
>>>> 11 files changed, 974 insertions(+)
>>>> create mode 100644 arch/arm/dts/rk3288-phycore-rdk.dts
>>>> create mode 100644 arch/arm/dts/rk3288-phycore-som.dtsi
>>>> create mode 100644 board/phytec/phycore_rk3288/Kconfig
>>>> create mode 100644 board/phytec/phycore_rk3288/MAINTAINERS
>>>> create mode 100644 board/phytec/phycore_rk3288/Makefile
>>>> create mode 100644 board/phytec/phycore_rk3288/phycore-rk3288.c
>>>> create mode 100644 configs/phycore-rk3288_defconfig
>>>> create mode 100644 include/configs/phycore_rk3288.h
>>> This unfortunately causes build errors on various rockchip boards.
>> This seems to be a problem because I am using of_machine_is_compatible().
>> I think we will need a CONFIG_TARGET_PHYCORE_RK3288 guard there.
> You mean that it does not work? You should be able to call that function.
When I build, for example the chromebook_jerry_defconfig, I get the
following error:
drivers/built-in.o: In function `of_machine_is_compatible':
u-boot/chromebook/../drivers/core/device.c:845: undefined reference to
`fdt_node_check_compatible'
It works with the phycore-rk3288_defconfig and fails with other rockchip
configs.
This is why I wanted to use CONFIG_TARGET_PHYCORE_RK3288 in the first place.
>
>>> Try 'buildman rockchip' to see it.
>>>
>>> Please see below.
>>>
>>> [...]
>>>
>>>> diff --git a/arch/arm/mach-rockchip/rk3288-board-spl.c
>>>> b/arch/arm/mach-rockchip/rk3288-board-spl.c
>>>> index 74f3379..724dcb4 100644
>>>> --- a/arch/arm/mach-rockchip/rk3288-board-spl.c
>>>> +++ b/arch/arm/mach-rockchip/rk3288-board-spl.c
>>>> @@ -8,6 +8,7 @@
>>>> #include <debug_uart.h>
>>>> #include <dm.h>
>>>> #include <fdtdec.h>
>>>> +#include <i2c.h>
>>>> #include <led.h>
>>>> #include <malloc.h>
>>>> #include <ram.h>
>>>> @@ -25,6 +26,7 @@
>>>> #include <dm/test.h>
>>>> #include <dm/util.h>
>>>> #include <power/regulator.h>
>>>> +#include <power/rk8xx_pmic.h>
>>>>
>>>> DECLARE_GLOBAL_DATA_PTR;
>>>>
>>>> @@ -157,6 +159,38 @@ static int configure_emmc(struct udevice *pinctrl)
>>>> }
>>>> #endif
>>>>
>>>> +void phycore_init(void)
>>>> +{
>>>> + struct udevice *dev;
>>>> + uint reg;
>>>> + int ret;
>>>> +
>>>> + ret = uclass_get_device(UCLASS_I2C, 0, &dev);
>>>> + if (ret) {
>>>> + debug("I2C init failed: %d\n", ret);
>>>> + return;
>>>> + }
>>>> +
>>>> + ret = i2c_get_chip(dev, 0x1c, 1, &dev);
>>> We should not be hard-coding the bus address here.
>>>
>>> See veyron_init() for an example of another way to do this.
>>>
>>>> + if (ret) {
>>>> + debug("Cannot find RK818: %d\n", ret);
>>>> + return;
>>>> + }
>>>> +
>>>> + reg = dm_i2c_reg_read(dev, REG_USB_CTRL);
>>>> +
>>>> + /*
>>>> + * Increase USB input current selection to 2A and close charger
>>>> + * when usb lower then 3.4V.
>>>> + */
>>>> + reg |= 0x77;
>>>> + ret = dm_i2c_reg_write(dev, REG_USB_CTRL, reg);
>>> Here you are hacking registers in the PMIC. This should go in the PMIC
>>> driver. See rk8xx_spl_configure_buck() for how to do this in SPL
>>> without bringing in the whole regulator framework.
>> For this I have to enable CONFIG_SPL_POWER_SUPPORT to get the pmic part
>> compiled.
>> Unfortunately, I am not able to boot my board with POWER_SUPPORT
>> enabled, because the dm_init_and_scan() function which is called from
>> spl_early_init() is failing.
>> Do you have any hints on what the problem could be?
> Do you know what the error is? Also you could put #define DEBUG in
> lists.c pehaps?
With CONFIG_SPL_POWER_SUPPORT=y enabled the device_bind_common()
function is failing for
the I2C (fail_uclass_post_bind) and PMIC (fail_bind) device. See the
output below:
[...]
bind node dmc@ff610000
- attempt to match compatible string 'rockchip,rk3288-dmc'
- found match at 'rockchip_rk3288_dmc'
Bound device dmc@ff610000 to root_driver
bind node i2c@ff650000
- attempt to match compatible string 'rockchip,rk3288-i2c'
- found match at 'i2c_rockchip'
bind node pmic@1c
- attempt to match compatible string 'rockchip,rk818'
- found match at 'rk8xx pmic'
bind node power-management@ff730000
- attempt to match compatible string 'rockchip,rk3288-pmu'
- found match at 'rk3288_syscon'
Bound device power-management@ff730000 to root_driver
[...]
bind node config
Device 'config' has no compatible string
### ERROR ### Please RESET the board ###
Regards,
Wadim
>
>> Regards,
>> Wadim
>>
>>>> + if (ret) {
>>>> + debug("Unable to set RK818 REG_USB_CTRL: %d\n", ret);
>>>> + return;
>>>> + }
>>>> +}
>>>> +
>>> Regards,
>>> Simon
> Regards,
> Simon
_______________________________________________
U-Boot mailing list
U-Boot@lists.denx.de
https://lists.denx.de/listinfo/u-boot
