Hi Julia,

>>
>> Ahh yes, I see this here! (Actually sounds like a reasonable thing to do!)
>> @@ -207,37 +207,6 @@ enum chips { lm90, adm1032, lm99, lm86, max6657,
>> max6659, adt7461, max6680,
>>  #define MAX6696_STATUS2_LOT2   (1 << 7) /* local emergency limit tripped */
>>
>>  /*
>
> Is there really no - here?

Now that's a keen eye you have!

Yes - that is correct, but fear not ... its my fault for cropping the hunk.

There is a comment section below, and the hunk removes the first /*
there instead.


Full hunk below: I'll try not to 'cut corners' when pasting next time :)


diff --git a/drivers/hwmon/lm90.c b/drivers/hwmon/lm90.c
index c9ff08d..5b76055 100644
--- a/drivers/hwmon/lm90.c
+++ b/drivers/hwmon/lm90.c
@@ -207,37 +207,6 @@ enum chips { lm90, adm1032, lm99, lm86, max6657,
max6659, adt7461, max6680,
 #define MAX6696_STATUS2_LOT2   (1 << 7) /* local emergency limit tripped */

 /*
- * Driver data (common to all clients)
- */
-
-static const struct i2c_device_id lm90_id[] = {
-       { "adm1032", adm1032 },
-       { "adt7461", adt7461 },
-       { "adt7461a", adt7461 },
-       { "g781", g781 },
-       { "lm90", lm90 },
-       { "lm86", lm86 },
-       { "lm89", lm86 },
-       { "lm99", lm99 },
-       { "max6646", max6646 },
-       { "max6647", max6646 },
-       { "max6649", max6646 },
-       { "max6657", max6657 },
-       { "max6658", max6657 },
-       { "max6659", max6659 },
-       { "max6680", max6680 },
-       { "max6681", max6680 },
-       { "max6695", max6696 },
-       { "max6696", max6696 },
-       { "nct1008", adt7461 },
-       { "w83l771", w83l771 },
-       { "sa56004", sa56004 },
-       { "tmp451", tmp451 },
-       { }
-};
-MODULE_DEVICE_TABLE(i2c, lm90_id);
-
-/*
  * chip type specific parameters
  */
 struct lm90_params {



--
Regards

Kieran


>
> julia
>
>> - * Driver data (common to all clients)
>> - */
>> -
>> -static const struct i2c_device_id lm90_id[] = {
>> -       { "adm1032", adm1032 },
>> -       { "adt7461", adt7461 },
>> -       { "adt7461a", adt7461 },
>> -       { "g781", g781 },
>>
>>
>>
>>
>>
>>
>> > It would need to observe that the comment does not start at the beginning
>> > of the line, and treat it differntly.  Just to check, are you using
>> > Coccinelle 1.0.2?  I made a change in this regard recently, but I'm not
>> > sure it was an overall improvement.
>>
>> No - I'm afraid I'm using the version shipped with ubuntu 15.04:
>>
>> spatch --version
>> spatch version 1.0.0-rc22 with Python support and with PCRE support
>>
>>
>> --
>> Regards
>>
>> Kieran
>>
>> >
>> > julia
>>
_______________________________________________
Cocci mailing list
[email protected]
https://systeme.lip6.fr/mailman/listinfo/cocci

Reply via email to