[PATCH 5/8] xhci: mediatek: Add support for MTK xHCI host controller

Simon Glass sjg at chromium.org
Mon Apr 20 01:38:09 CEST 2020


Hi Marek,

On Fri, 10 Apr 2020 at 13:57, Marek Vasut <marex at denx.de> wrote:
>
> On 4/10/20 9:14 PM, Simon Glass wrote:
> > Hi Marek,
>
> Hi,
>
> > On Fri, 10 Apr 2020 at 12:45, Marek Vasut <marex at denx.de> wrote:
> >>
> >> On 3/31/20 6:05 PM, Marek Vasut wrote:
> >>> On 3/31/20 4:16 PM, Simon Glass wrote:
> >>>> HI Marek,
> >>>
> >>> Hi,
> >>>
> >>> [...]
> >>>
> >>>>>>>> OK I have updated the coding style page with all of this.
> >>>>>>>
> >>>>>>> Which page ?
> >>>>>>
> >>>>>> https://www.denx.de/wiki/U-Boot/CodingStyle
> >>>>>
> >>>>> " U-Boot typically uses a C structure to map out the registers in an I/O
> >>>>> region, rather than offsets. The reasons for this are: " is misleading
> >>>>> and suggests that using structures is the best practice. This should be
> >>>>> reworded to make it clear both options are equally valid.
> >>>>
> >>>> I'd like to see a preference to use struct where it makes sense and
> >>>> not use it when it doesn't, with the different tradeoffs clearly
> >>>> written. Are asking that we say nothing about which is better in each
> >>>> situation?
> >>>
> >>> Correct, because I don't see a clear agreement on which one is better
> >>> and should be preferred.
> >>
> >> I see the wiki has still not been correctly reworded, can you please
> >> repair it ?
> >
> > What sort of 'repairs' are you thinking of?
>
> "
>  This may need to change to the kernel model if we allow for more
> run-time detection of what drivers are appropriate for what we're
> running on.
> "
>
> This was removed. I believe this is quite important to leave there.

OK it seemed to repeat the first point, but I suppose it is saying it
a different way. I updated it.

>> > I did add your counterpoints but did not change the text in the first
> > part, which has been long-standing. I think to do that we really would
> > need a wider discussion than just the two of us, particularly as we
> > seem not to agree on the 'default' position.
>
> ACK.

Regards,
Simon


More information about the U-Boot mailing list