lkml.org 
[lkml]   [2015]   [May]   [19]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    From
    Subject[PATCH v3] Added tables i2c_device_id and platform_device_id for checking. Extend checking on tables containing structures which are initialized without specifying member name.
    Date
    Signed-off-by: Daniel Granat <d.granat@samsung.com>
    ---
    scripts/coccinelle/misc/of_table.cocci | 26 +++++++++++++++++++++-----
    1 file changed, 21 insertions(+), 5 deletions(-)

    diff --git a/scripts/coccinelle/misc/of_table.cocci b/scripts/coccinelle/misc/of_table.cocci
    index 3c93404..74f7dbb 100644
    --- a/scripts/coccinelle/misc/of_table.cocci
    +++ b/scripts/coccinelle/misc/of_table.cocci
    @@ -1,6 +1,6 @@
    -/// Make sure of_device_id tables are NULL terminated
    +/// Make sure (of_device_id | i2c_device_id | platform_device_id) tables are NULL terminated
    //
    -// Keywords: of_table
    +// Keywords: of_table i2c_table platform_table
    // Confidence: Medium
    // Options: --include-headers

    @@ -13,7 +13,7 @@ virtual report
    identifier var, arr;
    expression E;
    @@
    -struct of_device_id arr[] = {
    +struct \(of_device_id \| i2c_device_id \| platform_device_id\) arr[] = {
    ...,
    {
    .var = E,
    @@ -24,7 +24,8 @@ struct of_device_id arr[] = {
    identifier var, arr;
    expression E;
    @@
    -struct of_device_id arr[] = {
    +(
    +struct \(of_device_id \| i2c_device_id \| platform_device_id\) arr[] = {
    ...,
    {
    .var = E,
    @@ -32,19 +33,34 @@ struct of_device_id arr[] = {
    + },
    + { }
    };
    +|
    +struct \(of_device_id \| i2c_device_id \| platform_device_id\) arr[] = {
    + ...,
    + { ..., E, ... },
    ++ { },
    +};
    +)

    @r depends on org || report@
    position p1;
    identifier var, arr;
    expression E;
    @@
    -struct of_device_id arr[] = {
    +(
    +struct \(of_device_id \| i2c_device_id \| platform_device_id\) arr[] = {
    ...,
    {
    .var = E,
    }
    @p1
    };
    +|
    +struct \(of_device_id \| i2c_device_id \| platform_device_id\) arr[] = {
    + ...,
    + { ..., E, ... },
    + @p1
    +};
    +)

    @script:python depends on org@
    p1 << r.p1;
    --
    1.9.1


    \
     
     \ /
      Last update: 2015-05-19 14:21    [W:4.256 / U:0.632 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site