ing() isn't a great argument alone,
because this adds three times the number of __match_string() calls than
there are __sysfs_match_string() calls. It's not a good model to follow.
Arguably both should be renamed.
BR,
Jani.
--
Jani Nikula, Intel
I will be happy.
>
> OK, I'll go queue it in tip/locking/core after -rc1. Thanks!
Is this where the series fell between the cracks, or was there some
follow-up that I missed?
I think this would still be useful. Andrzej, would you mind rebasing and
resending if there are no objections?
BR,
Jani.
--
Jani Nikula, Intel
o functional changes.
*are
Reviewed-by: Jani Nikula
>
> Signed-off-by: Thomas Zimmermann
> ---
> arch/powerpc/include/asm/backlight.h| 5 ++--
> arch/powerpc/platforms/powermac/backlight.c | 26 -
> 2 files changed, 2 insertions(+), 29 deletions(-)
; Closes:
> https://lore.kernel.org/dri-devel/ca+g9fysak5tbqqxfc2w4ohlga0cbthmxbeq8qayfxtu75yi...@mail.gmail.com/
> Signed-off-by: Thomas Zimmermann
> Fixes: 11b4eedfc87d ("fbdev: Do not include in header")
> Cc: Thomas Zimmermann
> Cc: Jani Nikula
> Cc: Helge Deller
On Tue, 05 Mar 2024, Michael Ellerman wrote:
> I prefer Thomas' patch, but I'm fine with either to go in via drm.
Thanks!
--
Jani Nikula, Intel
On Mon, 04 Mar 2024, Jani Nikula wrote:
> Removal of the backlight include from fb.h uncovered an implicit
> dependency in powerpc asm/backlight.h. Add the explicit include.
>
> Reported-by: Naresh Kamboju
> Closes:
> https://lore.kernel.org/r/ca+g9fysak5tbqqxfc2w4ohlga0cbt
v: Do not include in header")
Cc: Thomas Zimmermann
Cc: Helge Deller
Cc: linux-fb...@vger.kernel.org
Signed-off-by: Jani Nikula
---
Not even compile tested!
---
arch/powerpc/include/asm/backlight.h | 1 +
1 file changed, 1 insertion(+)
diff --git a/arch/powerpc/include/asm/backlight.h
b/ar
nal(struct eventfd_ctx *ctx)
> {
> - return eventfd_signal_mask(ctx, n, 0);
> + return eventfd_signal_mask(ctx, 1, 0);
> }
> EXPORT_SYMBOL_GPL(eventfd_signal);
>
--
Jani Nikula, Intel
On Wed, 21 Jun 2023, Joel Granados wrote:
> On Wed, Jun 21, 2023 at 02:16:55PM +0300, Jani Nikula wrote:
>> On Wed, 21 Jun 2023, Joel Granados wrote:
>> > Remove the empty end element from all the arrays that are passed to the
>> > register sysctl calls. In som
proc_handler = proc_dointvec_minmax,
> + .extra1 = SYSCTL_ZERO,
> + .extra2 = _sample_rate_hard_limit,
> + }
> };
The existing indentation is off, but fixing it doesn't really belong in
this patch.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
if (sinfo->backlight)
> return;
>
> @@ -173,7 +156,6 @@ static void init_backlight(struct atmel_lcdfb_info *sinfo)
> sinfo->backlight = bl;
>
> bl->props.power = FB_BLANK_UNBLANK;
> - bl->props.fb_blank = FB_BLANK_UNBLANK;
> bl->props.brightness = atmel_bl_get_brightness(bl);
> }
--
Jani Nikula, Intel Open Source Graphics Center
On Tue, 23 Nov 2021, Luis Chamberlain wrote:
> There is no need to user boiler plate code to specify a set of base
> directories we're going to stuff sysctls under. Simplify this by using
> register_sysctl() and specifying the directory path directly.
\o/
Acked-by: Jani Nikula
>
alse;
> - fallthrough;
> + break;
> case DRM_FORMAT_MOD_LINEAR:
> case I915_FORMAT_MOD_X_TILED:
> case I915_FORMAT_MOD_Y_TILED:
Acked-by: Jani Nikula
for merging via whichever tree seems best.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
be using
static tables for this. It's far from obvious.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
instance in
Documentation/gpu/i915.rst
However there is no "layout" label in i915.rst. The one being warned
about I can dig into based on the line number, but not the second
one. You have to resort to grepping the source. And avoiding duplicate
subsection headings in completely isolated places is a
anted. Nobody needs to know or care it's
specifically an unsigned long.
BR,
Jani.
--
Jani Nikula, Intel Open Source Graphics Center
just one conflict down the line because it touches
the kernel all over the place, then IMO it already wasn't worth
it. Merge conflicts are inevitable, but there's no reason to make life
harder just to cater for a cleanup patch. It's not that important.
Had it been split up, the drm parts would've
On Tue, 19 Dec 2017, Joe Perches <j...@perches.com> wrote:
> drivers/gpu/drm/i915/i915_sysfs.c | 12 ++--
For i915,
Acked-by: Jani Nikula <jani.nik...@intel.com>
--
Jani Nikula, Intel Open Source Technology Center
the option depending on it is
built-in. Obviously something that's built-in can't depend on something
built as a module.
BR,
Jani.
--
Jani Nikula, Intel Open Source Technology Center
___
Linuxppc-dev mailing list
Linuxppc-dev@lists.ozlabs.org
https
On Wed, 22 Oct 2014, Tomi Valkeinen tomi.valkei...@ti.com wrote:
On 18/10/14 00:13, Jani Nikula wrote:
Documentation/kbuild/kconfig-language.txt warns to use select with care,
and in general use select only for non-visible symbols and for symbols
with no dependencies, because select will force
: Benjamin Herrenschmidt b...@kernel.crashing.org
Cc: Jens Frederich jfreder...@gmail.com
Cc: Daniel Drake d...@laptop.org
Cc: Jon Nettleton jon.nettle...@gmail.com
Cc: Jean-Christophe Plagniol-Villard plagn...@jcrosoft.com
Cc: Tomi Valkeinen tomi.valkei...@ti.com
Signed-off-by: Jani Nikula jani.nik
21 matches
Mail list logo