In 129-clickpad-threshold.patch I notice your patch reverses a chunk that the opensuse patch was removing. This bit of code:
@@ -524,6 +512,9 @@
pars->area_top_edge = set_percent_option(opts, "AreaTopEdge", height, priv->miny);
pars->area_bottom_edge = set_percent_option(opts, "AreaBottomEdge", height, priv->miny);
+ /* in clickpad mode, we don't want to sense the button area as default */
+ if (pars->area_bottom_edge == 0 && priv->is_clickpad)
+ pars->area_bottom_edge = b;
pars->area_left_edge = set_percent_option(opts, "AreaLeftEdge", width, priv->minx);
pars->area_right_edge = set_percent_option(opts, "AreaRightEdge", width, priv->minx);
I'm wondering if this was just an oversight, or if it's actually something you found was necessary to add? Can you explain why this chunk is being added in your patch? I'm going to omit it for now.
In 129-clickpad- threshold. patch I notice your patch reverses a chunk that the opensuse patch was removing. This bit of code:
@@ -524,6 +512,9 @@
pars- >area_top_ edge = set_percent_ option( opts, "AreaTopEdge", height, priv->miny); >area_bottom_ edge = set_percent_ option( opts, "AreaBottomEdge", height, priv->miny); area_bottom_ edge == 0 && priv->is_clickpad) bottom_ edge = b; >area_left_ edge = set_percent_ option( opts, "AreaLeftEdge", width, priv->minx); >area_right_ edge = set_percent_ option( opts, "AreaRightEdge", width, priv->minx);
pars-
+ /* in clickpad mode, we don't want to sense the button area as default */
+ if (pars->
+ pars->area_
pars-
pars-
I'm wondering if this was just an oversight, or if it's actually something you found was necessary to add? Can you explain why this chunk is being added in your patch? I'm going to omit it for now.