[lkml]   [2016]   [Mar]   [27]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
Messages in this thread
    SubjectRe: [PATCH 01/30] ACPICA: Linuxize: reduce divergences for 20160212 release
    On Mon, 2016-03-28 at 03:02 +0000, Zheng, Lv wrote:
    > Hi,


    > > So why not fix the process script first?
    > > Maybe add something like:
    > > $ grep -E "^typedef\s+\w+\s*\*?\s*acpi_\w+" include/acpi/actypes.h | \
    > >   grep -Eoh "\bacpi_\w+"
    > >
    > > to the acpi_types variable in the lindent_single function
    > [Lv Zheng] 
    > I don't think this can work given:
    > 1. we are not only dealing with typedefs, but structs, struct xxx will be converted into types during the release process.
    > 2. we have only upper cased type names in ACPICA upstream, but have the lower cased type names in Linux, and this doesn't solve that.
    > So I guess you didn't test your idea.

    Good guess.

    The "maybe add something like" should give you a clue.

    > You need to pull ACPICA repo and do the followings to confirm if this is working:

    No, I disagree.  _I_ don't need to.  You need to.

    You shouldn't have a process that generates defective patches
    and then sends them to the list.

    cheers, Joe

     \ /
      Last update: 2016-03-28 05:41    [W:2.922 / U:0.340 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site