Re: [patch -mm 20/20 RFC] chardev: GPIO for SCx200 & PC-8736x: add sysfs-GPIO interface



Jim Cromie <jim.cromie@xxxxxxxxx> wrote:

Heres a better version

Well..

***************
*** 312,317 ****
}

pc8736x_init_shadow();
return 0;
}

--- 422,428 ----
}

pc8736x_init_shadow();
+ pc8736x_sysfs_init(&pdev->dev);
return 0;
}

I don't know how to fix that - pc8736x_gpio_init() doesn't call
pc8736x_init_shadow().

Plus the patch you sent still has spaces all over the place where we should
have tabs. You can fix that by editing the diff, search for "^+ ".

So I'll drop this patch altogether. Which is probably for the best at this
stage, because we're expecting a pile of teeny fixes from you agaisnt the
other 19 patches as a result of review comments (yes?)

Once all that has landed we can look at the sysfs-GPIO interface.
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/



Relevant Pages

  • RE: Importing Data
    ... You will need to fix this later. ... EXCEL or you can ignore them and add them into the table later, ... Create an select query based upon your Import table. ... Otherwise you need to review the data and fix the errors. ...
    (microsoft.public.access.tablesdbdesign)
  • Re: [PATCH 1/4] Blackfin: arch patch for 2.6.18
    ... We'll try to fix them ASAP. ... This is the blackfin architecture for 2.6.18, ... Please review it and give you comments, ... Best regards, ...
    (Linux-Kernel)
  • Re: Dpreview delays review to allow Canon to fix problem?
    ... allow Canon to come up with a fix for the focusing problems it has. ... cameras, the problems only the the USERS catch? ... Lets see what this review looks like if and when it finally ...
    (rec.photo.digital.slr-systems)
  • Re: Dpreview delays review to allow Canon to fix problem?
    ... allow Canon to come up with a fix for the focusing problems it has. ... cameras, the problems only the the USERS catch? ... Lets see what this review looks like if and when it finally ...
    (rec.photo.digital.slr-systems)
  • Re: [patch] slab: resize the alien caches too
    ... when user changes the slab tunables. ... Appended patch ... tries to fix this. ... Please review and let me know if I missed anything. ...
    (Linux-Kernel)