lkml.org 
[lkml]   [2010]   [Feb]   [23]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
Date
From
SubjectRe: [PATCH v3 1/3] MFD: introduce lpc_sch for Intel SCH LPC bridge
Hi Samuel,
Regarding renaming of sch* to isch* do you want incremental patch, or fresh version?

Denis

Jean Delvare wrote:
>> +static struct mfd_cell lpc_sch_cells[] = {
>> + {
>> + .name = "isch_smbus",
>> + .num_resources = 1,
>> + .resources = &smbus_sch_resource,
>> + },
>> + {
>> + .name = "sch_gpio",
>> + .num_resources = 1,
>> + .resources = &gpio_sch_resource,
>> + },
>> +};
>
> These names are nicely inconsistent. What about "isch_gpio"?
>

>> +obj-$(CONFIG_LPC_SCH) += lpc_sch.o
>
> I don't like this name either. There is another vendor (SMSC) shipping
> LPC devices with "SCH" in their names, so there is room for confusion.
> "isch" makes it clearer that we are talking about the Intel ones.
>



\
 
 \ /
  Last update: 2010-02-23 10:27    [W:0.064 / U:0.776 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site