Hey Cédric,
On Wed, 15 Jul 2020 at 15:07, Cédric Le Goater <c...@kaod.org> wrote:
> > Signed-off-by: erik-smit <erik.lucas.s...@gmail.com>
> > --
> > checkpatch was complaining about the length of
> > aspeed_machine_supermicrox11_bmc_class_init(ObjectClass *oc, void *data)
>
> in which way ? line too long ?
Yes.
> If so, you can insert a return inside the parameter list.
Ok. Next patch coming up. Thanks for the input.
>
> > so I renamed it to aspeed_machine_smx11_bmc_class_init. Not sure if
> > that's the right way to go, since then it's out of sync with the machine
> > name "supermicrox11-bmc". But renaming the machine name to "smx11-bmc"
> > makes it less descriptive...
>
> You should put the patch comments below the '---'
Thank you.
> > ---
> > hw/arm/aspeed.c | 34 ++++++++++++++++++++++++++++++++++
> > 1 file changed, 34 insertions(+)
> >
> > diff --git a/hw/arm/aspeed.c b/hw/arm/aspeed.c
> > index 379f9672a5..e74a89f427 100644
> > --- a/hw/arm/aspeed.c
> > +++ b/hw/arm/aspeed.c
> > @@ -57,6 +57,20 @@ struct AspeedMachineState {
> > SCU_HW_STRAP_VGA_SIZE_SET(VGA_16M_DRAM) | \
> > SCU_AST2400_HW_STRAP_BOOT_MODE(AST2400_SPI_BOOT))
> >
> > +/* TODO: Find the actual hardware value */
>
> If you can access the BMC console, you can do that from u-boot.
I don't own this hardware anymore. Missing playing with this hardware
is one of the reasons I'm emulating it.
--
Best regards,
Erik Smit
