Hi Guenter,
> >>> switch (index) {
> >>> +case 0: /* temp1_input */
> >>> +if (drv->get_temp)
> >>> +mode |= S_IRUGO;
> >>
> >> This should be mandatory. Sorry, I don't really understand what you
> >> are trying to accomplish here.
> >>
>
Hi Guenter,
switch (index) {
+case 0: /* temp1_input */
+if (drv-get_temp)
+mode |= S_IRUGO;
This should be mandatory. Sorry, I don't really understand what you
are trying to accomplish here.
Can you give me a real world
hi Vivien,
On 04/16/2015 03:05 PM, Vivien Didelot wrote:
Hi Guenter,
switch (index) {
+case 0: /* temp1_input */
+if (drv->get_temp)
+mode |= S_IRUGO;
This should be mandatory. Sorry, I don't really understand what you are
trying to
Hi Guenter,
> > switch (index) {
> > +case 0: /* temp1_input */
> > +if (drv->get_temp)
> > +mode |= S_IRUGO;
>
> This should be mandatory. Sorry, I don't really understand what you are
> trying to accomplish here.
>
> Can you give me a
On Thu, Apr 16, 2015 at 02:38:19PM -0400, Vivien Didelot wrote:
> A switch driver may only provide one of the temperature limit accessors,
> or the temperature alarm getter. So register the hwmon subsystem if any
> of the related functions is provided.
>
> Thus, check get_temp to set the
A switch driver may only provide one of the temperature limit accessors,
or the temperature alarm getter. So register the hwmon subsystem if any
of the related functions is provided.
Thus, check get_temp to set the visibility of temp1_input.
Signed-off-by: Vivien Didelot
---
net/dsa/dsa.c | 7
On Thu, Apr 16, 2015 at 02:38:19PM -0400, Vivien Didelot wrote:
A switch driver may only provide one of the temperature limit accessors,
or the temperature alarm getter. So register the hwmon subsystem if any
of the related functions is provided.
Thus, check get_temp to set the visibility of
Hi Guenter,
switch (index) {
+case 0: /* temp1_input */
+if (drv-get_temp)
+mode |= S_IRUGO;
This should be mandatory. Sorry, I don't really understand what you are
trying to accomplish here.
Can you give me a real world
hi Vivien,
On 04/16/2015 03:05 PM, Vivien Didelot wrote:
Hi Guenter,
switch (index) {
+case 0: /* temp1_input */
+if (drv-get_temp)
+mode |= S_IRUGO;
This should be mandatory. Sorry, I don't really understand what you are
trying to
A switch driver may only provide one of the temperature limit accessors,
or the temperature alarm getter. So register the hwmon subsystem if any
of the related functions is provided.
Thus, check get_temp to set the visibility of temp1_input.
Signed-off-by: Vivien Didelot
10 matches
Mail list logo